C语言程序循环与if语句失效问题求助

我帮你梳理下代码里的核心问题，然后给出完整的修复方案：

1. if语句失效的关键原因：用了赋值运算符而非比较运算符 #

你所有if (paycode = 1)这类写法都是错误的——=是赋值运算符，它会把右边的数值赋给paycode，并且整个表达式的结果永远为真。这就导致代码会依次执行每个if/else if分支（每次赋值都会改变paycode的值，让后续条件也成立），所以你会看到所有printf语句都被输出。

修复方法：把所有判断条件里的=改成比较相等的==运算符，比如：

if (paycode == 1)

同理，把paycode = 2、paycode = 3、paycode = 4都改成paycode == 2、==3、==4。

2. 循环无法重复计算的原因：错误的循环条件 #

你原来的循环条件while (paycode >= 0 && paycode != 1)有两大问题：

• 当用户输入1（经理类型）时，paycode !=1不成立，循环直接退出，没法继续计算下一位员工
• 初始paycode设为0，虽然第一次能进入循环，但逻辑不够直观，不符合“先输入员工类型，再判断是否结束”的流程

修复方法：改用do-while循环结构（确保至少执行一次输入操作），并把循环条件改为while (paycode != -1)，这样用户输入-1时才会退出循环，其他情况都能重复计算。

额外的隐藏bug：小时工薪资的硬编码错误 #

你代码里小时工的正常工时薪资写了40 * 16.78，这是硬编码了时薪，不管用户输入多少时薪，正常工时的薪资都是固定值，明显不符合需求。我已经在修复代码里改成了用用户输入的worker_salary计算。

完整修复后的代码 #

#include <stdio.h>

int main() {
    int paycode = 0; 
    double manager_salary; 
    double worker_salary; 
    double worker_hours; 
    double ot; 
    double worker_wage; 
    double sales; 
    double commission; 
    double pieces; 
    double piece_wage; 
    double pieceworker; 

    do {
        printf("\nEnter the employee paycode (1-4) (-1 to end): ");
        scanf_s("%d", &paycode);

        if (paycode == -1) {
            break; // 提前退出循环，避免执行无效的类型判断
        }

        if (paycode == 1) {
            printf("\nManager Selected");
            printf("\nEnter weekly Salary: ");
            scanf_s("%lf", &manager_salary);
            printf("\nManagers Pay is $%.2f", manager_salary);
        } else if (paycode == 2) {
            printf("\nHourly Worker Selected");
            printf("\nEnter the hourly salary: ");
            scanf_s("%lf", &worker_salary);
            printf("\nEnter the total hours worked: ");
            scanf_s("%lf", &worker_hours);
            
            if (worker_hours > 40.00) {
                ot = (worker_hours - 40.00) * worker_salary * 1.5;
                worker_wage = 40 * worker_salary + ot;
            } else {
                ot = 0.00;
                worker_wage = worker_hours * worker_salary;
            }
            printf("\nHourly Worker's Pay is $%.2f", worker_wage);
        } else if (paycode == 3) {
            printf("\nCommission Worker Selected");
            printf("\nEnter gross weekly sales: ");
            scanf_s("%lf", &sales);
            commission = sales * 0.057 + 250;
            printf("\nCommission Worker's Pay is $%.2f", commission);
        } else if (paycode == 4) {
            printf("\nPieceworker Selected");
            printf("\nEnter number of pieces: ");
            scanf_s("%lf", &pieces);
            printf("\nEnter wage per piece: ");
            scanf_s("%lf", &piece_wage);
            pieceworker = pieces * piece_wage;
            printf("\nPieceworker's pay is $%.2f", pieceworker);
        } else {
            printf("\nInvalid paycode! Please enter 1-4 or -1 to end.");
        }
    } while (paycode != -1);

    return 0;
}

修复后的效果 #

• if语句现在能正确匹配员工类型，只会执行对应分支的代码
• 计算完一位员工薪资后，程序会自动回到输入paycode的步骤，直到用户输入-1才退出
• 小时工薪资计算不再有硬编码问题，完全基于用户输入的参数计算
• 新增了无效paycode的提示，提升了程序的健壮性

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