依存性注入 - ロジックをオーバーロードされたコンストラクタに配置しますか？

Question

今日、私はこのクラスを持っている：

public class SmtpEmailProvider : IMessageProvider 
{ 
    private readonly SmtpClientWrapper _smtpClientWrapper; 

    public SmtpEmailProvider(SmtpClientWrapper smtpClientWrapper) 
    { 
     _smtpClientWrapper = smtpClientWrapper; 
    } 

はSmtpClientを模擬できるようにするために、私はこのようにそれを包みました：

public class SmtpClientWrapper 
{ 
    private readonly SmtpClient _smtpClient; 

    public SmtpClientWrapper(SmtpClient smtpClient) 
    { 
     _smtpClient = smtpClient; 
    } 

    public virtual void Send(MailMessage msg) 
    { 
     if (_smtpClient == null) throw new InvalidOperationException("SmtpClient must be passed to the constructor before calling Send."); 

     _smtpClient.Send(msg); 
    } 
} 

今私はSmtpEmailProviderを開始するには、これを行うことができますクラス、およびそこSmtpClientロジックを配置します。

public IMessageProvider LocateProviderByName(string providerName) 
{ 
    var client = new SmtpClient 
        { 
         Host = "127.0.0.1", 
         Port = 25 
        }; 
    client.Credentials = new NetworkCredential("...", "..."); 
    return new SmtpEmailProvider(new SmtpClientWrapper(client)); 
} 

をしかし、私はそれを交換したい：

public IMessageProvider LocateProviderByName(string providerName) 
{ 
    return IoC.Resolve<IMessageProvider>(providerName); 
} 

次に、ロジックをパラメータなしでコンストラクタに配置する必要があります。しかし、私はその時、コンストラクターでずっとやっているような気持ちになります。

他の方法はありますか？

Answer 1

私はちょうどあなたが達成しようとしているものについて少し混乱しています。あなたがSmtp電子メールを提供する必要があり、IoCを使用していると仮定するなら、IoCフレームワークを使ってオブジェクトグラフ全体を作成して配線する必要があります。つまり、SmtpClientを作成するようにIoCフレームワークを設定し、SmtpClientWrapperを作成して、SmtpEmailProviderを作成します。 SmtpEmailProviderコンストラクタに依存関係作成ロジックを配置する必要はありません。ここで

は、あなたが提供するコード与えられ、城ウィンザーでの例です：

public IMessageProvider LocateProviderByName(string providerName) 
{ 
    return IoC.Resolve<IMessageProvider>(providerName); 
} 

var messageProvider = LocateProviderByName("smtpProvider"); 

キーポイント：

<configuration> 
    <component id="smtpClient" type="System.Net.Mail.SmtpClient, System"> 
    <parameters> 
     <Host>127.0.0.1</Host> 
     <Port>25</Port> 
    </parameters> 
    </component> 
    <component id="smtpClientWrapper" type="Naespace.SmtpClientWrapper, Assembly"> 
    <parameters> 
     <smtpClient>${smtpClient}</smtpClient> 
    </parameters> 
    </component> 
    <component id="smtpProvider" service="Namespace.IMessageProvider, Assembly" type="Namespace.SmtpEmailProvider, Assembly"> 
    <parameters> 
     <smtpClientWrapper>${smtpClientWrapper}</smtpClientWrapper> 
    </parameters> 
    </component> 
</configuration> 

を上記ウィンザーの構成では、単にそのようなあなたのIMessageProviderを作成することができますここでは、IoCコンテナを使用することです：あなたのために完全なオブジェクトグラフを作成する必要がある依存関係の作成と管理システム。これは、コンストラクタ内の依存性管理ロジックが多すぎるという問題を軽減します。