SmtpClient.SendAsync code revue

Question

Quelle est la bonne façon de disposer de SmtpClient et MailMessage tout en utilisant SendAsync?

J'ai copié mon code ci-dessous.

{ 
... 
var client = new SmtpClient {Host = _smtpServer}; 
client.SendCompleted += SendCompletedCallback; 
var userState = mailMessage; 
client.SendAsync(mailMessage, userState); 
... 
} 

private static void SendCompletedCallback(object sender, 
    AsyncCompletedEventArgs e) 
{ 
    // Get the unique identifier for this asynchronous operation. 
    var mailMessage= (MailMessage)e.UserState; 

    if (e.Cancelled) 
    { 
     Log.Info(String.Format("[{0}] Send canceled.", mailMessage)); 
    } 
    if (e.Error != null) 
    { 
     Log.Error(String.Format("[{0}] {1}", mailMessage, e.Error)); 
    } 
    else 
    { 
     Log.Info("Message sent."); 
    } 
    mailMessage.Dispose(); 
} 

le MailMessage après Mise au rebut du client.SendAsync(...) lève une exception. Je dois le disposer dans le gestionnaire de rappel.

Answer 1

Cela semble correct.

Notez que MailMessage ne remplace pas ToString, donc vos journaux indiqueront simplement [MailMessage] Send cancelled.
Vous pouvez utiliser le proeprty Subject (ou une autre propriété) à la place.

Answer 2

Je pense que cela pourrait aider

client.SendCompleted += (s, e) => { client.Dispose(); message.Dispose(); };