Analiza kodu narzeka, że ​​nie wyrzucam przedmiotów. Co jest nie tak?

Question

rozważyć ten kod

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; 
     } 

prostu biorę mój typ bazy danych i konwersji na MailMessage. To jest wysłane w innej funkcji.

Analiza kodu mówi, że nie wyrzucam "msg", co jest poprawne. Ale jeśli zrobię to tutaj - otrzymuję wyjątek, kiedy próbuję go wysłać.

Ponadto, narzeka i nie wyrzucaj MemoryStream tutaj:

msg.Attachments.Add (nowy załącznik (new MemoryStream (attachment.Data) attachment.Name));

Nie mam pojęcia, jak prawidłowo go usunąć. Próbowałem różnych rzeczy, ale był coraz wyjątki podczas wysyłania poczty mówiąc: „Stream zamyka”

Answer 1

Zasadniczo nie należy - usuwania wiadomości e-mail później będzie wyrzucać każdego załącznika, która będzie dysponować każdego strumienia. Poza tym nieudana próba pozbycia się numeru MemoryStream, który nie jest używany do zdalnego dostępu, nie wyrządzi szkody.

Proponuję wyłączyć ostrzeżenie dla tej metody.

EDYTOWANIE: Podejrzewam, że można użyć [SuppressMessage], aby wyłączyć komunikat.

---

Należy pamiętać, że istnieje ryzyko, że niektóre kod wygeneruje kod półmetku tej metody, więc w końcu nie mogąc pozbyć się komunikatu nawet jeśli masz using oświadczenie w kodzie wywołującym. Jeśli naprawdę się o to martwisz, możesz napisać:

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(); 
     } 
    } 
} 

Osobiście mogę powiedzieć, że to przesada.

Answer 2

Jeśli chodzi o "nie wyrzucać" msg "", jedyny sposób, jaki mogę wymyślić, to zamiast zwracać MailMessage, a raczej przekazać odwołanie do MailMessage. Coś takiego. Nie jestem pewien, czy to taki dobry pomysł.

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

Answer 3

Twórca obiektu jednorazowego użytku również powinien go wyrzucić. Jeśli nie możesz przekazać wiadomości tutaj, to powinna ona zostać przekazana przez innego twórcę. Analiza kodu jest w tym przypadku właściwa, a jeśli zignorujesz te wiadomości, możesz otrzymać bardzo niefortunne i trudne do usunięcia błędy.