C# - 无法保存包含新元素的自定义 linq-to-xml 文件？

Question

我正在创建一个新的 XDocument 并在其中插入根元素“profiles”，然后保存。

if (!System.IO.File.Exists("profiles.xml"))
{
    XDocument doc = new XDocument(
        new XElement("profiles")
    );
    doc.Save("profiles.xml", SaveOptions.None);
}

然后我想接受用户输入并将配置文件添加到已经创建的 xml 文件中：

XElement profile =
        new XElement(Player.Name,
            new XElement("level", Player.Level),
            new XElement("cash", Player.Cash)
        );

XDocument doc = XDocument.Load("profiles.xml");
List<XElement> profiles = doc.Root.Elements().ToList();

for (int i = 0; i < profiles.Count; i++)
{
    if (profiles[i].Name.ToString() == Player.name)
    {
        profiles[i] = profile;
        return;
    }
}
profile.Add(profile);
doc.Save("profiles.xml", SaveOptions.None);

但由于某种原因，它永远不会添加任何新的配置文件？

编辑：另外，如果我手动在 xml 文件中创建一个新的配置文件，它也不会自定义，因此问题出在保存文件中？

Answer 1

您实际上从未执行任何操作来更改 doc 引用的 XDocument 中的任何元素：

• 如果您找到具有现有名称的元素，您'重新修改列表，但这不会修改文档。您可能想使用 XElement.ReplaceWith：

  profiles[i].ReplaceWith(profile);
  

  请注意，在这种情况下，您甚至没有尝试再次保存 XML 文件（由于 return 语句），因此并不清楚您在这种情况下想要实现的目标.

• 如果您没有找到该元素，则您将 profile 元素添加到其自身，这当然不会修改文档。我怀疑你想要：

  doc.Root.Add(配置文件);
  

  换句话说，将新的配置文件元素添加为根元素的新最终子元素。

编辑：这里有一种不同的方法可以尝试 - 我假设任何一个名称只应该出现一次：

XDocument doc = XDocument.Load("profiles.xml");
var existingElement = doc.Root
                         .Elements()
                         .Where(x => x.Name.ToString() == Player.name)
                         .FirstOrDefault();
if (existingElement != null)
{
    existingElement.ReplaceWith(profile);
}
else
{
    doc.Root.Add(profile);
}
doc.Save("profiles.xml", SaveOptions.None);

另外，我强烈建议您不要使用玩家名称作为元素名称。将其用作属性值或文本值，例如

XElement profile =
        new XElement("player",
            new XAttribute("name", Player.Name),
            new Attribute("level", Player.Level),
            new XAttribute("cash", Player.Cash)
        );

这样，如果玩家名称包含空格等，您就不会遇到问题。然后您需要将查询更改为：

var existingElement = doc.Root
                         .Elements()
                         .Where(x => (string) x.Attribute("name)" == Player.name)
                         .FirstOrDefault();