C#（Unity）でクイズゲームを作る際の、ランダム出題用の配列について

Question

前提・実現したいこと

1. Unityでクイズゲームを作る。

過去にこのWebページを参考にJavaでクイズゲームを作成したので、Unityで作成しなおし、ブラッシュアップしたい。
0. 問題はCSV管理する。
0. 問題はランダムに10問出題する。←ここで詰まっています。

発生している問題・エラーメッセージ

1問目はランダム出題されるのですが、1問目を回答し、2問目の出題を行う際に停止してしまいます。

1. 空のオブジェクトにQuizMgrスクリプトをアタッチし、1問目を出題します。
2. 配列Order1を作成し、OからCSVの行数分数字を入れます。
3. Order2でOrder1をシャッフルします。
4. Order2の前から2番目の問題を出題します。
5. ４つの回答ボタンにJudgeスクリプトをアタッチし、

正解の回答ボタンをクリックするとScoreが10点加算されるところまではできています。

しかし、Judgeクラスから、QuizMgrクラスのNextQuizSetメソッドを呼ぶところで、以下のエラーコードが表示されます。

JudgeクラスからNextQuizSetメソッドを呼んだ場合は、Order2の要素が元から入っていないと判断されているのはなんとなくわかるのですが、QuizMgrクラスのStartメソッドが実行された時点で、Order2の要素は固定することはできないのでしょうか。
大変わかりにくい説明となっていしまいましたが、ご教示いただけると幸いです。

