while文で無限ループが起きる

Question

###<実現したいこと>
クリックされるとcount3nextという関数を実行したい。

<エラー>

その処理(エラーが起きている処理)までの処理はうまくいくが、その処理
(
public void count3()
{
while(!startflag&&gameStartB)
{
if (Startb)
{
count3next();
break;
}
}

}

)
が実行されたときにUnityがフリーズする。

<ソースコード>

using System;
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using UnityEngine.SceneManagement;


public class GameScript : MonoBehaviour
{
    // ロングタップ判定で使用する変数
    float longTapTime = 2.0f;       // 実行直後は、longTapTimeを過ぎると、ロングタップとみなされる
                                    // 1回目の問題が終わると、タップする目標時間
    float nowTapTime;               // タップし始めてから現在までの時間
    float safeTime = 1.0f;          // 目標時間に対して許容される誤差
    bool isLongtap;                 // ロングタップ中かどうか(タップする時間がlongTapTimeを越えるとtrueになる)
    bool isLongtapclear;            // ロングタップが終わったか

    // ゲームで使用する変数
    int challengecount;             // 挑戦回数
    int clearcount=0;          // クリアした回数
    int level;                      // 難易度
    bool startflag = false;         // ゲーム開始されていたらtrue。開始前(カウントダウン中も含む)ならfalse;
    public string count;            // 秒読みで表示する値

    // デバッグ用表示テキスト
    private Text countText;         // カウントダウン表示
    private Text Text1;             // longTapTime表示(ロングタップとみなされる時間)
    private Text Text2;             // safeTime表示(タップ時間の許容誤差)
    private Text Text3;             // clearCount表示(クリアした回数)
    private Text Text4;             // nowTapTime(現在のタップ時間)            
    private Text Text5;             // nowTapTime2(判定後のタップ時間)
    private Text Text6;             // 説明のText
    public string Text1s;           // longTapTime
    public string Text2s;           // safeTime
    public string Text3s;           // clearCount
   
    public Slider slider;
    public AudioClip clearClip;
    public AudioClip outClip;
    public AudioSource audioSource;
    public bool Startb=false;
    public bool gameStartB = false;

    // Start is called before the first frame update
    void Start()
    {

        // 表示するTextのセット
        countText = GameObject.Find("CountText").GetComponent<Text>();
        Text1 = GameObject.Find("LongtapTimeText").GetComponent<Text>();
        Text2 = GameObject.Find("SafeTimeText").GetComponent<Text>();
        Text3 = GameObject.Find("clearcount").GetComponent<Text>();
        Text4 = GameObject.Find("NowTapTime").GetComponent<Text>();
        Text5 = GameObject.Find("結果text").GetComponent<Text>();
        
   
        level = PlayerPrefs.GetInt("level");
        kabe();                                     // レベルに合わせて許容誤差を計算
        count3next();                                   // カウントダウン開始

        // Textの初期値
        Text1s = longTapTime.ToString();
        Text2s = safeTime.ToString();
        Text3s = clearcount.ToString();

    }

    // Update is called once per frame
    void Update()
    {
        Text1.text = "目標："+Text1s;
        Text2.text = "許容範囲：" + Text2s;
        Text3.text = "clearcount: " + Text3s;
        
        if (startflag)
        {
            gameStartB = true;
            // ゲームが開始されていたら....
            Text1.text = "目標：" + Text1s;
            Text2.text = "許容範囲：" + Text2s;
            Text3.text = "clearcount: " + Text3s;
            if (Input.GetMouseButton(0))
            {
                slider.maxValue = longTapTime;
                slider.value = nowTapTime;
                // ボタンが押されていたら...
                isLongtapclear = false;         // ロングタップが終わっていない状態にして
                Text1.text = "目標：" + Text1s;
                Text2.text = "許容範囲：" + Text2s;
                Text3.text = "clearcount: " + Text3s;
                nowTapTime += Time.deltaTime;   // 秒数をカウント
                Text4.text = nowTapTime.ToString();

                // タップし続けた時間が規定値を超えたらロングタップとして扱う
                if (nowTapTime >= longTapTime && !isLongtap)
                {
                    isLongtap = true;                       // ロングタップしたフラグを立てる             // Prefabの作成
                    Debug.Log("Long Tap");
                }
                
                        

            }
            else if (Input.GetMouseButtonUp(0))
            {
                // 「ボタンが離されたら、時間を測って、結果表示」をUpdate()内で行うため、
                //  WaitASecond()から以下の部分を移動
                
                if (longTapTime-safeTime<nowTapTime && longTapTime + safeTime > nowTapTime && !isLongtapclear)
                {
                    audioSource.PlayOneShot(clearClip);
                    isLongtapclear = true;
                    Debug.Log("tapclear!");
                    longTapTime = UnityEngine.Random.Range(2, 6);
                    Text1.text = Text1s;
                    clearcount += 1;
                    Text3.text = "clearcount: " + Text3s;
                    Text1s = longTapTime.ToString();
                    Text2s = safeTime.ToString();
                    Text3s = clearcount.ToString();
                    challengecount = challengecount + 1;
                    if (challengecount > 10)
                    {
                        PlayerPrefs.SetInt("clearcount", clearcount);
                        SceneManager.LoadScene("ResultScene");
                    }
                    startflag = false;
                    count3();
                    
                }
                else
                {
                    audioSource.PlayOneShot(outClip);
                    Debug.Log("OUT");
                    challengecount = challengecount + 1;
                    if (challengecount > 10)
                    {
                        PlayerPrefs.SetInt("clearcount", clearcount);
                        SceneManager.LoadScene("ResultScene");
                    }
                    count3();
                }


                // クリックを終えたら初期化
                nowTapTime = 0;
                isLongtap = false;
            }
            if(Input.GetMouseButtonUp(0)&&!isLongtap)
            {
                
                startflag = false;
                count3();
            }
        }
        if (Input.GetMouseButtonDown(0) && !startflag&&gameStartB)
        {
            Startb = true;
        }
        else if(!startflag&&gameStartB)
        {
            Startb = false;
        }
    }
    public void kabe()
    {
        safeTime = 1.0f / (level + 1);
        safeTime *= 1.5f;
        Debug.Log(safeTime);
    }
    public void count3()
    {
        while(!startflag&&gameStartB)
        {
            if (Startb)
            {
                count3next();
                break;
            }
        }
         
        
    }
    public void count3next()
    {
        count = "3";
        countText.text = count;
        Invoke("count2", 1);
    }
    public void count2()
    {
        count = "2";
        countText.text = count;
        Invoke("count1", 1);
    }
    public void count1()
    {

        count = "1";
        countText.text = count;
        Invoke("start", 1);
    }
    public void start()
    {
        count = "start!";
        countText.text = count;
        Invoke("startsetting", 1);
    }
    public void startsetting()
    {
        startflag = true;

    }
    //IEnumerator WaitASecond()
    //{
    //    yield return new WaitForSeconds( 1f );

