【C#】ページ遷移時の処理について。分岐方式を改善したい。

Question

#前提・実現したいこと
C# WPFにてクイズゲームを開発しています。
以下のコードを改善したいです。
プログラムの内容としては、

【mainクラス】

1. ローカルファイルからテキスト、画像、音声ファイルパスを取得する。
2. txtファイルはテキストに変換、jpg,mp3ファイルはファイルパスのままstring型変数に格納
   し、インスタンス化、これをListの1要素とする。
3. 1,2,をフォルダ内に存在する分繰り返す。

※カテゴリーごとにA_フォルダ、B_フォルダ,C_フォルダで分かれており、
A_フォルダはA_List,B_フォルダはB_List,C_フォルダはC_Listに格納

【categoryクラス】

1. ボタンが3つあり、それぞれのクリックイベントでquizクラスページに遷移。

遷移時に渡すパラメータにより、カテゴリーを識別、対応したListを操作。

【quizクラス】

1. mainクラスのListを参照し、要素の順番と、要素内 配列 の順番をシャッフルする。
2. 対応したコントロールのプロパティに代入。

長いコードなので、コードの内容を説明しながら、改善したい箇所を書いていきたいと思います。

まず、次のような、テキスト、ファイルパスを格納するためのクラスがあります。

    public class QuizData
    {
        public QuizData(string quizText, string[] choiceText, string answerText, string imgFilePath, string voiceFilePath)
        {
            QuestionText = quizText;
            ChoiceText = choiceText;
            AnswerText = answerText;
            QuizImage = imgFilePath;
            QuizVoice = voiceFilePath;
        }
        public string QuestionText = "";//テキスト(問題文)
        public string[] ChoiceText = null;//テキストの配列(選択肢)
        public string AnswerText = null;//テキスト(答え)
        public string QuizImage = "";//ファイルパス(画像ファイルパス)
        public string QuizVoice = "";//ファイルパス(音声ファイルパス)
    }

下記のコードは、ローカルファイルからテキスト、ファイルパスを取得し、上記QuizDataクラスに格納、1問ずつインスタンス化してListに格納するコードです。

public partial class MainWindow : Window    
{
    public List<QuizData> A_QuizData = new List<QuizData>(); //1要素につき、下記 QuizData classのインスタンスを格納
    public List<QuizData> B_QuizData = new List<QuizData>(); //同上
    public List<QuizData> C_QuizData = new List<QuizData>(); //同上

    public void CreateQuizDataList()
    {
        string[] A_quizFilePath = Directory.GetFiles(@"C:\Users...\category_A", "Question*.txt", SearchOption.AllDirectories);
        string[] A_quizText = new string[A_quizFilePath.Length];
        for (var i = 0; i < A_quizFilePath.Length; i++)
        {
            A_quizText[i] = File.ReadAllText(A_quizFilePath[i]);
        }
        string[] A_answerFilePath = Directory.GetFiles(@"C:\Users...\category_A", "Answer*.txt", SearchOption.AllDirectories);
        for (var i = 0; i < A_answerFilePath.Length; i++)
        {
            var j = 0;
            foreach (var line in File.ReadAllLines(A_answerFilePath[i]))
            {
                A_answerText[i, j] = line;
                j++;
            }
        }
        string[] A_imgFilePath = Directory.GetFiles(@"C:\Users...\category_A", "Img*.jpg", SearchOption.AllDirectories);

        string[] A_voiceFilePath = Directory.GetFiles(@"C:\Users...\category_A", "Question.mp3", SearchOption.AllDirectories);
        for (var i = 0; i < A_quizFilePath.Length; i++)
        {
            A_QuizData.Add(new QuizData(A_quizText[i], new string[] { A_answerText[i, 0], A_answerText[i, 1], A_answerText[i, 2] }, A_answerText[i, 3], A_imgFilePath[i], A_voiceFilePath[i]));
        }
    }

次は、カテゴリー選択画面についてのコードです。
押下されたボタンによってカテゴリーを決定し、遷移先のページは同一のものですが、
遷移時に渡すパラメータによって、表示する内容が変化します。

    public partial class CategoryPage : Page
    {
        public CategoryPage()
        {
            InitializeComponent();
        }

        private void ButtomAtA__Click(object sender, RoutedEventArgs e) //Quizページ遷移時、"A_"を渡す。
        {
            var category = "A_";
            var quiz = new Quiz(category);
            NavigationService.Navigate(quiz);
        }
        private void ButtomAtB__Click(object sender, RoutedEventArgs e) //Quizページ遷移時、"B_"を渡す。
        {
            var category = "B_";
            var quiz = new Quiz(category);
            NavigationService.Navigate(quiz);
        }
        private void ButtomAtC__Click(object sender, RoutedEventArgs e) //Quizページ遷移時、"C_"を渡す。
        {
            var category = "C_";
            var quiz = new Quiz(category);
            NavigationService.Navigate(quiz);
        }
    }

次が、改善したいコードとなります。
下記のQuizクラスは、mainクラスを生成し、List要素の順番と、要素内の
string[] choiceText の順番をランダムにシャッフルし、画面に表示するものですが、
3つあるうちのListのどれをシャッフルするかを、switch文にて指定している形になっています。

public partial class Quiz : Page
{
    readonly MainWindow _main = new MainWindow();
    private readonly string _category = ""; //switch分岐用.
    public QuizPage(string category)
    {
        InitializeComponent();
        _category = category; //生成時、受け渡される "A" or "B" or "C".
        ShuffleOrderQuizData(_category);
    }

    public void ShuffleOrderQuizData(string _category)
    {
        switch (_category)
        {
            case "A":
                ShuffuleOrder_A_QuizData();
                break;
    
            case "B":
                ShuffleOrder_B_QuizData();
                break;

            case "C":
                ShuffleOrder_C_QuizData();
                break;
        }
    }

    void ShuffuleOrderA_QuizData()
    {
        _main.A_QuizData = _main.A_QuizData.OrderBy(a => Guid.NewGuid()).ToList();
        for (var i = 0; i < _main.A_QuizData.Count; i++)
        {
            _main.A_QuizData[i].ChoiceText = _main.A_QuizData[i].ChoiceText.OrderBy(a => Guid.NewGuid()).ToArray();
        }
    }

    void ShuffleOrderB_QuizData()
    {
        _main.B_QuizData = _main.B_QuizData.OrderBy(a => Guid.NewGuid()).ToList();
        for (var i = 0; i < _main.B_QuizData.Count; i++)
        {
            _main.B_QuizData[i].ChoiceText = _main.B_QuizData[i].ChoiceText.OrderBy(a => Guid.NewGuid()).ToArray();
        }
    }

