【Unity】自動で迎撃する固定砲台の処理について

Question

前提・実現したいこと

Unityを用いて2Dゲームを作っています。固定砲台の索敵範囲内に敵がいればその中からランダムに1体を選んでそちらを向いて発砲する、ということを実現したいと思っています。

該当のソースコード

まずオブジェクト同士の親子構造として、Turret.csがアタッチされたゲームオブジェクトTurretがあり、その子オブジェクトEnemySearchAreaには索敵用のスクリプトEnemySearchArea.csとサークルコライダーがアタッチされています。

索敵から攻撃までの流れは以下の通りです。
1、EnemySearchAreaのサークルコライダーに敵が接触したら、その敵のゲームオブジェクトをリストに追加
2、EnemySearchAreaのサークルコライダーから敵が出たら、その敵のゲームオブジェクトをリストから削除
3、リストの長さが0でない（＝索敵範囲内に敵がいる）場合、その中からランダムに1体を選んで、そちらの方を向いて発砲
4、発砲するとゲームオブジェクトBulletが生成され、敵に当たるとダメージ処理を行う

以下がEnemySearchArea.csとなります。このスクリプトは索敵および索敵範囲内にいる敵リストの保持と、親オブジェクトであるTurretへの回転と発砲の指示を担当しています。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class EnemySearchArea : MonoBehaviour
{
    public bool setTarget; //ターゲットが設定されているか否か
    private List<GameObject> targetList = new List<GameObject>(); //索敵範囲内にいるターゲットのリスト

    // Start is called before the first frame update
    void Start()
    {
        setTarget = false;
    }

    // Update is called once per frame
    void Update()
    {
        if (!setTarget && targetList.Count > 0) //ターゲットがセットされておらず、かつ索敵範囲内に敵がいる場合
        {
            GameObject target = targetList[Random.Range(0, targetList.Count)]; //ターゲットをランダムに選択
            setTarget = true;
            this.transform.parent.GetComponent<Turret>().CoRotate(target);
            this.transform.parent.GetComponent<Turret>().CoShot();
        }
    }

    private void OnTriggerEnter2D(Collider2D collision)
    {
        targetList.Add(collision.gameObject);
    }

    private void OnTriggerExit2D(Collider2D collision)
    {
        targetList.Remove(collision.gameObject);
    }
}

EnemySearchAreaからの指示を受けてTurret.csではゲームオブジェクトTurretの回転と発砲（ゲームオブジェクトBulletの生成）を行います。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Turret : MonoBehaviour
{
    public GameObject bullet;
    public float coolTime; //発砲間隔
    public int shotsNum; //連射数
    public float turningSpeed; //旋回速度

    IEnumerator Shot() //発砲コルーチン
    {
        for(int i = 0; i <= shotsNum; i++){ //0.1秒ごとに発射
            Instantiate(bullet, transform.position, transform.rotation, transform);
            yield return new WaitForSeconds(0.1f);
        }
        yield return new WaitForSeconds(coolTime); //連射後のクールタイム
        this.transform.GetChild(1).GetComponent<EnemySearchRange>().setTarget = false; //ターゲットを未セットにしてコルーチン終了
        yield break;
    }

    IEnumerator Rotate(GameObject target) //targetの方を向くまで一定速度で回転
    {
        while (Vector2.Angle(target.transform.position - this.transform.position, transform.up.normalized) > 5f)
        {
            this.GetComponent<Rigidbody2D>().angularVelocity = (Vector2.SignedAngle(transform.up.normalized, target.transform.position - this.transform.position) < 0 ? -1f : 1f) * turningSpeed;
            yield return null;
        }
        this.GetComponent<Rigidbody2D>().angularVelocity = 0f;
        yield break;
    }

    public void CoShot() //外部からShotコルーチンを呼ぶためのメソッド
    {
        StartCoroutine("Shot");
    }

    public void CoRotate(GameObject target) //外部からRotateコルーチンを呼ぶためのメソッド
    {
        StartCoroutine(Rotate(target));
    }
}

Turretから発射されたゲームオブジェクトBulletは敵と接触するとダメージ処理を行います。ダメージ処理の内容は敵にアタッチされたスクリプトCharacter.csのDamaged()メソッドに書かれています。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Bullet : MonoBehaviour
{
    public float speed; //弾速
    public float damage; //1発当たりのダメージ量

    private void OnTriggerEnter2D(Collider2D collision)
    {
        string layerName = LayerMask.LayerToName(collision.gameObject.layer); //衝突したオブジェクトのレイヤー名を取得

        if (layerName == "Enemy" || layerName == "Player" )
        {
            collision.GetComponent<Character>().Damaged(damage, GetComponent<Rigidbody2D>().velocity);
            Destroy(this.gameObject);
        }
    }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Character : MonoBehaviour
{
    public float maxHP;
    private float HP;


    public void Damaged(float damage, Vector2 dmgDirection) //ダメージ処理
    {
        this.HP = this.HP - damage;

        if (this.HP < 0)
        {
            Destroy(gameObject);
        }

        this.GetComponent<Rigidbody2D>().AddForce(dmgDirection.normalized, ForceMode2D.Impulse); //ノックバック

    }

    // Start is called before the first frame update
    void Start()
    {
        this.HP = maxHP;
    }

}

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

実際に上記のコードを実行してみたところEnemySearchAreaの中のCoRotate()にtargetを渡すところで以下のようなエラーが発生しました。

MissingReferenceException: The object of type 'GameObject' has been destroyed but you are still trying to access it.
Your script should either check if it is null or you should not destroy the object.

先ほどのEnemyearchArea.csでは索敵範囲内の敵がHP0になって消滅するパターンを考えていなかったため、既にDestroyされた敵をターゲットにしようとしてエラーが発生したようです。

教えていただきたいこと

それぞれのEnemySearchAreaクラスのインスタンスのtargetListからある敵のゲームオブジェクトを一斉に削除するような方法はないでしょうか？もしそのような方法があればCharacter.csのDamaged()メソッドのDestroyの直前に行うことで、上記の問題が解決するのではないかと思っています。
またもし別の解決方法（コードを根本的に変更する必要のあるものでも構いません）があればぜひとも教えていただけないでしょうか。

Answer 1

GameObjectがいつの間にかいなくなってたはゲーム制作あるあるですね。

基本的にGameObjectをDestroyするときはGameObject自身でDestroyせず、DestroyManagerのようなオブジェクトにDestroyを依頼する形にします。DestroyManagerはフレームの最後（GameObject同士のやり取りがなくなったとき）にまとめてDestroyします。
その場合、EnemySearchAreaはUpdate毎に範囲内のGameObjectを探すようにします。

別の方法としてはGameObjectがDestroyするときにEnemySearchAreaにリストから自分自身を削除するよう依頼するようにします。

Comments

[shukrin]

DestroyManagerにDestroyを依頼したとしても、結局targetListにはDestroyされたゲームオブジェクトが残ったままではないでしょうか？

[tatamyiwathy]

スミマセン。まさにその通りでございます。回答を修正しました。

[shukrin]

何度もすみません。

＞別の方法としてはGameObjectがDestroyするときにEnemySearchAreaにリストから自分自身を削除するよう依頼するようにします。

こちらの方法に関してなのですが、すべてののインスタンスのリストから自身を削除するには具体的にどうすればよろしいのでしょうか。

[tatamyiwathy]

ゲームの進行を管理するGameObject（GameMasterとする)が存在すると思いますが、そのGameMasterに配列を持たせて
DestroyするGameObjectはGameMasterの配列に自身を追加します。その後、フレームの最後でGameMasterが配列に登録されているGameObjectをDestroyします。