[java]クイズプログラムを分かりやすくしたい

Question

import java.io.*;

public class quiz   {
    public static void main (String[]args) throws IOException{
        InputStreamReader is = new InputStreamReader(System.in);       
        BufferedReader br = new BufferedReader(is);  
        int[] mondai01 = new int [9];
        mondai01[0] = 1;
        mondai01[1] = 1;
        mondai01[2] = 1;
        switch (mondai01[0]) {          //第1問
        case 1:
        System.out.println("第1問 これを作っている言語の名前はなんでしょう。");
        System.out.println("1.C# 2.Java 3.Unity");
        System.out.println("なお、数字のみ入力してください。(以後共通)例：3");
        String kotae1 = br.readLine();

        if (kotae1.equals("1")){
        System.out.println("不正解です。残念。");
        }else if(kotae1.equals("2")){
        System.out.println("正解!第２問に進みます。");
         switch (mondai01[1])    {                  //第2問
        case 1:
        System.out.println("第2問 ドラえもんの誕生日はいつでしょう。");
        System.out.println("1.2109年9月11日 2.2110年9月4日 3.2112年9月3日");
        String kotae2 = br.readLine();

        if(kotae2.equals("1")){
        System.out.println("不正解です。残念。");
        }else if(kotae2.equals("2")){
        System.out.println("不正解です。残念。");
        }else if(kotae2.equals("3")){
        System.out.println("正解です!第三問に進みます。");
        }

        switch (mondai01[2]) {          //第3問
        case 1:
        System.out.println("第3問 将棋に使う駒の数は敵味方合わせていくつでしょう？");
        System.out.println("1.30個 2.40個 3.50個");
        String kotae3 = br.readLine();

        if(kotae3.equals("1")){
        System.out.println("不正解です。残念。");    
        }else if(kotae3.equals("2")){
        System.out.println("正解です!第4問に進みます。");
        }else if  (kotae3.equals("3")){
        System.out.println("不正解です。残念。");
        }
        }
    }
        }else if(kotae1.equals("3")){
        System.out.println("不正解です。残念。");
        }else{
        System.out.println("数字のみ入力してください。");
        break;
            }
        }
    }
}   
              
    

このようなコードをVisual studio Code で書いています。
見ての通り、数字のみ入力してください。という処理がkotae2とkotae3にはありません。
このようなミスをなくすためにコメントなどをつけてみましたが、やっぱりわかりませんでした。
このようなミスをなくすには、どうしたらいいでしょうか。

Answer 1

まずは、コードとデータを分離することを考える必要があります。
問題文を出したり、答えをチェックしたりするプログラムとクイズのデータを分けることを考えるべきです。
コードの中に問題文や答えの文字列が埋まっていると同じロジックを繰り返すということができません。

以下にテキストファイルに3行で1問を表すデータファイルを作って、データを分離する例を示します。テキストファイルは1行目は問題文、2行目は選択肢、3行目は正解を繰り返すというフォーマットです。

import java.io.*;

public class quiz   {
  private String question;
  private String candidates;
  private int answer;
  
  quiz(String q, String c, int a) {
    question = q;
    candidates = c;
    answer = a;
  }
  
  public static quiz readNewQuiz(BufferedReader quizDataReader) throws IOException {
    String q;
    if ((q = quizDataReader.readLine()) == null) {
      return null;
    }
    String c;
    if ((c = quizDataReader.readLine()) == null) {
      return null;
    }
    String a;
    if ((a = quizDataReader.readLine()) == null) {
      return null;
    }
    return new quiz(q, c, Integer.parseInt(a));
  }
  
  public void printQuestion(int count, PrintStream ps) {
    ps.println("第" + count +"問 " + question);
    ps.println(candidates);
    ps.println("なお、数字のみ入力してください。(以後共通)例：3");
  }
  
  public boolean checkAnswer(int ans) {
    return answer == ans;
  }
  
  public static int readNumber(BufferedReader br, PrintStream errorStream) throws IOException {
    while (true) {
      try {
        int ans = Integer.parseInt(br.readLine());
        if (ans < 1 || ans > 3) {
          errorStream.println("1 から 3 の範囲で入力してください");
        } else {
          return ans;
        }
      } catch (NumberFormatException e) {
        errorStream.println("数値を入力してください");
      }
    }
  }
  
  public static void main (String[]args) throws IOException {
    BufferedReader quizDataReader = new BufferedReader(new FileReader("quizdata.txt"));
    InputStreamReader is = new InputStreamReader(System.in);
    BufferedReader br = new BufferedReader(is);
    int count = 1;
    while (true) {
      quiz q = readNewQuiz(quizDataReader);
      if (q == null) break;
      q.printQuestion(count, System.out);
      int ans = readNumber(br, System.out);
      if (q.checkAnswer(ans)) {
        System.out.println("正解!第" + ++count +"問に進みます。");
      } else {
        System.out.println("不正解です。残念。");
        break;
      }
    }
  } 
}

同じフォルダに quizData.txt という名前で以下の内容のファイルを置いておきます。

これを作っている言語の名前はなんでしょう。
1.C# 2.Java 3.Unity
2
ドラえもんの誕生日はいつでしょう。
1.2109年9月11日 2.2110年9月4日 3.2112年9月3日
3
将棋に使う駒の数は敵味方合わせていくつでしょう？
1.30個 2.40個 3.50個
2

ここでは、簡単のため独自フォーマットのテキストファイルにしましたが、JSONやXML形式のファイルにするほうが、もっと構造化ができて良いでしょう。
本格的なアプリケーションになれば、データベースにこれらのデータを入れるかもしれません。

あと、細かくはいろいろとありますが、quiz を1問ずつオブジェクトにする点など、コードを参考にしていただければと思います。

Comments

[asadako]

なるほど。コードとデータを分離すればいいのですね。
ご回答ありがとうございました！

Answer 2

同じような処理を繰り返すのであれば、その同じような処理はまとめてしまいましょう。

現状のソースコードは
・問題表示
・入力
・正答判定
・
・
・・の、繰り返しになっています。

これをもっとシンプルに、
private static boolean 問題と解答(String 問題文, String 回答選択肢) {
・問題文の表示
・（数字を入力してください表示）
・入力
return 回答選択肢.equals(入力)
}

の、ようなメソッドを作り、繰り返し呼べばよいかと。

Comments

[asadako]

なるほど。繰り返し呼べばいいと。参考になりましたが、今回は別の方をベストアンサーにさせていただくことをご了承ください。