嗨，我帮你揪出了代码里几个导致if语句失效的关键问题，咱们一步步来解决：

1. action变量定义错误 #

你把action写成了数组int action[10];，但实际上你只需要一个存储玩家输入数字的普通整数。数组在scanf中会被当作指针处理，这会导致输入的值被写到错误的内存位置，根本没法正确判断玩家的选择。

修复：改成普通int变量：

int action; // 去掉数组定义

2. 用比较运算符==代替了赋值运算符= #

这是最核心的错误！你所有的操作都用了==（用来判断两个值是否相等），而不是=（给变量赋值）。比如：

(monsterhealth == monsterhealth - 40); // 这只是做了一次无意义的比较，完全不会修改怪物血量

这种写法只会计算右边的值和左边比较，结果是个布尔值，但你没有保存这个结果，等于白写了一行代码。

修复：把所有==改成=，或者用更简洁的复合赋值运算符：

• 攻击怪物：monsterhealth = monsterhealth - 40; 或 monsterhealth -= 40;
• 降低受击伤害：damage = damage - 30; 或 damage -= 30;
• 恢复生命值：health = health + 100; 或 health += 100;

3. else语句的语法完全错误 #

你写的else (action == 3); {(health == health + 100);};不符合C语言语法规则。else后面要么跟if条件（即else if(...)），要么直接跟代码块，不能先写条件再加个分号。

修复：改成标准的else if结构：

else if(action == 3) {
    health += 100;
}

4. 战斗逻辑顺序不合理 #

现在的代码是玩家刚选完操作，立刻就扣血，相当于怪物在玩家行动前就发起攻击了，不符合正常对战逻辑。而且防御效果会永久生效——玩家防御一次后，怪物永远只打20血，这可能不是你想要的设计。

优化：

• 把扣血步骤放在玩家行动之后，代表怪物的反击
• 每次循环重置damage为50，让防御效果只生效一轮
• 给生命值加个上限（比如不超过初始的100），避免回血过度

完整修复后的代码 #

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main() {
    int monsterhealth = 100;
    int health = 100;

    while(health > 0 && monsterhealth > 0) {
        int action;
        int damage = 50; // 每次循环重置默认伤害

        printf("Player has %d health \n", health);
        printf("Monster has %d health\n", monsterhealth);
        printf("Act: attack[1] | defend[2] | potion[3]\n");
        scanf("%d", &action); // 注意这里要加&，scanf需要变量地址才能修改值

        if(action == 1) {
            monsterhealth -= 40;
            printf("You attacked the monster for 40 damage!\n");
        } else if(action == 2) {
            damage -= 30;
            printf("You defended! Monster's damage will be reduced by 30 this round.\n");
        } else if(action == 3) {
            health += 100;
            if(health > 100) health = 100; // 限制生命值不超过初始值
            printf("You used a potion! Health restored to %d.\n", health);
        } else {
            printf("Invalid action! Monster attacks you directly!\n");
        }

        // 怪物没死才会攻击玩家
        if(monsterhealth > 0) {
            health -= damage;
            printf("Monster attacks you for %d damage!\n", damage);
        }

        // 修正生命值为非负数，显示更合理
        if(health < 0) health = 0;
        if(monsterhealth < 0) monsterhealth = 0;
    }

    // 战斗结束提示
    if(health <= 0) {
        printf("You died! Game over.\n");
    } else {
        printf("You defeated the monster! Victory!\n");
    }

    return 0;
}

另外别忘了scanf("%d", &action);里的&——你之前的代码不管是数组还是普通变量都没加这个符号，这也是输入无法正确存储的重要原因！

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