IoCコンテナを使用してServiceクラスのServiceConfigを解決する適切な設計

Question

私の設計については、適切かどうかは疑問です。 IoC autofacコンテナのXML構成で

public class Service 
{ 
    private ServiceConfig Config { get; set; } 

    //inner class to store configuration 
    public class ServiceConfig 
    { 
    //a lot of configuration properties 
    } 

    //ctor - self resolve Config dependency - wondering whether this is very bad ? 
    public Service() 
    { 
    //get configuration class from IoC container - accessible as Singleton 
    Config = ContainerResolver.Instance().Resolve<Service.ServiceConfig>(); 
    } 

    //other code that provides functionality making use of Config property 
    //... 
} 

あなたは私が私の構成プロパティを維持するつもりだ内部ServiceConfigクラスがある見ることができるように私が登録し、それらのすべて：。

私はクラスを持っています。可能であれば、いくつかのOODの教祖がそれについていくつか言いたいと思います。

、 ルーク最初

Answer 1

は、はい、これは悪いデザインです。インスタンスが「ルート」である場合や、型がコンテナを介して簡単に解決できない場合にのみ、コンテナと直接対話する必要があります。ルートインスタンスは通常、スタートアップコードまたはブートストラップです。この場合

、あなたのコンテナが明確にすでに、なぜこのようにそれをしない、ServiceConfigを解決する方法を知っているので：

public class Service 
{ 
    private ServiceConfig Config { get; set; } 

    public class ServiceConfig 
    { 
    } 

    public Service(ServiceConfig serviceConfig) 
    { 
     Config = serviceConfig; 
    } 

    ... 
}