阶乘与斐波那契函数可正常运行，CallFibOrFac函数异常求助

我帮你排查了代码里的几个关键问题，这些问题直接导致CallFibOrFac无法正常工作，咱们一步步拆解解决：

问题点梳理 #

• Factorial函数的输入逻辑混乱：你在Factorial内部又调用了input()，这会导致递归过程中反复弹出输入提示，完全忽略了CallFibOrFac传进来的参数，逻辑完全走偏。
• 输入未做类型转换：CallFibOrFac中获取的num和n都是字符串类型，直接传给需要整数参数的Factorial和Fibonacci会触发类型错误。
• Fibonacci函数的双重问题：
  • 循环逻辑有偏差，最后输出的i和nextterm对应关系不对；
  • 函数没有返回值，所以print(Fibonacci(num))会额外输出None。
• CallFibOrFac的参数冗余：函数定义里的s参数根本没用到，因为你在函数内部重新用input()获取了输入，完全是多余的。

修正后的完整代码 #

def Factorial(n):
    # 移除内部输入，直接使用外部传入的整数参数
    if n == 0:
        return 1
    else:
        return n * Factorial(n - 1)

def Fibonacci(num):
    i = 0
    present = 1
    previous = 0
    # 调整循环条件，确保计算到对应项
    while i < num:
        nextterm = present + previous
        present = previous
        previous = nextterm
        i += 1
    # 返回计算结果，让调用方统一处理打印
    return previous

def CallFibOrFac():
    s = input('Fib or Fac? ')
    # 处理输入的空格，避免用户输入" Fib "这类带空格的内容导致判断失败
    if s.strip() == 'Fib':
        num = int(input('Up to what number? '))
        result = Fibonacci(num)
        print(f"The fibonacci number for {num} is {result}")
    elif s.strip() == 'Fac':
        n = int(input('What number? '))
        result = Factorial(n)
        print(f"The factorial of {n} is {result}")

# 调用无参函数
CallFibOrFac()

修改说明 #

1. Factorial函数：把输入逻辑移到CallFibOrFac中，让Factorial只专注于阶乘计算，递归时不会再反复弹出输入框。
2. Fibonacci函数：
   • 调整循环条件为i < num，确保计算结果对应你输入的数字项；
   • 改为返回计算结果，避免调用时输出多余的None。
3. CallFibOrFac函数：
   • 移除冗余的s参数；
   • 对输入的数字做int()转换，确保传给子函数的是正确的整数类型；
   • 用strip()处理输入的空格，提升用户输入的容错性；
   • 统一在CallFibOrFac中处理打印逻辑，让子函数只负责计算，职责更清晰。

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