一、保留XML注释的更优修改方案 #

你之前用XmlReader并设置IgnoreComments=true的方式会丢失注释，维护两个XML副本（带注释/不带注释）的方法不仅繁琐，还容易出现同步错误。推荐直接使用**XmlDocument或LINQ to XML（XDocument）**，这两个API默认会保留所有节点（包括注释），修改后直接保存即可保留原注释。

方案1：使用XmlDocument #

XmlDocument加载XML时会完整保留所有节点类型，包括注释。你可以直接定位到需要修改的节点进行操作，最后保存时注释会被原样保留：

// 加载XML文件，默认保留注释
XmlDocument doc = new XmlDocument();
doc.Load(path);

// 定位到需要修改的节点（示例：修改Node2下childnode的someattr2属性）
XmlNode node2 = doc.SelectSingleNode("//Node2");
XmlNode childNode = node2.SelectSingleNode("./childnode");
childNode.Attributes["someattr2"].Value = "NewBook";

// 保存文件，注释完全保留
doc.Save(path);

方案2：使用LINQ to XML（XDocument） #

如果你更倾向于LINQ语法，XDocument同样会保留注释节点，操作更简洁：

// 加载XML文件
XDocument doc = XDocument.Load(path);

// 修改目标节点属性
var childNode = doc.Descendants("Node2").Elements("childnode").FirstOrDefault();
if (childNode != null)
{
    childNode.Attribute("someattr2").Value = "NewBook";
}

// 保存文件，注释保留
doc.Save(path);

这两种方案都不需要维护两个XML副本，直接在原文档结构上修改，既避免了同步问题，又能确保注释完整保留，比你的初始方案高效得多。

二、节点跳转操作的优化方式 #

你当前使用node.PreviousSibling和node.NextSibling实现节点跳转，这个方法本身可行，但有两个可以优化的点：

1. 过滤非元素节点（跳过注释、空白文本节点） #

PreviousSibling/NextSibling会返回所有类型的节点（包括注释、空白文本），如果只想跳转元素节点，可以写一个辅助方法来跳过无关节点：

// 获取下一个兄弟元素节点
public static XmlNode GetNextElementSibling(XmlNode node)
{
    XmlNode next = node.NextSibling;
    while (next != null && next.NodeType != XmlNodeType.Element)
    {
        next = next.NextSibling;
    }
    return next;
}

// 获取上一个兄弟元素节点
public static XmlNode GetPreviousElementSibling(XmlNode node)
{
    XmlNode prev = node.PreviousSibling;
    while (prev != null && prev.NodeType != XmlNodeType.Element)
    {
        prev = prev.PreviousSibling;
    }
    return prev;
}

使用时直接调用这两个方法，就能精准跳转到下一个/上一个元素节点，不用手动判断节点类型。

2. 缓存同级节点列表（适合频繁跳转场景） #

如果需要多次在同级节点间跳转，建议提前把同级元素节点缓存到一个列表中，这样可以通过索引直接访问，比反复遍历Sibling更高效：

// 假设当前节点是某个元素，获取其父节点下的所有元素子节点
XmlNode parentNode = node.ParentNode;
List<XmlNode> siblingElements = parentNode.ChildNodes
    .Cast<XmlNode>()
    .Where(n => n.NodeType == XmlNodeType.Element)
    .ToList();

// 获取当前节点在列表中的索引
int currentIndex = siblingElements.IndexOf(node);

// 跳转到下一个元素节点
if (currentIndex < siblingElements.Count - 1)
{
    XmlNode nextElement = siblingElements[currentIndex + 1];
}

// 跳转到上一个元素节点
if (currentIndex > 0)
{
    XmlNode prevElement = siblingElements[currentIndex - 1];
}

如果使用XDocument，可以直接用Elements()获取同级元素集合，然后通过LINQ的ElementAt()或索引访问：

XElement currentElement = ...; // 当前元素节点
IEnumerable<XElement> siblingElements = currentElement.Parent.Elements();

// 转换为列表以便索引访问
List<XElement> siblingList = siblingElements.ToList();
int index = siblingList.IndexOf(currentElement);

XElement nextElement = index < siblingList.Count - 1 ? siblingList[index + 1] : null;
XElement prevElement = index > 0 ? siblingList[index - 1] : null;

这种缓存方式在频繁跳转时能显著提升性能，避免每次遍历兄弟节点的开销。

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