Question

请考虑以下代码：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Xml;
using System.Xml.Linq;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            XmlDocument xmlDoc = new XmlDocument();

            xmlDoc.LoadXml(@"<Parts>
  <Part name=""DisappearsOk"" disabled=""true""></Part>
  <Part name=""KeepMe"" disabled=""false""></Part>
  <Part name=""KeepMe2"" ></Part>
  <Part name=""ShouldBeGone"" disabled=""true""></Part>  
</Parts>");

            XmlNode root = xmlDoc.DocumentElement;
            List<XmlNode> disabledNodes = new List<XmlNode>();

            try
            {

                foreach (XmlNode node in root.ChildNodes.Cast<XmlNode>()
                                             .Where(child => child.Attributes["disabled"] != null &&
                                                             Convert.ToBoolean(child.Attributes["disabled"].Value)))
                {
                    Console.WriteLine("Removing:");
                    Console.WriteLine(XDocument.Parse(node.OuterXml).ToString());
                    root.RemoveChild(node);
                }
            }
            catch (Exception Ex)
            {
                Console.WriteLine("Exception, as expected");
            }

            Console.WriteLine();
            Console.WriteLine(XDocument.Parse(root.OuterXml).ToString());

            Console.ReadKey();
        }
    }
}

当我在 visual studio express 2010 中运行此代码时，我没有像预期的那样得到异常。我期待一个因为我在迭代它时从列表中删除了一些内容。

我得到的是一个仅删除了第一个子节点的列表：

enter image description here

为什么我的操作异常无效？

请注意，IDEOne.com中的等效代码确实提供了预期的异常：http://ideone.com/qoRBbb

另请注意，如果我删除所有LINQ（.Cast().Where()），我会得到相同的结果，只删除了一个节点，没有例外。

我的VSExpress设置有问题吗？

请注意，我知道延迟执行是涉及的，但我希望在迭代时使用where子句迭代源枚举（子注释），这将给出我期望的异常。

我的问题是我在VSexpress中没有得到那个例外，但是在IDEOne中做了（我希望在两个/所有情况下，或者至少如果没有，我希望得到正确的结果）。

从Wouter's answer开始，当删除第一个子节点时，它似乎使迭代器无效，而不是给出异常。有什么官方说的吗？在其他情况下会出现这种行为吗？我会默默地使迭代器无效，而不是“无声但致命”的例外。

Answer 1

因为您正在迭代ChildNodes，所以删除第一个子句会使迭代器无效。因此，迭代将在第一次删除后停止。

如果您拆分过滤和迭代，您的代码将删除所有项目：

var col = root.ChildNodes.Cast<XmlNode>()
                             .Where(child => child.Attributes["disabled"] != null &&
                                             Convert.ToBoolean(child.Attributes["disabled"].Value)).ToList();

foreach (XmlNode node in col)
{
    Console.WriteLine("Removing:");
    Console.WriteLine(XDocument.Parse(node.OuterXml).ToString());
    root.RemoveChild(node);
}

Answer 2

即使以下代码也不会抛出任何异常：

foreach (XmlNode node in root.ChildNodes)
    root.RemoveChild(node);

它将删除恰好一个元素。我不是100％，我的解释是正确的，但它是在正确的轨道上。迭代集合时，将检索其枚举器。对于XmlNode，它是一个集合，这是一个名为XmlChildEnumerator的自定义类。

如果要通过Reflector查找MoveNext实现，您会看到枚举器会记住它当前正在查看的节点。当你调用MoveNext时，你会移动到下一个兄弟。

上面的代码中发生的是你从集合中获得第一个节点。在foreach循环体中隐式生成的枚举器将第一个节点作为其当前节点。然后，在foreach循环的主体中删除该节点。

现在该节点已与列表分离，执行将再次调用MoveNext。但是，由于我们刚从集合中删除了第一个节点，因此它与集合分离，并且节点没有兄弟节点。由于节点没有兄弟节点，迭代停止并且foreach循环退出，因此只删除单个元素。

这不会抛出异常，因为它不会检查集合是否已更改，它只是想继续查找它可以找到的下一个节点。但由于删除的（分离的）节点不属于集合，因此循环停止。

希望这能解决问题。

Linq，XmlNodes，foreach和例外

2 个答案: