C++使用for循环实现阶乘计算的程序编写及现有代码问题求助

原有代码错误梳理 #

你提交的代码存在以下几类基础语法和逻辑错误：

• 命名空间声明错误：include namespace std; 应为 using namespace std;，缺少关键字using
• 输入运算符使用错误：cin<<factorial; 中<<是输出运算符，输入需要用>>，即cin >> factorial;
• 循环条件逻辑错误：for循环的条件i=factorial是赋值操作，不是等值判断，即使改成i==factorial也不合理，因为你用同一个变量factorial同时存储输入的目标值和阶乘计算结果，计算过程中factorial的值会持续变大，循环终止条件完全失控
• 基础语法缺失：factorial=factorial*i 末尾缺少C++语句必须的分号;
• 变量重复定义：外层已经定义了long long i，for循环内部重复定义了同名变量i，不符合C++语法规范
• 冗余头文件：math.h在当前逻辑中没有用到，可以直接删除

需求说明与数据类型补充 #

你提到的long long是当前常用C++编译器支持的最大内置有符号整数类型，为64位，取值范围是 -9223372036854775808 ~ 9223372036854775807，最多可以存储20!的结果，21!就会超出范围发生溢出。
如果你要实现的是计算当前机器long long支持的最大阶乘，不需要用户输入数值，直接通过循环计算直到即将溢出时停止即可。

正确实现代码 #

版本1：自动计算最大支持阶乘 #

#include <iostream>
#include <cstdlib>
#include <climits> // 导入LLONG_MAX常量，即long long的最大值
using namespace std;

int main()
{
    long long fact_result = 1;
    int max_n = 1;
    // 提前判断下一次乘法是否会溢出，避免有符号整数溢出的未定义行为
    while (true) {
        if (fact_result > LLONG_MAX / (max_n + 1)) {
            break;
        }
        max_n++;
        fact_result *= max_n;
    }
    cout << "当前机器long long类型可支持的最大阶乘为：" << max_n << "! = " << fact_result << endl;

    system("PAUSE");
    return EXIT_SUCCESS;
}

版本2：输入指定数值计算阶乘 #

如果你需要实现输入一个数计算它的阶乘，可以用以下版本：

#include <iostream>
#include <cstdlib>
using namespace std;

int main()
{
    long long fact_result = 1;
    int input_n;
    cout << "请输入要计算阶乘的正整数：";
    cin >> input_n;

    // 提前判断输入是否超出long long支持范围
    if (input_n > 20) {
        cout << "输入数值过大，long long类型无法存储" << input_n << "!的结果" << endl;
        system("PAUSE");
        return EXIT_SUCCESS;
    }

    for (int i = 1; i <= input_n; i++) {
        fact_result *= i;
    }
    cout << input_n << "! = " << fact_result << endl;

    system("PAUSE");
    return EXIT_SUCCESS;
}

学习建议 #

• 写代码时尽量保证单一变量只承担一个职责，不要用同一个变量同时存储输入值、计算结果等不同含义的内容
• 遇到编译报错时优先看编译器的错误提示，基础语法错误一般都会明确标注错误行和错误类型，比如少分号、运算符误用都能直接定位
• 如果需要计算更大的阶乘，可以用数组模拟大数乘法，就可以摆脱内置数据类型的长度限制

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