analyse du code met en garde contre un potentiel de disposer d'un objet à deux reprises

Question

juste couru une analyse de code sur l'une des applications dont j'ai hérité, et il a soulevé des avertissements sur un code similaire à ce qui suit:

using (StreamWriter tw = File.AppendText("Log.txt")) 
{ 
    try 
    { 
     tw.WriteLine(DateTime.Now.ToString("------------------------"); 
     tw.WriteLine(data); 
    } 
    catch(Exception ex) 
    { 
     Console.WriteLine(String.Format(CultureInfo.InvariantCulture, "Error writing to the log file: {0}", ex.Message)); 
    } 
    finally 
    { 
     tw.Close(); 
    } 
} 

Si je en commentaire le bloc finally, l'avertissement n'est pas déclenché. J'étais sous l'impression que la fermeture d'un écrivain de flux fermait seulement le fichier sous-jacent, mais ne disposait pas réellement de l'objet écrivain. Est-ce que l'analyse de code est cassée, ou est-ce que je ne comprends pas comment le rédacteur de flux devrait être utilisé?

Voici un autre exemple où l'analyse de code se plaint possible Mise au rebut à la fois écrivain et le flux sous-jacent deux fois:

using (TextReader tr = new StreamReader(File.OpenRead(filename))) 
{ 
    while (tr.ReadLine() != null) 
    { 
     counter++; 
    } 

    tr.Close(); 
} 

Il se plaint à la fois tr et File.OpenRead(filename)

Answer 1

Close just calls Dispose on the object, donc oui, vous disposer de l'objet deux fois. (Le fait de disposer d'un objet deux fois est problématique, juste redondant.)