MVC Async-Methode in Konstruktor-Controller

Question

Ich versuche, dynamische Menü (in DB gespeichert) zu machen, die auf allen Web-App-Seiten angezeigt wird. Mit Google fand ich, dass es besser ist, die Menüansicht als Teil der Master-Ansicht (_Layout.cshtml) zu machen. Und deshalb muss jede Aktionsmethode des Controllers Daten mit dem Menümodell enthalten. Um zu vermeiden, Code-Duplizierung fand ich die Lösung, die eine Basis-Controller zu erstellen und liefern Daten über den Konstruktor:

https://docs.microsoft.com/en-us/aspnet/mvc/overview/older-versions-1/views/passing-data-to-view-master-pages-cs

Auch ich versuche async/verwenden warten Möglichkeiten und meine PageService (Menü) wird mit ToListAsync() um Daten von der DB zu bekommen. So, jetzt habe ich ein Problem, dass Base Konstruktor hat eine Asynchron-Methode:

public class BaseController : AsyncController, IBaseController 
{ 
    private readonly IPageService _pageService; 

    public BaseController(IPageService pageService) 
    { 
     _pageService = pageService; 
     SetBaseViewModelAsync(); 
    } 

    private async Task SetBaseViewModelAsync() 
    { 
     ViewData["Pages"] = await _pageService.GetAllAsync(); 
    } 
} 

Ich weiß, dass dies BAD CODE, aber ich weiß nicht, wie diese Situation richtig zu gestalten. Vielleicht gibt es einen anderen besseren Weg, um das dynamische Menü oder eine andere Möglichkeit, Daten asynchron zu bekommen, zu erstellen?

Auch fand ich diesen Artikel, aber ich weiß nicht, ob ich seine Lösungen anwenden kann, weil ich weiß nicht, ob ich Controller-Instanz Schöpfung umgehen kann:

http://blog.stephencleary.com/2013/01/async-oop-2-constructors.html

Answer 1

Statt alles abzuleiten von einer Basissteuerung (die eine Menge zusätzlicher Arbeit und Prüfung sein kann) Sie einen Controller einfach erstellen können MenuController, erstellen Sie eine Methode Default genannt und dann rufen Sie es von Ihrem Plan genannt:

[ChildActionOnly] 
public Default() 
{ 
    var viewModel = _pageService.GetAllAsync(); 
    return Partial(viewModel); 
} 

in Ihrem Layout:

@{Html.RenderAction("Default", "Menu");} 

Dies ist wirklich die einfachste und sauberste Lösung. Die größte PRO ist, dass Sie den Cache für das Menü getrennt von der Methode aufrufen können. Es gibt keine gute Lösung für asp.net-mvc (1-5), Async-Code auf diese Weise auszuführen. (ActionFilters can't be async und (Render) Partials nicht async sein. Sie können nach wie vor ein Asynchron-Methode aufrufen, es wird nur Sync laufen.

Render vs Non-Render Performance.

Answer 2

Ich änderte Funktionalität Html.RenderAction in meiner Ansicht zu nennen, wie Erik Philips vorgeschlagen:

@{ 
    Html.RenderAction("Index", "Pages"); 
} 

und Controller:

public class PagesController : AsyncController, IPagesController 
{ 
    private readonly IPagesService _pagesService; 

    public PagesController(IPagesService pagesService) 
    { 
     _pagesService = pagesService; 
    } 

    [HttpGet] 
    [Route("")] 
    public async Task<ActionResult> IndexAsync() 
    { 
     var viewModel = await _pagesService.GetAllAsync(); 
     return PartialView("MenuPartial", viewModel); 
    } 
} 

Aber Render funktioniert nicht mit Asynchron-Controller Aktionen:

Async PartialView causes "HttpServerUtility.Execute blocked..." exception

scheint So wie sync Aufruf der einzige hier möglich ist.