Unity（２D）問題を重複なく出題する方法がわかりません。

Question

実現したいこと

Unityでクイズゲームを作成しています。
現在できていること
CSVファイルを読み込み問題をUIに反映させ、正誤判定、スクリプト間のデータを受け渡してスコアを表示させるというところまででき、クイズゲームとしては完成に近いのですが、問題が重複して出題されてしまい困っております。

発生している問題・分からないこと

QuizMgrというスクリプトを使ってCSVの問題を取得してUIに表示させています。下記補足ソースコードを追加すると問題と答えがバラバラにUIに反映されるようになってしまいました。

該当のソースコード

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

public class QuizMgr : MonoBehaviour
{
    public string dataName;
    public string title;
    public TextAsset csvFile;
    public List<string[]> csvDatas = new List<string[]>();
    public int height = 0;
    public int i, j = 0;
    public int k = 0;
    internal Text ansLabel;
    public string AnswerText;
    const int size = 5;
    private static string correctAnswer;

    void Awake()
    {

        title = "theme_";
        csvFile = Resources.Load("CSV/CSVDatas") as TextAsset;
        StringReader reader = new StringReader(csvFile.text);

        while (reader.Peek() > -1)
        {
            string line = reader.ReadLine();
            csvDatas.Add(line.Split(','));
            Debug.Log("reading:" + height);
            height++;
        }

        for (i = 0; i < height; i++)
        {
            for (j = 0; j < size; j++)
            {
                Debug.Log("csvDatas[" + i + "][" + j + "]:" + csvDatas[i][j]);
            }
        }

        QuestionLabelSet();
        AnswerLabelSet();
        AnswerSet();
    }


    public void QuestionLabelSet()
    {
        csvDatas[k] = csvDatas[Random.Range(0, 10)];
        //特定の名前のオブジェクトを検索してアクセス
        Text qLabel = GameObject.Find("Quiz/QLabel").GetComponentInChildren<Text>();
        //データをセットすることで、既存情報を上書きできる
        qLabel.text = csvDatas[k][0];
    }

    public void AnswerLabelSet()
    {
        //問題文に対応した答えをそれぞれのuGUIボタンにセット
        string[] array = new string[] { csvDatas[k][1], csvDatas[k][2], csvDatas[k][3], csvDatas[k][4] };

        //問題文をシャッフル
        array = array.OrderBy(x => System.Guid.NewGuid()).ToArray();

        //ボタンが4つあるのでそれぞれ代入
        for (int i = 1; i <= 4; i++)
        {
            Text ansLabel = GameObject.Find("Quiz/AnsButton" + i).GetComponentInChildren<Text>();
            ansLabel.text = array[i - 1];
        }
    }
  
    public void AnswerSet()
    {
        //答えとなるcsvデータを変数として型に代入する
        AnswerText = csvDatas[k][1];
        Debug.Log(AnswerText);

        correctAnswer = csvDatas[k][1];
    }

    public static string GetCorrectAnswer()
        {
            return correctAnswer;
        }

    void Update() { }
}

//追加したもの一覧

 public List<string[]> shuffledCsvDatas = new List<string[]>();

 shuffledCsvDatas = csvDatas.OrderBy(x => Random.value).ToList();

  public void QuestionLabelSet()
    {
        // シャッフルしたデータを順番に表示
        Text qLabel = GameObject.Find("Quiz/QLabel").GetComponentInChildren<Text>();
        qLabel.text = shuffledCsvDatas[k][0];
    }

public void AnswerSet()
    {
        //答えとなるcsvデータを変数として型に代入する
        AnswerText = shuffledCsvDatas[k][1];
        correctAnswer = shuffledCsvDatas[k][1];
    }

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

問題とクイズのそれぞれがシャッフルされ本来の問題とその答えがバラバラになってしまいました。

補足

特になし

Comments

[Refrain]

問題視しているのは"重複すること"でしょうか。
それとも"問題と回答がバラバラになること"でしょうか。

[Meron_]

重複することです。

[juner]

