C++读取XML时仅获取到第一个check_id节点值问题求助

嘿，我之前在使用类似的DOM库遍历节点时也踩过一模一样的坑！咱们来拆解下你代码里可能的问题，以及对应的解决办法：

1. 遍历逻辑的误区：Next()方法的调用对象错了 #

你的循环里用了oCheckNode = oNodeList.Next()，但很多自定义的CXmlDomNodeList实现中，Next()并不是基于当前节点来推进，而是始终从列表的起始位置往后找——这就会导致你每次都拿到第二个节点，然后循环条件可能因为某种原因提前终止，看起来像是只拿到了第一个。

正确的做法应该是让当前节点自己找下一个兄弟节点，而不是从列表调用Next()：

CXmlDomNodeList oNodeList = m_oInputXmlData.pXmlDomRecvd->GetElementsByTagName("check_info");
CXmlDomNode oCheckNode = oNodeList.First();
// 用节点的有效性判断替代直接的bool转换（更稳妥）
while (oCheckNode.IsValid()) {
    i64CheckId = _atoi64(oCheckNode.GetChildNodeText("check_id"));
    // do your stuff here

    // 关键：让当前节点跳转到下一个兄弟节点
    oCheckNode = oCheckNode.NextSibling();
}

2. 另一种可靠的遍历方式：按索引访问节点列表 #

如果你的CXmlDomNodeList支持通过索引获取节点，这种方式会更直观也更不容易出错：

CXmlDomNodeList oNodeList = m_oInputXmlData.pXmlDomRecvd->GetElementsByTagName("check_info");
int nodeCount = oNodeList.GetLength(); // 假设存在获取列表长度的方法
for (int i = 0; i < nodeCount; ++i) {
    CXmlDomNode oCheckNode = oNodeList.Item(i); // 通过索引取节点
    if (oCheckNode.IsValid()) {
        i64CheckId = _atoi64(oCheckNode.GetChildNodeText("check_id"));
        // do your stuff here
    }
}

3. 排查XML结构本身的问题 #

如果上面的方法还是不行，建议你先确认XML的结构是否符合预期：

• 是不是所有的<check_info>节点都包含<check_id>子节点？
• 有没有部分<check_id>是空值，导致_atoi64返回0，看起来像是没拿到？
• 可以临时加日志输出每个节点的XML内容，比如oCheckNode.GetXml()（如果有这个方法），确认遍历确实到了每个节点。

4. 注意DOM节点对象的拷贝问题 #

有些自定义DOM库的CXmlDomNode是浅拷贝的句柄对象，赋值操作可能会导致原节点的状态被覆盖。如果你用的库有指针或引用形式的节点访问，也可以试试用指针遍历：

CXmlDomNode* pCheckNode = &oNodeList.First();
while (pCheckNode && pCheckNode->IsValid()) {
    i64CheckId = _atoi64(pCheckNode->GetChildNodeText("check_id"));
    // do your stuff here
    pCheckNode = &pCheckNode->NextSibling();
}

你可以先试试第一种方法，这是最常见的问题所在。如果还是不行，再排查XML结构或者库的特殊用法~

内容的提问来源于stack exchange，提问作者K.Krunk