回答編集履歴

追記

2018/01/16 06:07

投稿

fiveHundred

スコア10480

answer CHANGED Viewed

@@ -16,4 +16,75 @@
 - Vector4がどうたらこうたら言っておりますが、そもそもVector4は向きや位置を示すのには使われておりません。
   QuaternionかVector3の間違いだと思います。
-回転しているかどうかの判定については、``transform.rotation``を比較するだけだと、完全に止まっていないと回転していないと判断されないため、[Quaternion.Angle()](https://docs.unity3d.com/2017.3/Documentation/ScriptReference/Quaternion.Angle.html)を使って角度を取得し、それが一定値以下かどうかで処理を分けるのがよろしいかと思います。
+回転しているかどうかの判定については、``transform.rotation``を比較するだけだと、完全に止まっていないと回転していないと判断されないため、[Quaternion.Angle()](https://docs.unity3d.com/2017.3/Documentation/ScriptReference/Quaternion.Angle.html)を使って角度を取得し、それが一定値以下かどうかで処理を分けるのがよろしいかと思います。
+---
+追記：
+> コルーチンは、void updateはあまり使わないほうが良いと書いていたので、コルーチンで動作を区切るために使っております。
+そうであったとしても、``yield return``で待ち合わすという処理をしていないのであれば、コルーチンでも関数でも同じです。
+また、Update()を重い処理を避ける理由は、毎フレーム処理されるためですが、そう考えると
+- 毎フレーム、コルーチンを呼び出している場合は、そもそも意味がない
+- そうでない場合、フレーム間の時間を表す``Time.deltaTime``を使うのは不適切
+となるので、素直にUpdate()内に書いてしまったほうがよろしいかと思います。
+> Quaternion.Angle()を使う具体的な方法がわかる方、もしおられましたらご一報ください。
+以下のようにすればよいです。
+```C#
+    Quaternion targetRotation; // 追加
+    // …（中略）
+    void Update()
+    {
+        //全てのアニメーションをリセット
+        moveResaet();
+        //初期値からいくつ回転したのかを調べる
+        float angle = Quaternion.Angle(transform.rotation, targetRotation); // 変更
+        //ターゲットへ旋回
+        rb.rotation = Quaternion.Slerp(transform.rotation, Quaternion.LookRotation
+            (targetP.transform.position - transform.position), Time.deltaTime * 10f);
+        //見るだけなので動かない
+        rb.velocity = Vector3.zero;
+        if (angle < 10f * Time.deltaTime)
+        {
+            animator.SetBool("Idle", true);
+        }
+        else
+        {
+            animator.SetBool("Walk", true);
+        }
+        targetRotation = transform.rotation; // 追加
+    }
+```
+ちなみに、以下のようにした場合は、transformのインスタンス＝targetのインスタンスとなり、transformの変更がtargetにも反映されてしまうため、うまくいきません。
+```C#
+    Transform target;
+    // …（中略）
+    void Update()
+    {
+        // …（中略）
+        float angle = Quaternion.Angle(transform.rotation, target.rotation);
+        // …（中略）
+        target = transform;
+    }
+```