C#异步发送邮件：资源释放与正确实现方式咨询

你的核心疑问及代码 #

你通过按钮触发异步发送邮件，当前在SendEmail任务的finally块中释放MailMessage对象，想知道这种实现是否正确，是否需要在SendEmail内加catch块、或在按钮点击事件中用finally释放资源，另外SmtpClient等对象是否需要释放。你的代码如下（已修正明显语法错误）：

按钮点击事件代码：

private async void closeorder_Click(object sender, EventArgs e)
{
    try
    {
        await SendEmail();
    }
    catch (Exception)
    {
        MessageBox.Show("mail has not been sent");
    }
}

SendEmail方法代码：

private async Task SendEmail()
{
    SmtpClient smtpClient = new SmtpClient();
    MailMessage message = new MailMessage();

    if (System.IO.File.Exists(emailfile))
    {
        String[] Lines = System.IO.File.ReadAllLines(emailfile);
        foreach (string line in Lines)
            message.To.Add(line);
    }
    else
    {  
        System.IO.File.WriteAllText(emailfile,"");
        TextWriter Email = new StreamWriter(emailfile, true);
        Email.WriteLine("foo@email.com");
        Email.Close();
    }

    try
    {
        MailAddress fromAddress = new MailAddress("goo@email.com","order");
        smtpClient.Host = "";
        smtpClient.Port = xx;
        smtpClient.Credentials = new NetworkCredential("","");
        smtpClient.DeliveryMethod = SmtpDeliveryMethod.Network;
        message.From = fromAddress;
        message.Subject = "";
        message.IsBodyHtml = true;
        message.Body = Mybody();
        message.DeliveryNotificationOptions = DeliveryNotificationOptions.OnFailure;
        await smtpClient.SendMailAsync(message);
    }
    finally 
    {
        message.Dispose();
    }
}

逐个解答你的疑问 #

1. MailMessage在finally块释放是否正确？
   这种方式是正确的，finally块无论try内代码是否抛出异常都会执行，能保证MailMessage资源被释放。不过更简洁可靠的写法是用using语句，它会自动在代码块结束时调用Dispose，避免手动写finally可能出现的遗漏：

   using (MailMessage message = new MailMessage())
   {
       // 所有操作message的逻辑都放在这里
       try
       {
           await smtpClient.SendMailAsync(message);
       }
       // 无需手动写finally，using会自动处理释放
   }
   

2. 是否需要在SendEmail内加catch块，或是在按钮点击事件中通过finally释放资源？

   • 不需要在SendEmail内加catch块，你已经在按钮点击事件里统一捕获异常并提示用户，这样异常处理逻辑更集中，代码更清晰。
   • 也不需要在按钮点击事件的finally里释放资源，MailMessage的生命周期局限在SendEmail方法内，应该在方法内完成释放，避免跨方法的资源管理混乱。

3. SmtpClient等其他对象是否需要释放？

   • SmtpClient必须释放：它实现了IDisposable接口，使用完后不释放会导致资源泄漏，同样推荐用using语句包裹：

     using (SmtpClient smtpClient = new SmtpClient())
     {
         // 配置SMTP参数、发送邮件的代码
         await smtpClient.SendMailAsync(message);
     }
     

   • StreamWriter需要释放：你在else分支里创建的StreamWriter，目前用Close()虽能释放，但更规范的是用using，它会自动处理资源释放，无需手动调用Close：

     else
     {  
         System.IO.File.WriteAllText(emailfile,"");
         using (TextWriter emailWriter = new StreamWriter(emailfile, true))
         {
             emailWriter.WriteLine("foo@email.com");
         } // 代码块结束自动释放资源
     }
     

优化后的完整SendEmail方法示例 #

private async Task SendEmail()
{
    // 用using包裹所有实现IDisposable的对象，自动处理资源释放
    using (SmtpClient smtpClient = new SmtpClient())
    using (MailMessage message = new MailMessage())
    {
        if (System.IO.File.Exists(emailfile))
        {
            string[] lines = System.IO.File.ReadAllLines(emailfile);
            foreach (string line in lines)
                message.To.Add(line);
        }
        else
        {  
            System.IO.File.WriteAllText(emailfile, "");
            using (TextWriter emailWriter = new StreamWriter(emailfile, true))
            {
                emailWriter.WriteLine("foo@email.com");
            }
        }

        MailAddress fromAddress = new MailAddress("goo@email.com", "order");
        smtpClient.Host = ""; // 填写你的SMTP主机地址
        smtpClient.Port = xx; // 填写对应端口，比如587
        smtpClient.Credentials = new System.Net.NetworkCredential("你的邮箱账号", "授权码/密码");
        smtpClient.DeliveryMethod = SmtpDeliveryMethod.Network;
        message.From = fromAddress;
        message.Subject = "";
        message.IsBodyHtml = false; // 纯文本邮件请设为false
        message.Body = Mybody();
        message.DeliveryNotificationOptions = DeliveryNotificationOptions.OnFailure;
        
        await smtpClient.SendMailAsync(message);
    }
}

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