如何简化含变量的C语言if-else代码？求错误排查与优化方案

第一段代码的错误分析与修复 #

存在的问题 #

• 未初始化变量违规使用：当yos <= 3时，代码直接打印bonus，但bonus从未被赋值，属于未初始化变量，会输出随机垃圾值，触发未定义行为。
• 三元运算符逻辑完全倒置：当前代码仅在yos>3时给bonus赋值100却不打印，yos<=3时打印未初始化的bonus，完全不符合奖金计算的正常逻辑。

修复后的代码 #

#include <stdio.h>
int main()
{
    int bonus, cy, yoj, yos;
    printf("Enter current year and year of joining ");
    scanf("%d %d", &cy, &yoj);
    yos = cy - yoj;
    
    // 修正逻辑：满足条件赋值100，否则设为0（可根据需求调整），统一打印结果
    bonus = (yos > 3) ? 100 : 0;
    printf("Bonus = Rs. %d\n", bonus);
    
    return 0;
}

第二段代码的简化方案 #

简化方向 #

• 移除冗余变量，直接计算服务年限
• 合并条件判断与输出逻辑，缩短代码长度
• 保留代码可读性，避免过度简化导致维护困难

简化后的代码（兼顾简洁与可读性） #

#include <stdio.h>
int main()
{
    int cy, yoj;
    printf("Enter current year and year of joining ");
    scanf("%d %d", &cy, &yoj);
    
    // 直接计算并判断，无需额外变量存储服务年限
    if (cy - yoj > 3)
        printf("Bonus = Rs. 2500\n");
    
    return 0;
}

极致简洁版（利用逻辑短路特性） #

#include <stdio.h>
int main()
{
    int cy, yoj;
    printf("Enter current year and year of joining ");
    scanf("%d %d", &cy, &yoj);
    // 逻辑与短路：仅当条件为真时执行打印
    cy - yoj > 3 && printf("Bonus = Rs. 2500\n");
    return 0;
}

内容的提问来源于stack exchange，提问作者MD. Tanvir Ahmed Akash