csvDatas[k] = csvDatas[Random.Range(0, 10)];
の様に直接元のデータ上書きしてしまうと既にあったデータが別のデータに上書きされ、滅茶苦茶になると思うのですが意図していますでしょうか？
せめてやるなら入れ替えであるべきではないでしょうか？

[fana]

「これがコードです．できません」だけじゃなくて，まず「どういう方法でやろうとしているのか」という実装しようとしているアルゴリズムの話をするべきなんじゃないかなぁ．

例えば，コードだけ見せられると，
[k] が頻繁に出てくる様子だけどこの k というのは一体何のための存在なのか？　とか疑問に思う．

例えば「複数の問題がある中から，k番目の問題を出題する」といったような意図なのであれば，やるべきことは「 k の値を何かしらの方法で（重複しないように）決定していくこと」になると思うのだけど，提示コードを見る限り k の値は常に 0 であるように見えるから，そういう物ではないのですよね？きっと．

[Meron_]

確かにその通りです。ご指摘ありがとうございます。
Kの値はcsvのリスト内のk番目の要素を表していて、[k][1]だったらk行目の2番目の要素です。
作ろうとしているのは、CSVのデータを一次元配列に入れてそれをシャッフルし上から順番に読み込んで重複しないようにするというものです。

[fana]

> 作ろうとしているのは、CSVのデータを一次元配列に入れてそれをシャッフルし上から順番に読み込んで重複しないようにするというものです。

であれば，
* データをシャッフルすること
* k の値を出題する毎に更新していくこと
が必要となるであろうと想像しますが，前述のように後者に相当する部分がコード内に見当たりません．
その点はどうなのでしょう？
本当に k の値の更新が必要なのであれば，「それ（必要な処理の実装）が無いコードでは所望の動作をしないのは当たり前」という話にしかならない気がするのですが．

[fana]

とりあえず
「データをCSVファイルから読込んだら，その場で即シャッフルする」だけをまずは実装し，想定通りにシャッフルできることが確認できたならば（その時点で「問題とその答えがバラバラ」という問題は無いハズであるから），
次に「それを順番に使う」という部分を実装する，
というステップを経るのが良さそうかな，と思います．

また，そういう原理確認をする場合には Unity みたいな要素は 不要/余計 であろうと思うので，適当なコンソールアプリとかで試して→できたら Unityの本番コードに移植する みたいな手順を経るのが良いのではないか，とか思います．

[Meron_]

該当のソースコードにある、QUizMgrだけなら想定通りにシャッフルできているのですが補足コードを追加すると問題と回答がバラバラになるということです。
ですので現時点では想定通りにシャッフルするというのはできています。
QuizMgrに以下のコードを追加したところ問題と解答がバラバラになってしまったということです。
public List<string[]> shuffledCsvDatas = new List<string[]>();

shuffledCsvDatas = csvDatas.OrderBy(x => Random.value).ToList();

public void QuestionLabelSet()
{
// シャッフルしたデータを順番に表示
Text qLabel = GameObject.Find("Quiz/QLabel").GetComponentInChildren<Text>();
qLabel.text = shuffledCsvDatas[k][0];
}

public void AnswerSet()
{
//答えとなるcsvデータを変数として型に代入する
AnswerText = shuffledCsvDatas[k][1];
correctAnswer = shuffledCsvDatas[k][1];
}

[Meron_]

コメント、回答をご丁寧にありがとうございました。ベストアンサーにさせていただきます。

[fana]

いや，あなたが考えている方法と私の回答の内容は若干違いますよね？
「それはそれ，これはこれ」としてご自身の方法で行くような回答を待っても（あるいはご自身で検討を続けても）良くないですか？

※ベストアンサーを付けてしまうと，以降，回答等が付きにくくなる気がします．
本当に「もういい」となったのでなければ特に急ぐ必要は無いと思いますよ．

[Meron_]

確かにそうですね。回答を待っている間、もっといろいろ試してみようと思います。ありがとうございます。

Answer 1

どういう方法で　やりたい/やろうとしている のか？　というのが読み取れませんが……

