C# 如何正确处理 SmtpClient？

Question

VS 2010 代码分析报告如下：

警告 4 CA2000：Microsoft.Reliability：在方法“Mailer.SendMessage()”中，对象“client”并未沿所有异常路径进行处理。在对对象“客户端”的所有引用超出范围之前调用 System.IDisposable.Dispose。

我的代码是：

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

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

我应该如何正确处置客户？

更新：回答 Jons 的问题，这里是处理附件功能：

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

上次更新完整的课程列表（简短）

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

这样，即使在Send 方法调用期间抛出异常，客户端也会被释放。您应该很少需要显式调用Dispose - 它应该几乎总是在using 语句中。

但是，不清楚附件是如何在这里涉及的。你的班级是否实现了IDisposable 本身？如果是这样，那可能是处理可能是成员变量的附件的地方。如果您需要绝对确保它们在这里得到处理，您可能需要：

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

Answer 2

.NET 4.0 中的 SmtpClient 类现在实现了 IDisposable，而 .NET 2.0 中的 SmtpClient 类缺少此接口（如 Darin 所述）。这是框架中的一项重大更改，您应该在迁移到 .NET 4.0 时采取适当的措施。但是，在迁移到 .NET 4.0 之前，可以在代码中缓解这种情况。这是一个例子：

var client = new SmtpClient();

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

此代码将在 .NET 2.0（+3.0 和 3.5）和 .NET 4.0 下正确编译和运行。

Answer 3

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

有趣——与.NET 3.5相反，SmtpClient在.NET 4.0中实现IDisposable，每天都在学习新东西。

Answer 4

我会这样做：

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

如果您想发送多封邮件，这将允许重用 SmtpClient 对象。

Answer 5

这是通过代码警察测试的更简洁的解决方案（如果发送失败，将始终调用 dispose）：

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

Answer 6

我最初的问题是间歇性发送失败。 例如。第一个 Send() 成功，第二个 Send() 失败，第三个 Send() 成功。 起初我以为我没有正确处理。 于是我求助于using()。

反正后来我加了UseDefaultCredentials = false,，Send()终于稳定了。我仍然保留了using() 功能。

Answer 7

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

        }
    }
    finally
    {
        DisposeAttachments(); 
    }
}