Unityでモグラ叩きゲームを作っています。プレハブのモグラを叩いて消えたら3秒後に生成する仕組みを作りたいです。

Question

StageViewスクリプトで、monsterViewPrefabをインタンシエイトしています。

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

public class StageView : MonoBehaviour
{
    [SerializeField] GameObject monsterViewPrefab;
    [SerializeField] GameObject monster;

    public void SpawnMonster()
    {
        monster = Instantiate(monsterViewPrefab);
        monster.transform.SetParent(transform, false);
        //return monster.GetComponent<MonsterView>();
    }
    
    
}

その後、MonsterViewスクリプトを用いてmonsterViewPrefabにボタン機能をアタッチしており、生成されたmonsterViewPrefabをクリックすると、Destroyされます。Destoryされ3秒後にまたmonsterViewPrefabを生成したいのですが、うまくいきません。

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

public class MonsterView : MonoBehaviour
{
    //[SerializeField] StageView stageView;
    /*Action onTap; // 
	public void AddListenerOnTap(Action action)
	{
		onTap += action;
	}*/

    public void OnTap()
	{
		Debug.Log("MonsterをTap");
        Destroy(this.gameObject);
        //Invoke("SpawnLead",3.0f);
	}

    /*void SpwanLead()
    {
        stageView.SpawnMonster();
    }*/
}

MonsterViewスクリプトでOnTap関数の中で、ボタンが押されたら、 SpwanLead関数を作って、その中にmonsterViewPrefabを生成するSpawnMonster関数を入れて見ましたが、Playボタンを押しても反応がありませんでした。

生成する関数SpawnMonsterは、下のMainPresenterスクリプトで行っています。
StageViewクラスでは、monster変数にInstantiateしたものを入れているので、
これがnullで何も入っていなかったらInvoke関数を用いて、Invoke("SpawnMonster", 3.0f);このように呼び出せるかと思いましたが、「stageView.monster == null」のmonsterはアクセスできない保護レベルになっていてできませんでした。

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

public class MainPresenter : MonoBehaviour
{
    [SerializeField] PlayerStatusView playerStatusView;
    [SerializeField] StageView stageView1;
    [SerializeField] StageView stageView2;
    [SerializeField] StageView stageView3;
    [SerializeField] GameObject menuView;
    [SerializeField] GameObject finalScoreText;
    
    [SerializeField] GameObject leftTimeText;
    //TimeController timeController;
    PlayerModel playerModel;
    StageView stageView;
    public float leftTime = 20;



    public void OnRetryButton()
    {
        SceneManager.LoadScene("Main");
    }

    public void OnToTitleButton()
    {
        SceneManager.LoadScene("Title");
    }

    void Start()
    {
        playerModel = new PlayerModel();
        stageView = new StageView(); 

        //timeController = new TimeController();

        stageView1.SpawnMonster();
        stageView2.SpawnMonster();
        stageView3.SpawnMonster();
        //playerStatusView.UpdateText(playerModel);
        leftTimeText.GetComponent<Text>().text = ((int)leftTime).ToString();
    }

    void Update()
    {
      
        leftTime -= Time.deltaTime;
        
        if (leftTime < 0) leftTime = 0;
        //GetComponent<Text>().text = "残り時間：" + ((int)leftTime).ToString();


        if(stageView.monster == null)
        {
            Debug.Log("nothing");
            Invoke("SpawnMonster", 3.0f);
        }



        if (leftTime <= 0)
        {
            menuView.SetActive(true);
            finalScoreText.SetActive(true);
        }

        leftTimeText.GetComponent<Text>().text = "残り時間：" + ((int)leftTime).ToString();
    }
       
}

行いたいことは、プレハブのモグラを叩いて消えたら3秒後に生成することです。
どなたか、ご教示よろしくお願いいたします。
何か不十分な点があれば、情報を追記いたします。

Answer 1

StageViewのmonsterには[SerializeField]が付いています。
こいつはざっくり言うと「この変数はprivate（StageViewの中からしかアクセス出来ない）けど、インスペクタ上には表示されてそこから値を設定出来るよ」という効果があります。
なのでStageViewの外（MainPresenter）からアクセス出来ないのは当然というか仕様通りの動きです。

考えられる方法は3つ。

１．monsterをpublicにする
[SerializeField]を取ってpublicにしてしまう方法。
但しこうするとStageViewの外からmonsterを書き換えられるので、設計上あまりよくありません。

２．Getterを作る／チェック用メソッドを作る
StageViewにpublic void GetMonster() {return monster;}的なメソッドを作って呼び出し。
Nullかどうかのチェックをしたいだけならpublic void IsNullMonster() {return (monster == null);}でもいいかもしれません。
ただ今回の本質はここではないです。

３．全部StageViewに管理させる【最適】
今のInvokeの書き方上、これだとMainPresenterのSpawnMonsterを呼び出そうとするので間違っています（＝StageViewのSpawnMonsterを呼び出すようになっていない）。
更にUpdateの中に組み込んでいるので、monsterがnullになってから実際にメソッドが実行されるまでの3秒間、ずっとInvokeされ続けるので3秒後に大量に生成されることでしょう。

やりたいことは「プレハブのモグラを叩いて消えたら3秒後に生成」なので、全部StageViewに任せましょう。
つまり「自分のモグラの生成」と「自分のモグラが存在しているかのチェック」と「自分のモグラが消えてたら3秒後に生成」ということだけ注視させます。
本当はコルーチンにした方がいいかもですが雑に書くと以下の通り。（動作未チェック）
後はどこかからSpawnMonsterメソッドを呼び出せば後は延々勝手に動作し続けます。（停止機能が欲しければ任意で付け足してください）

public class StageView : MonoBehaviour
{
    [SerializeField] GameObject monsterViewPrefab;
    [SerializeField] GameObject monster;
    bool isThereMonster; //monster存在フラグ。待機時間を設ける為にフラグを持つ

    public void SpawnMonster()
    {
        monster = Instantiate(monsterViewPrefab);
        monster.transform.SetParent(transform, false);
        isThereMonster = true;
    }

    void Update()
    {
        // monsterがさっきまで存在していたが今は無い場合、3秒後に生成
        if (isThereMonster && monster == null) {
            isThereMonster = false;
            Invoke("SpawnMonster", 3.0f);
        }
    }
}

Comments

[defour]

StageViewに管理させて解決できました。丁寧にサンプルコードまで載せていただきありがとうございました。