Unityでwhile文を追加するとフリーズする。

Question

実現したいこと

Unityで敵に8秒間待たせた後、今のplayerの位置にスムーズに移動したい。
なお、8秒間待たせた後のその時のplayerの位置に移動させたいので、playerが移動すると、よけれるようにしたい。

発生している問題・分からないこと

while文を追加すると、8秒間立った後にフリーズします。

エラーメッセージ

フリーズ（タスクマネージャーなどで強制終了しなければならなくなっている状況）になっています。

該当のソースコード

using System.Collections;
using System.Collections.Generic;
using UnityEditor;
using UnityEngine;
using System.Threading.Tasks;

public class mobMove : MonoBehaviour
{
    public Kusuri kusuri; //ゲーム内で使う「クスリ」という物体のスクリプト
    public GameObject pl; //player
    public GameObject ka; //自分自身(mob)

    float startTime;
    [SerializeField] private float duration = 100;


    // Start is called before the first frame update
    void Start()
    {
        transform.position = new Vector2(14f, 4f); //モブの初期位置
        startTime = Time.time;
    }

    // Update is called once per frame
    async void Update()
    {
        if (kusuri.stage == 3)
        {
            await waitTime(8);
            int sw;
            sw = Random.Range(1, 2);
            Transform target = pl.transform;
            switch (sw)
            {
                case 1:
                    while (target.position != ka.transform.position) //←ここから
                    {
                        var t = (Time.time - startTime) / duration;
                        var xPos = Mathf.SmoothStep(ka.transform.position.x, target.position.x, t);
                        var yPos = Mathf.SmoothStep(ka.transform.position.y, target.position.y, t);
                        var zPos = Mathf.SmoothStep(ka.transform.position.z, target.position.z, t);
                        ka.transform.position = new Vector3(xPos, yPos, zPos);
                    }
                    break;
            }
            Debug.Log(sw);
        }
    }

    async Task waitTime(float time)
    {
        int T = (int)time * 1000;
        await Task.Delay(T);
    }
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

検索して出てきたのを見ても、自分の状況とは違うかったので参考になりませんでした。（本当はなったのかもしれないけど）

補足

while文の内容だけ残して実行してもフリーズはしないので、while文に原因があると思っています。（違うかもしれません。）

ぼくは、モブを動かす際にMathf.SmoothStep();を使っているので、ループしないと目的地まで移動できなくなります。

Answer 1

そのwhile条件の部分しか見てませんが、計算で求めた実数値同士が完全に一致することは希です。
「座標が完全に一致するまでループ」じゃなくて「距離がxx以下になるまでループ」にするのが普通でしょう。

ループが何時までも終わらないというバグはよくあります。
その場合は、継続条件/終了条件を構成する値がループ内でどう変化しているか、変化していないかを調べます。
今回だと、ループ前にtarget.positionの座標値を出力して、ループ内でka.transform.positionの座標値を出力する。

Comments

[UPUSCB]

回答ありがとうございます。
ループ前にDebug.Logで、ループ前にtarget.positionの座標値を出力して、ループ内でka.transform.positionの座標値を出力しようとすると、ループ前のtarget.positionの座標値を出力するプログラムが実行される前にフリーズしました。
ですが、「距離がxx以下になるまで」という条件式を参考にさせてもらいます。→ ```Vector2.Distance(target.position, ka.transform.position) > 0.1f```

Answer 2

そのスクリプトを入れると、まずまずループの意味がなくなってしまいます。

とのことですが、そもそもUpdate()自体、「何度も呼び出されるメインループの中の関数」のようなものなので、その考え自体が間違いです。

len_soukoさんの内容を踏まえると以下のようになります。
動作未確認ですが、これを参考にすればよいと思います。

    // Start()でもUpdate()でも使うので、メンバー変数化
    int sw;

    void Start()
    {
        transform.position = new Vector2(14f, 4f); //モブの初期位置
        startTime = Time.time;
        
        // 最初の1回のみ行う処理はStart()で行えばよい
        if (kusuri.stage == 3)
        {
            sw = Random.Range(1, 2);
        }
    }

