通过SmtpClient异步发送邮件的两种方式，结果不同

Question

这里有简单的概念。这是针对使用 MVC 3 和 Entity Framework 4 构建的网站。用户在网站上注册后，系统会向他们的电子邮件地址发送一封电子邮件。我首先使用 SmtpClient.Send() 实现了这一点，并且效果很好。然后我想到了一个好主意：尝试异步发送电子邮件。我尝试过的两种异步方法遇到了问题。

第一次实现（来自这个未答复的帖子：https://stackoverflow.com/ questions/7558582/how-to-dispose-using-smtpclient-send-and-asynccallback）：

public bool Emailer(){
    .
    .
    .
    using (var smtpClient = new SmtpClient())
    {
        smtpClient.EnableSsl = true;
        smtpClient.Host = "smtp.gmail.com";
        smtpClient.Port = 587;
        smtpClient.UseDefaultCredentials = false;
        smtpClient.Credentials = new NetworkCredential("[email protected]", "mypassword");

        var sd = new SendEmailDelegate(smtpClient.Send);
        var cb = new AsyncCallback(SendEmailResponse);
        sd.BeginInvoke(message, cb, sd);

        return true;
    }
}

private delegate void SendEmailDelegate(System.Net.Mail.MailMessage m);

private static void SendEmailResponse(IAsyncResult ar)
{
    try
    {
        SendEmailDelegate sd = (SendEmailDelegate)(ar.AsyncState);
        sd.EndInvoke(ar); // "cannot access a disposed object" errors here
    }
    catch (Exception e)
    {
        _logger.WarnException("Error on EndInvoke.", e);
    }
}

这有效了一半的时间。另一半我会在回调中收到“无法访问已处置的对象”错误。

下一个实现（来自具有良好声誉的成员：在 .NET 4.0 下使用 SmtpClient、SendAsync 和 Dispose 的最佳实践是什么）：

var smtpClient = new SmtpClient();
smtpClient.EnableSsl = true;
smtpClient.Host = "smtp.gmail.com";
smtpClient.Port = 587;
smtpClient.UseDefaultCredentials = false;
smtpClient.Credentials = new NetworkCredential("[email protected]", "mypassword");

smtpClient.SendCompleted += (s, e) =>
    {
        smtpClient.Dispose();
        message.Dispose();
    };
smtpClient.SendAsync(message, null);

使用此实现我没有收到任何错误，但是当 smtpClient.SendAsync() 执行时，调试模式下有明显较长的延迟（约 5 秒），导致我认为它不是异步发送的。

问题：

1）第一个方法有什么问题导致“已处置对象”错误？

2）第二个实现是否存在导致电子邮件无法异步发送的问题？ 5秒的延迟没有意义吗？

可能还需要注意的是，虽然该网站不需要支持发送大量电子邮件（仅用户注册、选择加入时事通讯等），但我们确实预计未来会有大量用户，因此我决定异步发送电子邮件。

谢谢。

Answer 1

由于 USING 块，您的第一种方法将无法正常工作。 using-block 结束后，SmtpClient 对象将被释放。因此您无法在事件处理程序中访问它。

Answer 2

尖端：
1-不要对 MailMessage 对象使用“using block”，它会在发送邮件之前处理您的对象
2-在 SmtpClient.SendCompleted 事件上处理 MailMessage 对象：

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

3-为 smtpClient 对象设置 SendCompletedEventHandler

smtpClient.SendCompleted += new SendCompletedEventHandler(SendCompletedCallback);

4-更多代码：

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

        if (e.Cancelled)
        {
            //write your code here
        }
        if (e.Error != null)
        {
            //write your code here
        }
        else //mail sent
        {
            //write your code here
        }

        mailSent = true;
    }

Answer 3

SmtpClient.SendAsync 是异步电子邮件发送的首选方法，因为它使用专门为此目的设计的 SmtpClient 方法。它也更容易实施，并且已被证明可以工作数千次。

您的 5 秒延迟很奇怪，表明存在需要解决的问题。第一段代码只是掩盖了问题，但并没有消除它。

如果您的传送方法不是SpecifiedPickupDirectory 或PickupDirectoryFromIis，SmtpClient.SendAsync 实际上只会异步发送。在这些情况下，它会在返回之前将消息文件写入拾取文件夹。检查配置文件的 部分。我的猜测是您正在使用其中一种方法，问题出在拾取文件夹上。删除那里可能有的旧文件，并检查问题是否出在您的防病毒软件上，该软件很可能会在每个新文件中搜索病毒。检查是否设置了加密或压缩属性。也可以是其他东西。测试该文件夹是否是问题根源的最佳方法是手动将电子邮件文件复制到其中。