【Unity】敵を倒してくれる仲間のNPCを作りたい

Question

###前提・実現したいこと
Unity3Dを使ってTPS視点のアクションゲームを作っています。
プレイヤーと一緒に戦うノンプレイヤーキャラを作っているのですが、一匹目のモンスターを倒したあと
次の行動に移行するまでに時間がかかります。

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

2匹めの敵も攻撃するようにりましたが、Update内がうまく言っておりません。
今は、原因究明中です。

###該当のソースコード(2018/01/19改定)

    [Header("基本設定")]
    Rigidbody rb;
    AudioSource aud;

    [Header("対象オブジェクト")]
    GameObject targetP;
    public float PlayerLength;             //プレイヤーとの設定距離
    public float PlayerDistance;           //プレイヤーとの今の距離

    public Transform target;

    public Transform[] enemyList;   //敵のリスト
    public Transform targetE;       //現在選択されている敵
    public float EnemyDistance;            //一番近い敵との距離

    [Header("行動しているか？")]
    public bool nowExecCoroutine;

    [Header("再攻撃と再移動")]
    //再攻撃までの時間
    public float atackInterval = 0;
    public float atackIntervalMax = 2.0F;

    //再移動までの時間
    public int moveInterval = 0;
    public int TukiBunki;

    //アニメーション関係
    public Animator animator;

    [Header("スクリプト参照")]
    private EnemyState EState;              //EnemyStateスクリプトへの参照
    private PlayerGuardChecker PGCheck;     //PlayerGuardCheckerスクリプトへの参照
    private NonPcState NPState;


    [Header("巡回")]
    private NavMeshAgent agent;

    // Use this for initialization
    void Start () {
        //ターゲットを取得
        targetP = GameObject.FindGameObjectWithTag("Player");

        //targetE = GameObject.FindGameObjectWithTag("Enemy");
        atackInterval = 0;
        moveInterval = 200;

        //スクリプト参照
        EState = GetComponent<EnemyState>();
        PGCheck = GetComponent<PlayerGuardChecker>();

        //音とアニメーション
        animator = GetComponent<Animator>();
        aud = gameObject.GetComponent<AudioSource>();
        rb = GetComponent<Rigidbody>();

        //初期モーション
        animator.SetBool("Idle", true);

        //NPCの位置チェック
        //StartCoroutine(NpcPositionCheck());
    }

    // Update is called once per frame
    void Update()
    {
        enemyList = GameObject.FindGameObjectsWithTag("Enemy").Select(g => g.transform).OrderBy(g => (g.position - target.position).sqrMagnitude).ToArray();
            
        //なんらかの行動をしているか？
        if (!nowExecCoroutine)
        {
            //プレイヤーが近くにいるか？
            //いる
            if (PlayerDistance >= 100)
            {
                //敵がいない
                if (enemyList.Length == 0)
                {
                    if (PlayerDistance > 50)
                    {
                        nowExecCoroutine = true;
                        StartCoroutine(NpcNormalWalk());
                        Debug.Log("敵がいないのでついていく");

                    }
                    else //if(Vector3.Distance(targetP.transform.position, transform.position) >= 7)
                    {
                        //すごく近いので停止
                        nowExecCoroutine = true;
                        StartCoroutine(NpcNormalKaiten());
                        Debug.Log("敵がいないので安心の停止");
                    }
                }
                //敵がいる
                else if (enemyList.Length > 0)
                {
                    SelectEnemy(0);
                    Debug.Log("tagE" + targetE + "｜" + "enemyList0" + enemyList[0]);

                    //敵が近くにいるか？
                    //いる
                    if (EnemyDistance >= 80)//80
                    {
                        //敵がいるので抜刀
                        StartCoroutine(NpcKamaeIdle());
                        Debug.Log("構える");
                    }
                    //殴れない、近づく
                    else if (EnemyDistance >= 50 && EnemyDistance > 5)//50
                    {
                        nowExecCoroutine = true;
                        StartCoroutine(NpcKamaeWalk());
                        Debug.Log("敵に近づく");
                    }
                    else if (EnemyDistance <= 5)//20
                    {
                        nowExecCoroutine = true;
                        StartCoroutine(NpcKamaeTuki());
                        Debug.Log("敵につき");
                    }
                }
            }
            //いないが30メートル以内にいるか？)
            else if (PlayerDistance <= 110)
            {
                nowExecCoroutine = true;
                StartCoroutine(NpcNormalKaiten());
                Debug.Log("遠くから見る");
            }
            //いない
            else
            {
                nowExecCoroutine = true;
                StartCoroutine(NpcNormalIdle());
                Debug.Log("待機");
            }
            
            
            
        }
    }
    //LateUpdateは、Updateが実行されたあと一回だけ実行される
    void LateUpdate()
    {
        //Find all enemies within range and add them to list. Order list from closest to furtherest
        //範囲内のすべての敵を見つけてリストに追加します。furtherestに最も近いものからの注文リスト
        enemyList = GameObject.FindGameObjectsWithTag("Enemy").Select(g => g.transform).OrderBy(g => (g.position - target.position).sqrMagnitude).ToArray();

        //一番近い敵との距離を算出
        if (targetE != null)
        {
            EnemyDistance = Vector3.Distance(targetE.transform.position, transform.position);
        }
        //プレイヤーとの距離を算出
        PlayerDistance = Vector3.Distance(targetP.transform.position, transform.position);
    }

    void SelectEnemy(int i)
    { // 0 = Nearest / New List, 1 = Forwards, -1 = Backwards
        switch (i)
        {
            case 0:
                targetE = enemyList.FirstOrDefault();
                break;
            default:
                targetE = enemyList[((System.Array.FindIndex(enemyList, g => g == targetE) + i) % enemyList.Length + enemyList.Length) % enemyList.Length];
                break;
        }
        Debug.Log("敵がいる" + targetE);
    }

    public void NpcDead()
    {
        StartCoroutine(NpcResurrection());
    }

    void moveResaet()
    {
        animator.SetBool("Idle", false);
        animator.SetBool("Walk", false);
        animator.SetBool("Kamae", false);
        animator.SetBool("Dead", false);
    }

