移動しているのにNavMeshAgent.velocityが初回だけ０になる

Question

#やりたいこと
Unityで2Dゲームを作っています。
NevMeshを使ってキャラクター（エージェント）を動かす際に、スプライトアニメーションを再生したいのですが、初回の移動だけうまくいきません。

#質問
NaMeshを使ってキャラクターを移動させた際、初回だけ、移動するのにNavMeshAgent.velocityが(0.0, 0.0, 0.0)になってしまいます。
２回目以降の移動では(1.0, 0.0, -0.2)という風に、値が入ってきます。
velocityだけではなくhasPathなども試してみたんですが、同様に初回だけfalseになってしまいます。
ちゃんと移動はしているのにパスがないって、どういうこと？って感じなのですが、こういうものなのでしょうか？
NavMeshを初めて使うので、何か変なところがないか見ていただけますでしょうか。

#該当するコード
キャラクターをドラッグして、MouseUpしたときにルート上なら、一旦ドラッグ開始地点に戻してから移動させてAnimatorを操作（Walk())します。
ドラッグ部分は問題ないので省略しています。当たり判定も問題ありません。

変数start_positionはドラッグ開始時のpositionを代入しています。
ドラッグしたキャラクターを最初の位置に戻すためです。

Characterクラス

	Animator animator;
	NavMeshAgent agent;
	Vector3 start_position;

	void Update ()
	{
		if (Input.GetMouseButtonUp (0)) {
			OnStopDrag ();
		}
	}

	void OnStopDrag ()
	{	
			Ray ray = Camera.main.ScreenPointToRay (Input.mousePosition);
			RaycastHit hit_info = new RaycastHit ();
			float max_distance = 100f;
			int layer = LayerMask.NameToLayer ("Route");
			int layer_mask = 1 << layer;
			bool is_hit = Physics.Raycast (ray, out hit_info, max_distance, layer_mask); 

			transform.position = start_position;
			if (is_hit) {
				if (hit_info.transform.name.IndexOf ("DirectionChanger") != -1) {
					//移動開始
					agent.SetDestination (hit_info.point);
					StartCoroutine (Walk ());
				}
			}
	}

	IEnumerator Walk ()
	{
		animator.SetBool ("attack", false);
		animator.SetBool ("stop", false);

		while (true) {	
			Debug.Log (agent.velocity);

			if (agent.velocity.z > 0.5f) {
				animator.SetInteger ("direction", 1);
			} else if (agent.velocity.z < -0.5f) {
				animator.SetInteger ("direction", 2);
			} else if (agent.velocity.x > 0.5f) {
				animator.SetInteger ("direction", 3);
			} else if (agent.velocity.x < -0.5f) {
				animator.SetInteger ("direction", 4);
			}
			now_direction = animator.GetInteger ("direction");

			if (agent.velocity == Vector3.zero) {
				animator.SetBool ("stop", true);
				yield break;
			}
			yield return null;
		}
	}

#コンポーネント

#出力結果
初回 (0.0, 0.0, 0.0)
Walkコルーチンはvelocityが０になったら抜けるようにしてあるので１度しか呼び出されません。
もちろん、アニメーションは変化しません。
移動はします。

２回目(1.0, 0.0, -0.2) ・・・・・ (0.0, 0.0, 0.0)
数値が変化しながら最後は０で終わります。
アニメーションも意図した通りに動いてくれています。

情報が不足していたら申し訳ございません。
よろしくお願いいたします。

Answer 1

経路探索が完了しない内にwhileに入ってしまい、そのまま抜けているのではないでしょうか。
Unity - スクリプトリファレンス: NavMeshAgent.SetDestination
上記ページを確認の上、agent.pathPending が false になるのを待ってから以後の処理を開始してみてください。

Comments

[nico29]

早速のご返信ありがとうございます。
以下のように変更してみたのですが、やはり初回だけ０になってしまいます。
やり方、間違ってますでしょうか？

```C#
//移動開始
bool request = agent.SetDestination (hit_info.point);
if (request) {
StartCoroutine (Walk ());
}
```