    void ShuffleOrderC_QuizData()
    {
        _main.C_QuizData = _main.C_QuizData.OrderBy(a => Guid.NewGuid()).ToList();
        for (var i = 0; i < _main.C_QuizData.Count; i++)
        {
            _main.C_QuizData[i].ChoiceText = _main.C_QuizData[i].ChoiceText.OrderBy(a => Guid.NewGuid()).ToArray();
        }
    }
}

質問するに至った経緯ですが、現状だと、項目が増えた場合や処理に変更があった場合などに、caseひとつひとつを修正しなければならないことと、インスタンスに引数を持たせ、遷移先のコンストラクタで分岐する、というやり方が、適切ではないように感じたためです。
開発経験が浅い為、的外れかもしれませんが、どのように改善すればよいか、ご意見を頂けないでしょうか。
宜しくお願い致します。
##追記・修正
質問の仕方が悪く、申し訳ありませんでした。
おもにお聞きしたい内容としては、「ページ遷移時、インスタンスに引数を持たせ、遷移先のコンストラクタで分岐する」というやり方について、別の方法があるならばご教示頂きたい。といったものがメインとなります。
このやり方が最適ではないように感じ、調べたところstateパターンというのが理想に近いと感じましたが、具体的な実装方法に詰まってしまい、教えて頂きたく、質問を致しました。

Comments

[退会済みユーザー]

単なる思い付きですが、

(1) テキスト、画像、音声ファイルパスにカテゴリー列を追加してデータベースに格納する（データベースとかは使いたくないなら例えば CSV ファイルに格納する）

(2) データベースから DISTINCT を使ってカテゴリーを取得し ComboBox で表示し、それからユーザーに選択してもらう。

(3) ユーザーのカテゴリー選択結果に基づき、データベースからそのカテゴリーに属するレコードをすべて取得する。

(4) 上記 (3) の結果を処理して出題する。

[Hottopia]

こんにちは。
条件等記載し忘れていましたが、DB,CSVは使えず、ローカルファイルでの構築が前提となっております。
ですが考え方については参考にさせていただきます。ありがとうございます。

[退会済みユーザー]

ローカルファイルに CSV は含まれるはず。CSV って何だかわかってますか？　ただのテキストファイルですよ。

何かの理由で CSV を使いたくないとしても、ローカルファイルなら XML でも JSON でも使えるはず。

[Hottopia]

失礼しました。
テキストはtxtファイル、画像はjpg、音声はmp3、という指定です。
混乱させてしまい申し訳ありません。

[退会済みユーザー]

あなたの言うカテゴリーには増減があるから今回のような質問になっていると思いますが、そうであればそのカテゴリーを質問のようにコードの中にハードコーディングしてしまっては、増減があるたびコードを書き直さなければならないのはやむを得ません。そういうことをしなくて済む提案をしているつもりですけど、理解してもらっているでしょうか？

[退会済みユーザー]

例えば、

カテゴリー,テキストファイルのパス,画像ファイルのパス,音声ファイルのパス

というデータをその数 CSV ファイルに格納しておくということはできないのですか？　それさえできれば、上に述べた (1) ～ (4) はできるはずで、switch 文のハードコーディングは避けられると思いますけど。

[Hottopia]

理解力が足らず申し訳ありません。
現状の.txtファイルで問題文・選択肢・回答テキストを管理する方法は好ましくなく、CSVファイルで取り扱ったほうが色々と便利ということでしょうか。
一点教えて頂きたいのですが、CSV形式でデータを管理する場合、画像ファイル・音声ファイルはファイルパスとしてCSVファイルに格納するということでしょうか？

[退会済みユーザー]

> 現状の.txtファイルで問題文・選択肢・回答テキストを管理する方法は好ましくなく、CSVファイルで取り扱ったほうが色々と便利ということでしょうか。

その「.txtファイル」というのは、質問に書いてあった、

> 1. ローカルファイルからテキスト、画像、音声ファイルパスを取得する。

の「ローカルファイル」のことですよね。その内容がどうなっているのか不明ですが上に私が書いた「テキストファイルのパス,画像ファイルのパス,音声ファイルのパス」の情報は含まれているのではないですか？　であれば、それに「カテゴリー」を追加して形式を以下のようにし、

カテゴリー,テキストファイルのパス,画像ファイルのパス,音声ファイルのパス<改行＞

それを必要な数だけ複数行作る、すなわち CSV 形式にすることは可能だと思うのですが。それができれば CSV ファイルを読んできて List<T> 型のオブジェクトを作成して、Linq を使って扱うのが便利で、switch 文のハードコーディングを避けるという目的も果たせると思いますけど。

[Hottopia]

お付き合いいただきありがとうございます。

今回は、「インスタンスに引数を持たせ、遷移先のコンストラクタで分岐する。」というやり方が、適切ではないように感じていた為、この場にて皆様のご意見を伺っておりました。

今回のプログラムについては、問題文等のテキストリソースは全てメモ帳で作成し、txtファイルで管理したい、という要望があったため、このような形式にこだわっていました。
このようなローカルファイルでデータを管理するようなケースでは、SurferOnWww様のおっしゃるようにCSVファイルでの管理が適切なようですので、そういった設計にしたい思います。私自身不勉強で、説明能力も足りていませんでしたが、丁寧に教えてくださってありがとうございました。

[退会済みユーザー]

> 今回は、「インスタンスに引数を持たせ、遷移先のコンストラクタで分岐する。」というやり方が、適切ではないように感じていた為、この場にて皆様のご意見を伺っておりました。

意味不明です。話が変わって来てませんか？

タイトルに「switch文による分岐を改修したい」とあって、現状ハードコーディングしてある switch 文をカテゴリーの増減によって書き直さなくても済むようにしたいという話ではなかったのですか？

どうもお互い話が通じなかったみたいですね。

[退会済みユーザー]

とりあえず回答いくつか出てますが、放置せずに回答の内容で解決するのか、解決しないのであればその理由を書くか、何かコメントつけたらどうですか？回答した人に失礼ですよ。

[Hottopia]

SurferOnWww様　
質問を見返してみましたが、確かにそう受け取られてしまう書き方だったと思います。修正しました。

[Hottopia]

radian様 失礼致しました。質問に追記をしましたが、データ構造についてもお聞きしたいとは思っていましたが、メインであるのは遷移時のやり方についてです。お手を煩わせてしまい申し訳ありませんでした。

[Zuishin]

「遷移時のやり方」が何を意味しているのかわかりません。私には十分な回答が出ているように見えるので、何に引っかかっているのかをもう少し具体的に質問した方がいいと思います。

[退会済みユーザー]

結局どういう回答を期待しているのかさっぱり分からないのですが。
[質問するときのヒント]
https://teratail.com/help/question-tips
ここの2と3を読んでください。
回答者はエスパーではないので、疑問点やどういう答えが欲しいかを相手に判るように具体的に伝えないと期待した答えは返せません。

[ozwk]

(勘違いのため削除)

[dodox86]

Zuishinさんと同意見。いただいている回答を理解すればクローズするかと思ったのですが、そうでもないのですね。「遷移」という言葉が出てきていますが、これは画面遷移のことでしょうか。「カテゴリー選択画面」という言葉が質問中にありますね。ページとは、Pageクラスを使っている画面を指しているのでしょうか。コードが断片的で、質問者さんの意図が正確に伝わらないのだと思います。（私もいまだ、分かりません）

[退会済みユーザー]

次回質問することがあるなら、ぜひこれをよく読んでからにしてください⇒ https://teratail.com/help/question-tips

特にこのあたり：

"人に質問をするには、自分が何を尋ねたいかを知っている必要があります。これは、「自分が今『何がわからないのか』がわかっていて、言語化できている」ということです。一見簡単なようですが、実はとても難しいことです。何がわからないかわからない人から質問を受けても、答える側も困ってしまいます。"

[Hottopia]

はい、自分でも、途中から何を伝えたいのか、意味の伝わらない質問を重ねてしまっていたと反省しております。。
精進します。

Answer 1

たくさん回答も付いてるうえBAもされているようですが^^;

結局やりたいことって「問題の順番・選択肢の順番をランダムにしたい」ということですよね？
難しく考えすぎというか、外野であれこれしようとしすぎというか。

今回の場合は単に取得時に、都度シャッフルした配列・リストを返せばいいだけに見えます。
カテゴリもなぜ文字列で渡さなければならないかが、コードから読み取れません（リスト自体を渡せばいいだけでは？）

xamlがわからないので何とも言えませんが、ItemsControl等をうまく使うとコレクション内の個数を気にせず表示できます（回答は選択肢だけで使っていますが、カテゴリでも同じように作れます）

コードがないとイメージしにくそうなので、非常に雑ですがサンプルです（ファイル云々は省略）

MainWindow

<NavigationWindow
  x:Class="Questions370911.MainWindow"
  xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
  xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
  Width="800"
  Height="450"
  ShowsNavigationUI="False"
  Source="/CategoryPage.xaml" />

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

namespace Questions370911
{
    public class QuizData
    {
        public string Question { get; }

