PrefabにアタッチしたスクリプトでFind関数を使うことについて

Question

現在Unityにてキャラクターを移動させ、落ちてきたアイテムを拾いスコアを稼ぐゲームを作成しています。

落ちてくるアイテムはプレハブ化したものをインスタンス化して使用していて、以下のスクリプトをアタッチしているのですが

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

public class ItemController : MonoBehaviour
{
    private PlayerController _playerController;
    private ItemGenerator _itemGenerator;
    private ModeManager _modeManager;
    private UIController _uiController;
    private ScoreController _scorecontroller;
    private AudioSource _meron;
    private AudioSource _specialMeron;
    private AudioSource _bad;

    void Start()
    {
        //prefabなのでFind
        _playerController = GameObject.Find("Player").GetComponent<PlayerController>();
        _itemGenerator = GameObject.Find("ItemGenerator").GetComponent<ItemGenerator>();
		_modeManager = GameObject.Find("ModeManager").GetComponent<ModeManager>();
        _uiController = GameObject.Find("UIController").GetComponent<UIController>();
        _scorecontroller = GameObject.Find("ScoreText").GetComponent<ScoreController>();
        AudioSource[] audioSources = GameObject.Find("SE").GetComponents<AudioSource>();
        _meron = audioSources[0];
        _specialMeron = audioSources[1];
        _bad = audioSources[2];
    }

    void FixedUpdate()
    {
        //アイテム落下
        transform.Translate(0, _itemGenerator.Speed * Time.deltaTime, 0);
    }

    void OnTriggerEnter2D(Collider2D collision)
    {
		//床より下に行ったとき
		if (collision.gameObject.tag == "DestroyFloor")
		{
			Destroy(gameObject);
		}

        //床についとき
        if (collision.gameObject.tag == "Floor")
        {
            if (gameObject.tag == "Meron" || gameObject.tag == "SpecialMeron")
            {
                if (!_modeManager.MaxMode && _modeManager.ItemDrop)
                {
                    _bad.PlayOneShot(_bad.clip);
                    _playerController.GameOver = true;
                }
            }
            else
            {
                Destroy(gameObject);
            }
        }

        //プレイヤーに当たったとき
        if (collision.gameObject.tag == "Player")
        {
            if (!_playerController.GameOver)
            {
                if (gameObject.tag == "Meron")
                {
                    _meron.PlayOneShot(_meron.clip);
                    _uiController.PlusScoreSetActiveTrue(10);
                    _scorecontroller.ScorePlus(gameObject);
                    Destroy(gameObject);
                }

                if (gameObject.tag == "SpecialMeron")
                {
                    _specialMeron.PlayOneShot(_specialMeron.clip);
                    _uiController.PlusScoreSetActiveTrue(100);
                    _scorecontroller.ScorePlus(gameObject);
                    _modeManager.MaxMode = true;
                    _modeManager.ItemDrop = false;
                    Destroy(gameObject);
                }
            }

            if (gameObject.tag == "Onigiri")
            {
                _bad.PlayOneShot(_bad.clip);
                _playerController.GameOver = true;
            }
        }
    }
}

毎回FindでほかのManagerがアタッチされたオブジェクトを探しており、メモリに負荷をかける原因になるのではないだろうかと思いました。
Publicで登録するなどにしたいのですがPrefabなのでそれもできず、どのようにすればいいか悩んでおります。
Findで検索する以外にどのようなやり方がありますでしょうか？何かアドバイスいただけますと幸いです。

Comments

[hermit19901127]

ちょっと暇つぶしに質問主さんの過去の質問を見てましたが、シングルトンパターンに抵抗があるように見受けられました。

その場合、Game.Sceneみたいに、シーンのオープンと開始に本ゲームが始まるのであれば
・Stage生成時に「StageManager」を生成
・StageManagerのStartで、主要６つをFindで取得。StageManagerのメンバ変数とする。
・主要６つの相互のやり取りをStageManagerで統括する

こうするとゲーム終了で、Game.Sceneの終了と同時に
StageManagerも破棄され、変に生きてる実体も無くなり、すっきりした処理になる思います。

一番きれいな回答ですが、説明が難しく、言葉足らずなのはご容赦ください…

Answer 1

質問主様のおっしゃる通り、Findは重く、Update処理等では使わない方が賢明です。
ですが、Startのような、1フレームで終わるような処理であれば、似たような処理を行うものがゲーム中に100個も200個も作られるようなことが無い限りは、致命的なフレーム落ち等は起こらない…ですが、

いちいちFindで参照を持たず、「どのクラスからでもアクセスが簡単に行える機構が欲しい」となれば
シングルトンパターンを使うのもありかと思います。
https://unity.moon-bear.com/unitychan-coin/singleton-gamemanager/

このシングルトンパターンで作ったクラスは、特に前処理も必要なく、ほぼすべてのクラスが自由にアクセス可能となります。

そのシングルトンクラスに

       _playerController = GameObject.Find("Player").GetComponent<PlayerController>();
        _itemGenerator = GameObject.Find("ItemGenerator").GetComponent<ItemGenerator>();
        _modeManager = GameObject.Find("ModeManager").GetComponent<ModeManager>();
        _uiController = GameObject.Find("UIController").GetComponent<UIController>();
        _scorecontroller = GameObject.Find("ScoreText").GetComponent<ScoreController>();
        AudioSource[] audioSources = GameObject.Find("SE").GetComponents<AudioSource>();

シングルトンの初期化タイミングや、これらが生成されたタイミングでシングルトンクラスに代入すれば、
それ以降、どのクラスからも自由に上記のクラスへアクセスが可能です。

---

ここまでは聞こえがよく、凄く強い機能に見えますが、諸刃の剣のようなもので
「どこからでも書き換えられる」事は「自分が修正なりして消したはずの所から急に書き換えられる」等で
特定し辛いエラーの原因にもなったりするので、運用には気を付けてください。

それでも、シーンの切替等が発生しても値を保持できる点で、ゲーム中のデータ保存の方法としては強力だとは思います…

---

追記

これは、古い知識かもしれませんが、Unityのプロジェクトセッティングか、それに類するものに
「シングルトンオブジェクト」を専用に登録できるメニュー等があった気がします…
余力があったら、探してみるのも一考…

---

与太話（聞き流しても大丈夫です）

普通に作れば大丈夫ですが、シングルトンオブジェクト、下手な実装をすると
ゲームを終了してもシングルトンオブジェクトが生きて、再度ゲームを実行すると
前のゲームのデータを保持したまま実行するみたいなトラブルが起こる事もあります…。

実は、UnityEditor自体が、一つのゲームのプログラムになっていて、
そこで再生してるゲームは、そのプログラムが再生してる現象でしかありません…
static関数をどう管理するかにもよりますが、やり方次第で、
「UnityEditorが起動中は存在し続ける」実装も可能なので、
運悪く、そんな実装をすると、いくら再生してるゲームを終了しようが
UnityEditorそのものを終了しないと初期化処理が走らず、前のゲームのデータを引き継いだまま処理を行う、のような、珍妙な動作が起こります…。

…まぁ、エディタ再生の終了、またはエディタ再生を感知して、シングルトンを常に初期化するようにすれば、そのような問題は起きませんが…

一応、知見というか、豆知識として…