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

Estoy un poco perplejo sobre cómo administrar SmtpClient ahora que es desechable, especialmente si hago llamadas usando SendAsync. Es de suponer que no debería llamar a Dispose hasta que se complete SendAsync. Pero, ¿debería llamarlo alguna vez (por ejemplo, usando "using"). 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 del correo electrónico puede tardar aproximadamente un segundo, por lo que sería preferible Async.

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

Actualización En caso de que marque la diferencia, cada correo electrónico siempre está personalizado para el usuario. WCF está alojado en Azure y Gmail se usa como correo.

Nota: .NET 4.5 SmtpClient implementa el async awaitablemétodo SendMailAsync. Para versiones inferiores, utilice SendAsynccomo se describe a continuación.

Siempre debe deshacerse de las IDisposableinstancias lo antes posible. En el caso de las llamadas asíncronas, esto está en la devolución de llamada después de que se envía 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 SendAsyncque no acepte una devolución de llamada.

La pregunta original se hizo para .NET 4, pero si ayuda a partir de .NET 4.5 SmtpClient implementa un método asincrónico en espera SendMailAsync.

Como resultado, enviar correo electrónico de forma asincrónica es el 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 del método SendAsync.

En general, los objetos desechables deben desecharse lo antes posible; La implementación de IDisposable en un objeto pretende comunicar el hecho de que la clase en cuestión contiene recursos costosos que deberían liberarse de forma determinista. 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: desechar y Async: usingobviamente no se puede usar . En su lugar, normalmente desecha el objeto en el evento SendCompleted:

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

Ok, vieja pregunta, lo sé. Pero me encontré con esto yo mismo cuando necesitaba implementar algo similar. Solo quería compartir un código.

Estoy iterando sobre varios SmtpClients para enviar varios correos de forma asincrónica. Mi solución es similar a TheCodeKing, pero estoy desechando el objeto de devolución de llamada. También estoy pasando MailMessage como userToken para obtenerlo en el evento SendCompleted para que pueda llamar a dispose también. Me gusta esto:

foreach (Customer customer in Customers)
{
    SmtpClient smtpClient = new SmtpClient(); //SmtpClient configuration out of this scope
    MailMessage message = new MailMessage(); //MailMessage configuration out of this scope

    smtpClient.SendCompleted += (s, e) =>
    {
        SmtpClient callbackClient = s as SmtpClient;
        MailMessage callbackMailMessage = e.UserState as MailMessage;
        callbackClient.Dispose();
        callbackMailMessage.Dispose();
    };

    smtpClient.SendAsync(message, message);
}

Puede ver por qué es particularmente importante deshacerse de SmtpClient mediante el siguiente comentario:

public class SmtpClient : IDisposable
   // Summary:
    //     Sends a QUIT message to the SMTP server, gracefully ends the TCP connection,
    //     and releases all resources used by the current instance of the System.Net.Mail.SmtpClient
    //     class.
    public void Dispose();

En mi escenario, enviando varios correos electrónicos usando Gmail sin deshacerme del cliente, solía obtener:

Mensaje: Servicio no disponible, cerrando canal de transmisión. La respuesta del servidor fue: 4.7.0 Problema temporal del sistema. Vuelve a intentarlo más tarde (WS). oo3sm17830090pdb.64 - gsmtp