        // プロパティにして取得のたびにシャッフルした配列を返す
        public string[] Choices
        {
            get
            {
                return _Choices.OrderBy(_ => Guid.NewGuid()).ToArray();
            }
        }
        // ↑↓同じ意味
        //public string[] Choices => _Choices.OrderBy(_ => Guid.NewGuid()).ToArray();

        public string Answer { get; }

        private readonly string[] _Choices;

        public QuizData(string question, params string[] choices)
        {
            Question = question;
            _Choices = choices;
            Answer = choices[0];
        }
    }


    public partial class MainWindow // : Window
    {
        // ここも
        public List<QuizData> A_QuizData => _A_QuizData.OrderBy(_ => Guid.NewGuid()).ToList();
        public List<QuizData> B_QuizData => _B_QuizData.OrderBy(_ => Guid.NewGuid()).ToList();
        public List<QuizData> C_QuizData => _C_QuizData.OrderBy(_ => Guid.NewGuid()).ToList();

        private readonly List<QuizData> _A_QuizData = new List<QuizData>
        {
            new QuizData("史上初の家庭用ゲーム機とされるものは？", "オデッセイ（マグナボックス）", "ホーム・ポン（アタリ）", "カラーテレビゲーム15（任天堂）"),
            new QuizData("ファミコンのコントローラーにはマイクが付いている？", "○", "×"),
            new QuizData("スーパーファミコンソフト「ドラゴンクエスト6 幻の大地」の定価(税抜）は？", "11,400円", "14,800円", "9,700円", "5,980円"),
            new QuizData("PlayStationの同時発売ソフトは？", "リッジレーサー（ナムコ）", "モータートゥーン・グランプリ（SCE）", "チョロQ（タカラ）"),
        };
        private readonly List<QuizData> _B_QuizData = new List<QuizData>
        {
            new QuizData("hogehoge", "A", "B", "C"),
        };
        private readonly List<QuizData> _C_QuizData = new List<QuizData>
        {
            new QuizData("fugafuga", "1", "2", "3", "4"),
        };


