双数组节点校验异常：仅nod节点可被读取，求技术协助

核心问题排查 #

1. 遍历逻辑错误：原代码中for (let node of nodes || nod)的写法存在缺陷——如果nodes是空数组（属于JavaScript假值），逻辑或||会直接返回nod，导致nodes的所有元素完全被忽略，这就是只有nod节点能被读取的根本原因。
2. 未声明变量：monto未提前声明，运行时会抛出ReferenceError。
3. return位置错误：return new_nodes写在函数体外，函数无法返回结果。
4. 无效判断：child.length是无意义的判断——单个DOM节点没有length属性，该条件永远不成立。
5. 大小写敏感问题：硬编码判断nodeName的大小写，容易因为节点名称大小写不一致漏判目标节点。

修复后的代码 #

function validatePagosNodes(nodes, nod) {
  let new_nodes = [];
  let monto = 0; // 声明并初始化金额变量

  // 合并两个输入节点集合，确保两者都被遍历
  const allNodes = [...(nodes || []), ...(nod || [])];

  for (let node of allNodes) {
    // 安全读取并累加Monto属性值
    const montoValue = node.getAttribute("Monto");
    if (montoValue) {
      monto += Number(montoValue);
    }

    // 遍历当前节点的所有子节点
    for (let child of node.children) {
      // 统一转小写判断节点名称，兼容大小写差异
      if (child.nodeName.toLowerCase() === "pago20:doctorelacionado") {
        new_nodes.push(child);
      }
    }
  }

  return new_nodes; // 正确返回收集到的节点
}

关键修复说明 #

• 合并节点集合：使用扩展运算符...将nodes和nod统一合并为一个数组，无论其中一个是空数组还是undefined，都能保证两个集合的节点都被遍历。
• 变量声明与安全处理：声明monto并初始化为0，同时判断Monto属性是否存在，避免非数字值导致金额变成NaN。
• 修正return位置：将返回语句移到函数内部，确保函数能正常返回结果。
• 简化子节点遍历：直接遍历node.children（类数组对象可直接用for...of遍历），移除无效的child.length判断。
• 兼容大小写：将nodeName转成小写后再判断，避免因大小写差异漏判目标节点。

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