Unityで、一番近くの敵に自動で弾を撃つタレットをつくりたいのですが…

Question

現在、「指定した１つのオブジェクトに向かって弾を撃つ」スクリプトと「指定したタグを持つオブジェクトの中で一番近いものを見つける」スクリプトを拾ったので、どうにかくっつけて「指定したタグをもつオブジェクトの中で一番近いものに向かって弾を撃つ」スクリプトにしたいのですが、うまく組み合わせることができません。

以下の二つを組み合わせて使う方法、あるいは他の方法でも今回やりたいことが実現できる方法があれば教えてください。

####【指定した１つのオブジェクトに向かって弾を撃つ】

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

public class LookAt_Pod : MonoBehaviour
{
    public GameObject BulletPrefab;
    public float bulletSpeed;
    public float span = 1f;
    private int timeCount = 0;
    private GameObject targetObj; // 注視したいオブジェクト

    void Update()
    {
        this.transform.LookAt(targetObj.transform);

        timeCount += 1;

        if (timeCount > span)
        {
            timeCount = 0;
            // 敵の弾を生成する
            GameObject Bullet = Instantiate(BulletPrefab, transform.position, Quaternion.identity);

            Rigidbody BulletRb = Bullet.GetComponent<Rigidbody>();

            // 弾をforwardに飛ばす
            BulletRb.AddForce(transform.forward * bulletSpeed);

            // ３秒後に弾を削除する。
            Destroy(Bullet, 3.0f);
        }
    }
}

####【指定したタグを持つオブジェクトの中で一番近いものを見つける】

    GameObject SerchEnemy(string Enemy)
    {
        float dis = 0;           //距離保存用
        float nearDis = 0;          //最も近いオブジェクトの距離        
        GameObject targetObj = null; //オブジェクト

        //  指定したタグのオブジェクトをすべて取得する
        foreach (GameObject obj in GameObject.FindGameObjectsWithTag(Enemy))
        {
            //  敵との距離を計算
            dis = Vector3.Distance(obj.transform.position, transform.position);

            //  より近いオブジェクトか、距離が０のオブジェクトなら更新
            if (nearDis > dis || dis == 0)
            {
                nearDis = dis;          //  距離を保存            
                targetObj = obj;        //  ターゲットを更新
            }
        }
        //最も近かったオブジェクトを返す
        return targetObj;
    }

よろしくお願いいたします。

Answer 1

LookAt_Pod側にSetTargetObj関数を実装し、
SerchEnemyの返り値を渡す感じじゃ無理なのでしょうか？

LookAt_Pod pod;
EnemyManager enemyManager;

pod.SetTargetObj(enemyManager.SerchEnemy());

また、単純にLookAt_PodクラスにSerchEnemy関数を実装しちゃえば
こんな感じでできそう

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

public class LookAt_Pod : MonoBehaviour
{
    public GameObject BulletPrefab;
    public float bulletSpeed;
    public float span = 1f;
    private int timeCount = 0;
    private GameObject targetObj; // 注視したいオブジェクト

    void Update()
    {
        this.transform.LookAt(SerchEnemy().transform);

        timeCount += 1;

        if (timeCount > span)
        {
            timeCount = 0;
            // 敵の弾を生成する
            GameObject Bullet = Instantiate(BulletPrefab, transform.position, Quaternion.identity);

            Rigidbody BulletRb = Bullet.GetComponent<Rigidbody>();

            // 弾をforwardに飛ばす
            BulletRb.AddForce(transform.forward * bulletSpeed);

            // ３秒後に弾を削除する。
            Destroy(Bullet, 3.0f);
        }
    }

   GameObject SerchEnemy(string enemy)
   { 
      // 省略
   }

}

でもこれ、懸念点があるとすれば、
LookAt_Podクラスが複数あると、
複数のLookAt_Podが全員、敵を全検索してしまうので、
やはりSerchEnemy関数はEnemyManagerのようなもので持っておいて、
LookAt_Podにセットする形がふさわしいかなと思います！

Comments

[thaya]

丁寧にいろいろなご提案をいただき、誠にありがとうございます。