        public MainWindow() => InitializeComponent();
    }
}

CategoryPage

<Page
  x:Class="Questions370911.CategoryPage"
  xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
  xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml">
  <UniformGrid Columns="3">
    <Button Click="ButtonA_Click" Content="おじさんゲーマークイズ" />
    <Button Click="ButtonB_Click" Content="hogehoge" />
    <Button Click="ButtonC_Click" Content="fugafuga" />
  </UniformGrid>
</Page>

using System.Windows;
using System.Windows.Controls;
using System.Windows.Navigation;

namespace Questions370911
{
    public partial class CategoryPage : Page
    {
        // こうしてしまうと「今」表示しているMainWindowとは別のMainWindowを作ってしまっている！
        //readonly MainWindow _main = new MainWindow();
        private readonly MainWindow _main;

        public CategoryPage()
        {
            InitializeComponent();

            // 手軽にやるならこう
            _main = (MainWindow)Application.Current.MainWindow;
        }

        private void ButtonA_Click(object sender, RoutedEventArgs e)
            => NavigationService.Navigate(new QuizPage(_main.A_QuizData));
        private void ButtonB_Click(object sender, RoutedEventArgs e) 
            => NavigationService.Navigate(new QuizPage(_main.B_QuizData));
        private void ButtonC_Click(object sender, RoutedEventArgs e) 
            => NavigationService.Navigate(new QuizPage(_main.C_QuizData));
    }
}

QuizPage

<Page
  x:Class="Questions370911.QuizPage"
  xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
  xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
  xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
  xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
  d:DesignHeight="450"
  d:DesignWidth="800"
  mc:Ignorable="d">
  <Page.Resources>
    <Style TargetType="TextBlock">
      <Setter Property="HorizontalAlignment" Value="Center" />
      <Setter Property="VerticalAlignment" Value="Center" />
    </Style>
    <Style TargetType="Button">
      <Setter Property="HorizontalAlignment" Value="Center" />
      <Setter Property="VerticalAlignment" Value="Center" />
    </Style>
  </Page.Resources>
  <Grid>
    <!--
      TabControlを使った深い意味はありません。
      比較的シンプルでデザインも確認しやすいって程度です。
    -->
    <TabControl x:Name="tabControl" BorderThickness="0">
      <TabItem Header="問題" Visibility="Collapsed">
        <Grid>
          <Grid.RowDefinitions>
            <RowDefinition />
            <RowDefinition />
          </Grid.RowDefinitions>
          <TextBlock x:Name="questionTextBlock" Text="問題文" />
          <ItemsControl
            x:Name="itemsControl"
            Grid.Row="1"
            HorizontalAlignment="Center"
            VerticalAlignment="Center"
            d:ItemsSource="{d:SampleData}"
            Button.Click="AnswerButton_Click">
            <ItemsControl.ItemsPanel>
              <ItemsPanelTemplate>
                <WrapPanel />
              </ItemsPanelTemplate>
            </ItemsControl.ItemsPanel>
            <ItemsControl.ItemTemplate>
              <DataTemplate>
                <Button Margin="5" Content="{Binding}" />
              </DataTemplate>
            </ItemsControl.ItemTemplate>
          </ItemsControl>
        </Grid>
      </TabItem>

      <TabItem Header="正解" Visibility="Collapsed">
        <Grid>
          <Grid.RowDefinitions>
            <RowDefinition />
            <RowDefinition />
          </Grid.RowDefinitions>
          <TextBlock Text="正解" />
          <Button
            Grid.Row="1"
            Click="NextButton_Click"
            Content="次へ" />
        </Grid>
      </TabItem>

      <TabItem Header="不正解" Visibility="Collapsed">
        <Grid>
          <Grid.RowDefinitions>
            <RowDefinition />
            <RowDefinition />
          </Grid.RowDefinitions>
          <TextBlock Text="不正解" />
          <Button
            Grid.Row="1"
            Click="NextButton_Click"
            Content="次へ" />
        </Grid>
      </TabItem>