```C#
IEnumerator Walk ()
{
animator.SetBool ("attack", false);
animator.SetBool ("stop", false);

yield return !agent.pathPending;

Debug.Log (agent.pathPending);

while (true) {
Debug.Log (agent.velocity);

if (agent.velocity.z > 0.5f) {
animator.SetInteger ("direction", 1);
} else if (agent.velocity.z < -0.5f) {
animator.SetInteger ("direction", 2);
} else if (agent.velocity.x > 0.5f) {
animator.SetInteger ("direction", 3);
} else if (agent.velocity.x < -0.5f) {
animator.SetInteger ("direction", 4);
}
now_direction = animator.GetInteger ("direction");

if (agent.velocity == Vector3.zero) {
animator.SetBool ("stop", true);
yield break;
}

yield return null;
}
}
```

出力結果
------------ 初回
True
False
(0.0, 0.0, 0.0)

------------- ２回目
True
False
(0.9, 0.0, 0.3) .... (0.0, 0.0, 0.0)

コメント欄でのコードの整形の仕方がわからなくて、見づらくなってしまい申し訳ございません。

[sakura_hana]

リファレンス読み間違えてました。
「経路探索中はagent.pathPending が true」なので、「falseになるまで待つ」が正しいです。

それと
yield return !agent.pathPending;
この部分を

while (agent.pathPending) {
yield return new WaitForEndOfFrame();
}

に置き換えてみてください。
（ただDebug.Logが同じ物を返しているのでこれが原因ではない可能性もありますが）

[nico29]

お返事ありがとうございます。
リファレンス、私も最初読み間違えました。ややこしいですね。

ご指摘の通りに書き換えてみましたが、結果は同じでした。

while (agent.pathPending) {
yield return new WaitForEndOfFrame ();
}

Debug.Log (agent.pathPending);

Debug.Log (agent.pathPending)でTrueが返されなくなりました。
velocityの出力値は０のままです。
どうやら一瞬だけアニメーションが表示されたり・されなかったりしているみたいです。
移動はしてます。

[sakura_hana]

Unity - スクリプトリファレンス: NavMeshAgent.velocity
https://docs.unity3d.com/ja/540/ScriptReference/NavMeshAgent-velocity.html
上記を見ると「値を設定する場合、次の更新で効果が現れます。」とありますので、
速度取得についても１フレームかかるのかもしれません。

while (agent.pathPending) {
yield return new WaitForEndOfFrame ();
}
yield return new WaitForEndOfFrame ();

こんな感じでどんな場合でも１フレーム待ってみてください。
これでダメなら強硬手段としてboolで管理ですかね……一見よく分からないコードになりますが……

bool isMoveStart = false;

while(true) {
〜中略〜

if (agent.velocity != Vector3.zero) {
isMoveStart = true;
}

if (agent.velocity == Vector3.zero && isMoveStart) {
animator.SetBool ("stop", true);
yield break;
}
}

アニメーションが表示されるのは、velocityが０のままでも１回は animator.SetBool ("stop", true); が呼ばれるからではないでしょうか？

[nico29]

待つ方法で、できました！
velocityもちゃんと値が入ってます！
処理に時間がかかるから待ってあげれば良かったんですね。
すごく勉強になりました。自力では無理でした。
本当にありがとうございました。

while (agent.pathPending) {
yield return new WaitForEndOfFrame ();
}
yield return new WaitForEndOfFrame ();

Debug.Log (agent.pathPending);

animator.SetBool ("attack", false);
animator.SetBool ("stop", false);

[sakura_hana]

解決して良かったです。

ちょっとだけ補足すると、最後に追記したWaitForEndOfFrameについては、
「処理に時間がかかるから」待つのではなく、
「まだ実行（反映）されていないから」待つ、というイメージです。

私も正確には把握していないので「そういうものなんだ」という認識ですが、コルーチンの時間（フレーム）の関係を確認すると後学になるかもしれません。

[nico29]

補足までありがとうございます！
まだyied returnもよく理解できていなくて、整理していたところでした。
反映されていないから待つ、なのですね。
理解できるかどうかわかりませんが、じっくり勉強してみたいと思います。
ありがとうございました。