如果我处置其底层 Stream，是否需要处置 XmlReader？

Question

我有以下方法 GetData 从文件创建 StreamReader 。

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) 调用以下内容：

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

    return xmldoc;
}

StreamReader 由 GetData 处置。这是否意味着我不必处置 XmlReader 因为（我相信）这会处置其唯一的非托管资源？

Answer 1

最好的“经验法则”是：

如果某些东西实现了 IDisposable，请始终将其包装在 using() 块中，以确保任何非托管资源拥有的已正确处置。

依赖“某物”的当前实现来处置底层资源这一事实是危险的，将所有内容包装在 中不会有什么坏处使用，只是为了安全 =)

Answer 2

你是对的，你不必必须处置读者。但在给出的代码中，它也不会造成伤害。

我不会在 LoadDocument() 中放置一个 using 块，因为它的设计是为了“借用”它的流（它不会创建它）。

但无论如何，还是有一些争论认为要处置 XmlReader，只是因为它是 IDisposable。我认为这里没有明显的赢家，因为 Reader（和 Writer）系列的设计存在争议：他们在没有明确成为这些流的所有者的情况下处置其 baseStreams。