      <TabItem Header="成績" Visibility="Collapsed">
        <Grid>
          <Grid.RowDefinitions>
            <RowDefinition />
            <RowDefinition />
          </Grid.RowDefinitions>
          <TextBlock x:Name="resultTextBlock" Text="〇問中〇問正解" />
          <Button
            Grid.Row="1"
            Click="BackButton_Click"
            Content="戻る" />
        </Grid>
      </TabItem>
    </TabControl>
  </Grid>
</Page>

using System;
using System.Collections.Generic;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Navigation;

namespace Questions370911
{
    public partial class QuizPage : Page
    {
        private readonly List<QuizData> quizData;
        private int index;
        private int correct;

        // わざわざ文字列でカテゴリを渡すより、直接リストをもらったほうが早いでしょう
        // カテゴリ名も欲しければ、KOZ6.0さんの回答のようにクラスにまとめましょう
        public QuizPage(List<QuizData> quizData)
        {
            InitializeComponent();

            this.quizData = quizData;

            questionTextBlock.Text = quizData[index].Question;
            itemsControl.ItemsSource = quizData[index].Choices;
            tabControl.SelectedIndex = 0;
        }

        private void AnswerButton_Click(object sender, RoutedEventArgs e)
        {
            if (e.OriginalSource is Button button)
            {
                if ((string)button.Content == quizData[index].Answer)
                {
                    tabControl.SelectedIndex = 1;
                    correct++;
                }
                else
                {
                    tabControl.SelectedIndex = 2;
                }

                index++;
            }
        }

        private void NextButton_Click(object sender, RoutedEventArgs e)
        {
            if (index < quizData.Count)
            {
                questionTextBlock.Text = quizData[index].Question;
                itemsControl.ItemsSource = quizData[index].Choices;
                tabControl.SelectedIndex = 0;
            }
            else
            {
                resultTextBlock.Text = $"{quizData.Count}問中{correct}問正解";
                tabControl.SelectedIndex = 3;
            }
        }