    //    print( ( longTapTime - safeTime < nowTapTime ) + " : " + ( longTapTime + safeTime > nowTapTime ) + " : " + !isLongtapclear );
    //    if ( ( longTapTime - safeTime < nowTapTime )
    //        && ( longTapTime + safeTime > nowTapTime ) && !isLongtapclear ) {
    //        isLongtapclear = true;
    //        Debug.Log( "tapclear!" );
    //        longTapTime = UnityEngine.Random.Range( 2, 6 );
    //        Text1.text = Text1s;
    //        clearcount += 1;
    //        Text3.text = "clearcount: " + Text3s;
    //        Text1s = longTapTime.ToString();
    //        Text2s = safeTime.ToString();
    //        Text3s = clearcount.ToString();

    //    }
    //    else {
    //        Debug.Log( "OUT" );
    //    }
    //}
    //IEnumerator MyLoop()
    //{
    //    for ( isLongtap = false; ; ) {
    //        yield return new WaitForSeconds( 0.2f );
    //    }
    //    StartCoroutine( "WaitASecond" );
    //}
}

Comments

[y_waiwai]

その処理とはどの処理？
無限ループとはコードのどの部分で起きる？
あまりにも説明不足ですぜ

[Hamburger_Man]

すみません。編集します。

Answer 1

演算子の優先順位
https://docs.microsoft.com/ja-jp/dotnet/csharp/language-reference/operators/#operator-precedence

演算子の優先順位に従って&&より先に!が 処理されるから意図せず while (true && true)と評価されている可能性があります。

演算子の優先順位を変更するには先に評価させたい式を()で括ります。

例えば A && B の結果全体を真偽反転させたいのであれば !(A && B) と記述します。

（あくまで演算子の優先順位を理解しようって話で、そう書いたからといってあなたの意図したとおりに動くかはわかりません）

Answer 2

このようにすればフリーズせずに原因を絞れると思います。

public void count3()
{
	int brk = 100;
	while(!startflag&&gameStartB)
	{
		if (Startb)
		{
			count3next();
			break;
		}

		brk--;
		if( brk==0 ){
			Debug.LogError( "startflag:" + startflag + "\ngameStartB:" + gameStartB + "\nStartb:" + Startb );
			Debug.Break();
			break;
		}
	}
}

Comments

[Hamburger_Man]

while文の中で Debug.LogErrorを実行してみて、コンソールを見ましたが、startflagがfalse,gameStartBがtrueと出ました。それであってるのですが、なぜ無限ループするのででしょうか?

[i_zna]

while文は()の中がtrueである限り処理を繰り返します。Startbで脱出させたいみたいですが、Startbがfalseだったときの処理が何もないので永久に空回りします。

[Hamburger_Man]

すみません。説明が足りませんでした。trueになるまで待つというスクリプトを書きたくて、このようなスクリプトを書きました。

[i_zna]

Unityが処理の終了を待っているからです。
Update()は毎フレーム呼ばれているので、待つ目的ならそもそもループ文は必要ありません。
失礼ですがかなり基礎的なことですので、このような複雑なコードや非同期処理に触れるのはまだ早いかと…

[Hamburger_Man]

ということは、Update内でループ分は書かなくてよいといことですね。