「EnemyManager」などを作って管理する方法はとてもよさそうですが、初心者すぎて結局組み込むことは叶いませんでした…
もう少しプログラミングを理解して、ちゃんと扱えるようになったら活用させていただきたいと思います。

[MasujimaRyohei]

2つ目の方法でもできませんでしたか？

[thaya]

２つ目にご提示いただいた方法は、"Enemy"を狙ってほしい場合は、
this.transform.LookAt(SerchEnemy().transform);
の()のところに"Enemy"を入れて
this.transform.LookAt(SerchEnemy("Enemy").transform);
みたいにして使用するのでしょうか？
何か入れないと
LookAt_Pod.SerchEnemy(string)' の必要な仮パラメーター 'Enemy' に対応する特定の引数がありません。
というエラーになってしまいます。

"Enemy"と入れたとしても、エラーなくゲームが再生できるものの、foeward方向に弾を飛ばすだけで求めていた動きはしてくれませんでした…

[MasujimaRyohei]

> LookAt_Pod.SerchEnemy(string)' の必要な仮パラメーター 'Enemy' に対応する特定の引数がありません。
こちらはあなたの作った通りstring型の引数を渡してあげてください。

今回、この二つのスクリプトはきちんと動いている、という認識でしたが、
それは合っていますでしょうか？

あと、タグの処理を含んでいるので、敵のオブジェクトにきちんとEnemyタグがついているかどうかも確認お願いいたします！

[thaya]

すみません、スクリプトがちゃんと動くかどうかもテストしていませんでした。

【指定した１つのオブジェクトに向かって弾を撃つ】のほうは動作しますが、【指定したタグを持つオブジェクトの中で一番近いものを見つける】のほうは拾い物ののスクリプトをそのままコピペしただけです。
http://hellomoneyworld.seesaa.net/article/449510462.html
ブログに書くくらいだから、エラーもでてないし当然動作しているんだろうと、安易に思い込んでしまっていました。

Dbag.Logをうまく組み込んだりして、動作の確認をすることすら今の自分にはできないので、とりあえず質問してみれば答えがもらえるのかなと…考えが甘かったです…

タグは一応ちゃんとついてます。

[MasujimaRyohei]

一旦頑張って各機能が動くかどうかを試してみたから合成させたほうがいいですね
これは単体テストっていうもので、
一つの機能ごとに動くことを確認しないと、
何が間違っているのかがわからないので手の出しようがなくなってしまうからです…！

Answer 2

public class LookAt_Pod : MonoBehaviour
{
    // （中略）
    
    GameObject SerchEnemy(string Enemy)
    {
        // 中身は2つ目のコードと同じ
    }
    
    void Update()
    {
        targetObj = SerchEnemy("タグ名");
        
        // 続きは1つ目のコードと同じ
    }
}

Comments

[thaya]

ご回答いただき、ありがとうございます。

なるほど！とおもい、targetObj = SerchEnemy("タグ名");　を追加してみたのですが、あまりうまくいきませんでした。

visual studio側ではエラーになっていないので、スクリプトではなくインスペクターの内容などに不備があるのかもしれません…

ちなみに、
GameObject targetObj = null; //オブジェクト
と
this.transform.LookAt(targetObj.transform);
に対して
NullReferenceException: Object reference not set to an instance of an object LookAt_Pod.Update
のエラーが出ています。

もし、なにか改善案などがございましたら、ご教授いただけますと幸いです。

[fiveHundred]

おそらくですが、「GameObject.FindGameObjectsWithTag(Enemy)」で1つも見つからなかったのが原因では無いかなと思います。
敵のオブジェクトが存在しているかどうか、敵のオブジェクトにタグが設定しているかを確認してみてください。
もし、敵が存在しない時があるのであれば、場合分けをする必要があります。

[thaya]

なるほど、敵がずっといる場合でないとこのスクリプトではダメなんですね…

敵を最初からずっといる状態にしたら、
GameObject targetObj = null; //オブジェクト
のほうのエラーは出なくなりました。
ありがとうございます。

