だいたい以下の様にしているかと思います。
using System;
public class Monster {
private int hp = 100;
public int HP(){
return hp;
}
}
class Sample{
static void Main(){
Monster monster = new Monster();
int hp = monster.HP();//モンスターの現在のHPを取り出す
int damage = 20;
while(hp > 0){
Console.WriteLine("モンスターの現在のHPは{0}です。", hp);
hp -= damage;//ダメージの計算
Console.WriteLine("ダメージを受けてHPは{0}になりました。", hp);
}
}
}
値をメソッド(monster.HP()
)で取り出していることから
外部のメソッドで実行していると思いますが、
HPはmonster
の属性値ですから書き戻す必要があります。
例えば以下の様に属性値をセットするメソッドを追加します。
using System;
public class Monster {
private int hp = 100;
public int HP(){
return hp;
}
public void HP(int value){
hp = value;
}
}
class Sample{
static void Main(){
Monster monster = new Monster();
int hp;
int damage = 20;
do {
hp = monster.HP();//読み出し
Console.WriteLine("モンスターの現在のHPは{0}です。", hp);
hp -= damage;
monster.HP(hp);//書き出し
Console.WriteLine("ダメージを受けてHPは{0}になりました。", hp);
} while(hp > 0);
}
}
こうすると、モンスターの値を取り出し、ダメージの計算をして、書き戻すというループにできます。
C#では、こういう属性値でprivateにしておいて直接値を扱わせたくない(のでメソッド経由でやりとりする)ような場合にプロパティとしての構文があります。
プロパティを使って同じように書くと以下の様になります。
using System;
public class Monster {
private int hp = 100;
public int HP{
get {
return hp;
}
set {
if(value < 0){
hp = 0;
} else {
hp = value;
}
}
}
}
class Sample{
static void Main(){
Monster monster = new Monster();
int hp;
int damage = 20;
do {
hp = monster.HP;
Console.WriteLine("モンスターの現在のHPは{0}です。", hp);
hp -= damage;
monster.HP = hp;
Console.WriteLine("ダメージを受けてHPは{0}になりました。", hp);
} while(hp > 0);
}
}
外部からは直接属性値を扱っているように見えるので、単純に以下の様に書けます。
Monster monster = new Monster();
int damage = 20;
do {
Console.WriteLine("モンスターの現在のHPは{0}です。", monster.HP);
monster.HP -= damage;
Console.WriteLine("ダメージを受けてHPは{0}になりました。", monster.HP);
} while(monster.HP > 0);
個人的にはこういうのを外部でやるより
monster.damage(damege);
のような呼び出しでやる方がいいような気がします。
ref
とありますが、質問中のコードからはメソッド等の設計が読み取れないのでもう少し加筆してください。int monstarLEFTHP = (monstarHP -= damage);
とかint monstarLEFTHP = monstarHP;... monstarLEFTHP -= damage;
とか?monstar.HP();
の様にしてmonstar
がHP自体をプロパティなりなんなりで持っていて取り出しているなら、一回の戦闘の結果のHPをmonstar
に書き戻してやる必要があるかと思います。そうすればHPを取り出した時に前回の値を取り出すことになりますよね。この関数内で繰り返し戦闘の解決をするような場合は、whileなりforなりで条件を満たす間、あるいは無限ループにしておいて、条件が成立したらbreak
すればいいと思います。(その後ステータスの書き戻し)