User.GetUserId() kończy się niepowodzeniem w konstruktorze kontrolera

Question

Otrzymuję następujący błąd: Wartość nie może być pusta. Nazwa parametru: główny

Jak uzyskać dostęp do Tożsamości (userId) wewnątrz konstruktora kontrolera? Mogę go uruchomić tylko przez zawinięcie nieudanego połączenia w funkcję (obie podświetlone poniżej).

Czy jest coś, co muszę wstrzyknąć?

public class BaseController : Controller { 
    protected readonly MylDbContext dbContext; 
    protected readonly string userId; 

    public BaseController(MylDbContext dbContext) { 
     this.dbContext = dbContext; 
     userId = User.GetUserId(); // fails here 
    } 

    public string GetUserId() { 
     return User.GetUserId(); // works here 
    } 
} 

Answer 1

Jak wspomniano @Henk, konstruktor kontroler będzie wykonywany przed ActionContext został ustawiony, więc nie będziesz mieć dostępu do usług takich jak Context, Request lub User. Musisz pobrać identyfikator użytkownika w kontekście żądania.

Można użyć staroświeckiego podejścia z action filters, które są nadal częścią potoku MVC6 (który obsługuje także filtry akcji asynchronicznej za pośrednictwem IAsyncActionFilter).

Ponieważ chcesz ustawić właściwość w kontrolerze, najprostszym sposobem, w jaki można to zaimplementować, jest przesłonięcie metody OnActionExecuting w klasie kontrolera. Działa to, ponieważ dziedziczysz po Controller, która już implementuje IActionFilter.

public override void OnActionExecuting(ActionExecutingContext context) 
{ 
    //Get user id 
    userId = User.GetUserId(); 
} 

Edit

Jeśli zaznaczysz DefaultControllerFactory widać, że:

1. pierwszy kontroler jest stworzony

2. wtedy ActionContext jest ustawiony (Za pomocą DefaultControllerPropertyActivator który jest jeden z aktywatorów właściwości):

   var controller = _controllerActivator.Create(actionContext, controllerType); 
   foreach (var propertyActivator in _propertyActivators) 
   { 
       propertyActivator.Activate(actionContext, controller); 
   } 
   

Answer 2

Po utworzeniu kontrolera nie ma gwarancji, że informacje o żądaniu są dostępne w jeszcze HttpContext. Może wcale nie być prośby. Czy istnieje powód, dla którego potrzebujesz tej informacji w konstruktorze?

Edytuj
2 Rozumiem twój problem. Co zwykle zrobić w sytuacji jak ta, jest utworzyć właściwość z polem podkładowej, która tylko raz na kontrolerze zapytania:

private int? _userId; 
public int UserId 
{ 
    get 
    { 
     if (!_userId.HasValue) 
     { 
      // query from db. 
      _userId = 42; 
     } 
     return _userId.Value; 
    } 
}