C# cómo desechar correctamente un SmtpClient?

Question

VS 2010 de análisis de código informa de lo siguiente:

Advertencia 4 CA2000: Microsoft.Reliability: En el método de 'Mailer.SendMessage()', objeto 'cliente' no está dispuesto a lo largo de todos los caminos de excepción. Call System.IDisposable.Dispose en el objeto 'client' antes de que todas las referencias estén fuera del alcance.

Mi código es:

public void SendMessage() 
    { 
     SmtpClient client = new SmtpClient(); 

     client.Send(Message); 
     client.Dispose(); 
     DisposeAttachments(); 
    } 

¿Cómo debo desechar correctamente cliente?

Actualización: a responder a la pregunta Jons, aquí está el adjuntos funcionalidad disponer:

private void DisposeAttachments() 
{ 
    foreach (Attachment attachment in Message.Attachments) 
    { 
     attachment.Dispose(); 
    } 
    Message.Attachments.Dispose(); 
    Message = null; 
} 

Última actualización lista de clases completo (su corta)

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 
using System.Net.Mail; 

public class Mailer 
    { 
    public MailMessage Message 
    { 
     get; 
     set; 
    } 

    public Mailer(MailMessage message) 
    { 
     this.Message = message; 
    } 

    public void SendMessage() 
    { 
     using (SmtpClient client = new SmtpClient()) 
     { 
      client.Send(Message); 
     } 
     DisposeAttachments(); 
    } 

    private void DisposeAttachments() 
    { 
     foreach (Attachment attachment in Message.Attachments) 
     { 
      attachment.Dispose(); 
     } 
     Message.Attachments.Dispose(); 
     Message = null; 
    } 
    } 

Answer 1

public void SendMessage() 
{ 
    using (SmtpClient client = new SmtpClient()) 
    { 
     client.Send(Message); 
    } 
    DisposeAttachments(); 
} 

De esta forma el cliente estará dispuesto incluso si se produce una excepción durante la llamada Send método. Rara vez debe llamar al Dispose explícitamente, casi siempre debe estar en una declaración using.

Sin embargo, no está claro cómo los anexos están involucrados aquí. ¿Su clase implementa IDisposable? Si es así, ese es probablemente el lugar para deshacerse de los archivos adjuntos que son, presumiblemente, variables miembro. Si tiene que estar absolutamente seguro de que quedan dispuestas aquí, es probable que tenga:

public void SendMessage() 
{ 
    try 
    { 
     using (SmtpClient client = new SmtpClient()) 
     { 
      client.Send(Message); 
     } 
    } 
    finally 
    { 
     DisposeAttachments(); 
    } 
} 

Answer 2

using (SmtpClient client = new SmtpClient()) 
{ 
    client.Send(Message); 
    DisposeAttachments(); 
} 

Interesante - contrario a .NET 3.5, SmtpClient implementa IDisposable en .NET 4 .0, aprendiendo cosas nuevas todos los días.

Answer 3

Esta es (siempre será llamado y disponer si Enviar falla) la solución más ordenado que pasar la prueba de la policía código:

public void SendMessage() 
{ 
    using (SmtpClient client = new SmtpClient()) 
    { 
     client.Send(Message); 
     DisposeAttachments(); 
    } 
} 

Answer 4

que haría algo así:

class Attachments : List<Attachment>, IDisposable 
{ 
    public void Dispose() 
    { 
    foreach (Attachment a in this) 
    { 
     a.Dispose(); 
    } 
    } 
} 

class Mailer : IDisposable 
{ 
    SmtpClient client = new SmtpClient(); 
    Attachments attachments = new Attachments(); 

    public SendMessage() 
    { 
    [... do mail stuff ...] 
    } 

    public void Dispose() 
    { 
    this.client.Dispose(); 
    this.attachments.Dispose(); 
    } 
} 


[... somewhere else ...] 
using (Mailer mailer = new Mailer()) 
{ 
    mailer.SendMail(); 
} 

Esto permitiría reutilizar el objeto SmtpClient si desea enviar varios mensajes.

Answer 5

La clase SmtpClient en .NET 4.0 ahora implementa IDisposable, mientras que la clase SmtpClient en .NET 2.0 carece de esta interfaz (como señaló Darin). Este es un cambio radical en el marco y debe tomar las medidas adecuadas al migrar a .NET 4.0. Sin embargo, es posible mitigar esto en su código antes de migrar a .NET 4.0. Aquí es un ejemplo de tales:

var client = new SmtpClient(); 

// Do not remove this using. In .NET 4.0 SmtpClient implements IDisposable. 
using (client as IDisposable) 
{ 
    client.Send(message); 
} 

Este código compilar y ejecutar correctamente tanto bajo .NET 2.0 (3.0 y 3.5) y bajo .NET 4.0.

Answer 6

public void SendMessage() 
{ 
    try 
    { 
     using (SmtpClient client = new SmtpClient()) 
     { 
      client.Send(Message); 
     client.dispose() 

     } 
    } 
    finally 
    { 
     DisposeAttachments(); 
    } 
}