しかし、敵を感知することができているのに、
this.transform.LookAt(targetObj.transform);
のほうが機能しないのはどうしてなのでしょうか…

なにかお気づきでしたら、よろしくお願いいたします。

[fiveHundred]

「機能しない」というのは「NullReferenceExceptionが出てしまう」ということでしょうか？

[thaya]

はい、そうです。
あいまいな書き方をしてしまって申し訳ありません。

[fiveHundred]

そうであれば、見つからない場合はtargetObjはnullなので、その場合は（「this.transform.LookAt(targetObj.transform);」などの）処理をしない、ということをする必要があります。
（先ほど言った「場合分け」とはこういう意味です）

[thaya]

正直ちゃんと理解できてはいませんが、自分がやりたかった「指定したタグをもつオブジェクトの中で一番近いものに向かって弾を撃つ」という動きが単純にふたつのスクリプトを組み合わせるだけでは、動作しないということはわかりました。

ちゃんと動かすためには、他にもいろいろと処理が必要なんですね…
何度も返信してくださって、ありがとうございます。

Answer 3

https://blog.beatdjam.com/entry/2014/10/22/032751
このサイトに載っていたスクリプトを使用することで、やりたかった動きを実現することができたので、自己解決ということにさせていただきたいと思います。

回答やアドバイスをくださった方々、ありがとうございました。
とても参考になりました。

今回つくったタレットは、範囲の指定がなくシーン内の対象タグを持つオブジェクトをすべて狙ってしまうので、実際にゲームで使用するためには、もう一工夫必要だと思いますが、自分と同じようなものをつくりたいと思った初心者の方のためにコードを置いておこうと思います。

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

public class LookAt_Enemy : MonoBehaviour
{
    public GameObject BulletPrefab;
    public float bulletSpeed;
    public float span = 1f;
    private int timeCount = 0;

    private GameObject nearObj;         //最も近いオブジェクト
    private float searchTime = 0;    //経過時間

    // Use this for initialization
    void Start()
    {
        //最も近かったオブジェクトを取得
        nearObj = serchTag(gameObject, "Enemy");
    }

    // Update is called once per frame
    void Update()
    {

        //経過時間を取得
        searchTime += Time.deltaTime;

        if (searchTime >= 0.1f)
        {
            //最も近かったオブジェクトを取得
            nearObj = serchTag(gameObject, "Enemy");

            //経過時間を初期化
            searchTime = 0;
        }

        if (nearObj == null)
        {
            this.transform.rotation = Quaternion.identity;   //敵がいない場合は回転をリセット
        }
        else
        {
            //対象の位置の方向を向く
            transform.LookAt(nearObj.transform);

            timeCount += 1;

            if (timeCount > span)
            {
                timeCount = 0;
                // 敵の弾を生成する
                GameObject Bullet = Instantiate(BulletPrefab, transform.position, Quaternion.identity);

                Rigidbody BulletRb = Bullet.GetComponent<Rigidbody>();

                // 弾をforwardに飛ばす
                BulletRb.AddForce(transform.forward * bulletSpeed);

                // ３秒後に弾を削除する。
                Destroy(Bullet, 3.0f);
            }
        }
    }

    //指定されたタグの中で最も近いものを取得
    GameObject serchTag(GameObject nowObj, string tagName)
    {
        float tmpDis = 0;           //距離用一時変数
        float nearDis = 0;          //最も近いオブジェクトの距離
        GameObject targetObj = null; //オブジェクト

        //タグ指定されたオブジェクトを配列で取得する
        foreach (GameObject obs in GameObject.FindGameObjectsWithTag(tagName))
        {
            //自身と取得したオブジェクトの距離を取得
            tmpDis = Vector3.Distance(obs.transform.position, nowObj.transform.position);

            //オブジェクトの距離が近いか、距離0であればオブジェクト名を取得
            //一時変数に距離を格納
            if (nearDis == 0 || nearDis > tmpDis)
            {
                nearDis = tmpDis;
                targetObj = obs;
            }

        }
        //最も近かったオブジェクトを返す
        return targetObj;
    }
}