        private void BackButton_Click(object sender, RoutedEventArgs e)
            => NavigationService.Navigate(new Uri("/CategoryPage.xaml", UriKind.Relative));
    }
}

---

それより_main = new MainWindow();のほうが気になります。
今表示しているMainWindowとは別のMainWindowを作ってしまっています。
これでは無駄に何度もファイルを読むことになります。

Comments

[TN8001]

個人的にはNavigationWindowやFrameは、挙動が独特（ブラウザ寄り）すぎて好きじゃないですね。

特に↓の「イヤな点」が嫌すぎます。
[WPFでシンプルな独自ナビゲーション処理のサンプルを書いてみた - SourceChord](https://sourcechord.hatenablog.com/entry/2016/02/01/003758

「じゃあどうすんだ？」と言われると、これと言ってベストな解があるわけでもないんですが^^;

[Hottopia]

まさに、ページ遷移の方法についても、不便さを感じていました。リンクまで貼っていただいてありがとうございます。

Answer 2

正直，意味がよくわからないが……

「CategoryPage でどのボタンが押されたか？」次第で，以降の処理で取り扱うべきデータの種類（category）が定まる
↓
のであれば，それ以降（categoryが定まって以降）に生成される Quiz とか MainWindow とかいうやつらは，その特定のcategoryのデータのみを用意して扱えば良いのではないのでしょうか？

すなわち，そもそも
A_QuizData, B_QuizData, C_QuizData という３つのListを保有する意味はないのでは？
Listは単独で良く，単にそのListに選ばれたcategoryに相当するデータファイルを読み込んでやればよいだけでは．

Comments

[fana]

極端なことを言えば，CategoryPage の各ボタンのイベントにて，

new Quiz( そのカテゴリ用のファイル群があるディレクトリのパス );

とかいう形のことをするとしたら，
Quiz は MainWindow にこのパスをそのまま渡せばよく，
MainWindow は渡されたこのパスを用いて，単一のカテゴリ用のデータを List に読み込めばよく，
Quiz は MainWindow が保有している List を単にいじくりまわせばいい．

Quiz も MainWindow も，「選択された Category は何なのか？」なんてことは全く把握していないし，把握する必要もないのでは．

[Hottopia]

皆様ご回答ありがとうございました、
もっと質問を整理して出直します。なんとなく理想に一番近い気がしたので、fana様をベストアンサーとさせて頂きます。質問力など技量が足らず混乱させてしまったかと思います。すみませんでした。

[fana]

考えの整理がついてからベストアンサーを選ぶのでも良いのでは…という気がします．
一旦この質問は閉じて仕切り直したいのだとしても，
他の既存回答の 意味/意義 などを踏まえたうえで閉じ方を判断するのが良いでしょう．

コメントを拝見するに他回答の内容をデータ構造の話としか見えていない様子とも見受けますが，
そもそもデータ構造が変わればそれを扱う処理コード（「遷移時のやり方」等）も相応に変わる（というか，変えることができる）わけですから，少し考えてみてはどうでしょうか．
仮にそれらのデータ構造を採用したならば，あなたのコードはどうなるだろうか？　というのを．

Answer 3

QuizDataにカテゴリを含めてしまえば、このような事が出来ます。

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

public class QuizData
{
    public QuizData(string category, string quizText)
    {
        Category = category;
        QuestionText = quizText;
    }
    public string Category { get; } //カテゴリ
    public string QuestionText { get; } //テキスト(問題文)
}

class Program
{

    static void Main()
    {
        //テストデータ作成（カテゴリは A,B,C,A,B,C…となる）
        var listQuiz = new List<QuizData>();
        for( int i = 0; i<10; i++ )
        {
            listQuiz.Add(
                new QuizData(
                    ((char)('A' + (i % 3))).ToString(),
                    $"QuestionText{i}"
                )
            );
        }

        //カテゴリをKey、そのカテゴリに含まれるQuizDataのListをValueとするDictionaryを作成
        var dictQuiz = listQuiz.ToLookup(quiz => quiz.Category)
            .ToDictionary(quiz => quiz.Key, quiz => quiz.ToList());

        //カテゴリ一覧を出力
        foreach (var key in dictQuiz.Keys)
        {
            Console.WriteLine($"Category:{key}");
        }

        //カテゴリ"B"のクイズを出力
        foreach(var quiz in dictQuiz["B"])
        {
            Console.WriteLine($"QuizData(Category, QuestionText):{quiz.Category}, {quiz.QuestionText}");
        }

        Console.ReadKey();
    }
}

(出力結果)
Category:A
Category:B
Category:C
QuizData(Category, QuestionText):B, QuestionText1
QuizData(Category, QuestionText):B, QuestionText4
QuizData(Category, QuestionText):B, QuestionText7

Answer 4

こんな感じですべてのクイズを持たせる Dictionary を作れば良いのでは？

public Dictionary<string, List<QuizData>> All_QuizData 
                = new Dictionary<string, List<QuizData>>();

All_QuizData["A_"] = A_QuizData;
All_QuizData["B_"] = B_QuizData;
All_QuizData["C_"] = C_QuizData;

ShuffleOrderQuizData(string _category) は

public void ShuffleOrderQuizData(string _category) {
    List<QuizData> target = _main.All_QuizData[_category];
    target = target.OrderBy(a => Guid.NewGuid()).ToList();
    for (var i = 0; i < target.Count; i++) {
        target[i].ChoiceText = target[i].ChoiceText.OrderBy(a => Guid.NewGuid()).ToArray();
    }
}

のような感じになります。

私だったら List<QuizData> の部分は、以下のようなクラスを作って置き換えるかも。

public class QuizDataList : List<QuizData>
{
    public string Category { get; }
    public QuizDataList(string category) {
        Category = category;
    }
}

Comments

[Hottopia]

KOZ6.0様 失礼致しました。質問に追記をしましたが、データ構造についてもお聞きしたいとは思っていましたが、メインであるのは遷移時のやり方についてです。お手を煩わせてしまい申し訳ありませんでした。

Answer 5

public List<QuizData> A_QuizData = new List<QuizData>(); //1要素につき、下記 QuizData classのインスタンスを格納

public List<QuizData> B_QuizData = new List<QuizData>(); //同上
public List<QuizData> C_QuizData = new List<QuizData>(); //同上

って、List<QuizData>のリストにすればいいのでは
数値（変数）でアクセスできますよ

Comments

[Hottopia]

y_waiwai様 失礼致しました。質問に追記をしましたが、データ構造についてもお聞きしたいとは思っていましたが、メインであるのは遷移時のやり方についてです。お手を煩わせてしまい申し訳ありませんでした。