8

我有一个实现 IDisposable 接口来处理私有变量_MailMessage的类同一个类有一个使用私有 IDisposable 变量的异步方法，即async public Task<bool> Send我的问题是：普通的 IDisposable 实现会在异步方法完成后处理私有变量吗？这是我正在谈论的课程的示例：

public class Email : IEmail
{
    private readonly IEmailData _EmailData;
    private MailMessage _MailMessage = new MailMessage();

    public Email(IEmailData emailData)
    {
        if (emailData == null)
        {
            throw new ArgumentNullException("emailData");
        }
        if (String.IsNullOrEmpty(emailData.To))
        {
            throw new ArgumentNullException("emailData.To");
        }
        if (String.IsNullOrEmpty(emailData.From))
        {
            throw new ArgumentNullException("emailData.From");
        }
        if (String.IsNullOrEmpty(emailData.FromName))
        {
            throw new ArgumentNullException("emailData.FromName");
        }
        if (String.IsNullOrEmpty(emailData.Subject))
        {
            throw new ArgumentNullException("emailData.Subject");
        }
        if (String.IsNullOrEmpty(emailData.Body))
        {
            throw new ArgumentNullException("emailData.Body");
        }

        _EmailData = emailData;
    }


    async public Task<bool> Send()
    {
        return await Task.Run<bool>(() =>
        {
            using (SmtpClient smtp = new SmtpClient())
            {
                smtp.Send(_MailMessage);
            }
            return true;
        });
    }

    #region "IDisposable implementation"
    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }


    ~Email()
    {
        Dispose(false);
    }

    protected virtual void Dispose(bool disposing)
    {
        if (disposing)
        {
            if (_MailMessage != null)
                _MailMessage.Dispose();
        }
    }
    #endregion

}

我已经根据建议不要使用析构函数的答案之一更改了 IDisposable 实现：

#region "IDisposable implementation"
public void Dispose()
{
    Dispose(true);
    GC.SuppressFinalize(this);
}

protected virtual void Dispose(bool disposing)
{
    if (disposing)
    {
        if (_MailMessage != null)
            _MailMessage.Dispose();
    }
}
#endregion
4

2 回答 2

15

除了_MailMessage在Email.Dispose.

async没有做任何特别神奇的事情IDispose; 您唯一需要记住的是async方法可能会提前返回。

所以如果你这样称呼它：

using (var email = new Email(...))
{
  await email.Send();
}

然后您的调用代码将（异步）等待Send完成，然后再处理email。但如果你这样称呼它：

Task task;
using (var email = new Email(...))
{
  task = email.Send();
}

然后您的调用代码将email在Send完成之前处理。

于 2013-09-17T12:45:43.023 回答
12

你这样做从根本上是错误的。要记住的一个好规则是，如果你认为你需要一个析构函数，那么你在 99.9% 的情况下都是错误的。仅当您有需要释放的非托管类型的私有变量时才需要析构函数。你没有。你可以告诉你做错了的方式是当你发现如果处置论点是错误的，你实际上并没有做任何事情。或者换句话说，析构函数实际上并没有做任何事情。所以不需要。那么你也不需要一次性图案。

还有更多的错误，你需要继承IDisposable接口来实现你自己的Dispose()方法。你忘了。

您的 Dispose() 方法需要由创建您的 Email 类实例的客户端代码调用。您不能自己调用​​它，您不知道客户端代码何时停止使用您的 Email 对象。所以这是对您问题的快速回答，您不能在 Send() 方法中处理自己。无法保证客户端代码会实际调用它。您必须将其留给客户端代码才能正确处理。

于 2013-09-17T13:41:53.270 回答