Warum verursacht diese Methode Code-Analyse Fehler CA2000: Rufen Sie Dispose()

Question

ich mein Projekt baue mit dem „Microsoft Minimal-Regeln“ Code-Analyse-Set und es gibt mir CA2000 auf dieser Methode:

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

Diese Methode erstellt nur eine neue System.Timers.Timer aus einem Intervall in Sekunden. Ich habe drei solche Timer laufen (eine für jede Sekunde, eine jede Minute und jede halbe Stunde). Vielleicht ist es besser, einen Timer zu haben und den Event-Handler zu überprüfen, ob eine Minute oder eine halbe Stunde vergangen ist, aber ich weiß nicht, das ist zu diesem Zeitpunkt einfacher, es ist Code geerbt und ich möchte nicht alles kaputt machen noch.

Diese Methode gibt mir das berüchtigte

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. 

Jetzt bin ich Entsorgen im Fang Aufruf und dachte, dass dies ausreicht? Ich verfüge auch über alle Timer in der eigenen IDisposable-Implementierung der Klasse.

Was fehlt mir hier?

Answer 1

Sie rufen nur Dispose im Falle einer Ausnahme (die Sie nie mit einem Catch-All-Block BTW behandeln sollten, aber das ist eine andere Geschichte). Im Ausnahmefall verfügen Sie nicht über das Objekt Timer.

Entweder einen finally Block hinzufügen und den Dispose dorthin verschieben, oder einen using Block verwenden.

Answer 2

Die Warnung besagt, dass Sie ein Einwegobjekt erstellen und es nicht in allen Fällen entsorgen. Wenn Sie es ordnungsgemäß in einer anderen Methode entsorgen, können Sie diese Warnung sicher unterdrücken (Sie können das unter Verwendung der SuppressMessageAttribute) tun.

Answer 3

Okay, ich bearbeitet es wie folgt aus:

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

Dies ist nach den CA2000 docs und es keine Warnung geben. Ich hatte übersehen, dass die Objektinitialisierersyntax ein temporäres Objekt erzeugt, das nicht entsorgt werden kann.

Danke, Jungs!

Answer 4

Ich denke, es ist viel besser nutzen „können“ anstelle von „try/finally“ 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; 
}