C#中数学计算异常且MessageBox无法显示问题求助

问题现象 #

输入数字1或2时程序正常弹出结果；输入大于2的数字时程序冻结无响应；将所有数据类型改为int后，程序不再冻结，但计算结果错误。

待排查代码 #

double number, output;

if (!double.TryParse(txtNumber.Text, out number))
{
    MessageBox.Show("Input a valid number");
    Focus();
    return;
}
if (number < 0)
{
    MessageBox.Show("Enter a number greater than 0");
    Focus();
    return;
}
for (int i = 1; i < number; i++)
{ 
    number *= i;
}
output = number;
MessageBox.Show("Here is your number: " + output);

问题根源 #

循环逻辑存在致命错误：在for循环体内修改了循环条件依赖的变量number。

• 当输入大于2的数字（比如3），初始number=3，第一次循环i=1，执行number *=1后number仍为3；i自增到2，满足i<3，执行number *=2后number=6；i自增到3，此时i<6仍成立，继续循环，number会被不断放大，i永远追不上number的增长速度，导致死循环，程序因此冻结。
• 改为int类型时，虽然不会死循环，但number被修改后，循环的终止条件被破坏，实际循环次数远少于预期，最终阶乘计算结果错误。

修复方案 #

使用独立的临时变量保存初始输入数值，让循环条件依赖这个不变的变量，同时用单独变量存储阶乘计算结果：

double inputNumber, factorialResult;

if (!double.TryParse(txtNumber.Text, out inputNumber))
{
    MessageBox.Show("Input a valid number");
    Focus();
    return;
}
if (inputNumber < 0)
{
    MessageBox.Show("Enter a number greater than 0");
    Focus();
    return;
}
// 0和1的阶乘结果都是1
factorialResult = 1;
int n = (int)inputNumber;
// 从2开始计算，优化无意义的乘法步骤
for (int i = 2; i <= n; i++)
{ 
    factorialResult *= i;
}
MessageBox.Show("Here is your number: " + factorialResult);

修复说明 #

1. 用inputNumber存储初始输入值，循环条件不再依赖被修改的变量，彻底避免死循环。
2. 阶乘从2开始计算，跳过与1相乘的无意义步骤，提升效率。
3. 单独用factorialResult存储计算结果，与输入值分离，逻辑更清晰。
4. 兼容0和1的阶乘场景（0!和1!结果均为1）。

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