Java CUIでRPGの戦闘処理

Question

前提・実現したいこと

PG初心者です。
JavaでRPGの戦闘風プログラムを作成しているのですが
現在スキャナでコマンド入力をしその数字が7なら勇者１を、8なら勇者2を回復させるメソッドを呼んで回復処理をしています。
そこで、どちらかの勇者のHPが0の時、回復コマンドを入力しても、その勇者は回復しないようにしたいです。

該当のソースコード

言語はJavaです。

//メインメクラスで、Heroクラスから回復メソッドを呼ぶ処理
//jはコマンド入力の値です
else if(j==7) {
h.heal(h);
}

else if(j==8) {
h.heal(h2);
}

//Heroクラス内にある回復メソッド
void heal(Hero h) {
h.hp+=100;
System.out.println(h.name+"は100回復した");

}

試したこと

else if(j==7) {
if(h.hp>=0) {
h.heal(h);
}
}

else if(j==8) {
if(h2.hp>=0) {
h.heal(h2);
}
}

上記の様にhpが0でない場合のみ回復メソッドを呼ぶ分岐を書いてみたのですが
コマンド入力をすると回復してしまいます。

Answer 1

上記の様にhpが0でない場合のみ回復メソッドを呼ぶ分岐を書いてみた

それならhp >= 0ではなくhp > 0と書くべきですね。

---

healメソッドを書き換えてしまえば良いのではないでしょうか。

void heal() {         
    if(this.hp == 0) {
        System.out.println(this.name + "はもう死んでいる");
        return;
    }

    this.hp+=100;
    System.out.println(this.name+"は100回復した");
}

ついでに引数も不要だと思ったので取り払いました。
nameやhpをprivateフィールドに出来るというメリットもあります。

Comments

[k.k.0000]

ご丁寧な回答とアドバイスありがとうございます!
引数は、どちらの勇者を回復させるかを判別するために記述していたのですが、なくしてしまっても判別できるのですか？

[LouiS0616]

hを回復させたければh.heal()とすれば良いですし、h2を回復させたければh2.heal()とすれば良いです。

[k.k.0000]

ありがとうございます！
基本的なことがあいまいになっていました、確実に習得していけるようにします。