Java中fight方法的while循环未执行，求排查原因

核心问题1：while循环条件完全写反 #

你当前的while循环条件是yourHP <= 0 || theirHP <= 0，意思是只有任意一方血量归零或更低时才会进入循环，但初始化时yourHP是5-30的正数，theirHP是传入的有效血量（默认是正数），所以循环条件从一开始就不成立，自然不会执行。

正确逻辑是当双方血量都大于0时持续战斗，条件应修改为：

while (yourHP > 0 && theirHP > 0)

核心问题2：coinFlip的取值范围错误 #

你用ThreadLocalRandom.current().nextInt(1, 2 + 1)生成随机数，该方法遵循左闭右开规则，实际只会生成1或2，永远不会出现0，导致第二个if (coinFlip == 0)分支永远无法触发。

如果想让coinFlip取0和1，应改为：

int coinFlip = ThreadLocalRandom.current().nextInt(0, 2);

（或者保持1和2的取值，把第二个判断改成coinFlip == 2）

修复后的完整代码 #

public void fight(String person, int theirHP){
    int yourHP = ThreadLocalRandom.current().nextInt(5, 30 + 1);
    System.out.println("Your HP: " + yourHP + "\nTheirHP: " + theirHP);
    // 生成0或1的随机数，决定先手顺序
    int coinFlip = ThreadLocalRandom.current().nextInt(0, 2);
    int damage;
    System.out.println(coinFlip);
    
    if (coinFlip == 1) {
        // 双方都存活时继续战斗
        while (yourHP > 0 && theirHP > 0) {
            damage = ThreadLocalRandom.current().nextInt(1, 3 + 1);
            theirHP -= damage;
            System.out.println("Their HP: " + theirHP + "\nYour HP: " + yourHP);
            // 攻击后判断对方是否已死亡，避免无效攻击
            if (theirHP <= 0) break;
            
            damage = ThreadLocalRandom.current().nextInt(1, 3 + 1);
            yourHP -= damage;
            System.out.println("Their HP: " + theirHP + "\nYour HP: " + yourHP);
        }
    }
    if (coinFlip == 0) {
        while (yourHP > 0 && theirHP > 0) {
            damage = ThreadLocalRandom.current().nextInt(1, 3 + 1);
            yourHP -= damage;
            System.out.println("Their HP: " + theirHP + "\nYour HP: " + yourHP);
            // 攻击后判断自己是否已死亡
            if (yourHP <= 0) break;
            
            damage = ThreadLocalRandom.current().nextInt(1, 3 + 1);
            theirHP -= damage;
            System.out.println("Their HP: " + theirHP + "\nYour HP: " + yourHP);
        }
    }
}

额外优化建议 #

• 攻击后增加死亡判断，避免一方死亡后另一方仍继续掉血，逻辑更贴合实际战斗场景。
• 可将重复的战斗逻辑提取为独立方法，减少代码冗余，提升可维护性。

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