###試したこと

###補足情報(言語/FW/ツール等のバージョンなど)
【環境】
Unity5.6.1f1 personal
win10
MicrosoftVisualStudio
上記が作成環境になっております。

Answer 1

敵をDestroyしたのに、UpDate内ではその破棄されたゲームオブジェクトに継続アクセスし続けているが参照できないというエラーのようです。

targetE = GameObject.FindGameObjectWithTag("Enemy");

を

Vector3.Distance(targetE.transform.position, transform.position)で参照し続けているからではないでしょうか。

敵がnullであったなら計算しないという処理を追加してみるのはどうでしょうか。

ゲームオブジェクトの全滅判定ならGameObjectの配列に敵を格納してGameObject.FindGameObjectsWithTag.Lengthで数えるといいと思います。

FindGameObjectは重い処理らしいのでUpDate内で毎フレームでの取得はやめておいた方が良いです。
コルーチンで定期的にとか、何かのアクションが終わるタイミングとかで使うといいと思います。

コルーチンでシーン内にいる敵の数を１秒毎にカウントしてコンソールに表示するサンプルスクリプトを考えてみました。

public class Ecount : MonoBehaviour {

    GameObject[] targetE;

    void Start () {
        StartCoroutine("EnemyCount");
    }
	
    IEnumerator EnemyCount() {

        for (;;) {
        targetE = GameObject.FindGameObjectsWithTag("Enemy");
        Debug.Log(targetE.Length);
        
        yield return new WaitForSeconds(1.0f);
        }
    }
}

これを応用すれば全滅判定は出来ると思います。

追加サンプルコード

using UnityEngine;

public class ExistEnemyCount : MonoBehaviour {

    GameObject[] targetE;

    void Start() {

        ExistEnemy();
    }

    void Update() {

        if (Input.GetKeyDown(KeyCode.G)) {

            ExistEnemy();

            if (targetE.Length == 0) {
                Debug.Log("敵はいない");
            }else
            if (targetE.Length >= 1) {
                Debug.Log("敵はまだ"+ targetE.Length+"体存在する");
            }
        }
    }

    //シーン内の敵の数をサーチする
    void ExistEnemy() {
        targetE = GameObject.FindGameObjectsWithTag("Enemy");
    }
}

Comments

[jum6948]

すいません、技術不足で応用できませんでした。
質問なのですが、【for(;;)】というのはどういう条件なのでしょうか？

【現状報告】
敵が2体いる時に、1体目を倒したあと2体目を狙ってくれなかったのですが
ネットで拾ったLinQというものを使うのを試しました。
```
//Find all enemies within range and add them to list. Order list from closest to furtherest
//範囲内のすべての敵を見つけてリストに追加します。furtherestに最も近いものからの注文リスト
enemyList = GameObject.FindGameObjectsWithTag("Enemy").Select(g => g.transform).OrderBy(g => (g.position - target.position).sqrMagnitude).ToArray();
```
というものを足して、2体目を狙うようになってくれました。
またtargetEをgameobjectからtransformに変更しました。

[cloudmaterial]

