Python实现Collatz程序输入1无法退出的原因及修复方法

你的代码存在4个直接导致故障的逻辑错误：

• collatz 函数仅执行了打印操作，没有通过return返回计算结果。Python中无显式返回值的函数默认返回None，因此变量b永远是None，b == 1的判断条件永远不会成立，退出逻辑从始至终不会被触发。
• 循环逻辑完全不符合Collatz序列的要求：当前while True循环每轮都会重新要求用户输入新数字，没有对同一个初始输入值持续递推计算直到得到1的流程。
• 调用了sys.exit()但没有提前导入sys模块，就算判断条件命中，运行到这行也会抛出NameError。
• 没有处理初始输入值为1的边界场景：输入1时按照你当前的函数逻辑，会判定为奇数计算3*1+1=4并打印，永远不会生成结果1，自然无法触发退出。

按以下逻辑调整代码即可：

1. 修改collatz函数，计算完对应结果后先打印，再把结果作为返回值传出
2. 拿到用户输入的合法整数后，先判断边界值，再进入递推循环：每次把上一轮的计算结果传入collatz，直到返回值为1时终止循环即可，不需要调用sys.exit()
3. 删掉重复的int(person)转换，把转换后的整数存为变量复用
4. 不需要额外导入模块，靠循环条件控制终止就行

修复后的可运行代码：

def collatz(number):
    if number % 2 == 0:
        res = number // 2
    else:
        res = 3 * number + 1
    print(res)
    return res

try:
    while True:
        print('please enter a number')
        person = input()
        num = int(person)
        # 处理输入为1的边界情况
        if num == 1:
            print(1)
            break
        while num != 1:
            num = collatz(num)
        break
except ValueError:
    print('You must enter an integer')

测试输入1时，会直接打印1后退出；输入其他整数时会逐行打印Collatz序列直到出现1后自动终止，输入非整数时会正常抛出提示。

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