कोड विश्लेषण शिकायत करता है कि मैं वस्तुओं का निपटारा नहीं कर रहा हूं। यहाँ क्या गलत है?

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 को बदलने। यह किसी अन्य फ़ंक्शन में भेजा जाता है।

कोड विश्लेषण मुझे बताता है कि मैं "संदेश" का निपटारा नहीं कर रहा हूं जो सही है। लेकिन अगर मैं इसे यहां करता हूं - जब मैं इसे भेजने की कोशिश कर रहा हूं तो मुझे अपवाद मिलता है।

इसके अलावा, इसके बारे में यहाँ MemoryStream निपटान नहीं शिकायत:

msg.Attachments.Add (नई अनुलग्नक (नई MemoryStream (attachment.Data), attachment.Name));

मुझे इस बात का कोई अंदाजा नहीं है कि इसे ठीक तरीके से निपटाने के तरीके के बारे में कोई जानकारी नहीं है। मैंने अलग-अलग चीजों की कोशिश की, लेकिन "स्ट्रीम बंद हो गया है" कहने पर मेल भेजते समय अपवाद प्राप्त हो रहा था, "

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

"डिस्पोजेक्ट नहीं" संदेश के संबंध में, "एकमात्र तरीका जिसे मैं सोच सकता हूं, मेलमैसेज को वापस करने के बजाय मेलमेसेज के संदर्भ में पास करने के बजाय है। कुछ ऐसा है। यकीन नहीं है कि यह एक अच्छा विचार है।

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

Answer 3

डिस्पोजेबल ऑब्जेक्ट के निर्माता को भी इसका निपटान करना चाहिए। यदि आप यहां संदेश का निपटान नहीं कर सकते हैं तो इसे कहीं और निर्माता से पास किया जाना चाहिए। कोड विश्लेषण इस मामले में सही है और यदि आप इन संदेशों को अनदेखा करते हैं तो आप लीक को डीबग करने के लिए बहुत दुर्भाग्यपूर्ण और कठिन हो सकते हैं।