Ninjectを使ってサービスを設定する

Question

まず、Ninjectを新しくしましたが、この質問はNinjectを対象としていますが、一般的にDIに当てはまるようです。

ここに何か不足していると思います。これまでに提案されたソリューションはすべて、ひどく複雑なようです。

私はこのようなものだった：

public class MyController : Controller 
{ 
    private IMyService _Service; 

    public MyController() 
     :this(null) 
    { } 

    public MyController(IMyService service) 
    { 
     _Service = service ?? new MyService(ModelState); 
    } 
} 

public IMyService 
{} 

public class MyService : IMyService 
{ 
    private ModelStateDictionary _Model; 

    public MyService(ModelStateDictionary model) 
    { 
     _Model = model; 
    } 
} 

をそして私は、私はそれにNinjectを行くだろうと思っていました。そして、この思い付いた：

public class MyController : Controller 
{ 
    private IMyService _Service; 

    public MyController() 
     :this(null) 
    { 
     _Service = Locator.Kernel.Get<IMyService>(new Ninject.Parameters.ConstructorArgument("model", ModelState)); 
    } 
} 

public class MyServiceModule : NinjectModule 
{ 
    public override Load() 
    { 
     Bind<IMyService>().To<MyService>(); //here 
    } 
} 

それは、私はそれが結合ビット（ここではマーク）を変更することができるはずです、しかし私には思えるので、それはむしろ私が欲しいときよりも、にModelStateを取得するには、その時点で知っています具体的なサービスクラスの事前知識が必要なコンストラクタのインスタンス。

私は不必要に心配していますか、これを行う良い方法がありますか？

サイモン

Answer 1

MyServiceが本当にModelStateDictionaryを構築する必要がありますか？

私はリファクタリングを検討しているので、Serviceクラスの構築ではなく、私が呼び出していたメソッドにModelStateDictionaryを渡していました。

なリファクタリングが不合理である場合、あなたはおそらくModelStateDictionary

public interface IModelStateProvider { 
    ModelStateDictionary GetModelState(); 
} 

の上に抽象化レイヤーを追加し、セットアップが結合、現在のコンテキストのModelStateDictionaryを取得することができ、そのインタフェースの実装を作ることになるでしょうそのインターフェイスのコンストラクタにそれを取るためにあなたのサービスクラスを変更：

public class MyService : IMyService 
{ 
    private ModelStateDictionary _Model; 

    public MyService(IModelStateProvider modelStateProvider) 
    { 
     _Model = modelStateProvider.GetModelState(); 
    } 
}