Czy muszę pozbyć się XmlReadera, jeśli usunę jego strumień bazowy?

Question

Mam następującą metodę GetData, która tworzy StreamReader z pliku.

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) wywołuje następujące:

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

    return xmldoc; 
} 

StreamReader jest usytuowany przy GetData. Czy to oznacza, że ​​nie muszę się pozbywać XmlReader, ponieważ (jak sądzę) pozbędzie się to jedynego niezarządzanego zasobu?

Answer 1

Najlepszą "regułą" do pracy jest, jeśli coś implementuje IDisposable, zawsze należy ją opakować w blok using(), aby upewnić się, że wszelkie niezarządzane zasoby, które posiada, są usuwane poprawnie. Powołując się na fakt, że obecny realizacja „coś” zbywa zasobu bazowego jest niebezpieczne i nie zaszkodzi owinąć everythig w using, żeby być po bezpiecznej stronie =)

Answer 2

Masz rację, nie musisz mieć do pozbyć się czytnika. Ale w podanym kodzie też nie zaszkodzi.

Nie wstawiłbym bloku użytego do LoadDocument(), ponieważ został zaprojektowany w taki sposób, że "pożycza" strumień (nie tworzy go).

Ale istnieją argumenty, aby zrezygnować z XmlReader tak czy inaczej, tylko dlatego, że jest IDisposable. Nie sądzę, aby tutaj był wyraźny zwycięzca z powodu spornego projektu rodziny Reader (i Writer): Usuwają swoje baseStreams bez wyraźnego bycia właścicielem tych strumieni.