Question

我正在尝试通过我的emial安装程序填充我的电子邮件正文中的项目列表。当我试图在我的展望中打开电子邮件时，我可以看到只看到一个项目我期待的项目列表。

以下是我的代码：

 public class EmailSetup
{
    string toEmailSetup = string.Empty;
    string fromEmailSetup = string.Empty;
    string domainName = string.Empty;
    string emailServer = string.Empty;

    public void ApplicationFailedEmailSetup(List<string>ApplicationsInactive,DateTime dateRun)
    {
        toEmailSetup = ConfigurationManager.AppSettings["To mailid"];
        fromEmailSetup = ConfigurationManager.AppSettings["From mailid"];
        domainName = ConfigurationManager.AppSettings["Domain Name"];
        emailServer = ConfigurationManager.AppSettings["Email Server"];
        try
        {
            var messager = new MailMessage();
            messager.To.Add(toEmailSetup);
            messager.Subject = "Applications Crashed/Closed";
            messager.From = new MailAddress(fromEmailSetup);
            try
            {
                messager.Body = "Following applications you are monitoring are closed are crashed:";
                **foreach (var item in ApplicationsInactive)
                {
                    messager.Body = item;
                }**  // Here i am trying to populate list of applications.
            }
            catch (Exception)
            {
              throw;
            }

            var smtp = new SmtpClient(emailServer);
            smtp.EnableSsl = true;

            try
            {
                smtp.DeliveryMethod = SmtpDeliveryMethod.Network;
                smtp.UseDefaultCredentials = false;
                smtp.Send(messager);
            }
            catch (Exception)
            {

                throw;
            }

        }
        catch (SmtpException ex)
        {
            throw new ApplicationException
               ("SmtpException has occured: " + ex.Message);
        }

    }

}

Answer 1

循环中的这一行是问题所在：

messager.Body = item;

您每次都会覆盖Body属性，因此循环后只有最后一项。你想要追加：

messager.Body += item;

当然，还有更多方法可以做到这一点，而这个方法实际上有点草率。查看StringBuilder类以创建格式化字符串并构建您的电子邮件正文，然后将电子邮件正文设置为.ToString()对象的StringBuilder。

此外，作为附注，此代码无意为您服务：

catch (Exception)
{
    throw;
}

如果你没有以任何有意义的方式处理异常，为什么要抓住它呢？代码将抛出异常，所以让它抛出异常。绝对没有理由在这里抓住它，这只是在代码中产生噪音。

此外，这个也很糟糕：

catch (SmtpException ex)
{
    throw new ApplicationException
       ("SmtpException has occured: " + ex.Message);
}

您正在抑制原始异常并创建一个全新异常。您正在丢失堆栈跟踪以及原始异常中的任何其他有用信息。您是否有特殊原因希望将SmtpException转换为ApplicationException s？至少，将InnerException的{{1}}属性设置为ApplicationException，这样您就不会完全丢失该信息。

但是，更重要的是，就像上面一样，你实际上并没有以任何有意义的方式处理异常。没有添加任何上下文，也没有进行日志记录，无论如何都会抛出异常。同样，这只是代码中的噪音。如果您不打算实际处理它，就没有理由捕获异常。

Answer 2

您当前的代码会在每次迭代中覆盖正文，并为每个项目分配 - 导致最后一项成为正文。

您需要追加代替身体：

messager.Body = "Following applications you are monitoring are closed are crashed:";
messager.Body += string.Join(", ", ApplicationsInactive);

请注意 + = 运算符，而不仅仅是 = 。

此外，您根本不需要循环 - 只需使用Join()类的方便string方法，即可使用更少且更易读的代码来实现相同的结果。

填充电子邮件正文中的项目列表

2 个答案: