C语言阶乘计算器代码遇阻：逻辑问题求助排查

原代码存在的问题 #

• 变量未初始化：j未赋予初始值就参与计算，会触发未定义行为，最终输出垃圾值。阶乘的初始计算值需要合理赋值。
• for循环更新逻辑无效：i*1不会改变i的数值，循环条件i>0会一直成立，造成死循环。同时循环体内手动执行i--，和循环更新逻辑重复冲突。
• 阶乘计算逻辑错误：每次执行j=zahl*i是用原始输入值乘以i，而非用当前累积的阶乘结果继续相乘，完全不符合阶乘的计算逻辑。
• 字符串字面量格式错误：C语言中字符串字面量不能直接跨行书写，原代码的换行方式会导致编译报错，需用\n实现换行。
• 缺少边界值处理：未针对0、1或负数做特殊处理，不符合阶乘的数学定义（0!和1!均为1，负数没有阶乘）。

修正后的代码 #

#include<stdio.h>

int main(){
    
    int zahl;
    int i;
    int j;
    
    printf("Fakultaet Rechner\n===============\n");
    printf("Von welcher Zahl soll die Fakultaet berechnet werden: ");
    scanf("%d", &zahl);

    // 处理边界情况
    if(zahl < 0){
        printf("Fehler: Negative Zahlen haben keine Fakultaet!\n");
        return 1;
    } else if(zahl == 0 || zahl == 1){
        printf("Die Fakultaet von %d ist 1\n", zahl);
        return 0;
    }

    // 初始化阶乘结果为输入值
    j = zahl;
    // 从zahl-1开始累乘到1
    for(i = zahl - 1; i > 0; i--){
        j = j * i;
    }
    
    printf("Die Fakultaet von %d ist %d\n", zahl, j);
    
    return 0;
}

修正说明 #

1. 初始化j：将j的初始值设为输入的zahl，后续用j = j * i实现累积相乘，符合阶乘的计算逻辑。
2. 修复for循环：将循环更新表达式改为i--，移除循环体内的手动i--，避免逻辑冲突。
3. 修正字符串格式：用\n替代直接跨行，符合C语言字符串语法规范。
4. 增加边界判断：针对负数、0和1做特殊处理，输出符合数学定义的结果，避免错误计算。

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