C#中如何替换XML元素？现有代码运行出现NULL错误求助

看起来你的代码主要有两个核心问题导致了Null引用错误或者无效果，咱们一步步拆解并修复：

问题分析 #

1. XPath定位错误：你通过UserAccountData/user/ID" + IDcheck从根节点查找ID节点，但这个路径会匹配第一个user节点下的IDn元素，而不是你找到的那个匹配的user节点。如果第一个user里没有对应的IDn，就会返回null，触发Null错误。
2. 错误的元素操作方式：你用AppendChild添加新的stats元素，而不是替换已有的，这会导致一个ID节点下出现多个stats，既不符合你的需求，还可能破坏XML结构。
3. 依赖ID计数的风险：依赖IDcheck的循环计数来定位ID节点很脆弱——如果XML里的user顺序改变、或者有user被删除，这个计数就会完全失效。

修复后的代码 #

public partial class Game : MetroFramework.Forms.MetroForm
{
    private void Save_data_Progress_Click(object sender, EventArgs e)
    {
        XmlDocument User_Data_Exp_Use = new XmlDocument();
        User_Data_Exp_Use.Load("UserData.xml");
        
        // 直接找到匹配的user节点，不再依赖计数
        XmlNode matchedUser = null;
        foreach (XmlNode userNode in User_Data_Exp_Use.SelectNodes("UserAccountData/user"))
        {
            string username = userNode.SelectSingleNode("username").InnerText;
            string password = userNode.SelectSingleNode("password").InnerText;
            if (Account.Text == username && Log_In_Data.PassKey == password)
            {
                matchedUser = userNode;
                break;
            }
        }

        // 先检查是否找到匹配的用户
        if (matchedUser == null)
        {
            MessageBox.Show("未找到匹配的用户账号");
            return;
        }

        // 找到当前user下的ID节点（不管是ID1/ID2...，直接取第一个符合ID开头的子节点）
        XmlNode idNode = null;
        foreach (XmlNode child in matchedUser.ChildNodes)
        {
            if (child.Name.StartsWith("ID"))
            {
                idNode = child;
                break;
            }
        }

        if (idNode == null)
        {
            MessageBox.Show("用户节点下未找到ID元素");
            return;
        }

        // 移除已有的stats元素（如果存在）
        XmlNode existingStats = idNode.SelectSingleNode("stats");
        if (existingStats != null)
        {
            idNode.RemoveChild(existingStats);
        }

        // 创建新的stats元素并添加
        XmlElement Stats_Set = User_Data_Exp_Use.CreateElement("stats");
        Stats_Set.SetAttribute("Level", Lv.Text);
        Stats_Set.SetAttribute("HP", Hit_Points.Text);
        Stats_Set.SetAttribute("exp", XP.Text);
        idNode.AppendChild(Stats_Set);

        // 保存XML
        User_Data_Exp_Use.Save("UserData.xml");
        MessageBox.Show("数据保存成功");
    }
}

关键改进点 #

• 直接持有匹配的user节点：不再依赖循环计数，而是找到匹配的user后直接保存该节点，后续操作都基于这个节点，避免XPath定位错误。
• 安全的ID节点查找：遍历user的子节点找到以ID开头的元素，比硬编码IDn更灵活，也避免计数错误。
• 替换而非新增元素：先检查并删除已有的stats元素，再添加新的，保证XML结构正确。
• 增加空值检查：在每一步关键操作前都检查节点是否为null，避免NullReferenceException，同时给用户明确的提示信息。

内容的提问来源于stack exchange，提问作者Ihor Berezovskyy