इस विधि क्यों कारण नहीं संहिता विश्लेषण त्रुटि CA2000: निपटान() कॉल

Question

मैं "माइक्रोसॉफ्ट मिनिमल नियम" कोड विश्लेषण सेट के साथ अपने प्रोजेक्ट का निर्माण कर रहा हूँ और यह मेरे इस पद्धति पर CA2000 देता है:

private Timer InitializeTimer(double intervalInSeconds) 
{ 
    Timer timer = null; 

    try 
    { 
     timer = new Timer { Interval = intervalInSeconds * 1000, Enabled = true }; 
     timer.Elapsed += timer_Elapsed; 
     timer.Start(); 
    } 
    catch 
    { 
     if (timer != null) 
     { 
      timer.Dispose(); 
     } 
    } 
    return timer; 
} 

यह विधि केवल सेकंड में अंतराल से एक नया System.Timers.Timer बनाता है। मेरे पास तीन ऐसे टाइमर चल रहे हैं (प्रत्येक सेकेंड के लिए एक, एक मिनट और हर आधे घंटे)। हो सकता है कि एक टाइमर होना बेहतर हो और विलुप्त होने वाले ईवेंट हैंडलर में जांच करें कि क्या डेढ़ घंटे बीत चुका है, लेकिन मुझे नहीं पता, यह इस समय आसान है, यह विरासत कोड है और मैं सबकुछ तोड़ना नहीं चाहता अभी तक।

इस विधि को पकड़ने में मुझे बदनाम

Warning 21 CA2000 : Microsoft.Reliability : In method 'TimerManager.InitializeTimer(double)', call System.IDisposable.Dispose on object '<>g__initLocal0' before all references to it are out of scope. 

अब मैं बोल रहा हूँ निपटान देता है और यह पर्याप्त होगा सोचा है? मैं कक्षा के अपने आईडीएसओएसपीबल कार्यान्वयन में सभी टाइमर का निपटान भी कर रहा हूं।

मुझे यहां क्या याद आ रही है?

Answer 1

अपवाद के मामले में आप केवल Dispose पर कॉल करते हैं (जिसे आपको कैच-ऑल ब्लॉक बीटीडब्लू के साथ कभी भी संभालना नहीं चाहिए, लेकिन यह एक और कहानी है)। कोई अपवाद नहीं होने पर, आप Timer ऑब्जेक्ट का निपटान नहीं करते हैं।

या तो finally ब्लॉक जोड़ें और Dispose पर जाएं, या using ब्लॉक का उपयोग करें।

Answer 2

चेतावनी आपको बताती है कि आप एक डिस्पोजेबल ऑब्जेक्ट बना रहे हैं और सभी मामलों में इसका निपटारा नहीं कर रहे हैं। यदि आप इसे किसी अन्य विधि में ठीक से निपट रहे हैं, तो आप सुरक्षित रूप से इस चेतावनी को दबा सकते हैं (आप SuppressMessageAttribute का उपयोग करके ऐसा कर सकते हैं)।

Answer 3

ठीक है, मैं इसे इस तरह संपादित:

private Timer InitializeTimer(double intervalInSeconds) 
    { 
     Timer tempTimer = null; 
     Timer timer; 
     try 
     { 
      tempTimer = new Timer(); 
      tempTimer.Interval = intervalInSeconds * 1000; 
      tempTimer.Enabled = true; 
      tempTimer.Elapsed += timer_Elapsed; 
      tempTimer.Start(); 
      timer = tempTimer; 
      tempTimer = null; 
     } 
     finally 
     { 
      if (tempTimer != null) 
      { 
       tempTimer.Dispose(); 
      } 
     } 
     return timer; 
    } 

यह प्रति CA2000 डॉक्स है और यह एक चेतावनी नहीं देता है। मैंने इस तथ्य को अनदेखा कर दिया था कि ऑब्जेक्ट प्रारंभकर्ता वाक्यविन्यास एक अस्थायी वस्तु बनाता है जिसे निपटान नहीं किया जा सकता है।

धन्यवाद, दोस्तों!

Answer 4

मुझे लगता है कि इसका इस्तेमाल करने के बजाय "की कोशिश/अंत में" "का उपयोग कर" reference

private Timer InitializeTimer(double intervalInSeconds) 
{ 
    Timer timer; 
    using (var tempTimer = new Timer()) 
    { 
     tempTimer.Interval = intervalInSeconds * 1000; 
     tempTimer.Enabled = true; 
     tempTimer.Elapsed += timer_Elapsed; 
     tempTimer.Start(); 
     timer = tempTimer; 
    } 
    return timer; 
}