【Unity】動的に生成したPrefabについているボタンを押し、動的に生成された別PrefabをDestroyしたい

Question

前提・実現したいこと

私自身Unityやプログラミングに触れたことがほぼなく、独学での勉強のため言葉足らずになってしまう箇所などがあるかもしれませんが、ご了承くださいm(_ _)m

子供向けの英語の勉強ゲームを作っているのですが、Destroy関数を使用する際に問題にぶつかってしまいました。

ゲームの構成としては

●カエルオブジェクト×１
●題材オブジェクト×１
●ハエオブジェクト×３（それぞれのハエにはアルファベット1文字が振られている）

上記3種類のオブジェクトで成り立っており、
題材オブジェクトを英語で読んだ時の頭文字と同じ頭文字が書いてあるハエオブジェクトをクリックすると正解（カエルの表情が笑顔になる）、
違うハエオブジェクトをクリックすると不正解（カエルの表情が泣き顔になる）という内容のゲームを作ろうとしています。

流れとしては
ハエオブジェクトの子についているボタンを押すとPushButton()関数が実行され、カエルオブジェクトがDestroyされ、別の表情のカエルオブジェクトがInstantiateされるという流れになっています。
また、frogObj = Instantiate(frogNormal); というように、一度frogObjに格納した上でDestroy(frogObj);を実行しようとしています。

別の表情のカエルオブジェクトをInstantiateすることは成功するのですが、元々あったカエルオブジェクトを削除することがうまくいきません。

よろしくお願い致しますm(_ _)m

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

別の表情のカエルオブジェクトをInstantiateすることには成功するのですが、
元々あった変えるオブジェクトを削除することができません。

該当のソースコード

もしかするとInstantiateやDestroy以外の部分のコードが悪さをしているのかもしれませんので、
現状のコードをそのまま載せます。

↓FrogGameManager.cs↓

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

namespace frogGame
{
    public class FrogGameManager : MonoBehaviour
    {
        // Inspectorパネルで各ゲームオブジェクトにPrefabを設定
        // カエルは４種類の表情
        public GameObject frogNormal;
        public GameObject frogSmile;
        public GameObject frogSad;
        public GameObject frogCatch;

        // ハエオブジェクト
        public GameObject fly;

        // 題材オブジェクト（配列）
        public GameObject[] subject;

        // どの題材オブジェクトを使用するかを決めるための変数
        public int number;

        
        public GameObject frogObj;
        public GameObject flyObj;
        public GameObject subObj;

        // 題材オブジェクトの答えを格納するための変数
        public string answer;

        // ハエオブジェクトそれぞれのアルファベットを設定するための変数
        public string choise1;
        public string choise2;
        public string choise3;


        // Start is called before the first frame update
        void Start()
        {
            // 題材オブジェクトを生成
            subObj = Instantiate(subject[number], new Vector3(-6.0f, 2.5f), Quaternion.identity);
            // カエルオブジェクトを生成
            frogObj = Instantiate(frogNormal, new Vector3(-6.0f, -2.5f), Quaternion.identity);

            // ３匹のハエを生成し、それぞれのランダム移動範囲を指定
            for (int i = 0; i <= 2; i++)
            {
                flyObj = Instantiate(fly, new Vector3(Random.Range(-2.5f, 6.5f), Random.Range(-3.0f, 3.0f)), Quaternion.identity);
                var flyController = flyObj.GetComponent<FlyController>();

                if (i == 0)
                {
                    flyController.X_MAX_MOVE_RANGE = 4.0f;
                    flyController.X_MIN_MOVE_RANGE = -1.0f;
                    flyController.Y_MAX_MOVE_RANGE = 3.5f;
                    flyController.Y_MIN_MOVE_RANGE = 1.6f;

                    flyController.choise = choise1;
                }

                if (i == 1)
                {
                    flyController.X_MAX_MOVE_RANGE = 0.0f;
                    flyController.X_MIN_MOVE_RANGE = -2.0f;
                    flyController.Y_MAX_MOVE_RANGE = -1.2f;
                    flyController.Y_MIN_MOVE_RANGE = -3.0f;

                    flyController.choise = choise2;
                }

                if (i == 2)
                {
                    flyController.X_MAX_MOVE_RANGE = 6.5f;
                    flyController.X_MIN_MOVE_RANGE = 3.0f;
                    flyController.Y_MAX_MOVE_RANGE = -1.2f;
                    flyController.Y_MIN_MOVE_RANGE = -3.0f;

                    flyController.choise = choise3;
                }
            }

        }

        public void PushButton()
        {
            // ここのDestroyがうまくいきません。
            Destroy(frogObj);
            Debug.Log("Destoyed frogNormal.");

            // このInstantiateは正常に動きます。
            frogObj = Instantiate(frogCatch, new Vector3(-6.0f, -2.5f), Quaternion.identity);
            Debug.Log("Instantiated frogCatch.");
        }
    }
}

↓FlyController.cs↓

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using frogGame;
using UnityEngine.UI;

public class FlyController : MonoBehaviour
{
    private bool activeMethod;

    private bool caught;
    public float speed = 10.0f;

    private bool isReachTargetPosition;
    private Vector3 targetPosition;
    
    public float X_MIN_MOVE_RANGE;
    public float X_MAX_MOVE_RANGE;
    public float Y_MIN_MOVE_RANGE;
    public float Y_MAX_MOVE_RANGE;
    public float SPEED = 0.02f;

    public Text text;
    public string choise;

    private void Start()
    {
        activeMethod = true;
        caught = false;
        this.isReachTargetPosition = false;
        decideTargetPosition();

        text.text = choise;
    }

