配列のシャッフルがうまくできません

Question

クイズアプリ/C#/Unity/問題文をシャッフルしたい

csvからデータを読み込み、csvの配列通りに問題が出題されるようになっています。
これを重複しないランダムで表示したく、調べたらたくさん出てきたのですが、一通りやってみてもできなかったので質問させていただきます。
現在のスクリプトでは何もエラーは起こっていませんが、シャッフルされず配列通り出題されています。

##やりたいこと
csvの配列を丸ごとシャッフルさせたい。
→初めは問題文だけをシャッフルするようにしましたが、答えの選択肢までシャッフルされることを懸念して、csvの横列をランダムにしたい

##やったこと
"C# 配列　シャッフル"で検索し、画像のように試しました。試した内容はかなりあるのですが、どれも記述の仕方に間違いがありできていないようです。!

/Users/mugisama/Desktop/スクリーンショット 2021-11-19 13.00.57.png

/Users/mugisama/Desktop/スクリーンショット 2021-11-19 16.26.41.png

/Users/mugisama/Desktop/スクリーンショット 2021-11-24 16.19.32.png

/Users/mugisama/Desktop/スクリーンショット 2021-11-25 10.09.39.png

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

public class CSVScript : MonoBehaviour
{
    public TextAsset CSV;
    public Text value3, value4, value5;
    public static int nowIndex = 1;
    public static string CorrectAnswerText;


    //CSVから分解した問題クラスを代入する配列
    public Question[] questions = new Question[150];

    public static string QuestionText { get; private set; }


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

        //CSVを各行で区切る
        string[] csv = CSV.text.Split('\n');
        Debug.Log(csv.Length);

        //全ての行の数だけループする（１行目から開始）
        for (int i = 1; i < csv.Length; i++)
        {
            //各行の要素を,で区切る
            string[] values = csv[i].Split(',');

            //０番目：カテゴリ
            string category = values[0];

            //１番目：レベル
            int level = 0;
            if (values[1] == "初級")
            {
                level = 1;
            }
            else if (values[1] == "中級")
            {
                level = 2;
            }
            else if (values[1] == "上級")
            {
                level = 3;
            }

            //２番目：問題文
            string questionText = values[2];

            //3~5番目：選択肢　配列でまとめる
            //values[3]A values[4]B values[5]
            string[] answers = { values[3], values[4], values[5] };


            //６番目：正解の配列番号　Aが0、Bが1、Cが2
            int answerIndex = 0;
            if (values[6] == "A")
            {
                answerIndex = 0;
            }
            else if (values[6] == "B")
            {
                answerIndex = 1;
            }
            else
            {
                answerIndex = 2;
            }



            //７番目：解説
            string comment = values[7];

            Question q = new Question(category, level, questionText, answers, answerIndex, comment);

            //作成したquestionクラスを配列に入れる
            questions[i] = q;

        }

        //ここでシャッフル
        System.Random rng = new System.Random();
        int n = csv.Length;
        while (n > 1)
        {
            n--;
            int k = rng.Next(n + 1);
            string tmp = csv[k];
            csv[k] = csv[n];
            csv[n] = tmp;
        }

        questions[nowIndex].ShowLog();

        GetComponentInChildren<Text>().text = questions[nowIndex].question;
        value3.text = questions[nowIndex].answers[0];
        value4.text = questions[nowIndex].answers[1];
        value5.text = questions[nowIndex].answers[2];
    }


    public void OnClickAnswerButton(int answerIndex)
    {
        CorrectAnswerText = questions[nowIndex].GetCorrectAnswerText();

        if (questions[nowIndex].answerIndex == answerIndex)
        {
            SceneManager.LoadScene("correct");
        }
        else
        {
            SceneManager.LoadScene("incorrect");
        }

    }


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

    }
}

Comments

[BeatStar]

> 一通りやってみてもできなかった

とありますが、『どのようにやったか』を書きましょう。(複数個試しているはずなので)
もしかしたら調べ方が悪いのかもしれないし、参考サイトが間違っているかもしれないし、サイトでは間違っていないが条件が一致していないかもしれないし、単純に書き方が間違っているかもしれないし。

それに、プログラミングは『こう書けばいい』と言うものではありません。
ロジックを書くのです。

[BeatStar]

それと、ランダムにしている部分(シャッフル)では『csvのデータ』を変更しているっぽい。
でもやりたいことから考えると、『Questionクラスの配列』の方にアクセスすべきでは？

[tsukunemugi]

内容を一部変更しました。

シャッフルする部分に関してですが、初めはQuestionクラスの配列をシャッフルするよう記述sていたのですが、Questionクラスをシャッフルすると回答選択肢とバラバラになる可能性が出てくると思い、csvごとシャッフルするよう書き換えました。

[BeatStar]

画像が表示されていませんよ？(私の環境だけかな？)

[BeatStar]

必ず質問にある構成でしょうか？(たとえば課題とかみたいに問題文等で定義されている等)

[退会済みユーザー]

> string[] csv = CSV.text.Split('\n');

そもそもそれで期待通り CSV ファイルの各行の配列が取得できているか確認しましたか？　改行は "\r\n" とか、フィールドの中に改行コードが含まれていたらどうなのでしょう？

> string[] values = csv[i].Split(',');

これも同じく、そもそもこれで各行のフィールドが期待通り取得できるのかか確認しましたか？　フィールドの中にカンマ , が含まれていたらどうなるでしょう？

話はそのあたりは問題ないことを確認されてからにした方が良さそうです。

[tsukunemugi]

画像追加しました。
現在csvの配列通りですが、最後の画像の中に文字が格納され、問題なく動いています。

[BeatStar]

@ 質問者さん

とりあえずSurferOnWwwさんが仰るようなものを調べてください。(Console.WriteLineメソッド等で)

[BeatStar]

それと、まだ画像が表示されず、ユーザ名が出てますよ…

[dodox86]

既に色々と指摘が出ているのでこれ以上混乱させるつもりはないですが、

> Questionクラスをシャッフルすると回答選択肢とバラバラになる可能性が出てくると思い、

これがまず分かりませんね。Questionクラスの提示が無いので確証はないですが、Questionクラス自体が回答選択肢を持っているのであれば、Questionクラスを要素とする配列をシャッフルしてもバラバラにはならないように思えるのですが。

あれもこれも最初から全部やろうとするのではなく、まず配列の要素のシャッフルはどうやったらよいのかちゃんと理解したあと、今のコードに適用するようにした方が良いように思います。

[退会済みユーザー]

現時点では本題とは直接関係ない話かもしれませんが、CSV ファイルのテキストを改行・カンマで Split するのは止めた方が良いと思います。質問文に改行・カンマが含まれていたらそこで破綻しますよ。

現状でそういうことがあるとすると、質問の話を始める以前の問題があります。

興味がありましたら、以下の記事の「CSV形式とは？」を一読することをお勧めします。

CSV形式のファイルをDataTableや配列等として取得する
http://dobon.net/vb/dotnet/file/readcsvfile.html

[tsukunemugi]

みなさまの助言をもとにもう一度調べ直してみます。
ありがとうございます。

Answer 1

スクリプトをほぼまるっきり書き換え、フィッシャーイェーツを使ってシャッフルすることができました。たくさんのご助言、ありがとうございました。