ASP.NET MVC: HTTPContext и Dependency Injection

Question

В настоящее время у меня есть ActionFilter, который получает имя текущего пользователя из HttpContext и передает его в действие, которое использует его для метода службы. например:

Service.DoSomething(userName); 

У меня теперь есть причина сделать это не на уровне действия, а на уровне конструктора контроллера. В настоящее время я использую структуру структуры для создания контроллеров и ввода услуги. Я смотрю на что-то вроде:

public interface IUserProvider 
{ 
    string UserName { get; } 
} 

public class HttpContextUserProvider : IUserProvider 
{ 
    private HttpContext context; 

    public HttpContextUserProvider(HttpContext context) 
    { 
     this.context = context; 
    } 

    public string UserName 
    { 
     get 
     { 
      return context.User.Identity.Name; 
     } 
    } 
} 

Это сказал мой IoC Foo действительно слаб, как это первый проект, который я использовал его.

Итак, мой вопрос ... как я могу передать структурную карту для передачи в HttpContext в конструкторе HttpContextUserProvider? Это просто странно ... Я не уверен, как думать о HttpContext.

Answer 1

имеет интерфейс аннотацию HttpContext.Current. Expose только те методы, которые вам необходимы. GetUserName() назвал бы HttpContext.Current.User.Identity.Name в Например, сделайте это настолько тонким, насколько это возможно.

Возьмите эту абстракцию и введите ее в свой другой класс провайдера. Это позволит вам протестировать поставщика, насмехаясь над http контекстная абстракция. В качестве побочного преимущества вы можете делать другие отличные вещи с помощью абстракции HttpContext, помимо этого. Повторно используйте его, с одной стороны. Добавить общие параметры типа в мешки и т. Д.

Answer 2

Возможно, я что-то упустил, но вышеприведенный ответ не работает для меня (с тех пор он был удален - это был еще полезный ответ, хотя - он показал, как сообщить SM, чтобы передать аргументы конструктора). Вместо этого, если я это сделаю:

ObjectFactory.Initialize(x => 
{ 
    x.BuildInstancesOf<HttpContext>() 
     .TheDefault.Is.ConstructedBy(() => HttpContext.Current); 
    x.ForRequestedType<IUserProvider>() 
     .TheDefault.Is.OfConcreteType<HttpContextUserProvider>(); 
}); 

Я получаю его на работу. Я сделал это после того, как найти: http://codebetter.com/blogs/jeremy.miller/archive/2008/03/20/if-you-need-something-in-structuremap-but-you-can-t-build-it-with-new.aspx

---

редактировать:

Благодаря ответ Брэда я думаю У меня есть лучшая ручка на HttpContext. Его ответ определенно работает, я просто не уверен, что мне нравится иметь вызов HttpContext.Current внутри класса (похоже, он скрывает зависимость, но я далек от эксперта по этому поводу).

Вышеприведенный код должен работать для инъекции HttpContext, насколько я могу судить. Мэтт Хинзе добавляет, что если все, что мне нужно из HttpContext, является User.Identity.Name, мой проект должен быть явным в этом отношении (имея интерфейс вокруг HttpContext, только разоблачающий то, что мне нужно). Я думаю, что это хорошая идея.

Дело в том, что за день я понял, что мое обслуживание действительно зависит только от строки: userName. Если это зависит от IUserProvider, возможно, не имеет большой добавленной стоимости. Поэтому я знаю, что не хочу, чтобы он зависел от HttpContext, и я знаю, что все, что мне нужно, это строка (имя_пользователя). Мне нужно узнать, могу ли я достаточно изучить StructureMap foo, чтобы установить это соединение для меня. (ответ sirrocoo дает подсказку о том, с чего начать, но он удалил его: *().

Answer 3

Я не уверен, почему вы беспокоитесь. Похоже, что использование HttpContext.Current прямо в HttpContextUserProvider - это правильная вещь делать. вы никогда не будете подставляя в другом HttpContext ...

Answer 4

Похоже, что вы должны использовать HttpContextBase вместо HttpContextUserProvider.Это абстракция из HttpContext и позволяет создавать макет, писать UnitTests и вводить ваши зависимости.

public class SomethingWithDependenciesOnContext 
{ 
    public SomethingWithDependenciesOnContext(HttpContextBase context) { 
     ... 
    } 

    public string UserName 
    { 
     get {return context.User.Identity.Name;} 
    } 
} 

ObjectFactory.Initialize(x => 
      x.For<HttpContextBase>() 
      .HybridHttpOrThreadLocalScoped() 
      .Use(() => new HttpContextWrapper(HttpContext.Current));