为何变量无法添加到两个列表？Python奇偶列表问题求助

原代码的核心问题 #

• 提前终止循环：return语句写在for循环的判断分支里，导致函数处理完第一个元素就直接返回，后面的元素根本没机会被遍历。
• 重复生成数据：两次调用oddandeven函数，每次都会重新生成一个全新的随机列表，而且全局的even_numbers和odd_numbers会被重复填充。
• 参数冗余：定义的numbers_list参数完全没用到，属于无效代码。

修复后的代码（贴近原逻辑） #

import random

def oddandeven():
    # 只生成一次随机原始列表
    original_list = [random.randint(1, 10) for _ in range(10)]
    print('Original list:', original_list)
    
    even_numbers = []
    odd_numbers = []
    
    # 完整遍历所有元素
    for num in original_list:
        if num % 2 == 0:
            even_numbers.append(num)
        else:
            odd_numbers.append(num)
    
    # 遍历完成后统一排序，提升效率
    even_numbers.sort()
    odd_numbers.sort()
    
    # 一次性返回两个结果列表
    return even_numbers, odd_numbers

# 一次调用获取奇偶列表
evens, odds = oddandeven()
print('Even numbers:', evens)
print('Odd numbers:', odds)

更简洁的写法（列表推导式） #

如果想让代码更紧凑，可以用列表推导式直接筛选元素，再统一排序：

import random

original_list = [random.randint(1, 10) for _ in range(10)]
print('Original list:', original_list)

# 用列表推导式筛选+排序
even_numbers = sorted(num for num in original_list if num % 2 == 0)
odd_numbers = sorted(num for num in original_list if num % 2 != 0)

print('Even numbers:', even_numbers)
print('Odd numbers:', odd_numbers)

关键修改点说明 #

1. 把return移到循环外部，确保所有元素都被处理完。
2. 移除全局变量，改用函数内部变量存储结果，避免多次调用的数据污染。
3. 排序操作放在遍历完成后，避免每次添加元素都重复排序的无效操作。
4. 一次调用函数即可获取两个结果，避免生成多份不同的随机列表。

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