Unityでキャラクターを動かしたときに前に壁があった場合停止する処理が実行されません

Question

実現したいこと

キャラクターを前に動かしたいときに前に障害物があった場合停止する処理をしたいです。コードに不具合があるなど、どのような不備があるのか教えてください。

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

うまく作動せず延々と壁にぶつかり続け、操作不能になります。コライダーなどはしっかり設定しているため、何が不備なのかわかりません。

エラーメッセージ

延々とぶつかり続ける（エラーメッセージはなし）

該当のソースコード

using System.Collections;
using UnityEngine;
using UnityEngine.EventSystems;

public class chara_move : MonoBehaviour
{
    [SerializeField] LayerMask solidObjectsLayer;

    Animator animator;
    bool isMoving;

    private void Awake()
    {
        animator = GetComponent<Animator>();
    }

    void Update()
    {
        if (isMoving == false)
        {
            float x = Input.GetAxisRaw("Horizontal");
            float y = Input.GetAxisRaw("Vertical");

            if (x != 0)
            {
                y = 0;
            }

            if (x != 0 || y != 0)
            {
                animator.SetFloat("InputX", x);
                animator.SetFloat("InputY", y);
                StartCoroutine(Move(new Vector2(x, y)));
            }
        }
        animator.SetBool("IsMoving", isMoving);

    }

    IEnumerator Move(Vector3 direction)
    {
        isMoving = true;
        Vector3 targetPos = transform.position + direction;
        if (IsWalkable(targetPos) == false)
        {
            isMoving = false;
            yield break;
        }
        while ((targetPos - transform.position).sqrMagnitude > Mathf.Epsilon)
        {
            transform.position = Vector3.MoveTowards(transform.position, targetPos, 5f * Time.deltaTime);
            yield return null;
        }

        transform.position = targetPos;
        isMoving = false;
    }

    bool IsWalkable(Vector3 targetPos)
    {
        return Physics2D.OverlapCircle(targetPos, 0.2f, solidObjectsLayer) == false;
    }
}

試したこと・調べたこと

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

上記の詳細・結果

特に変わりはなかった

補足

特になし

Comments

[Eclipse216]

Ridibody2Dを使って、当たり判定を実装してはどうでしょうか？

[takopanda]

Rigidbody 2D は使っているんですけど、ぶつかっても止まらず延々とぶつかり続けるんですよね…

[fiveHundred]

Rigidbody2Dはどんな設定で、どのように使っていますか？
もしかしたら「Rigidbody2Dが邪魔になって移動できない」ということかもしれません。

[takopanda]

Rigidbody2Dの設定はボディタイプはダイナミックで、質量だけ1にしてます。後は衝突判定は連続的で、Z軸の回転は固定してます。

[fiveHundred]

この場合「Physics2D.OverlapCircle」で移動するか判定しているので、物理判定は不要だと思います。
むしろ、その物理判定によって「指定された位置まで移動できない」となってしまって現状の問題になっている可能性もあります。

[fiveHundred]

試しにRigidbody2Dなし（Collider2Dのみ）にしましたが、動作しました。
そのため「Rigidbody2Dによる物理的な挙動」は不要です。

そもそも、このスクリプトはRPGのようなマス目単位で移動するものですが、その点は理解していますか？

[takopanda]

マス移動のRPGを作っているので、大丈夫です。
Rigidbody2Dを試しに抜いたところ、タイルマップの建物にぶつからずに通過してしまいました。
TilemapCollider2Dは入れています。

Answer 1

動作の原因かどうかはわかりませんが、コードの不具合を指摘しておきます。

まず、 Update メソッドの中で入力があれば Move コルーチンを走らせています。
そして、 Move コルーチンの中では入力された方向に5unit/秒の速度で移動しようとしています。
入力値は-1から1の範囲ですから、最大1unit移動することになりますから、その場合には0.2秒かかることになります。

Update メソッドの呼ばれるサイクルはそれぞれですが、0.2秒の間に何回かは呼ばれます。
よって、同じ方向に入力された場合、 Move コルーチンが同時に多数走ることになります。

更に、同じ方向ではなく、急に逆方向に入力があった場合、多重にコルーチンが走っていれば前の方向に向かう移動力の方が勝っていますが、そのうちに逆方向へ向かうコルーチンが多くなるので、逆方向へ向かいます。
それ自体は見た目にはいい感じになるでしょうが、前に向かっていたコルーチンが残っていれば、目的地とは逆の方向に向かってしまうので、再度目的地に近づかない限りこのコルーチンが走り続けてしまうことになります。

次に当たり判定のとり方です。
移動先に半径0.2unitの範囲でコライダが無ければ、という判定になっていますが、先程も言ったとおり、最大1unit移動先との距離があることになりますが、となると最大0.8unitの間があることになります。
当然、その間にコライダがあっても、当たっていると判断されません。
(ただし、そんな小さなコライダはない、というのであれば、あまり問題にはならないかもしれません)
もしこの問題を回避したいのであれば、 Physics2D.OverlapCapsule を使って、現在位置と移動先とのカプセルで判定する方がいいでしょう。

次に Rigidbody2D に関してです。
Rigidbody2D を使っている場合には、基本的に移動に Transform.position などに位置を代入して移動してはいけません。 Rigidbody2D.AddForce や velocity などを使います。
この辺りの話は、「Unity Rigidbody2D 移動」で検索すればたくさん出てくると思うので、調べてください。

Rigidbody2D を使えば、壁に当たった際の処理もしてくれるので便利ですが、物理演算というのはリアリティに沿った動きをするので、想定外の動きをする場合もあります。
簡単な動きであれば Rigidbody2D に頼らず自分で処理する、という考えもありますが、そうであれば Rigidbody2D は基本有効にすべきではありません。(キネマティックにして使う、という考えもありますが、話がややっこしくなるので、とりあえず置いといて)

---

操作不能になる不具合の原因ですが、正直パッとはわかりませんが、上記のことを踏まえてあえて考えるのであれば、「障害物が小さかった場合」「 Rigidbody2D が有効であれば」、障害物を無視して移動しようとする力と、 Rigidbody2D による障害物の衝突による押し戻す力が同時に働いて、動けなくなっている、という可能性があります。
もしそうであれば、先に話したとおりカプセルで当たり判定を行い、 Rigidbody2D を無効にすれば解決しますが、コルーチンの多重起動の問題もありますので、一旦移動処理を見直すことをおすすめします。