エラーコード
NullReferenceException: Object reference not set to an instance of an object
QuizMgr.QuizSet () (at Assets/QuizMgr.cs:58)
QuizMgr.NextQuizSet () (at Assets/QuizMgr.cs:77)
Judge.CapText () (at Assets/Judge.cs:26)
UnityEngine.Events.InvokableCall.Invoke () (at C:/buildslave/unity/build/Runtime/Export/UnityEvent.cs:166)
UnityEngine.Events.UnityEvent.Invoke () (at C:/buildslave/unity/build/Runtime/Export/UnityEvent_0.cs:58)
UnityEngine.UI.Button.Press () (at C:/buildslave/unity/build/Extensions/guisystem/UnityEngine.UI/UI/Core/Button.cs:36)
UnityEngine.UI.Button.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) (at C:/buildslave/unity/build/Extensions/guisystem/UnityEngine.UI/UI/Core/Button.cs:45)
UnityEngine.EventSystems.ExecuteEvents.Execute (IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) (at C:/buildslave/unity/build/Extensions/guisystem/UnityEngine.UI/EventSystem/ExecuteEvents.cs:50)
UnityEngine.EventSystems.ExecuteEvents.Execute[IPointerClickHandler] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.EventFunction`1 functor) (at C:/buildslave/unity/build/Extensions/guisystem/UnityEngine.UI/EventSystem/ExecuteEvents.cs:261)
UnityEngine.EventSystems.EventSystem:Update()

using UnityEngine;
using System.Collections;
using System.Collections.Generic;// 追記
using UnityEngine.UI;//UI利用時必須
using System.Linq;//配列に必要
using System;//これがないとGuidクラスは使えない

public class QuizMgr: MonoBehaviour {
    public TextAsset csvFile;	　　　　　// GUIでcsvファイルを割当
　　　　List<string[]> csvDatas = new List<string[]>(); //ここは参考ブログとは違う
    public static string AnswerStr;//問題の答え
    int[] Order1 = null; //出題数を管理するメンバ変数
    public int[] Order2;　//出題をランダムにするメンバ変数
    public static int Count = 1; //今何問目か
    public int Score = 0; //得点

　　　　//スタート時、CSVファイルを読み込む
　　　　public void Start() {
　　　　　　　　// 格納
　　　　　　　　string[] lines = csvFile.text.Replace("\r\n", "\n").Split("\n"[0]);
        foreach (var line in lines) {
            if (line == "") { continue; }
            csvDatas.Add(line.Split(','));　　　　// string[]を追加している
　　　　　　　　}

        Order1 = new int[csvDatas.Count];　//配列の要素数をCSVの行数分にする
        //配列に順番に数字を入れる
        for(int i = 0; i < csvDatas.Count-1; i++) {
            Order1[i] = i;
        }
        //配列の数字をランダムに入れ替えた配列を作成
        Order2 = Order1.OrderBy(i => Guid.NewGuid()).ToArray();
        
        //問題をセットするメソッドを呼び出す
        QuizSet();
　　　　}
    //問題をセットするメソッド
    void  QuizSet() {

        //1から4の配列(ary1)を作成
        int[] ary1 = new int[] {1, 2, 3, 4};
        //ary1をランダムに並び替えたary2を作成
        int[] ary2 = ary1.OrderBy(i => Guid.NewGuid()).ToArray();

        //for (int i =0; i < csvDatas.Count; i++) {

        //答えをセット
        AnswerStr = csvDatas[Order2[Count]][1];
            //特定の名前のオブジェクトを検索してアクセス
            Text question = GameObject.Find("Quiz/Question").GetComponentInChildren<Text>();
            Text button1 = GameObject.Find("Quiz/Button1").GetComponentInChildren<Text>();
            Text button2 = GameObject.Find("Quiz/Button2").GetComponentInChildren<Text>();
            Text button3 = GameObject.Find("Quiz/Button3").GetComponentInChildren<Text>();
            Text button4 = GameObject.Find("Quiz/Button4").GetComponentInChildren<Text>();
            //データをセットすることで、既存情報を上書きできる
            question.text = csvDatas[Order2[Count]][0];//問題文セット
            button1.text = csvDatas[Order2[Count]][ary2[0]];//ary2の１番目に入っている数字の項目をボタン１にセット
            button2.text = csvDatas[Order2[Count]][ary2[1]];//ary2の２番目に入っている数字の項目をボタン２にセット
            button3.text = csvDatas[Order2[Count]][ary2[2]];//ary2の３番目に入っている数字の項目をボタン３にセット
            button4.text = csvDatas[Order2[Count]][ary2[3]];//ary2の４番目に入っている数字の項目をボタン４にセット
        //}
    }
    public void NextQuizSet() {
        Count++;
        Debug.Log(Count);
        if(Count == 11) {
        } else {QuizSet();}
    }
}

using UnityEngine;
using UnityEngine.UI;

public class Judge : MonoBehaviour {

    public static Text selectedBtn;

    //選択したボタンのテキストラベルと正解のテキストを比較して正誤を判定
    public void CapText() {

        QuizMgr quizMgr = new QuizMgr();//他のクラスのメソッドを使用するためにはオブジェクト作成が必要

        //選択したボタンのテキストラベルを取得する
        Text selectedBtn = this.GetComponentInChildren<Text>();
       
        //選択したボタンのテキストラベルと問題の答えを比較
        if (selectedBtn.text == QuizMgr.AnswerStr) {
            quizMgr.Score += 10;
        } 
        quizMgr.NextQuizSet();//上記で作成したオブジェクトを使用する
    }
}

試したこと

Judgeクラスに配列Order3を作成し、Order2の要素を渡す方法等も考えましたが、うまくいきませんでした。

Answer 1

58行目が以下だとすれば、
button2.text = csvDatas[Order2[Count]][ary2[1]];
「button2」「csvDatas」「Order2」「ary2」のどれかnullになっている可能性が高いです。
Debug.Logを使用して、具体的に何がnullなのかをまず確認してみてください。

ありえそうなのは、別の部分で「button2」のTextをオフにしている結果GameObject.Find("Quiz/Button2").GetComponentInChildren<Text>()に失敗しているとかでしょうか。

また、Startメソッドで設定した値は、シーンを変更しない限りは維持されます。
なので本来はcsvDatas、Order2もnullにならないはずなのですが、別スクリプトが影響していないかも確認してみてください。

---

（QuizMgr quizMgr = new QuizMgr();を見落としてたので追記）

そもそも現状は「MonoBehaviourに'new'キーワードを使うことは許されていません」という意味合いの警告が出ていると思います。

QuizMgrはオブジェクトに付いていると思うので、newではなく
QuizMgr quizMgr = GetComponent<QuizMgr>();
のように「オブジェクトに付いたクラスを取得」する必要があります。（※他オブジェクトに付いている場合は書き方がちょっと変わるので調べてください）

他のクラスのメソッドを使用するためにはオブジェクト作成が必要というのはちょっと語弊があります。

■最初からQuizMgrをGameObjectに付けてある場合
→シーン開始時にQuizMgrのStartメソッドが呼ばれます。
→既にオブジェクトが存在している為、新たにオブジェクトを作る必要はありません（GetComponentなどで「既存のGameObjectのコンポーネントを取得」する必要があります）。

■新たにQuizMgrをGameObjectに付ける場合
→QuizMgr quizMgr = gameObject.AddComponent<QuizMgr>();とします。（なおここを何度も呼ぶとその数だけ新たなQuizMgrがくっ付くので、一度だけにしたい場合はちゃんと条件分岐する必要があります）
→「新たにQuizMgrを生成してGameObjectに付与（＝新たにQuizMgrインスタンスを生成）」という意味になります。
→QuizMgrのStartメソッドは呼ばれますが、quizMgr.NextQuizSet();とどっちが早いかは保証されません（多分Startメソッドより前にNextQuizSetメソッドが呼ばれる可能性が高い）。

■new QuizMgr()する場合（QuizMgrがMonoBehaviourを継承していない場合）
→「新たにQuizMgrを生成」という意味になります。（GameObjectには付与しない、というかMonoBehaviourを継承していないと付与出来ない）
→QuizMgrのStartメソッドは呼ばれません。コンストラクタを使用するか初期設定用メソッドを作って自分で呼ぶ必要があります。

後者の２つだと毎回Order2を生成しているようなものなので、
今回はGetComponentを使用した形式が適していると思います。
（「unity インスタンス」とか「unity 他オブジェクトのスクリプト」とかで検索すると詳しい情報が出ます）

Comments

[yamasho69]

58行目はAnswerStr = csvDatas[Order2[Count]][1];です。

NextQuizSetメソッドでOrder2をDebug.Logした際にエラーが発生したので、おそらくnullになっていると思います。csvDatasは調べていないので、後ほど調べますが、どちらにせよStartメソッドで設定した値が保持されていないことになります。
本アプリは今のところ、QuizMgrスクリプトとJudgeスクリプトしかなく、シーンも一つしかありません。

[yamasho69]

「unity 他オブジェクトのスクリプト」で検索し、出てきたページ
（https://qiita.com/tsukasa_wear_parker/items/09d4bcc5af3556b9bb3a）を参考にし、 Judgeクラスに下記のコードを入れて、解決しました。お答えいただいたsakura_hana様、y_waiwai様、ありがとうございました。大変助かりました。

GameObject quizmanager;
QuizMgr quizMgr;

//選択したボタンのテキストラベルと正解のテキストを比較して正誤を判定
public void CapText() {

quizmanager = GameObject.Find("QuizManager"); //QuizManagerをオブジェクトの名前から取得して変数に格納する
quizMgr = quizmanager.GetComponent<QuizMgr>();//Mgrはスクリプト、managerはオブジェクト

Answer 2

Startメソッドが実行されてるところは見当たらないですが。

Comments

[yamasho69]

シーンが実行された際に、QuizMgrクラスが実行され、その歳に
public void Startメソッドが実行されます。
そこでOrder2の要素を決めているのですが、その時点で要素を固定できないのでしょうか。
JudgeクラスからQuizMgrクラスを呼び出した時に、再度Order2の要素を決めることになると、Order2の順序が変わってしまい、出題がかぶってしまう可能性があると思うのですが、間違っているでしょうか…。

【例】
StartメソッドでOrder2をランダムで｛３、１、２｝とします。
ファイル実行中はこの要素の順序を使い続けたいのです。

しかしJudgeクラスからQuizMgrクラスのNextQuizSetメソッドを
呼び出すとOrder2の要素がありません。
再度Order2の要素を決めると｛2、１、３｝
のように変わってしまうのではないでしょうか…。

[y_waiwai]

必要な初期化はコンストラクタで実行しときましょう

[sakura_hana]

@y_waiwaiさん
Unityの場合はStartがコンストラクタ的な動きをします。
（厳密には違うと思うのですが、MonoBehaviourを継承したクラスの場合はシーン上でこのインスタンスが稼働開始する時に勝手にStartメソッドが呼ばれる為）
なのでStartメソッドに関してはこのままでも問題無いかと思います。

[y_waiwai]

提示のコードのように、newして即何かをしようとする場合は、Startは呼ばれようがないと思うんですが。
まあ、そのstartメソッドが実行されるまで待つ、というテもあると思いますが

[sakura_hana]

すみません、「QuizMgr quizMgr = new QuizMgr();」の行見落としてました。
この場合だと確かにStartは（いつまで待とうが）呼ばれないですね。