Python3中函数操作全局列表及循环迭代问题求助

原代码核心问题拆解 #

• get_values()函数循环失效：

  1. 循环条件错误使用or，实际需要的是and——只有当输入数量未达10个且用户未输入quit时才继续循环。
  2. 循环启动前未定义value变量，直接在条件中引用会触发NameError。
  3. 未先判断用户输入是否为"quit"就强制转float，输入quit会直接报错崩溃。
  4. 第一次循环就执行return values，导致循环仅运行一次就终止，这是程序只弹出一次输入的直接原因。
  5. 依赖全局变量存储输入值，代码耦合性高，易引发意外问题。

• avg函数逻辑混乱：

  1. 使用Python内置关键字list作为参数名，违反命名规范。
  2. 函数内部未使用传入的参数，反而依赖全局变量，完全失去函数的复用意义。
  3. 调用时错误地将列表嵌套成[values]，导致sum()和len()计算对象错误，结果必然异常。

修改后的完整代码 #

MAX_NUM = 10

def get_values():
    values = []
    while len(values) < MAX_NUM:
        user_input = input('Enter a number or type quit to quit: ').strip().lower()
        if user_input == 'quit':
            break
        try:
            num = float(user_input)
            values.append(num)
        except ValueError:
            print('Invalid input, please enter a number or "quit".')
    return values

def calculate_average(num_list):
    if not num_list:
        return 0.0
    return sum(num_list) / len(num_list)

def main():
    input_values = get_values()
    average = calculate_average(input_values)
    print(f'\nThe average is: {average:,.1f}')

if __name__ == '__main__':
    main()

关键修改说明 #

1. get_values()优化：

   • 内部创建局部列表存储输入，彻底摆脱全局变量依赖。
   • 循环条件简化为判断输入数量，通过break处理quit输入，逻辑更直观。
   • 先处理用户输入（转小写、去空格），判断是否为quit后再尝试转数字，同时加入异常处理，避免非法输入导致程序崩溃。
   • 移除提前的return语句，确保循环能正常执行到退出条件。

2. calculate_average()优化：

   • 改用合法参数名num_list，并真正基于传入参数计算平均值。
   • 增加空列表判断，避免无输入时出现除以0的错误。

3. main()优化：

   • 正确接收get_values()返回的列表，直接传入计算函数，不再进行错误嵌套。
   • 添加if __name__ == '__main__':，确保脚本作为主程序运行时才执行逻辑，符合Python规范。

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