Unityで数字タッチゲームでの不具合

Question

Unityで数字順押しゲーム(タッチナンバー)を作る
（https://gamegame-game.com/archives/touch_number_1/）
を参考にして数字タッチゲームを作っています。

スクリプトをすべて書き終えましたが、
いざゲームを実行すると

①ボタンの数字がすべて「１」になる。
②OnClickがうまく動かない（NullReferenceException）

となってしまいます。

どの部分を修正するのかが分かりませんので、お教えください。

よろしくお願いします。

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

public class NumberButtonControllerScript : MonoBehaviour
{
    public GameDirectorScript GameDirectorScript { get; set; }
    public string Text { get; private set; }
    public int Number { get; private set; }

    public int Width = 40;
    public int Heigh = 40;


    // Start is called before the first frame update
    void Start()
    {
        this.GetComponent<Button>().onClick.AddListener(OnClick);
        this.GetComponent<RectTransform>().sizeDelta = new Vector2(this.Width, this.Heigh);
        SetButtonInfos(1, "1");


    }

    // Update is called once per frame
    void Update()
    {
        
    }

    public void SetButtonInfos(int number, string text)
    {
        this.Text =text;
        this.Number = number;
       // Debug.Log(number);

        this.GetComponentsInChildren<Text>()[0].text = text;
    }

    private void OnClick()
    {
        if (this.GameDirectorScript.CheckNumber(this.Number))
        {
          
            this.GameDirectorScript.ChangeNextValue();
            Destroy(gameObject);
        }
    }
}

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


public class NumberButtonGeneratorScript : MonoBehaviour
{
    public GameObject NumberButtonPrefab;

  
   

    // Start is called before the first frame update
    void Start()
    {
        
    }

    // Update is called once per frame
    void Update()
    {

    }

    public void GenerateNumberButtons(int rowCount, int colCount)
    {
        var canvas = GameObject.Find("Canvas");

         var numbers = Enumerable.Range(1, rowCount * colCount)
            .OrderBy(i => System.Guid.NewGuid()).ToArray();
        
        Debug.Log(string.Join(",",numbers));

        var buttons = GameObject.FindGameObjectsWithTag("NumberButton");
        foreach (var item in buttons) Destroy(item);

        float offsetCountX = (colCount - 1) / 2.0f;
        float offsetCountY = (rowCount - 1) / 2.0f;

        int index = 0;

        for (int y = 0; y < rowCount; y++)
        {
            for (int x = 0; x < colCount; x++)
            {
               
                int number = numbers[index++];
                //Debug.Log(number);

                var button = Instantiate(this.NumberButtonPrefab) as GameObject;

                
                var controller = button.GetComponent<NumberButtonControllerScript>();
                controller.SetButtonInfos(number, number.ToString());

               
                button.transform.SetParent(canvas.GetComponent<RectTransform>());

                

               
                button.transform.localPosition = new Vector3(
                    controller.Width * x - controller.Width * offsetCountX,
                    controller.Heigh * y - controller.Heigh * offsetCountY,
                    0);
            }
        }
    }




}
    

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

public class GameDirectorScript : MonoBehaviour
{
    public static int ButtonRowCount = 3;
    public static int ButtonColCount = 3;
    public static int ButtonAllCount = ButtonRowCount * ButtonColCount;

    public int NextNumber { get; set; } = 1;


    public bool CheckNumber(int number)
    {
        return number == this.NextNumber;
    }

    public void ChangeNextValue()
    {
        this.NextNumber++;
    }

    // Start is called before the first frame update
    void Start()
    {
        
        GameObject.Find("NumberButtonGenerator")
            .GetComponent<NumberButtonGeneratorScript>()
            .GenerateNumberButtons(ButtonRowCount, ButtonColCount);

    }

    // Update is called once per frame
    void Update()
    {
        
    }
}

Answer 1

質問内容を見る限り、Unityの仕様やC#にまだ不慣れな方だとお見受けします。
まずはUnityやC#について勉強した方がよいと思います。

今回の質問に対する回答は、

①NumberButtonControllerScript の Startに記述してある SetButtonInfos(1, "1");が原因だと思います。
記事２に記載されているNumberButtonControllerScript.csをよく確認してみてください。
Unityで数字順押しゲーム(タッチナンバー)を作る 2

②OnClickメソッドはスクリプトに記述しただけでは意味を成しません、ButtonコンポーネントのOnClikイベントに登録して初めてスクリプトが働きます。

登録の仕方は以下の記事を参考に実践してみてください。
また、下記のブログはUnityに関する様々な知識が紹介されているので、併せて読んで学んでみてください。
【Unity入門】Buttonの作り方!押された判定はどう取るの?

Comments

[Nanmotsu]

丁寧な回答ありがとうございます。
①は、回答者様のご指摘する通りでした。
②ですが、OnClick（）に登録しましたが、その関数内のbool値がの修正を求められているようです。
NullReferenceException: Object reference not set to an instance of an object
NumberButtonControllerScript.OnClick () (at Assets/Scripts/NumberButtonControllerScript.cs:43)

可能ならばお教えいただけますでしょうか。よろしくお願い致します。

[Y0241-N]

NullReferenceに関してですが、意味としては本来処理上で実行する際に必要となる値が参照できず、何もない（Null)ことが原因で発生するエラーです。

OnClikメソッドでNullエラーが発生している理由はおそらく、
【if (this.GameDirectorScript.CheckNumber(this.Number))】の部分で
GameDirectorScriptを取得できていないからでしょう。これを修正するにはthisで取得しているので
NumberButtonControllerScript と　GameDirectorScript を同じオブジェクトにアタッチする必要があります。

基礎的な理解が足りていないように感じるので、まずはエラーが出た際はそのエラー文をコピーして検索するなどしてみてください。

[Nanmotsu]

最後まで丁寧な回答ありがとうございます。
うまく動きました。

基礎的な部分勉強していきます