5

在下面的代码中，我XmlNodeList从XmlDocument.SelectNodes()

foreach (XmlNode node in doc.SelectNodes(xPath))
{
    // Do stuff
}

事实证明，XmlNodeList实现IDisposable. 这是否意味着每次我想迭代的返回值XmlDocument.SelectNodes()我都应该把它放在一个局部变量中并确保它被释放（即把它放在一个using块中）？

像这样：

using(XmlNodeList nodes = doc.SelectNodes(xPath))
{ 
    foreach (XmlNode node in nodes)
    {
        // Do stuff
    }
}
4

1 回答 1

8

System.Xml 命名空间是，呃，不稳定。我可以说的最好的方式。XmlNodeList 是一个抽象类，它继承了 IDisposable 并实现了一次性模式，但它本身不做任何事情。

有三个从 XmlNodeList 派生的内部类。其中之一实际上覆盖了 Dispose(bool) 方法 XmlElementList。该类有一个 XmlElementListener 类型的私有字段。另一个内部类，它的作用并不那么明显，但它似乎“听”了列表更改。Dispose 方法（等待它）取消订阅两个事件处理程序。

这打破了书中的每一条规则，为此滥用 IDisposable 是不正确的。不幸的是，您将不得不步行，几乎不可能解开这个问题以查看该侦听器是否被实例化，以及如果您不调用 Dispose()，这些事件处理程序是否会在您的程序中导致持久的泄漏。你最好叫它。

.NET Framework 中有很多很棒的代码。好的代码总是需要糟糕的代码才能让人们清楚地看到好的代码有多好。这是 System.Xml 的工作。

于 2013-10-10T21:52:02.563 回答