C++阶乘（factorial）循环程序问题求助：仅首次输入结果正确，后续计算值异常

嘿，我看到你的问题了——第一次计算阶乘结果正确，但后续所有迭代的结果全错，这其实是个很常见的变量初始化问题，咱们一步步来解决：

核心问题：阶乘变量未重置 #

你把factorial定义在main函数的开头，这意味着它的值会在整个程序运行过程中保留。第一次计算9的阶乘后，factorial的值变成了362880；第二次计算8的时候，程序是用362880乘以1到8，结果自然不对，而且还因为int类型的数值溢出，出现了奇怪的负数或大数。

还有几个小细节问题需要调整 #

1. 非法输入处理逻辑不完善：原来的while循环在检测到非法值时，只输出错误信息就break，没有让用户重新输入合法值，会导致程序直接进入下一轮循环。
2. 多余的n++操作：do-while末尾的n++会修改用户输入的数值，比如输入0后n会变成1，导致循环多执行一次，逻辑混乱。

修正后的完整代码 #

#include <iostream>
using namespace std;

int main (){
 int n;
 do{
 cout << "Enter a number between 1 and 10 (or 0 to end program): ";
 cin >> n;

 // 循环提示用户输入，直到输入合法值
 while (n < 0 || n > 10) {
 cout << "Error: " << n << " is outside the valid range \n";
 cout << "Please re-enter a number between 1 and 10 (or 0 to end program): ";
 cin >> n;
 }

 if (n > 0 && n <= 10) {
 int factorial = 1; // 每次计算阶乘前重新初始化
 for(int i = 1; i <= n; ++i){
 factorial *= i;
 }
 cout << "Factorial of " << n << " is " << factorial << endl;
 } else if (n==0){
 cout << "Good bye! \n";
 break;
 }
 } while (true); // 用无限循环配合break控制退出
 return 0;
}

关键修改点说明 #

• 将factorial的定义移到阶乘计算代码块内：这样每次计算前，factorial都会被重新初始化为1，彻底解决了上一次结果残留的问题。
• 完善非法输入处理：现在程序会持续提示用户输入，直到输入1-10之间的数或0为止，避免流程混乱。
• 移除多余的n++：让用户输入的数值完全控制程序流程，不再额外修改n的值。
• 改用while(true)循环：配合n==0时的break，逻辑更清晰，避免因n的修改导致循环条件出错。

现在运行程序，输入9、8、7、6就能得到你期望的正确结果了。

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