Muss ich XmlReader entsorgen, wenn ich den zugrunde liegenden Stream entsorge?

Question

Ich habe die folgende Methode GetData, die eine StreamReader aus einer Datei erstellt.

private void GetData(string file) 
{ 
    string filename = Path.GetFileNameWithoutExtension(file); 
    XmlDocument xmldoc = new XmlDocument(); 

    using (StreamReader sr = new StreamReader(file)) 
    { 
     Stream bs = sr.BaseStream; 
     Stream cl = mainParser.CleanMarkup(bs); 
     try 
     { 
      xmldoc = mainParser.LoadDocument(bs); 
     } 
     catch (XmlException ex) 
     { 
      // Exceptions are usually caused by non-compliant documents. 
      // These errors are not critical to the operation of this program. 
      Console.WriteLine(filename + " " + ex.Message); 
     } 
    } 
    Msdn msdnParser = new Msdn(xmldoc); 

    ListViewItem lvitem = new ListViewItem(filename); 
    lvitem.SubItems.Add(filename); 
    foreach (string item in msdnParser.Subitems) 
    { 
     lvitem.SubItems.Add(item); 
    } 
    listView.Items.Add(lvitem); 
} 

mainParser.LoadDocument(bs) ruft folgende:

public XmlDocument LoadDocument(Stream file) 
{ 
    XmlDocument xmldoc = new XmlDocument(); 
    XmlReader xmlread = XmlReader.Create(file); 
    xmldoc.Load(xmlread); 

    return xmldoc; 
} 

StreamReader durch GetData angeordnet ist. Bedeutet dies, dass ich XmlReader nicht entsorgen muss, da (glaube ich) dies über seine einzige nicht verwaltete Ressource verfügen würde?

Answer 1

Die beste "Faustregel" zu arbeiten ist, wenn etwas implementiert IDisposable, wickeln Sie es immer in einen using() Block, um sicherzustellen, dass alle nicht verwalteten Ressourcen, die es gehört, ordnungsgemäß entsorgt werden. Unter Berufung auf der Tatsache, dass die aktuelle Umsetzung von „etwas“ verfügt über eine zugrunde liegende Ressource ist gefährlich, und es wird nicht wickeln everythig in einem using verletzt, nur um auf der sicheren Seite =)

Answer 2

Sie haben Recht, haben Sie nicht müssen den Leser entsorgen. Aber in dem gegebenen Code würde es auch nicht weh tun.

Ich würde keinen Verwendungsblock innerhalb LoadDocument() setzen, weil es so entworfen ist, dass es seinen Strom 'leiht' (es schafft es nicht).

Aber es gibt Argumente, den XmlReader sowieso zu disponieren, nur weil es IDisposable ist. Ich glaube nicht, dass es hier einen eindeutigen Gewinner gibt, wegen des strittigen Designs der Reader (und Writer) -Familie: Sie entsorgen ihre baseStreams, ohne eindeutig der Besitzer dieser Streams zu sein.