Unity シングルトンの実装

Question

毎回シングルトンを作るのを楽にするために作ったのですがこのコードは何か問題がありますか？

using UnityEngine;

public static class SingletonInstance
{

	public static bool Instance<T>(ref T instance, T script, GameObject obj)
	{
		if (instance == null)
		{
			instance = script;
			Object.DontDestroyOnLoad(obj);
			return true;
		}
		else
		{
			Object.Destroy(obj);
			return false;
		}
	}
}

using UnityEngine;

public class program : MonoBehaviour
{

	static public program instance;

	void Awake()
	{

		if (!SingletonInstance.Instance(ref instance, this, gameObject))return;
		
			//シングルトン作成時に何か処理があれば
		

	}
}

Comments

[y_waiwai]

実際に動かしてみてそのコードでは問題があるんでしょうか。

[terateraterater]

エラーは出ていませんが処理的にこのコードは問題ないのでしょうか？
オブジェクト削除時にOnDestroy()でinstanceをnullした方がいいですか？

Answer 1

OnDestroy()でinstanceをnull

普通に使う分にはこのままでもいいと思いますが、気になるなら入れておけばいいと思います（無駄かもしれないけどあっても大した負荷ではないので）。

その他、問題点は思い付きません。
staticクラスではなく継承にしたら数文字減るなとかは思いますが、設計や用途によるので何とも言えません。
「特に目立った問題が無いならこれでいいのでは？」「問題が起こったら都度デバッグすればいいのでは？」と思います。