SmtpClient、SendAsync、Disposeを.NET 4.0で使用するためのベストプラクティス

Question

SmtpClientを使い捨てにできるようになったので、特にSendAsyncを使用して呼び出しを行うと、管理が少し難解です。おそらく、SendAsyncが完了するまでDisposeを呼び出すべきではありません。しかし、私はそれを呼び出すべきかどうか（例えば、 "使用する"を使用して）。このシナリオは、通話が行われると定期的に電子メールを送信するWCFサービスです。計算の大半は高速ですが、電子メールの送信には1秒ほどかかる場合がありますので、非同期が望ましいでしょう。

メールを送信するたびに新しいSmtpClientを作成する必要がありますか？私はWCF全体のために1つ作成する必要がありますか？助けて！

更新場合によっては、それぞれの電子メールが常にユーザーに合わせてカスタマイズされます。 WCFはAzureでホストされ、Gmailはメーラーとして使用されます。

Answer 1

注：.NET 4.5 SmtpClientがasync awaitable方法SendMailAsyncを実装します。下位バージョンの場合は、下記のようにSendAsyncを使用してください。

---

できるだけ早く、IDisposableインスタンスを廃棄してください。非同期呼び出しの場合、これはメッセージが送信された後のコールバックにあります。

var message = new MailMessage("from", "to", "subject", "body")) 
var client = new SmtpClient("host"); 
client.SendCompleted += (s, e) => { 
          client.Dispose(); 
          message.Dispose(); 
         }; 
client.SendAsync(message, null); 

SendAsyncはコールバックを受け付けません。

Answer 2

一般に、IDisposableオブジェクトはできるだけ早く処理する必要があります。 IDisposableをオブジェクトに実装することは、問題のクラスが確定的にリリースされるべき高価なリソースを保持しているという事実を伝えることを意図しています。しかし、これらのリソースを作成するのはコストがかかり、これらのオブジェクトをたくさん作成する必要がある場合、メモリ内に1つのインスタンスを保持して再利用するほうがパフォーマンスが向上します。違いがあるかどうかを知るための唯一の方法はあります：それをプロファイルしてください！

Re：disposingとAsync：明らかにusingは使用できません。代わりに、一般的にSendCompletedイベントでオブジェクトを配置：

var smtpClient = new SmtpClient(); 
smtpClient.SendCompleted += (s, e) => smtpClient.Dispose(); 
smtpClient.SendAsync(...); 

Answer 3

元の質問は.NET 4のために求められましたが、.NET 4.5のように役立つ場合は、SmtpClientは非同期で実行可能なメソッド SendMailAsyncを実装しています。

その結果、非同期でメールを送信するためには、次の通りである：

public async Task SendEmail(string toEmailAddress, string emailSubject, string emailMessage) 
{ 
    using (var message = new MailMessage()) 
    { 
     message.To.Add(toEmailAddress); 

     message.Subject = emailSubject; 
     message.Body = emailMessage; 

     using (var smtpClient = new SmtpClient()) 
     { 
      await smtpClient.SendMailAsync(message); 
     } 
    } 
} 

それは、SendAsyncメソッドを使用して避ける方が良いでしょう。