带默认回退的列表数值过滤出现无限循环问题求助

需求背景 #

我有两个数值列表：

• real_solutions_sols1 = [-53.2909210236, -8.31658000998, 1.87689837129, 1.4]
• real_solutions_sols2 = [-21.1439685227, -19.2]

需要实现的逻辑是：筛选出列表中0.1到4.0之间的数值；如果没有符合条件的数值，就返回原列表。

测试情况 #

测试real_solutions_sols2的代码（可正常运行） #

我写了以下代码测试real_solutions_sols2，因为没有元素满足条件，代码执行了else分支，输出符合预期：

real_roots_zero_to_four = []
for i in real_solutions_sols2:
    if (i >= 0.1) and (i <= 4.0):
        real_roots_zero_to_four.append(i)
    else:
        real_roots_zero_to_four = real_solutions_sols2
print 'real_roots_zero_to_four = ', real_roots_zero_to_four

测试real_solutions_sols1时的问题 #

但是用同样的代码测试real_solutions_sols1时，代码陷入了无限循环，我找不到原因，请求帮忙排查。

问题排查与解决方案 #

问题根源 #

你的代码逻辑存在两个致命问题：

1. else分支的时机完全错误：你在for循环的每一次迭代中，只要遇到不满足条件的元素，就直接把结果列表替换成原列表。这会导致后续的操作直接关联到原列表对象。
2. 遍历过程中修改原列表：当real_roots_zero_to_four被赋值为原列表后，后续的append操作会直接修改原列表的长度——原列表在循环中不断变长，for循环的迭代器永远遍历不完，这就是无限循环的直接原因！

举个具体的执行流程：

• 第一次循环：i=-53.29，不满足条件，执行else，real_roots_zero_to_four和real_solutions_sols1指向同一个列表对象；
• 第二次循环：i=-8.31，同样不满足条件，再次赋值（还是同一个对象）；
• 第三次循环：i=1.87，满足条件，执行append(i)——直接把1.87添加到原列表中，原列表长度从4变成5；
• 第四次循环：i=1.4，满足条件，append后原列表长度变成6；
• 此时for循环还在遍历原列表，而原列表在不断变长，循环永远不会结束。

正确的实现方式 #

你应该先完成所有元素的筛选，再判断结果列表是否为空。如果为空，返回原列表；否则返回筛选后的列表。

方法1：用for循环规范实现

def filter_between_01_and_4(lst):
    filtered = []
    for num in lst:
        if 0.1 <= num <= 4.0:
            filtered.append(num)
    # 遍历完成后再判断：筛选结果为空则返回原列表副本，否则返回筛选结果
    return filtered if filtered else lst.copy()

# 测试
real_solutions_sols1 = [-53.2909210236, -8.31658000998, 1.87689837129, 1.4]
real_solutions_sols2 = [-21.1439685227, -19.2]

print(filter_between_01_and_4(real_solutions_sols1))  # 输出 [1.87689837129, 1.4]
print(filter_between_01_and_4(real_solutions_sols2))  # 输出 [-21.1439685227, -19.2]

方法2：用列表推导式更简洁

def filter_between_01_and_4(lst):
    filtered = [num for num in lst if 0.1 <= num <= 4.0]
    return filtered if filtered else lst.copy()

# 测试同上

关键注意点 #

• 绝对不要在遍历列表时修改原列表：这会彻底打乱迭代器的正常行为，引发各种异常；
• 判断时机要正确：必须在所有元素遍历完成后，再决定返回筛选结果还是原列表；
• 返回原列表时用副本：使用lst.copy()（或lst[:]）可以避免后续操作意外修改原始列表的内容。

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