Python enumerate遍历空findall结果计数错误问题排查

你的错误根源 #

你用enumerate统计时得到错误结果，核心问题是空迭代时enumerate不会触发循环，导致你依赖的index变量保留了上一次的旧值。

举个你可能的错误代码示例：

total = 0
index = 0
for country in root.findall('country'):
    # 遍历当前country下的neighbor标签
    for index, neighbor in enumerate(country.findall('neighbor')):
        pass
    # 把当前country的neighbor数量（index+1）加到总数
    total += index + 1

当遇到没有neighbor的country（比如Iceland）时，country.findall('neighbor')返回空列表，enumerate遍历空列表时，内部的for循环一次都不会执行。这时候index还是上一次循环的数值（比如上一个country有3个neighbor，index就是2），你再加index+1（也就是3）到total里，就凭空多算了，最终总数就会偏大。

为什么findall空时enumerate不返回0？ #

enumerate的作用是遍历可迭代对象的每一个元素，生成(索引, 元素)的元组。如果可迭代对象是空的（比如空列表），它不会生成任何元组，对应的for循环体根本不会运行。所以你的index变量不会被重新赋值，自然不会变成0——这不是enumerate的用法错误，是你误用了它的场景：你误以为不管有没有元素，enumerate都会给你一个索引0，但实际上空迭代不会触发任何迭代动作。

正确的解决方法 #

方法1：直接用len()统计（最简洁可靠） #

这是最直接的方式，直接取每个country下符合条件的neighbor列表长度，空列表的长度就是0，不会出错：

total = 0
for country in root.findall('country'):
    # 只统计country的直接子节点neighbor
    neighbors = country.findall('./neighbor')
    total += len(neighbors)
print(total)  # 输出7，符合预期

方法2：如果非要用enumerate（修正逻辑） #

如果一定要用enumerate，要避免依赖循环外的变量，而是在每次循环country时初始化计数，或者实时累加：

total = 0
for country in root.findall('country'):
    current_count = 0
    # 从1开始计数，直接拿current_count作为当前country的neighbor数量
    for current_count, _ in enumerate(country.findall('./neighbor'), start=1):
        pass
    total += current_count

或者更简单的，每找到一个neighbor就加1：

total = 0
for country in root.findall('country'):
    for _, _ in enumerate(country.findall('./neighbor')):
        total += 1

总结 #

你之前的错误是错误地依赖了enumerate循环后的index变量，空迭代时该变量未更新导致旧值被错误累加。enumerate本身没问题，但不适合你这种“通过最后一个索引计算数量”的场景，直接用len()是最稳妥高效的方式。

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