Java文本RPG战斗系统HP更新仅生效一次问题求助

问题根源 #

1. 参数传递错误：你调用battlesystem时，提前一次性计算了playerHPUpdate和enemyHPUpdate，这两个值是初始状态下的固定结果，后续攻击不会重新计算，导致每次显示的都是第一次的HP数值。
2. 方法设计缺陷：battlesystem接收的是零散数值而非对象引用，无法实时修改和获取角色/敌人的最新HP状态。
3. Setter方法不规范：你的setHp返回值设计容易造成逻辑混淆，不符合Java标准Setter的设计习惯。

解决方案 #

• 直接将Characters和Enemies对象作为参数传入battlesystem，让方法内部可以直接操作对象的属性，实时更新HP。
• 移除提前计算HP更新值的冗余逻辑，改为每次攻击时实时计算并更新对象HP。
• 优化setHp为标准的void类型（可选，但更符合编码规范）。

修改后的代码示例 #

1. 优化对象类的setHp方法（推荐）

private int hp;
// 其他属性...

public int getHp() {
    return hp;
}

// 改为标准void类型Setter
public void setHp(int hp) {
    this.hp = hp;
}

2. 重构战斗系统方法

// 初始化角色与敌人
Characters Fiona = new Characters(100, 10, 50, "Fiona");
Enemies Goblin = new Enemies(50, 10, 10, "Goblin");

// 直接传入对象调用战斗系统
battlesystem(Fiona, Goblin);

// 重构后的战斗系统方法
public static void battlesystem(Characters player, Enemies enemy) {
    Scanner sc1 = new Scanner(System.in);
    System.out.printf(" Battle start! %n");

    // 速度判定：敌人先攻
    if (player.getSpd() <= enemy.getSpd()) {
        int enemyDmg = enemy.getDmg();
        int newPlayerHp = player.getHp() - enemyDmg;
        player.setHp(newPlayerHp);
        
        System.out.printf(" Goblin attacks! %n");
        System.out.printf(" You lose %d HP! %n", enemyDmg);
        System.out.printf(" You have %d HP left!%n", player.getHp());
    } 
    // 玩家先攻
    else {
        System.out.printf(" %n What is your Strategy? %n");
        boolean battleRunning = true;
        
        while (battleRunning) {
            System.out.printf(" %n%n (A)ttack %n (D)efend %n (I)tems %n (S)tats %n (F)lee %n");
            String choice = sc1.nextLine();
            
            switch (choice.toUpperCase()) {
                case "A":
                    int playerDmg = player.getDmg();
                    int newEnemyHp = enemy.getHp() - playerDmg;
                    enemy.setHp(newEnemyHp);
                    
                    System.out.printf(" You decide to attack the Goblin! %n");
                    System.out.printf(" %s attacks!", player.getName());
                    System.out.printf(" %s loses %d HP%n", enemy.getName(), playerDmg);
                    System.out.printf(" Goblin has %d HP left!%n", enemy.getHp());
                    // 可在此添加敌人反击、战斗结束判定逻辑
                    break;
                case "F":
                    battleRunning = false;
                    System.out.printf(" You fled from the battle!");
                    break;
                // 其他选项逻辑可在此补充
            }
        }
    }
    sc1.close();
}

关键修改说明 #

• 直接传递对象引用，方法内部可实时获取和修改角色/敌人的HP状态，每次攻击都会基于当前最新HP计算新值。
• 移除了冗余的setHPChar/setHPEnem静态方法，逻辑更简洁直观。
• 用battleRunning布尔变量替代原有的int i < 1循环条件，逻辑可读性更强。

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