您能提高这种 linq-to-xml 方法的性能吗？

Question

今天早上我真的需要去别的地方。因此，我决定在这里发布一个性能问题。

下面的代码有效，但它多次调用 Load 和 Save 方法。这似乎远没有效率。请有人提供到目前为止加载和保存行发生在循环之外的代码。我只想调用一次加载和保存。

谢谢各位:)

 public void RemoveNodes(IList<String> removeItems)

    {

        foreach (String removeItem in removeItems)

        {

            XDocument document = XDocument.Load(fullFilePath);

            var results = from item in document.Descendants(elementName)

                          let attr = item.Attribute(attributeName)

                          where attr != null && attr.Value == removeItem.ToString()

                          select item;

            results.ToList().ForEach(item => item.Remove());

            document.Save(fullFilePath);

        }

    }

Answer 1

您自己已经给出了答案 - 只需将 Load 和 Save 调用移到循环之外即可。我不清楚您自己在实现这一点时遇到了问题...

不过您也可以使您的查询稍微简单一些：

XDocument document = XDocument.Load(fullFilePath);
foreach (String removeItem in removeItems)
{
    var results = from item in document.Descendants(elementName)
                  where (string) item.Attribute(attributeName) == removeItem
                  select item;
    results.ToList().ForEach(item => item.Remove());
}
document.Save(fullFilePath);

这使用了从 XAttribute 到 string 的转换这一事实> 如果属性引用本身为 null，则返回 null。

您甚至不需要使用查询表达式：

var results = document.Descendants(elementName)
          .Where(item => (string) item.Attribute(attributeName) == removeItem);