ASP.NET MVC：のHttpContextと依存性注入

Question

現在、私はのHttpContextから、現在のユーザー名を取得し、サービスの方法でそれを使用するアクションにそれを渡すのActionFilterを持っています。例：

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は本当に弱いです。

だから私の質問は、私はHttpContextUserProviderのコンストラクタでのHttpContextに渡す構造マップを伝えることができるか...でしょうか？これはちょっと変わったようです...私はHttpContextの考え方がわかりません。

Answer 1

インターフェイスの抽象的なHttpContext.Currentが必要です。必要なメソッドだけを公開してください。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を注入するために働く必要があります。 Matt Hinzeは、HttpContextから必要なものがすべてUser.Identity.Nameである必要がある場合、私のデザインは明示的に（HttpContextの周囲にIが必要なものだけを公開するようにする）必要があるという点を追加しました。これは良い考えだと思う。ユーザー名を：

事は、私はちょっと文字列のみに依存する必要が本当に私のサービスを実現して昼食を超えています。それがIUserProviderに依存することは、あまり価値がないかもしれません。だから私はそれがHttpContextに依存することを望んでいないことを知っている、私は必要なすべてが文字列（userName）であることを知っている - 私はこの接続を作るために十分なStructureMap fooを学ぶことができるかどうかを確認する必要があります。 （*）。

Answer 3

なぜあなたは気にしないのですか？HttpContext.CurrentをHttpContextUserProviderで直接使用しているようですが、これは正しいことです。別のHttpContextに置き換えることは決してありません...

Answer 4

HttpContextUserProviderの代わりにHttpContextBaseを使用する必要があります。これはデフォルトの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));