C# while文の条件式について

Question

do{
    targetplayer = originallist[UnityEngine.Random.Range(0,originallist.Count)];
    debugcount++;
    Debug.Log("targetplayer"+targetplayer.PlayerName+"   attacker   "+attacker.PlayerName);
    Debug.Log(targetplayer.Team+""+targetplayer.isLive+"     "+attacker.Team+attacker.isLive);
    if(debugcount > 50){
        Debug.Log("無限ループ");
            break;
    }
}while((targetplayer.isLive == false) || (targetplayer.Team == attacker.Team))

do　while文で上記のような条件式を書いたのですが、targetplayer.isLive == trueで且つtargetplayer.Team != attacker.Teamのような条件の時でもwhile文から抜けずループしてしまうことがあります。
条件式の書き方でおかしいところがあるでしょうか？。

デバッグで確認したときの画像です。

分かりにくいですがdoの中身とデバックの結果を追記しました。

Comments

[gentaro]

実際にそのような動作をしている決定的な場面を提示できますでしょうか。
ループ文の中でConsole.WriteLineとか（Debug.WriteLineでもいいけど）でその変数の値を出力しつづけ、確実にループを抜けるはずなのになかったと判断できるエビデンスが欲しいです。

[gentaro]

変数の値と併せて
(targetplayer.isLive == false) と
(targetplayer.Team == attacker.Team)
の評価結果を出力した方がいいかも。

まぁ要するに「勘違いじゃない」という確認をどの程度しているのかわからんと、何とも言えない。

[gentaro]

クッソわかりにくい…

targetplayer.isLive == true
targetplayer.Team == 1
attacker.Team == 2

の時に
((targetplayer.isLive == false) || (targetplayer.Team == attacker.Team))

を満たさないから抜けるはずが次のループへ進んでる、ということかな。
まぁそれなら確かにおかしい気はする。

bool b1 = !targetplayer.isLive;
bool b2 = (targetplayer.Team == attacker.Team)
として、ログにb1とb2の評価結果を個別に出し、
while (b1 || b2)
という条件にしたらどうですか？

[Tomo0225]

分かりにくくて申し訳ありません。読んでいただきありがとうございます。
条件式の中身を個別に評価してみるんですね。やってみたいと思います。

[yudedako67]

もうひとつ、本当に抜けてないのか確認するためにループを抜けた場合に抜けたことがわかるようなログを出してみてほしいです。

Answer 1

皆さん回答ありがとうございます。
gentaroさんの
bool b1 = !targetplayer.isLive;
bool b2 = (targetplayer.Team == attacker.Team)
として、ログにb1とb2の評価結果を個別に出し、
while (b1 || b2)
として評価したら解決できました。
ありがとうございました。

Comments

[gentaro]

解決したのならそれはいいんだけど、コメントした内容ではロジックは代わってないので、それだけで解決するはずないんだけど。
実際にログはどのようになり、何が原因で、どのような修正をしたんですか？

Answer 2

|| という演算子は、かつ、ではなく、または、です。

VisualStudioを使ってるなら、ステップ実行させることができるので、その抜けない場合の条件式の値を確認してみてはどうでしょう

＃カッコの数があわないのはよこにおいといて

Comments

[Tomo0225]

回答ありがとうございます。
||はまたはという意味なのは知っています。この場合、(targetplayer.isLive == true)と(targetplayer.Team != attacker.Team)の２つの条件を満たすときに繰り返し処理を抜けるようにしたいので||を使っています。
実際に処理中でＤｅｂｕｇ．Ｌｏｇを使って値を確認したのですがtargetplayer.isLive == trueでtargetplayer.Team != attacker.Teamの時でも抜けれないときがあります。
条件式以外のところに問題があるかもしれませんが....

[y_waiwai]

具体的にどうLOGを出してどういう値を確認したんでしょうか
そこんとこを提示しましょう