whileの条件のつけかた

Question

while (DragonMAXHP>0 || DeathMAXHP>0) {

		DragonMAXHP -= DragonReceivedamage;

		DeathMAXHP -= DeathReceivedamage;

		Debug.Log (DragonMAXHP);
		Debug.Log (DeathMAXHP);
	}else break;

DragonMAXHPもしくはDeathMAXHPがどちらかが>0になると、その場で終了させたい
のですが、上記ではエラーがでてしまいます。。

Answer 1

while (DragonMAXHP>0 || DeathMAXHP>0)
この条件だと、どちらかが０以上なら実行されつづけるので、
どっちかが０になったら終わりとするなら
|| を && ですね。

while (DragonMAXHP>0 && DeathMAXHP>0)
これなら両方０以上じゃないと実行されないようになります。

Answer 2

else break; が余計なのではないでしょうか．

Comments

[user1041]

たしかにelse break;を削除すれば実行できますが、debuglogを見ると、
60,60,40,40,20,20,0,0 (HPの値)となっているので　どちらかが０になったら終わらせたいのですが、継続してるように思います。。

[swordone]

Debug.Logの段階では条件判定されていません．
DragomMAXHP，DeathMAXHP(以後，2値)が20の時
1.whileの条件判定→2値とも0より大きいのでtrue，ループ内に入る
2.2値が減算されて0になる
3.Debug.Logにより2値が出力される
4.先頭に戻り，whileの条件判定→2値とも0なのでfalse，ループは処理されず次に進む
こういう流れになります．0になったあとの条件判定で抜けてます．

[user1041]

なるほど。解決しました。

Answer 3

文法的には、コピペミスでないとしたら、else break;が不要です。

また、ロジック的には、c#のwhileは継続条件を指定しますので
「どちらかが0より大きくなったらループを抜ける」場合は、
「どちらも0以下である場合にループを継続する」という条件を指定する必要があります。
つまり
while(DragomMAXHP <= 0 && DeathMAXHP <= 0)
{
}
という感じです。
今の条件式全体に!演算子を付けても構いません。

変数の符号が分からないのでなんとも言えませんが、なんとなく不等号が逆の気が…。

Comments

[wakuwaku]

すいません。完全に勘違いしてました。

「変数の符号が分からないのでなんとも言えませんが、なんとなく不等号が逆の気が…。 」と書きましたが、
私の頭の中が逆でした。

目的の動作は else break; を削除すれば実現できますね。