Java XML递归解析问题：无法获取CustomerId元素值

我来帮你找出这个递归方法里的问题，你的代码有两个关键逻辑错误，导致无法正确拿到CustomerId的值：

错误点分析 #

• 递归判断逻辑完全错误：你在循环里写了parseXmlForCustomerId((Element) node).equals("CustomerId")，但这个方法返回的是节点的文本内容（比如目标值"123"）或者空字符串，永远不可能等于节点名称"CustomerId"，这就导致即使找到目标节点，也不会触发返回逻辑。
• 返回值逻辑错误：就算判断逻辑没问题，你返回的是当前循环节点的getTextContent()，但递归调用已经深入子节点找到了目标，应该直接返回递归的结果，而不是当前节点的内容。

修正后的代码 #

private String parseXmlForCustomerId(Element element) {
    // 先检查当前节点是否为目标节点
    if (element.getNodeName().equals("CustomerId")) {
        return element.getTextContent();
    }
    NodeList children = element.getChildNodes();
    for (int i = 0; i < children.getLength(); i++) {
        Node node = children.item(i);
        // 仅处理元素类型的节点，跳过文本、注释等非元素节点
        if (node.getNodeType() == Node.ELEMENT_NODE) {
            String result = parseXmlForCustomerId((Element) node);
            // 如果递归找到结果，立即返回，无需继续遍历
            if (!result.isEmpty()) {
                return result;
            }
        }
    }
    // 遍历完所有子节点仍未找到，返回空字符串
    return "";
}

修正后的逻辑说明 #

1. 首先检查当前元素是否是CustomerId，如果是直接返回它的文本内容（这部分你的原代码是正确的）。
2. 遍历当前元素的所有子节点，只处理元素类型节点（避免处理XML里的空白文本节点、注释等干扰项）。
3. 递归调用方法处理子节点，拿到返回结果后：
   • 如果结果非空，说明已经在子节点树中找到了目标CustomerId，直接返回这个结果，终止后续遍历。
   • 如果结果为空，继续遍历下一个子节点。
4. 所有子节点都遍历完成仍未找到目标，返回空字符串。

用你提供的XML结构测试时，传入根节点<Store>，这个方法会递归遍历到<CustomerId>节点，正确返回"123"。

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