    void Update()
    {
        if (kusuri.stage == 3)
        {
            // 8秒経過までは何もしない
            if (Time.time - startTime < 8f) return;
            
            Transform target = pl.transform;
            switch (sw)
            {
                case 1:
                    // 毎フレーム実行されるので、内部でループする必要なし
                    if (target.position != ka.transform.position)
                    {
                        var t = (Time.time - startTime) / duration;
                        var xPos = Mathf.SmoothStep(ka.transform.position.x, target.position.x, t);
                        var yPos = Mathf.SmoothStep(ka.transform.position.y, target.position.y, t);
                        var zPos = Mathf.SmoothStep(ka.transform.position.z, target.position.z, t);
                        ka.transform.position = new Vector3(xPos, yPos, zPos);
                    }
                    break;
            }
            Debug.Log(sw);
        }
    }

Comments

[UPUSCB]

回答ありがとうございます。
確かにそちらでも行けますが、敵モブのスクリプトなのでアニメーションを何個か対応させたいと思っています。スクリプトの構成を参考にさしてもらいます。

[fiveHundred]

> 確かにそちらでも行けますが、敵モブのスクリプトなのでアニメーションを何個か対応させたいと思っています。

このコードではアニメーションに対応できないということですか？
私はそう思いませんが。

[UPUSCB]

すみません。表現力がなかったですね。
「僕は、そのコードを参考にさしてもらってこっちのほうで多様なアニメーションに対応させます。」
まだ小学6年生なので、そのような風に読み取られたかもしれません。すみませんでした。

Answer 3

まず大前提として、MonoBehaviourを継承したクラスは毎フレームごとにUpdate()メソッドが呼び出されます
なので、その中で一瞬で終わるループなら大丈夫ですが、何フレームもまたがるループを行ってしまうのはルール違反です（多分フレーム更新が止まってしまうのかな？）

そこで用意された機能がコルーチンというものなので通常ではそちらを使うべしとなるところですが、今回の場合はそもそも8秒後に行いたいというだけなので、updateのところで現在時刻がstartTimeから8秒後未満の場合はupdateを抜ける（return;するだけ）で実現できると思います。
async void Update()の中の大事な場所

if (Time.time - startTime < 8秒) return;
// 8秒後以降にやりたいこと

Comments

[UPUSCB]

回答ありがとうございます。
そのスクリプトを入れると、まずまずループの意味がなくなってしまいます。こちらのほうの説明が足りていなかったようなので説明を補足しました。

Answer 4

皆さんの回答を参考にし、自分で改良したところフリーズはしなくなりました。ありがとうございます。コードはこちらです。↓

using System.Collections;
using System.Collections.Generic;
using UnityEditor;
using UnityEngine;

public class mobMove : MonoBehaviour
{
    public Kusuri kusuri;
    public GameObject pl;
    public GameObject ka;
    int sw; //メンバー変数にしました。
    bool Three = true; 

    Transform target;

    float startTime;
    [SerializeField] private float duration = 100;


    // Start is called before the first frame update
    void Start()
    {
        transform.position = new Vector2(14f, 4f);
        st_reset();
    }

    // Update is called once per frame
    void Update()
    {
        if(kusuri.stage == 3 && Three)
        {
            sT_reset();
            Three = false;
        }

        if (kusuri.stage == 3)
        {
            // 4秒経過までは何もしない
            if (Time.time - startTime < 4f) return; //4秒に変えました。
            attak();
        }
    }

    void sT_reset()
    {
        startTime= Time.time;
        target = pl.transform;
    }

    void attak()
    {
        sw = Random.Range(1, 2); //モブの攻撃方法の選択(1から1)

        switch (sw)
        {
            case 1:
                if (Vector2.Distance(target.position, ka.transform.position) > 0.1f)
                {
                    var t = (Time.time - startTime) / duration;
                    var xPos = Mathf.SmoothStep(ka.transform.position.x, target.position.x, t);
                    var yPos = Mathf.SmoothStep(ka.transform.position.y, target.position.y, t);
                    var zPos = Mathf.SmoothStep(ka.transform.position.z, target.position.z, t);
                    ka.transform.position = new Vector3(xPos, yPos, zPos);
                } 
                else
                {
                    sT_reset();
                }
                break;
        }
    }
    
    /*async Task waitTime(float time)
    {
        int T = (int)time * 1000;
        await Task.Delay(T);
    }*/
}

ここで一つ質問なんですが、元の文にもある通り、よけれるようにしたいので、playerが移動してもtargetには反映しないようにしているはずなんですが...ずっと移動してもついてきます。これを改良する方法を教えてください。

Comments

[UPUSCB]

改良する方法は別の記事で書いたほうがいいですかね。

[UPUSCB]

別の記事に書きました。