¿Cuáles son las mejores prácticas para usar SmtpClient, SendAsync y Dispose en .NET 4.0

Question

? Estoy un poco perplejo sobre cómo administrar SmtpClient ahora que es desechable, especialmente si realizo llamadas usando SendAsync. Presumiblemente, no debería llamar a Dispose hasta que SendAsync se complete. Pero debería llamarlo alguna vez (por ejemplo, usar "usar"). El escenario es un servicio WCF que envía correos electrónicos periódicamente cuando se realizan llamadas. La mayor parte del cálculo es rápido, pero el envío de correo electrónico puede tomar un segundo más o menos, por lo que Async sería preferible.

¿Debo crear un nuevo SmtpClient cada vez que envío un correo? ¿Debo crear uno para toda la WCF? ¡Ayuda!

Actualización En caso de que haga una diferencia, cada correo electrónico siempre se personaliza para el usuario. El WCF está alojado en Azure y Gmail se utiliza como el anuncio publicitario.

Answer 1

Nota: .NET 4.5 SmtpClient implementa async awaitable método SendMailAsync. Para versiones inferiores, use SendAsync como se describe a continuación.

---

siempre se debe disponer de IDisposable casos con la mayor brevedad. En el caso de las llamadas asincrónicas, esto se realiza en la devolución de llamada después de enviar el mensaje.

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); 

Es un poco molesto el SendAsync no acepta una devolución de llamada.

Answer 2

En general, los objetos descartables deben eliminarse lo antes posible; la implementación de IDisposable en un objeto tiene la intención de comunicar el hecho de que la clase en cuestión contiene recursos costosos que deberían ser liberados determinísticamente. Sin embargo, si la creación de esos recursos es costosa y necesita construir muchos de estos objetos, puede ser mejor (en términos de rendimiento) mantener una instancia en la memoria y reutilizarla. Solo hay una forma de saber si eso hace alguna diferencia: ¡perfilarlo!

Re: eliminación y Async: no se puede usar using obviamente. En su lugar, normalmente disponer el objeto en caso SendCompleted:

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

Answer 3

La pregunta original fue para .NET 4, pero si ayuda a partir de .NET 4.5 SmtpClient implementa el método asincrónico SendMailAsync.

Como resultado, para enviar correo electrónico de forma asíncrona es como la siguiente:

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); 
     } 
    } 
} 

Es mejor evitar el uso de método SendAsync.