如何重写导航节点列表的代码

Question

我刚刚编写了一些代码，在编写时我认为这将是搜索特定节点的一个很好的通用方法。当我完成时，我实际上意识到这是一团糟：D

public String sqlReading(String fileName, String path, String nodeId )
{
    XmlDocument doc = new XmlDocument();
    doc.Load(fileName);

    XmlNodeList names = doc.SelectNodes(path);
    foreach (XmlNode xmlDocSearchTerm in names)
    {
        //if the attribute of the node i start at is the same as where i am now
        if (xmlDocSearchTerm.Attributes.Item(0).Value.ToString().Equals(nodeId))
        {
            //get a list of all of its child nodes
            XmlNodeList childNodes = xmlDocSearchTerm.ChildNodes;

            foreach (XmlNode node in childNodes)
            {
                //if there is a node in here called gui display, go inside
                if (node.Name.Equals("GUIDisplay"))
                {
                    XmlNodeList list = node.ChildNodes;
                    //find the sqlsearchstring tag inside of here
                    foreach (XmlNode finalNode in list)
                    {
                        if (finalNode.Name.Equals("sqlSearchString"))
                        {
                            return node.InnerText;
                        }
                    }
                }
            }
        }
    }
    return "";
}

我打算做的是基于路径 - 我会开始检查该元素是否具有我正在寻找的 id，如果有，那么我想进入内部直到我到达深两层的 sqlsearchstring 标签，我才停止前进。我已经做到了，但这里的问题是，现在我似乎几乎硬编码了标签的路径，而不是在那里循环。我怎样才能改变我的代码来阻止我这样做？

在我看来，它是从第二个 foreach 开始出错的。

谢谢

Answer 1

还没有测试过，但我相信通过使用 xpath 这样的东西会起作用。但是我不确定属性的名称，或者它始终是第一个属性？

public String sqlReading(String fileName, String path, String nodeId)
{
    XmlDocument doc = new XmlDocument();
    doc.Load(fileName);

    XmlNode foundNode = doc.SelectNodes(path).SelectSingleNode("*[@id='" + nodeId + "']/GUIDisplay/sqlSearchString");
    if (foundNode != null)
        return foundNode.InnerText;
    return string.Empty;

}

Answer 2

我不确定这是否完全正确（因为我没有 XML 文档来尝试，但类似的东西应该可以工作

var innerTexts = XDocument.Load(fileName)
    .Elements(path)
    .Where(n => n.Attributes().ElementAt(0).Value == nodeId)
    .SelectMany(n => n.Elements())
    .Where(n => n.Name == "GUIDisplay")
    .SelectMany(n => n.Elements())
    .Where(n => n.Name == "sqlSearchString")
    .Select(n => n.ToString());

Answer 3

我想说递归是一个安全的选择（用于迭代嵌套的子节点），但是，根据我收集的信息，结构保持不变。考虑到这一点，为什么不使用 [XmlDocumentObj].SelectSingleNode("/[nodeId='"+nodeId+"']") （或某些传真）呢？它能够按属性名称进行搜索，除非 XML 结构总是更改并且您从没有常量标记（在这种情况下 XPath 可能是一个好主意）。