Unity 問題をシャッフルしても重複しないようにしたい

Question

前提・実現したいこと

下記のサイトを参考にクイズを作っています
https://moneytyping.com/archives/405

下のコードで作ってみましたがシャッフルはされますが
問題が重複してしまいます
問題がシャッフルされても重複しないようにしたいです
よろしくお願いいたします

該当のソースコード

using UnityEngine;
using System.Collections;
using UnityEngine.UI;//UI オブジェクトを扱う時は必須
using System.Collections.Generic;
using System.IO;
using System.Linq;
using Random = UnityEngine.Random;


public class Question : MonoBehaviour {

    public string[] questions = {
        "問題",
        "問題",
        "問題",
        "",

        };
    public int num;
    public string nQJ;
    public static  string answer;
    

 
     //アタッチしたオブジェクトが呼ばれた時に実行される。
    void Start () {
        num = Random.Range(0,questions.Length);
        nQJ = questions[num];
        QuestionLabelSet ();
        AnswerLabelSet();
    }
 
    private void QuestionLabelSet(){

        //特定の名前のオブジェクトを検索してアクセス
        Text qLabel = GameObject.Find("Quiz/QuestionField").GetComponentInChildren<Text> ();
        //データをセットすることで、既存情報を上書きできる
        qLabel.text = nQJ;
    }
    private void  AnswerLabelSet(){
        //回答文面の作成
        string[][] array = new string[questions.Length][];
        //正解の選択肢は必ず一番前に持ってきます。
        //下のアンサーで使います。
        array[0] = new string[4]{"選択肢","選択肢","選択肢","選択肢"};
        array[1] = new string[4]{"選択肢","選択肢","選択肢","選択肢"};
        array[2] = new string[4]{"選択肢","選択肢","選択肢","選択肢"};


        answer = array[num][0];
        array[num].Shuffle();


        //ボタンが4つあるのでそれぞれ代入
        for (int i=1; i<=4 ; i++){

            Text ansLabel = GameObject.Find("Quiz/AnsButton" + i).GetComponentInChildren<Text> ();
            ansLabel.text = array[num][i-1];
        }
    }
    public static string get()
    {
        return answer;
    }

}


public static class Extensions  //拡張メソッド用 static クラス（名前は任意）
{
    //配列の要素をシャッフルする (Fisher-Yates shuffle)
    public static void Shuffle<T>(this T[] arr)
    {
        for (int i = arr.Length - 1; i > 0; i--)
        {
            int j = Random.Range(0, i + 1);  //[0]～[i]
            T tmp = arr[i];     //swap
            arr[i] = arr[j];
            arr[j] = tmp;
        }
    }
}

試したこと

「シャッフル、重複」などで検索し
https://kan-kikuchi.hatenablog.com/entry/ListExtension
を参考に下記のコードも試してみました

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

/// <summary>
/// 重複しないように追加
/// </summary>
public static void AddToNotDuplicate<T>(this List<T> list, T t){
if(list.Contains(t)){
return;
}
list.Add (t);
}

/// <summary>
/// 重複を無くす
/// </summary>
public static void RemoveDuplicate<T>(this List<T> list){
List<T> newList = new List<T>();

foreach (T item in list) {
  newList.AddToNotDuplicate(item);
}

list = newList;

}

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

こちらの質問の
https://teratail.com/questions/92257
も試してみましたがうまくいきませんでした

Answer 1

出題時に乱数で選ぶのがまずいかと。

問題の配列（あるいは、問題のインデックスを格納した配列）をあらかじめシャッフルしといて、
最初から順番に出題するようにすればいいのでは

[C#] 配列をランダムソート(シャッフル)する方法 │ Web備忘録

Comments

[hanaGame]

回答ありがとうございます

上記のサイトを参考にコードを書き直してみましたがエラーが出ます

該当のソースコードは

using UnityEngine;
using System.Collections;
using UnityEngine.UI;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using Random = UnityEngine.Random;

public class Question : MonoBehaviour {

static void Main()
{

public string[] questions = {

"問題",
"問題",
"問題",
"",
string[] questions2 = questions.Shuffle().ToArray();

}

public int num;
public string nQJ;
public static string answer;

//アタッチしたオブジェクトが呼ばれた時に実行される。
void Start () {

//questionsのリスト内から選ばれた数字の問題を選んでいく
nQJ = questions[num];

QuestionLabelSet ();
AnswerLabelSet();

}

private void QuestionLabelSet(){

//特定の名前のオブジェクトを検索してアクセス
Text qLabel = GameObject.Find("Quiz/SafeAreaLayout/QuestionField").GetComponentInChildren<Text> ();

//データをセットすることで、既存情報を上書きできる
qLabel.text = nQJ;
}

private void AnswerLabelSet(){

//回答文面の作成
string[][] array = new string[questions.Length][];

//正解の選択肢は必ず一番前に持ってきます。
//下のアンサーで使います。

array[0] = new string[4]{"選択肢","選択肢","選択肢","選択肢"};
array[1] = new string[4]{"選択肢","選択肢","選択肢","選択肢"};
array[2] = new string[4]{"選択肢","選択肢","選択肢","選択肢"};

answer = array[num][0];

//選択肢のシャッフルを行う
array[num].Shuffle();

//ボタンが4つあるのでそれぞれ代入
for (int i=1; i<=2 ; i++){

Text ansLabel = GameObject.Find("Quiz/SafeAreaLayout/AnsButton" + i).GetComponentInChildren<Text> ();
ansLabel.text = array[num][i-1];
}
}

//正誤判断するスクリプト（Jadge）に答えを渡す
public static string get()
{
return answer;
}

}

public static class IEnumerableExtension //拡張メソッド用 static クラス
{

public static IEnumerable<T> Shuffle<T>(this IEnumerable<T> collection)
{
return collection.OrderBy(i => Guid.NewGuid());

}

}

です

エラーは
Assets/Question.cs(214,1): error CS1022: Type or namespace definition, or end-of-file expected
と
Assets/Question.cs(16,6): error CS1513: } expected
です
行数は問題などを非表示にしているため違っているかと思います
単純に　}　がぬかっているかと思いましたが見当たりません

何かミスを見落としているかもしれません
よろしくお願いいたします

[y_waiwai]

中身全く見てませんが、

string[] questions2 = questions.Shuffle().ToArray();

この記述おかしいですよ

[hanaGame]

コメントありがとうございます
「[C#] 配列をランダムソート(シャッフル)する方法 │ Web備忘録」のサイトのコードが

//シャッフルする配列
int[] ary1 = new int[] { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };
//シャッフルする
int[] ary2 = ary1.Shuffle().ToArray();

になっていたので参考に変えてみました
他のサイトも参考に色々とやってみましたがうまくいきません。
もう少し調べてみます
ありがとうございました

[y_waiwai]

記述がおかしいってより記述している場所がおかしいってことですな。
まだそこらへんの文法の理解が怪しいようなので、勉強しよう

[U_U_Jenkins]

シャッフル、重複させたいのなら既にステートメントに追加されてますが、
Linqを使うのが手っ取り早いと思います。

Answer 2

とりあえず他の方法を使うことにしました
コメントありがとうございました