Unity C# で[SerializeField] List<Vector3> の内容が吹っ飛ぶ話を解決したい

Question

実現したいこと

Unity C# で起きた問題を解決したいです。
詳しいことは次書きますが、
[SerializeField] List<Vector3> spawnPosList;
の内容が実行中に触っているつもりがないのに吹き飛んでしまうということです。
[SerializeField]なので他スクリプトからの干渉は考えられません。

なお現在は応急措置によって何とかなっています。
また、UnityC#やその他プログラミングは一応2年くらいやっていますが、独学なのでところどころ用語の使い方が変だったり、コードの細かい順番や書き方が変だったりするかもしれませんが、温かい目で見ていただけると幸いです。

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

ソースコード19行目の
[SerializeField] List<Vector3> spawnPosList;
の内容を消す操作を行っていないにも関わらず、内容が消失してしまう問題です。
コードにもコメントを書きましたが、Start()が実行されたときは長さが2で、ReStart()が実行されたときにもまだ長さが2でした。ですが、OnPlayerJoined()が実行されたときに長さが0となっていることを確認しています。
また、[SerializeField]なので他スクリプトからの干渉は考えられません。
添付したコード内で確実に何かが起きていると考えています。一応他スクリプトも検査しましたが、Listに触るようなコードは見つかりませんでした(privateなので当然ですが...)。

応急措置について。
宣言にインスペクタで設定していた初期値を設定。
List<Vector3> spawnPosList = new List<Vector3>()
{
new Vector3(-5, 3, 0),
new Vector3(5, -3, 0)
};

これだけではだめだったので、
public void ReStart()
{
playerCount = 0;
foreach (var player in playerList)
{
if(player != null)
{
Destroy(player.gameObject);
}
}
playerList.Clear();
spawnPosList = new List<Vector3>()
{
new Vector3(-5, 3, 0),
new Vector3(5, -3, 0)
};
EnableJoining();
}
と、無理やり設定しなおしました。

エラーメッセージ

エラーメッセージはありません。
しいて言うなら
if (playerCount >= spawnPosList.Count)
        {
            //Start()時点では2,ReStart()時点では0。なので弾かれてしまう。
            return;
        }
を書いていないときに発生した、
ctrl.Init((PlayerType)playerCount, spawnPosList[playerCount], wallTilemap, bombParent, fireParent,maxBombCount, inputThreshold, moveCooldown);
です。エラーはたぶん範囲外だと怒られていると思います。



ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
System.Collections.Generic.List`1[T].get_Item (System.Int32 index) (at <ed969b0e627d471da4848289f9c322df>:0)
PlayerSpawner.OnPlayerJoined (UnityEngine.InputSystem.PlayerInput playerInput) (at Assets/Scripts/PlayerSpawner.cs:81)
UnityEngine.InputSystem.LowLevel.<>c__DisplayClass7_0:<set_onUpdate>b__0(NativeInputUpdateType, NativeInputEventBuffer*)
UnityEngineInternal.Input.NativeInputSystem:NotifyUpdate(NativeInputUpdateType, IntPtr)

該当のソースコード

```C#
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.InputSystem;
using UnityEngine.Tilemaps;
using static GameManager;
public class PlayerSpawner : MonoBehaviour
{
    int playerCount = 0;
    List<PlayerCtrl> playerList = new List<PlayerCtrl>();
    PlayerInputManager playerInputManager;

    [SerializeField] Tilemap wallTilemap;
    [SerializeField] Transform bombParent;
    [SerializeField] Transform fireParent;
    [SerializeField][Min(1)] int maxBombCount = 2;
    [SerializeField][Range(0f,1f)] float inputThreshold = 0.7f;
    [SerializeField][Min(0)] float moveCooldown = 0.2f;

    [SerializeField] List<Vector3> spawnPosList;  ////今回の主役。インスペクタには二つ要素が入ってる

    public static PlayerSpawner playerSpawner;
    void Awake()
    {
        if (playerSpawner == null)
        {
            playerSpawner = this;
        }
        else
        {
            Destroy(gameObject);
        }

        playerInputManager = GetComponent<PlayerInputManager>();
        //Debug.Log($"[Awake] I'm {gameObject.name}, scene: {gameObject.scene.name}, spawnPosList.Count: {spawnPosList.Count}");
    }

