2点間でのワープ機能について【動画あり】

Question

前提

Unityでワープ機能、A地点からB地点、B地点からA地点を作りたくてコードを書きました。
しかし挙動がおかしくA地点からB地点までは正常に動作するのですが、B地点からA地点へ行こうとすると何故かB地点からB地点にワープとなってしまい理由もわからず困っています。
https://youtu.be/WwO_63G4hx0

実現したいこと

A地点からB地点、B地点からA地点へのワープ、またそのオブジェクトのプレハブ化

発生している問題・エラーメッセージ

該当のソースコード

public class Warp : MonoBehaviour
{

    public GameObject player;　//自分自身です、Unity自機を入れています

    public GameObject nextWarpObject;//次にワープしたいオブジェクトを入れるやつです

    private Vector3 trans;　//いろんなサイトを見て回った結果にできたnextWarpObjectのpositionの格納先です

    PlayerController script; //プレイヤーを動かすためのスクリプトを外部から引っ張ってきています

    PlayerInput playerInput; //Fキーを押したら移動するよという動作を作りたかったのでInputActioinを使用しています

    
    
    // Update is called once per frame

    private void Awake()
    {

        script = player.GetComponent<PlayerController>();　//代入しています

        playerInput = new PlayerInput();　//InputActionです



        playerInput.Player.CoverAction.started += CoverAction;//同

        playerInput.Enable();//同
    }

    private void Start()
    {
        trans = nextWarpObject.transform.position;　//ここでtransの中に”移動先”のオブジェクトの座標を代入しています。
        Debug.Log(trans); //移動先の座標の確認です、しっかりと別々の座標になってました。
    }
    private void CoverAction(InputAction.CallbackContext obj)
    {
        if (script.onWarpPoint)
        {
            Debug.Log("Warp");

            player.transform.position = trans; //押された時にplayerの座標を書き換えます
        }
    }


    private void Update()
    {
        
    }

    void OnTriggerStay(Collider other)　//これはBoxコライダーの中にいるときに自分の頭上に目印を出すスクリプトです
    {

        if (other.gameObject.tag == "Player")
        {
            Debug.Log("CanWarp");
            script.onWarpPoint = true;　



        }        
    }

    private void OnTriggerExit(Collider other)
    {
        script.onWarpPoint = false;
    }
}

試したこと

上記のように一行一行丁寧に見ました、またDebug.Logでお互いの座標が違うことを確認しました。
A地点とB地点をつなげるのではなく、移動先の指定を空のオブジェクトに指定しても同じ状況になります。

補足情報（FW/ツールのバージョンなど）

Unity2021.3.15f1

Comments

[Benway]

BAの内容を忘れないようにここにメモをします、
今回のバグが起こった原因は
①CoverActioinの登録はAwake以外で出来るということを知らなかった
②おんなじスクリプトを使いまわしていると一つの変数に2通りのパターンが登録されることを知らなかった
①についてはInputActioinの使い方の中でAwakeで登録した方がいいと書いてあったためその通りに使ったがほかの場所でも使えるということを知らなかったため起きてしまった。これは知識不足であった
②についてが一番重要で、同じスクリプト、（今回はWarp）を複数のオブジェクトアタッチしたときに複数のオブジェクトそれそれのスクリプトが独立していると考えていたが実際はそうではなく、それそれの値を独立していない、一つのスクリプトが管理していたという動作をしていた。そのため一つの変数の中に複数の値が入り、それらの値が2つあると2回実行する、3つあると3回実行する、という動作をしていたということになる。そのため今回のBAでは、値の登録を重複させないために、フラグを踏んだ時に変数に値を代入するという処理を行ったと思われます。

Answer 1

Awakeの中でplayerInputを通してPlayerにCoverAction（ワープのメソッド）を登録してますよね。

playerInput.Player.CoverAction.started += CoverAction;//同

で、今回はWarp（ワープオブジェクト）が2つ置いてありますよね。
ということは、WarpのAwakeは2回呼ばれてるってことですよね。
つまりゲームを起動時、Playerには2箇所のCoverAction（ワープ処理）が登録されるということですよね。

Acrtionを発火すると登録されている処理は全て実行されます。
今回でいうとPlayer.CoverAction.startedを実行するたびに「1-1のCoverAction」と「1-2のCoverAction」が順番に実行されるということです。
コンソールにもワープが行われるたびに"Warp"が2ずつ増えてますよね？
これは「1-2にワープした後、1-1にワープする」という2回のワープ処理が一度に行われているからです。
なので1-1を踏んでも1-1に飛ばされているんじゃないでしょうか？

故にワープ処理の登録＆削除を
「ワープに触れた瞬間、ワープ処理をPlayerに登録する」
「ワープが終わったら、ワープ処理をPlayerから削除する」
という風にして、「ワープする時、Playerには1つのワープ処理しか登録されていない状態」にすれば動くんじゃないでしょうか。

つまり

playerInput.Player.CoverAction.started += CoverAction;

をAwakeからOnTriggerStayに移し、CoverAction内に

playerInput.Player.CoverAction.started -= CoverAction;

を追加で書く的な感じです。

Comments

[Benway]

出来ました、ありがとうございます

Answer 2

B→AにワープしてからすぐにA→Bに再度ワープしていないでしょうか？
PlayerControllerにワープが有効状態かどうかのフラグ（仮にbool isWarpEnable)を持たせて、最初はtrueにしておき、

    private void CoverAction(InputAction.CallbackContext obj)
    {
        if (script.onWarpPoint && script.isWarpEnable )
        {
            Debug.Log("Warp");

            player.transform.position = trans; //押された時にplayerの座標を書き換えます
            script.isWarpEnable = false;
        }
    }

    private void OnTriggerExit(Collider other)
    {
        script.onWarpPoint = false;
        script.isWarpEnable = true;
    }

こんな感じで試してみてください。

Comments

[Benway]

出来なかったです、再起動してもダメです。