C++ 当main函数调用被重构时，C函数给出错误的结果

1. 失踪包括.

2. 正如@ RetiredNinja所说，问题是scanf()返回成功时匹配的参数数量，而不是n的值(如果出错，它会返回n). 我在这里添加了错误判断以进一步澄清这一点.

3. factorial():n是无符号的，所以n <= 0最好写成n == 0或!n.

4. factorial():return 0;语句的放置是奇怪的，因为它只为 case n <= 0执行.

5. factorial():return n并没有错，但由于它是一个基本情况，所以使用return 1更清楚.

6. (未修复)由于溢出，我的系统上的factorial()(带有sizeof(long) == 8)返回n > 65的0.或许再加一张上限支票？

7. (不固定)factorial():更喜欢迭代实现，因为C不能保证尾部调用优化. 如果输入足够大，在我的系统上，在n = 250k到275k之间的某个地方，程序将粉碎堆栈并(希望)segfault. (默认)堆栈大小是特定于操作系统的.

8. 比起main()，我更喜欢签名main(void).在实践中，他们将做同样的事情，但只有前者在标准中定义(与main(int, char *[])一起).

9. (不固定)你可能只想在n > 0的情况下调用factorial()，或者像你的初始实现一样将值存储在变量中，这样你就可以忽略n == 0的输出，而不是:

   0
   No factorial for zero
   0
   

#include <stdio.h>

unsigned long factorial(unsigned long n) {
    if(!n) {
        printf("No factorial for zero\n");
        return 0;
    }
    if(n == 1)
        return 1;
    return n * factorial(n - 1);
}

int main(void) {
    unsigned long n;
    if(scanf("%lu", &n) != 1) {
        printf("scanf failed\n");
        return 1;
    }
    printf("%lu\n", factorial(n));
}

示例运行:

6
720