    private void Update()
    {
        if(activeMethod == true)
        {
            decideTargetPosition();

            transform.position = Vector3.MoveTowards(transform.position, targetPosition, SPEED);

            if (transform.position == targetPosition)
            {
                isReachTargetPosition = true;
            }
        }

        if(caught == true)
        {
            float step = speed * Time.deltaTime;
            transform.position = Vector2.MoveTowards(transform.position, new Vector2(-6.0f, -2.5f), step);
        }
        
    }

    public void decideTargetPosition()
    {
        if (!isReachTargetPosition)
        {
            return;
        }

        targetPosition = new Vector3(Random.Range(X_MIN_MOVE_RANGE, X_MAX_MOVE_RANGE), Random.Range(Y_MIN_MOVE_RANGE, Y_MAX_MOVE_RANGE));
        isReachTargetPosition = false;      
    }

    public void PushedButton()
    {
        activeMethod = false;
        caught = true;
    }
}

###追記
実際の挙動を動画し、Youtubeに限定公開しました。
必要であればこちらもご覧くださいm(_ _)m

カエルゲーム挙動動画

Comments

[tsuki01]

Destroyに関しては特に問題ない様な気がします。
「発生している問題・エラーメッセージ」の欄に記載している、"元々あったカエルオブジェクト"というのは、Startメソッド内で生成しているオブジェクトでしょうか？
それとも、あらかじめUnityプロジェクトのHerarchy上に存在するオブジェクトでしょうか？
もし後者であれば、Destroyで消えないかもしれません。

[RyotaTakada]

ありがとうございます！
元々あったオブジェクトというのは、Startメソッド内で生成しているオブジェクトです。
上記のスクリプトでUnity側でもエラーがないため、原因が何なのかが理解できていない状態となっています。。。

[tsuki01]

返信と動画リンクありがとうございました。
動画を拝見しましたが、ログも出ているしInstantiateも記載頂いているように出来ていますね。

また、frogObj自体がGameObjectとして定義されているので、「Destroy(frogObj);」もしくは「Destroy(frogObj.gameobject);」のどちらでも消えそうですし。。

パッとお役に立てず申し訳ないですが、現状は以下などで地道にデバッグすることしか思い浮かばない状態です。

１、Canvas上に仮のボタンを配置し、そこからFrogGameManagerクラスのPushButtonメソッドを直接読んで確認してみる。もし消えるなら、FlyControllerクラスが影響している？

２、Destroyではなく、SetActive(false)などを指定した場合も状態が変わらないか確認してみる。

[RyotaTakada]

ご覧いただきありがとうございます！
ご提示いただいた１．２．の方法を試してみましたが、やはりうまく動きませんでした。。。

そのため今回は、新しく生成するカエルオブジェクトのOrder in Layerを１に設定し、無理やりオブジェクトが変わったように見せる形にしました。笑

なにか原因が分かり次第、こちらの投稿を更新させていただきますm(_ _)m

[tsuki01]

提示したものを試して頂いてありがとうございます。
やはりダメだったとのことで申し訳ありません。。

はい、もし原因分かりましたらぜひお願いいたします。
他の方からの回答もあることを祈ってます。

Answer 1

手元でうまく再現できなかったのでとりあえずの案として
まず原因の切り分けを行うためにFrogGameManager.cs単体で動作するかを見てみるとかですかね。

↓ 私がよくやるのはとりあえず画面クリックしたらメソッド叩くようにして動作確認する方法です

 void Update() {
   if (Input.GetMouseButtonDown(0)) {
     PushButton();
   }
 }

これをどこかに入れてもらうと、画面をクリックするとPushButtonが呼び出されて処理が走るかと思います。デバッグログが出ているのでおそらく動くとは思うんですが、念の為です。

次に、参照がちゃんと取れているかを確認してみます。消す前、消したあと、作ったあとでfrogObjの名前を表示してみることで参照がどうなってるかをチェックしてみます。

        public void PushButton()
        {
            // ここのDestroyがうまくいきません。
            Debug.Log("[BEFORE DESTROY] frogObj is " + frogObj.name);
            Destroy(frogObj);
            Debug.Log("Destoyed frogNormal.");

            Debug.Log("[AFTER DESTROY] frogObj is " + frogObj.name);

            // このInstantiateは正常に動きます。
            frogObj = Instantiate(frogCatch, new Vector3(-6.0f, -2.5f), Quaternion.identity);
            Debug.Log("[INSTANTIATE] frogObj is " + frogObj.name);
            Debug.Log("Instantiated frogCatch.");
        }

あとはそもそもDestroyがちゃんと動くのかを確かめるためにインスタンス化の部分を一旦コメントアウトしてみるのも手です。原因を切り分けます。

        public void PushButton()
        {
            // ここのDestroyがうまくいきません。
            Debug.Log("[BEFORE DESTROY] frogObj is " + frogObj.name);
            Destroy(frogObj);
            Debug.Log("Destoyed frogNormal.");

            // このInstantiateは正常に動きます。
            //frogObj = Instantiate(frogCatch, new Vector3(-6.0f, -2.5f), Quaternion.identity);
            //Debug.Log("[INSTANTIATE] frogObj is " + frogObj.name);
            //Debug.Log("Instantiated frogCatch.");
        }

Comments

[RyotaTakada]

お返事が遅れてしまい申し訳ございません。
ご回答ありがとうございます！

一つ一つ順を追って正常に動作しているのかを確かめるということですね。
こちらの方法で一度試してみようと思います！
ありがとうございます！