unityのMissingReferenceExceptionについて

Question

#エラー
玉ころがしゲームで一回のゲームが終わった後リトライのボタンを押してシーンを読み直すとスコアを表示するスクリプトでエラーが起きていましたどこを直せばいいのでしょうか？

MissingReferenceException: The object of type 'Text' has been destroyed but you are still trying to access it.
Your script should either check if it is null or you should not destroy the object.
UnityEngine.EventSystems.UIBehaviour.IsActive () (at C:/buildslave/unity/build/Extensions/guisystem/UnityEngine.UI/EventSystem/UIBehaviour.cs:28)
UnityEngine.UI.Graphic.SetVerticesDirty () (at C:/buildslave/unity/build/Extensions/guisystem/UnityEngine.UI/UI/Core/Graphic.cs:223)
UnityEngine.UI.Text.set_text (System.String value) (at C:/buildslave/unity/build/Extensions/guisystem/UnityEngine.UI/UI/Core/Text.cs:213)
ScoreTxt.OnAddScore (System.Int32 score) (at Assets/script/ScoreTxt.cs:27)
DataManager.AddScore (System.Int32 addScore) (at Assets/script/DataManager.cs:45)
item.OnTriggerEnter (UnityEngine.Collider hit) (at Assets/script/item.cs:48)

##ScoreTxt.cs

using UnityEngine;
using UnityEngine.UI;

public class ScoreTxt : MonoBehaviour
{
    /// <summary>
    /// テキストを設定
    /// </summary>
    [SerializeField] Text TxtRender;

    void Start()
    {
        //初期スコアを設定
        OnAddScore(DataManager.Instance.Score);

        // スコア追加イベントを受け取る
        DataManager.Instance.OnAddScore += OnAddScore;
        
    }

    /// <summary>
    /// スコアがカウントされると呼ばれる
    /// </summary>
    /// <param name="score">Score.</param>
    void OnAddScore(int score)
    {
        TxtRender.text = score.ToString();
    }
}

##GameResult.cs

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

public class GameResult : MonoBehaviour
{
    private int highScore;
    public static bool Result;
    public Text resultScoreText;
    public Text resultText;
    public Text bestScoreText;
    public GameObject resultUI;
    // Start is called before the first frame update
    void Start()
    {
        Result=false;
        if (PlayerPrefs.HasKey("HighScore"))
        {
            highScore=PlayerPrefs.GetInt("HighScore");
        }else
        {
            highScore=0;
        }
        
    }

    // Update is called once per frame
    void Update()
    {
        if (Timer.nowtime==0f)
        {
            Result=true;
            resultUI.SetActive(true);
            int result= Mathf.FloorToInt(DataManager.Instance.Score);
            resultScoreText.text="ResultScore:"+result;
            bestScoreText.text="BestScore:"+highScore;
            if (highScore<result)
            {
                resultText.text="HighScore!!";
                PlayerPrefs.SetInt("HighScore",result);
            }
        }
    }
    public void OnRetry(){
        DataManager.Instance.resetScore();
                SceneManager.LoadScene("Stage1");
    }
    public void BackTitle(){
        SceneManager.LoadScene ("Title");
    }
    
}

##DataManager.cs

using System;
using UnityEngine;

public class DataManager : MonoBehaviour
{
    //シングルトン化 DataManager.Instanceで何処からでもアクセスできます。
    public static DataManager Instance
    {
        get{
            if (_instance == null)
            {
                _instance = LoadInstance();
            }
            return _instance;
        }
    }
    static DataManager _instance;

    /// <summary>
    /// スコアが追加されたのを通知
    /// </summary>
    public event Action<int> OnAddScore = delegate {};

    /// <summary>
    /// 現在のスコア
    /// </summary>
    /// <value>The score.</value>
    public int Score{ get; private set; }

    void Awake()
    {
        if(_instance != null
           && _instance != this)
        {
            Destroy(gameObject);
        }
    }

    public void AddScore(int addScore)
    {
        if (!GameResult.Result)
        {
        Score += addScore;
        //通知
        OnAddScore(Score);         
        }
        //スコアを加算
   
    }
    public void resetScore()
    {
        if (!GameResult.Result)
        {
        Score = 0;
        //通知
              
        }
        //スコアを加算
   
    }

    /// <summary>
    /// インスタンスの設定
    /// </summary>
    public static DataManager LoadInstance()
    {
        var instance = FindObjectOfType<DataManager>();

        if(instance == null)
        {
            var go = new GameObject("DataManager", new []{typeof(DataManager) });
            instance =  go.GetComponent<DataManager>();
        }

        DontDestroyOnLoad(instance.gameObject);
        return instance;
    }
}

Answer 1

DataManager.csのDontDestroyOnLoad(instance.gameObject);をけしたら治りました。

Comments

[naotiki]

回答してくれた皆様ありがとうございました。

[naotiki]

http://razupai2468.starfree.jpでダウンロードできます

Answer 2

引用テキストThe object of type 'Text' has been destroyed but you are still trying to access it.

Your script should either check if it is null or you should not destroy the object.

直訳「'Text'型のオブジェクトは破壊されましたが、まだアクセスしようとしています。
あなたのスクリプトはそれがnullであるかどうかをチェックするべきであるか、あなたはオブジェクトを破壊するべきではありません。」

と書かれています。GameResult.csを見た感じですと、BackTitleの処理が走った後にもう一度Updateを読んでいる気がします。bool型のフラグ又は現在のシーンを判定する関数でも使って処理を切ればいけそうな気がします。

引き続きゲーム制作頑張って下さい。

Answer 3

DataManagerが破棄されないオブジェクトで、ScoreTxtはシーン繊維で破棄されるオブジェクトなのだと想定して、DataManager.Instance.OnAddScore += OnAddScore;で登録したものがScoreTxtが破棄されたときに解除されていないからだと思います。

Comments

[naotiki]

どうやったら解除できますか？(初心者ですいません)

[vo3]

タイミングやその他の実装によっては別の不具合を生むかもしれませんが、ScoreTxtにOnDestroyかOnDisableを定義して、その中でDataManager.Instance.OnAddScore -= OnAddScore;を実装してみてください。

OnDisable、OnDestroyなどのイベント関数がわからないのであれば、一度公式のリファレンスを確認してみるといいです。
https://docs.unity3d.com/jp/current/Manual/ExecutionOrder.html