从XmlDocument中删除节点

Question

下面的代码应该找到合适的项目标签并将其从XmlDocument中移除，但是当我测试它时，它会显示：

要移除的节点不是此节点的子节点。

有人知道正确的做法吗？

public void DeleteProject (string projectName)
{
    string ccConfigPath = ConfigurationManager.AppSettings["ConfigPath"];

    XmlDocument configDoc = new XmlDocument();

    configDoc.Load(ccConfigPath);

    XmlNodeList projectNodes = configDoc.GetElementsByTagName("project");

    for (int i = 0; i < projectNodes.Count; i++)
    {
        if (projectNodes[i].Attributes["name"] != null)
        {
            if (projectName == projectNodes[i].Attributes["name"].InnerText)
            {                                                
                configDoc.RemoveChild(projectNodes[i]);
                configDoc.Save(ccConfigPath);
            }
        }
    }
}

更新

已修复。我做了两件事：

XmlNode project = configDoc.SelectSingleNode("//project[@name='" + projectName + "']");

用XPath查询替换了for循环，这并不是为了修复它，而只是因为这是更好的方法。

实际的修复方法是：

project.ParentNode.RemoveChild(project);

感谢 Pat 和 Chuck 的建议。

- Jonathan Holland

---

我正在寻找这个东西。我在互联网上花了半天时间寻找XML相关的内容，但在Stack Overflow上只花了半分钟。这也是为什么Jeff和Joel很有远见的另一个原因。 - Anthony Potts

Answer 1

不是

configDoc.RemoveChild(projectNodes[i]);

尝试

projectNodes[i].parentNode.RemoveChild(projectNodes[i]);

- Pat

Answer 2

当你写长了之后，可能会感到很烦（对我来说很快就会），这时候你可以使用下面提供的帮助扩展方法。耶，新技术！

public static class Extensions {
    ...
    public static XmlNode RemoveFromParent(this XmlNode node) {
        return (node == null) ? null : node.ParentNode.RemoveChild(node);
    }
}
...
//some_long_node_expression.parentNode.RemoveChild(some_long_node_expression);
some_long_node_expression.RemoveFromParent();

- Eugene Ryabtsev

Answer 3

尝试

configDoc.DocumentElement.RemoveChild(projectNodes[i]);

- Jason

Answer 4

看起来在调用RemoveChild之前，您需要选择projectNodes [i]的父节点。

- user1921

Answer 5

项目节点可能不是子节点，而是孙子节点或更低级别的节点吗？如果我没记错，GetElementsByTagName将从子元素树的任何位置获取元素。

- Greg Hurlman

Answer 6

看到您正在处理的 XML 文件的样本会很方便，但我的猜测是您的文件大致如下：

<Root>
 <Blah>
   <project>...</project>
 </Blah>
</Root>

错误信息似乎是因为您试图从项目节点的祖先而不是直接父级中删除<project>

- David Hayes