for(;;)は単にEnemyCount()を随時機能させるための無限ループです。簡単にコルーチンで使うために使用しました、それを使うべきということではありません。ようするに、サンプルコードEcountは、targetE.Lengthで敵の数が取得できるということを言いたかっただけです。

GameObject.FindGameObjectsWithTagは重い処理なので、UpDate内で毎フレームでは使わない方が良いです。サンプルをコルーチンにしなかった方が良かったかもしれません。回答の方へ違うサンプルコードを追加しました。

サンプルコード「ExistEnemyCount」はGキーを押すとExistEnemy()を実行しシーン内のEnemyタブの付いたゲームオブジェクトを配列に格納します。そして、シーン内にいるEnemyの数によって条件分岐でメッセージが変わります。このような感じで、敵を倒してDestroyするタイミングなどでメソッドを呼び出してその都度サーチするようにすればUpdate内でも負荷をかけず条件分岐に使えるわけです。

シーン内にいる敵の中で最も近い敵の取得方法はLinQまで使わずとも実装は出来るとは思いますが、LinQを使う方法は当方の回答できる範囲を超えているので何とも言えません。「Unity5ゲーム開発レピシ」（翔泳社）という書籍の中の「一番近い敵をロックオンするには」という項目にLinQを使わずにシンプルに一番近い敵を取得する方法の解説が載っています。一度読んでみると参考になるかもしれません。

当方としては、今回の質問内容はエラーの原因とシーン内の敵の数の取得方法を知りたいということだと判断してそれについて回答しました。このように質問内容が変わっていくものだとは想定していませんでした。このような大規模なプロジェクトとなると付きっきりで完成までのお付き合いはちょっと出来ません。また、このように回答者が何に対して回答したかの質問を消していくのはどうかとも思います。問題が一つ解決したのなら、また新たに次の質問スレッドを立てていく方が質問の内容が残りますし、回答してくれる方も見つけやすいし回答しやすいと思いますが、どうでしょう。申し訳ありませんが、私の回答はここまでにさせて下さい。回答できる方がいらっしゃいましたら、どなたか引き継いでもらえたらと思います。

[jum6948]

修正した、文言をいれると、文字数オーバーで削らなければならなかったので、最初の内容は、解決したので、消してしまいました。
認識が甘かったです。申し訳ありませんでした。

Answer 2

Linqを使って解決しました。
最善の正解かどうかわかりませんが、「敵を倒す、次の敵を狙う」の行動はできるようになっていると思います。

using System.Collections;
using System.Collections.Generic;
using System.Linq;
using UnityEngine;
using UnityEngine.AI;

    GameObject targetP;             //プレイヤー
    public Transform target;        //プレイヤーの位置
    public Transform[] enemyList;   //敵のリスト
    public Transform targetE;       //現在選択されている敵

void Start()
{
    //ターゲットを取得
    targetP = GameObject.FindGameObjectWithTag("Player");
    target = targetP.transform;
}
void update
{
    //敵がいない
    if (enemyList.Length == 0)
    {
        //敵がいない時に実行する内容
    }
    //敵がいる
    else if (enemyList.Length > 0)
    {
        //対象を選択する
        SelectEnemy(0);
        //敵がいる時に実行する内容
    }
}
//LateUpdateは、Updateが実行されたあと一回だけ実行される
void LateUpdate()
{
    //Find all enemies within range and add them to list. Order list from closest to furtherest
    //範囲内のすべての敵を見つけてリストに追加します。furtherestに最も近いものからの注文リスト
    enemyList = GameObject.FindGameObjectsWithTag("Enemy").Select(g => g.transform).OrderBy(g => (g.position - target.position).sqrMagnitude).ToArray();

    //一番近い敵との距離を算出
    if (targetE != null)
    {
        EnemyDistance = Vector3.Distance(targetE.transform.position, transform.position);
    }
    //プレイヤーとの距離を算出
    PlayerDistance = Vector3.Distance(targetP.transform.position, transform.position);
}
//敵を選択(０になる対象を選択)
void SelectEnemy(int i)
{ // 0 = Nearest / New List, 1 = Forwards, -1 = Backwards
    switch (i)
    {
        case 0:
            //最初に選ばれた敵をデフォルトにする
            targetE = enemyList.FirstOrDefault();
            break;
        default:
            //デフォルトの条件：敵のリストの中から、
            targetE = enemyList[((System.Array.FindIndex(enemyList, g => g == targetE) + i) % enemyList.Length + enemyList.Length) % enemyList.Length];
            break;
    }
}

今回は、思っていた回答を得られなかったので、自分の文章がおかしかったのかと思い、質問の仕方を変更しました。不快に思われた方、申し訳ありませんでした。