一度生成した場所には生成させないコードを書きたい

Question

前提・実現したいこと

今現在的あてゲームを作っています。

何個かある板(的の生成位置)に、的を生成するコードを書きました。

既に生成した位置には、その的が消えるまでは生成させたくないのですが

手順をメモ帳に書き出してみましたが、いまいちコードが思いつきません。

一度生成した板には生成した的が消えるまで新しい的は生成しない
というコードが書きたいです。

何かいい方法があれば教えてください。

該当のソースコード

public class TargetSpawn : MonoBehaviour
{
    //Prefabのリスト 
    [Header("prefabリスト")]
    public GameObject[] targetPrefabs;

    //的の生成位置のリスト
    [Header("的の位置リスト")]
    public GameObject[] targets_L;
    public GameObject[] targets_M;
    public GameObject[] targets_S;

    private Vector3[] posList_L;
    private Vector3[] posList_M;
    private Vector3[] posList_S;

    ///ランダム生成///
    [Header("最小.最大インターバル時間設定")]
    //時間の最小値
    [SerializeField] float minTime;
    //時間の最大値
    [SerializeField] float maxTime;
    //経過時間
    private float time = 0f;
    //インターバル
    private float interval;

    private bool a;

    void Start()
    {

        interval = GetRandomTime();//ランダム生成したインターバルを入れる
        a = true;

        MakePosList();
    }

    void Update()
    {

        //経過時間を入れて計測する
        time += Time.deltaTime;//時間計測開始

        //経過時間がインターバルを超えたとき
        if(time > interval && a)
        {
            a = true;
            //ターゲットのサイズを決める数値をランダムに生成
            var targetSize = Random.Range(0,3);

            //的Prefabから配置する的の位置を生成
            var numPos = Random.Range(0, posList_L.Length);

            float posCheck = numPos;

            //Posリストのランダム生成値
            var pos_L = posList_L[Random.Range(0, posList_L.Length)];//PosリストLサイズ
            var pos_M = posList_M[Random.Range(0, posList_M.Length)];//PosリストMサイズ
            var pos_S = posList_S[Random.Range(0, posList_S.Length)];//PosリストSサイズ

            //Prefabリストのランダム生成値
            var targPref = targetPrefabs[Random.Range(0, targetPrefabs.Length)];

            switch (targetSize)
            {
                case 0: //Large
                    Instantiate(targetPrefabs[Random.Range(0, targetPrefabs.Length)], pos_L, transform.rotation); //的生成
                    break;
                case 1: //Medium
                    Instantiate(targetPrefabs[Random.Range(2, 5)], pos_M, transform.rotation); //的生成
                    break;
                case 2: //Small
                    Instantiate(targetPrefabs[Random.Range(4, 5)], pos_S, transform.rotation); //的生成
                    break;
            }

            time = 0f;
            interval = GetRandomTime();//ランダム生成したインターバルを入れる
        }
    }


    //ランダムに時間を生成
    private float GetRandomTime()
    {
        return Random.Range(minTime,maxTime);
    }

    void MakePosList()
    {
        //Vector3作成
        posList_L = new Vector3[targets_L.Length];
        posList_M = new Vector3[targets_M.Length];
        posList_S = new Vector3[targets_S.Length];

        //位置リストの中にターゲットのポジションリスト内のオブジェクト座標を入れる
        for (int i = 0; i < targets_L.Length; i++)
        {
            posList_L[i] = targets_L[i].transform.position;
        }

        //位置リストの中にターゲットのポジションリスト内のオブジェクト座標を入れる
        for (int i = 0; i < targets_M.Length; i++)
        {
            posList_M[i] = targets_M[i].transform.position;
        }
        
        //位置リストの中にターゲットのポジシsョンリスト内のオブジェクト座標を入れる
        for (int i = 0; i < targets_S.Length; i++)
        {
            posList_S[i] = targets_S[i].transform.position;
        }
    }
}

他にこのコードはもっと簡単に書ける点等、ありましたらご教示お願いします。

Comments

[m.ts10806]

そのメモ帳に書き出した手順とやらを転記してください。
設計とも言える部分ですし、そもそもその考え方が合っていなかったら実装にも繋がりません

[y0shida]

どの板に生成されたかという情報を保持
保持した情報と比較、すでに生成された板以外の板に生成
生成された的が消されたら、保持した情報も削除

というメモを書きました。

Answer 1

単純に生成中の的の位置をListあたりに入れるというルールで
生成成功したらListに追加、的が消えたらListから削除する。

生成しようとする時にFindやContainsで探して、見つかったら中断するような感じでいいのではないでしょうか。

【C#入門】Listの要素を検索する方法総まとめ(Find/IndexOf/Contains)
https://www.sejuku.net/blog/45381

Comments

[y0shida]

ありがとうございます、元々Listを使うことを考えていて、mahnyさんのやり方で試してみました。
Listにpos.magnitudeというVector3の情報をAddした後、Containsで前のフレームのpos.magnitudeを判定してみましたが
Vecotor3の(x,y,z)という情報ではなく、”pos.magnitude”が入っているかどうか という判定をしているらしく、思った通りの実装にはなりませんでした。

[退会済みユーザー]

pos.magnitudeにどんな値が入っているのかは提示されていないので分かりませんが、ListのFind等の挙動を考えたら、型が違うのですから単純比較できないのは当たり前じゃないでしょうか。

位置情報をどちらからも評価可能な値に変換して格納するのですよ。
一度JSONにするとか。もっと原始的には固定桁数の数字文字列にするとか。
{x:10, y:20, z:30} ⇒ "001000200030" みたいに。
この辺の仕様はそのゲーム開発者であるあなたが好きに決めて良いものです。

頑張ってください。