例えば私なら，「ファイルから読込んだデータ」というのを処理の進行に応じてごちゃごちゃといじくり回したくはない（読込んだままの状態で保持したい）気がするので，↓のような方法を考えるかもしれません．

これが　（何らかの意味で）良い方法か否か？　みたいな話は何かしらあり得るでしょうけども，
まず最初はこんな感じの　最もシンプルな部類の/限りなく混乱する要素のない　方法でトライしてみればどうでしょうか．

//何かこういう（CSVファイルから読込んだ）データがあって……
List< string[] > csvDatas = new List<string[]>
{
    new string[]{ "A0", "A1", "A2", "A3" },
    new string[]{ "B0", "B1", "B2", "B3" },
    new string[]{ "C0", "C1", "C2", "C3" },
    new string[]{ "D0", "D1", "D2", "D3" },
    new string[]{ "E0", "E1", "E2", "E3" }
};

//例えば，データの個数 (この例だと5) 回だけ出題を行うことを考えるとき，
//「各回にどのデータを用いるのか？」というのを乱数で決めておくのが簡単ではなかろうか．
int[] RandomOrderedDataIndexes;
{
    var Rnd = new Random(); //※ この例で使っているのは .NET Framework の Random
    RandomOrderedDataIndexes = Enumerable.Range( 0, csvDatas.Count ).OrderBy( x => Rnd.Next() ).ToArray();
}

//---------- ↑の RandomOrderedDataIndexes[] の使い方の例：

//少なくともデータの個数回だけ出題を行う範囲においては，「重複」はしないよね．
for( int iTimes=0; iTimes<csvDatas.Count; ++iTimes )
{
    int DataIndexForThisTime = RandomOrderedDataIndexes[ iTimes ];
    var DataForThisTime = csvDatas[ DataIndexForThisTime ];
    ...
}

---

CSVのデータを一次元配列に入れてそれをシャッフルし上から順番に読み込んで重複しないようにする

データ自体をシャッフルして良いのであれば，より単純になる：

//何かこういう（CSVファイルから読込んだ）データがあって……
List< string[] > csvDatas = new List<string[]>
{
    new string[]{ "A0", "A1", "A2", "A3" },
    new string[]{ "B0", "B1", "B2", "B3" },
    new string[]{ "C0", "C1", "C2", "C3" },
    new string[]{ "D0", "D1", "D2", "D3" },
    new string[]{ "E0", "E1", "E2", "E3" }
};

{//↑の csvDatas 自体をシャッフルしてよいならば，CSV読込直後にシャッフルしておけよい．
    var Rnd = new Random(); //※ この例で使っているのは .NET Framework の Random
    csvDatas = csvDatas.OrderBy( x => Rnd.Next() ).ToList();
}

//---

//あとはシャッフル済みの csvDatas を先頭から順に使うだけだ
for( int iTimes=0; iTimes<csvDatas.Count; ++iTimes )
{
    var DataForThisTime = csvDatas[ iTimes ];

    //※↓コレは確認用に DataForThisTime の中身をコンソールに出力してみているだけ
    Console.WriteLine( (iTimes+1).ToString() + "回目:" );
    foreach( var str in DataForThisTime )Console.Write( str + ", " );
    Console.WriteLine();
}

このコードの出力結果例：

1回目:
C0, C1, C2, C3,
2回目:
B0, B1, B2, B3,
3回目:
A0, A1, A2, A3,
4回目:
E0, E1, E2, E3,
5回目:
D0, D1, D2, D3,

質問文のコードでは，この例とは異なり， shuffledCsvDatas という別の変数にシャッフルした結果を受けているが，

問題とその答えがバラバラ

になるのは，コードの各所でこの shuffledCsvDatas を用いている個所と，元の（順序がシャッフルされていない）csvDatas を用いている個所とがあって整合性が取れていない，みたいな話なのではなかろうか？　と推測する．

Comments

[Meron_]

データ自体をシャッフルする方法でコードを書かせて頂きました。ご丁寧にありがとうございました。