代码分析抱怨我没有处理对象。这里有什么问题吗？

Question

考虑这段代码

private MailMessage GetMailMessageFromMailItem(Data.SystemX.MailItem mailItem)
        {

            var msg = new MailMessage();

            foreach (var recipient in mailItem.MailRecipients)
            {
                var recipientX = Membership.GetUser(recipient.UserKey);
                if (recipientX == null)
                {
                    continue;
                }

                msg.To.Add(new MailAddress(recipientX.Email, recipientX.UserName));
            }

            msg.From = new MailAddress(ConfigurationManager.AppSettings["EmailSender"],
                                   ConfigurationManager.AppSettings["EmailSenderName"]);

            msg.Subject = sender.UserName;
            if (!string.IsNullOrEmpty(alias)) msg.Subject += "(" + alias + ")";
            msg.Subject += " " + mailItem.Subject;
            msg.Body = mailItem.Body;
            msg.Body += Environment.NewLine + Environment.NewLine + "To reply via Web click link below:" + Environment.NewLine;
            msg.Body += ConfigurationManager.AppSettings["MailPagePath"] + "?AID=" + ContextManager.AccountId + "&RUN=" + sender.UserName;

            if (mailItem.MailAttachments != null)
            {
                foreach (var attachment in mailItem.MailAttachments)
                {
                    msg.Attachments.Add(new Attachment(new MemoryStream(attachment.Data), attachment.Name));
                }
            }

            return msg;
        }

，我只是将我的数据库类型转换为 MailMessage。 它在另一个函数中发送。

代码分析告诉我我没有处理“msg”，这是正确的。但如果我在这里这样做 - 当我尝试发送它时我会遇到异常。

另外，它还抱怨没有在这里处理 MemoryStream：

msg.Attachments.Add(new Attachment(new MemoryStream(attachment.Data), 附件.名称));

我不知道如何正确处置它。我尝试了不同的方法，但在发送邮件时遇到异常“流已关闭”

Answer 1

基本上你不应该 - 稍后处理邮件消息将处理每个附件，这将处理每个流。此外，未能处理未在远程处理中使用的 MemoryStream 不会造成任何损害。

我建议您抑制此方法的警告。

编辑：我怀疑您可以使用 [SuppressMessage] 抑制该消息。

---

请注意，某些代码可能会在方法中途抛出代码，因此即使调用代码中有 using 语句，您最终也永远无法处理该消息。如果你真的很烦恼，你可以写：

private MailMessage GetMailMessageFromMailItem(Data.SystemX.MailItem mailItem)
{
    bool success = false;
    var msg = new MailMessage();
    try
    {
        // Code to build up bits of the message
        success = true;
        return msg;
    }
    finally
    {
        if (!success)
        {
            msg.Dispose();
        }
    }
}

不过就我个人而言，我认为这有点过分了。

Answer 2

关于“不处理“msg””，我能想到的唯一方法是不返回 MailMessage，而是传递对 MailMessage 的引用。
像这样的东西。不确定这是否是一个好主意。

private void GetMailMessageFromMailItem(ref MailMessage msg, Data.SystemX.MailItem mailItem)

Answer 3

一次性物品的创造者也应该处置它。如果您无法在此处处理消息，则应将其从其他地方的创建者传入。在这种情况下，代码分析是正确的，如果您忽略这些消息，您可能会遇到非常不幸且难以调试的泄漏。