質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.79%

whileの条件のつけかた

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 1,066

score 34

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


            DragonMAXHP -= DragonReceivedamage;

            DeathMAXHP -= DeathReceivedamage;

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


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

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

0

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/07/01 22:53

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

    キャンセル

  • 2015/07/01 23:27

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

    キャンセル

  • 2015/07/02 01:36 編集

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

    キャンセル

0

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


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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/07/01 22:35

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

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

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

    キャンセル

0

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.79%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る