    public void ReStart()
    {
        //この時点でspawnPosList.Count()は2
        playerCount = 0;
        foreach (var player in playerList)
        {
            if(player != null)
            {
                Destroy(player.gameObject);
            }
        }
        playerList.Clear();
        EnableJoining();
    }

    


    void OnEnable()
    {
        playerInputManager.onPlayerJoined += OnPlayerJoined;
    }

    void OnDisable()
    {
        playerInputManager.onPlayerJoined -= OnPlayerJoined;
    }

    void OnPlayerJoined(PlayerInput playerInput)
    {
        if (playerCount >= spawnPosList.Count)
        {
            //Start()時点では2,ReStart()時点では0。なので弾かれてしまう。
            return;
        }
        if (gameManager.IsGameStateBool(GameStateFlag.GameStarted))
        {
            Debug.Log("すでにゲームが始まっています。");
            return;
        }
        if (gameManager.IsGameStateBool(GameStateFlag.NeededPlayers))//構造上あり得ないが一応書いておく。
        {
            Debug.Log("これ以上プレイヤーは参加できません。");
            return;
        }
        var ctrl = playerInput.GetComponent<PlayerCtrl>();
        Debug.Log($"Player {playerCount + 1} joined");
        //Debug.Log($"ctrl : {ctrl}");
        ctrl.Init((PlayerType)playerCount, spawnPosList[playerCount], wallTilemap, bombParent, fireParent, maxBombCount, inputThreshold, moveCooldown);
        playerList.Add(ctrl);
        playerCount++;
    }

    public List<Vector3> GetSpawnPosList()
    {
        return spawnPosList;
    }

    public void EnableJoining()
    {
        playerInputManager.joinBehavior = PlayerJoinBehavior.JoinPlayersWhenButtonIsPressed;
    }
    public void DisableJoining()
    {
        playerInputManager.joinBehavior = PlayerJoinBehavior.JoinPlayersManually;
    }

    public int GetPlayerCount()
    {
        return playerCount;
    }
}

### 試したこと・調べたこと
- [ ] teratailやGoogle等で検索した
- [x] ソースコードを自分なりに変更した
- [ ] 知人に聞いた
- [x] その他

##### 上記の詳細・結果
ChatGpt君にソースコード丸々渡しても解決できませんでした。

### 補足
特になし

Comments

[lehshell]

> [SerializeField]なので他スクリプトからの干渉は考えられません。
spawnPosList は参照変数です。
public List<Vector3> GetSpawnPosList()
{
    return spawnPosList;
}
と spawnPosList への参照を返していますから
★他スクリプトからの干渉は可能です★
List のコピーを返す以下のコードにしてしてここが原因か切り分ける
アプローチからしてみましょう。
public List<Vector3> GetSpawnPosList()
{
    return new List<Vector3>(spawnPosList);
}

[omotika710]

コメントありがとうございます。無事解決しました！
まさか参照型の変数であることが抜け落ちてるだなんて恥ずかしいミスを...穴があったら入りたい...
他スクリプトで似たような名前の変数があって混同してしまったのも原因の一つでした。
前にもTransformとVector3でミスをしたばかりなのに本当に懲りませんね()
解決したので正式に回答していただけないでしょうか？ベストアンサーに指名したいです。

[omotika710]

はぁ...掲示板でこんな痴態をさらすなんて...ほんとに恥ずかしい誰か助けてくれ()

Answer 1

調査のアプローチ方法をコメントして見えている要因を排除しようとしたところ原因だったとのこと

C#の参照変数は、代入やメソッドへの値渡しや return での返却で実体への参照情報を渡せてしまいます。
他スクリプトからの干渉を防ぐには GetSpawnPosList メソッドでリストのコピーを返す必要があります。

    public List<Vector3> GetSpawnPosList()
    {
        //return spawnPosList;
        return new List<Vector3>(spawnPosList);
    }

余談
C++の参照変数は、代入やメソッド（C++はメンバー関数というべきかな）への値渡しや return での値返却では実体のコピーが対象になります。
言語で動作が違うため注意が必要ですね。