是否需要手动释放System.Net.Mail.MailMessage对象？

你的判断是对的，这段代码确实存在不妥之处，咱们来拆解一下问题的核心：

为什么显式调用message.Dispose()是错误的？ #

主要有两个关键原因：

• 对象所有权原则：MailMessage实例是由调用Send方法的外部代码创建的，按照.NET的IDisposable设计规范，创建对象的一方才负责释放对象。如果你的Send方法擅自调用Dispose()，调用者后续如果还要使用这个message对象，就会触发“已释放的对象无法访问”的异常。
• SmtpClient的行为边界：SmtpClient.Send(MailMessage)方法只负责完成邮件发送操作，它不会接管MailMessage的所有权，也不会自动释放它，但这并不意味着你的方法需要承担释放责任——释放的控制权应该交还给调用者。

正确的写法应该是什么样的？ #

情况1：方法接收外部传入的MailMessage #

你的Send方法只需要专注于发送逻辑，不要处理message的释放：

public void Send(MailMessage message) 
{
    using(var smtpClient = new SmtpClient(_host)) 
    {
        smtpClient.Send(message);
    }
}

而调用者需要通过using语句来管理MailMessage的生命周期：

// 调用者创建并管理MailMessage
using(var message = new MailMessage("sender@example.com", "recipient@example.com", "测试主题", "测试内容"))
{
    var mailSender = new YourMailSender("smtp.example.com");
    mailSender.Send(message);
    // using块结束后，message会自动被释放
}

情况2：方法内部创建MailMessage #

如果你的方法是自己创建MailMessage实例，那你就需要负责释放它，此时可以嵌套using语句：

public void Send(string from, string to, string subject, string body) 
{
    using(var message = new MailMessage(from, to, subject, body))
    using(var smtpClient = new SmtpClient(_host)) 
    {
        smtpClient.Send(message);
    }
    // message和smtpClient都会在各自的using块结束后自动释放
}

总结 #

• 对于外部传入的IDisposable对象，除非方法明确声明会接管所有权，否则不要在方法内部调用Dispose()。
• 遵循“谁创建，谁释放”的原则，让对象的创建者通过using语句或手动调用Dispose()来管理生命周期，这是.NET中IDisposable模式的最佳实践。

内容的提问来源于stack exchange，提问作者FCin