为什么i.Parent.ReplaceWith(i)不抛出异常?

时间:2017-02-16 09:04:07

标签: c# linq-to-xml morelinq

在这些陈述中(与MoreLinq一起运行):

var xml = @"
<div>
<p>
    <h2>hey</h2>
</p>
<pre />
<h2 class=""cool"" />
<p>
    <h2>okay</h2>
</p>
</div>
".Trim();

var div = XElement.Parse(xml);
var h2Elements = div.Descendants("h2");
h2Elements.ToList().ForEach(i =>
{
    if(i.Parent.Name != "p") return;
    i.Parent.ReplaceWith(i);
});

我看到i.Parent.ReplaceWith(i)没有抛出异常,但是会抛出一个空引用异常(使用ForEach中的MoreLinq):

h2Elements.ForEach(i =>
{
    if(i.Parent.Name != "p") return;
    i.Parent.ReplaceWith(i);
});

我理解LINQ的ToList()正在制作列表的副本,但副本也不会抛出异常吗?此外,这里是否存在使用某种孤立引用的内存泄漏?

1 个答案:

答案 0 :(得分:4)

您根本不需要MoreLINQ来演示这一点 - 您也可以简化示例代码:

using System;
using System.Linq;
using System.Xml.Linq;

class Program
{    
    static void Main()
    {
        var element = new XElement(
            "root",
            new XElement("parent", new XElement("child")),
            new XElement("parent", new XElement("child"))
        );
        var children = element.Descendants("child");
        foreach (var child in children.ToList())
        {
            child.Parent.ReplaceWith(child);
        }
    }
}

如果没有ToList调用,则会抛出NullReferenceException。通过ToList()调用,也不例外。例外是:

Unhandled Exception: System.NullReferenceException: Object reference not set to an instance of an object.
   at System.Xml.Linq.XContainer.<GetDescendants>d__39.MoveNext()
   at Program.Main()

基本上,您通过在迭代树时修改树来使查询无效。这有点像在Add上调用RemoveList<T>时迭代它,但LINQ to XML更难发现问题并抛出有意义的异常。重要的是要注意,在调用ReplaceWith时不会出现异常 - 它是迭代部分失败,因为在您修改树后它无法正确遍历树。

当您致电ToList()时,您只是在列表中获得单独的XElement值 - 当您遍历该列表时,对元素的任何更改都不会更改出现在该列表中的引用列表。

至于内存泄漏:不,这就是垃圾收集器的用途......