(Unity)最も近いオブジェクトの取得について

Question

前提・実現したいこと

一番近いターゲットの方向へ移動。
それを破壊した後、次に近いターゲットへの接近。

ここに質問の内容を詳しく書いてください。
http://beatdjam.hatenablog.com/entry/2014/10/22/032751

上記サイトを参考にして、Unity２Dにて一番近いenemyへ接近するという機能を実装しようとしています。
一匹目のenemyに接近するのはうまくいくのですが、それを破壊した後、次の最も近いenemyへターゲットを移すことができていません。

おそらく、２Dなので敵位置の取得をLookAtの代わりで補っている部分が問題なのだろうとは思うのですが、どうすればうまく修正できるのかがわかりません。

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

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.

該当のソースコード

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

public class Ally2 : MonoBehaviour {

	Chara chara;
	Rigidbody2D rd;
	private GameObject nearEnemy;
	private float seachTime = 0;

	IEnumerator Start () {
		nearEnemy = searchTag(gameObject,"Enemy");
		chara = GetComponent<Chara> ();
		rd = GetComponent<Rigidbody2D> ();

		while (true) {
			Instantiate (bullet, AttackPos);
			yield return new WaitForSeconds (AttackDelay);
		}
	}

	void Update () {
		seachTime = Time.deltaTime;
		if (seachTime >= 1.0f) {
			nearEnemy = searchTag (gameObject, "Enemy");
			seachTime = 0;
			}
		//	
		Vector3 diff = (nearEnemy.transform.position - this.transform.position);
		this.transform.rotation = Quaternion.FromToRotation (Vector3.right,diff); 

		var v = new Vector2 (1.0f*Time.deltaTime, rd.velocity.y) * chara.Speed;
		transform.Translate (v);
	}

	GameObject searchTag(GameObject nowEnemy,string tagName){
		float tmpDis = 0;
		float nearDis = 0;
		GameObject targetObj = null;

		foreach (GameObject obs in GameObject.FindGameObjectsWithTag(tagName)) {
			tmpDis = Vector2.Distance (obs.transform.position, nowEnemy.transform.position);

			if (nearDis == 0 || nearDis > tmpDis) {
				nearDis = tmpDis;
				targetObj = obs;
			}
		}
		return targetObj;
	}
}

試したこと

LookAtの代わりは下記サイトを参考にしました。
http://karaagedigital.hatenablog.jp/entry/2016/09/11/192600

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

よろしくお願いします。

Comments

[Bongo]

オリジナルのコードでは「searchTime += Time.deltaTime;」となっている部分を「seachTime = Time.deltaTime;」としたのは意図があってのことでしょうか?ここが書き換えられていると「1秒おきに一番近い敵を再検索」という動きにならなくなりそうですが...

[ssss_grid]

あーそこでしたか！ありがとうございます。見落としでした。ただ、倒すたびに新たにターゲットを見つけるまでの一瞬だけ同じ箇所にnullのエラーが出ます（止まったりはしないですが）

Answer 1

※エラーの箇所を誤解していたので、回答を差し替えました。

回答の前に1点指摘を。

GameObject searchTag(GameObject nowEnemy,string tagName){ // nowEnemyではなく、
GameObject searchTag(GameObject player,string tagName){ // playerでは？

で、本題です。

エラー箇所はVector3 diff = (nearEnemy.transform.position - this.transform.position);の方でした。

nearEnemyがどの時点でnullになるかデバッガで確認し、その原因を調べたり考えたりしてみてください。

予想ですがおそらく、敵機が0になった結果、searchTagメソッドのGameObject.FindGameObjectsWithTag(tagName)の部分で取得できる敵機がなく、targetObjが初期値nullのままになってしまったのかなと思います。

もしそうであれば、敵機がいない場合はそちらを向かないよう、if文で制御すればいいかと思います。

Comments

[ssss_grid]

ご回答ありがとうございます！
そうですねエラーをきちんと把握できていませんでした。
エラー箇所は
Vector3 diff = (nearEnemy.transform.position -
this.transform.position);
の方でした。すいません。

nearEnemyの場合でも、上記エラー箇所の上にif文でnullの時には仮の敵を置く処理がよいのでしょうか。

[退会済みユーザー]

回答を差し替えました

Answer 2

Bongo様のご指摘の通り
「seachTime = Time.deltaTime;」
「searchTime += Time.deltaTime;」
私の見落としでした。
ただ、倒すたびに新たにターゲットを見つけるまでの一瞬だけ同じ箇所にnullのエラーが出ます（止まったりはしないですが、、、）

Comments

[Bongo]

では、敵の再検索を行う条件を「if (nearEnemy == null || seachTime >= 1.0f) { ...」としてみてはいかがでしょうか?
これなら、1秒経過した場合に加えて、nearEnemyが破壊済みの場合にも敵の再検索が行われるようになるかと思います。

さらに最後の敵を破壊した後には、ppnさんからご指摘のあった「敵が見つからずsearchTagがnullを返す」という状況になるでしょうから、ppnさんの案も折衷して、回転部分にさらにnullチェックを追加するのがよさそうです。

if (nearEnemy == null) {
Vector3 diff = (nearEnemy.transform.position - this.transform.position);
this.transform.rotation = Quaternion.FromToRotation (Vector3.right,diff);
}

※余談ですが、「破壊済みのオブジェクトを参照する変数をnullと比較するとtrueになる」という挙動はUnityEngine.Object特有でして、??演算子を使う場合など「本当のnull」と「破壊済みnull」の違いを考慮する必要のある状況もありますのでご注意ください。
参考:「Unityのnullはnullじゃないかもしれない」(https://qiita.com/satanabe1@github/items/e896303859be5d42c188 )

[ssss_grid]

ありがとうございます！
「if (nearEnemy == null || seachTime >= 1.0f) { ...」で一瞬のエラーは消えました。

「本当のnull」と「破壊済みnull」の違いを考慮する必要のある状況があるのですね。勉強になります。ありがとうございます！