Java 一度発火させたメソッドをif文で呼び出せない問題を解決したい。

Question

if文の中のメソッドを発火させたい

クイズプログラムを作っている。
A~Dの選択肢以外の答えが入力された際に、もう一度print();を実行したい。

2回目以降のprint();が発火しない

 if (checkAnswer(answer, st) == false) {
            System.out.println("選択肢の中から答えを入力してください。");
            print();
        }

該当のソースコード

public class Main {
    public static void main(String[] args) {
        Question q = new Question();
        q.print();
    }
}

import java.util.Arrays;
import java.util.Scanner;

public class Question {
    String answer;

    String str = "トランザクション処理プログラムが，データベース更新の途中で異常終了した場合，ロールバック処理によってデータベースを復元する。このとき使用する情報はどれか。";
    String A = "A：最新のスナップショット情報";
    String B = "B：最新のバックアップファイル情報";
    String C = "C：ログファイルの更新後情報";
    String D = "D：ログファイルの更新前情報";

    String[] st = { "A", "B", "C", "D" };

    public void print() {
        System.out.println(str);
        System.out.println(A);
        System.out.println(B);
        System.out.println(C);
        System.out.println(D);
        System.out.println("選択肢を入力してください。");

        input();
    }

    public void input() {
        Scanner scanner = new Scanner(System.in);
        String answer = scanner.nextLine().toUpperCase();

        if (checkAnswer(answer, st) == false) {
            System.out.println("選択肢の中から答えを入力してください。");
            print();
        }
    }

    public static boolean checkAnswer(String answer, String[] st) {
        if (Arrays.asList(st).contains(answer)) {
            return true;
        } else {
            return false;
        }
    }

}

実行結果

トランザクション処理プログラムが，データベース更新の途中で異常終了した場合，ロールバック処理によってデータベースを復元する。このとき使用する情報はどれか。
A：最新のスナップショット情報
B：最新のバックアップファイル情報
C：ログファイルの更新後情報
D：ログファイルの更新前情報
選択肢を入力してください。
⬛︎

試した事

該当しないfを入力した場合、プログラムが停止してprint();が実行されなかった。
a~dを入力した場合、プログラムが終了した。

開発・実行ツール

vscode
こちらを参考にvscodeでJavaを実行出来るようにしました。

Comments

[jimbe]

"1回目以降のprint();" では無く "2回目以降"の間違いでは無いでしょうか。

[You]

左様でございます。
ご指摘、ありがとうございます。

[jimbe]

あぁ、再帰してるんですね。失礼しました。

[jimbe]

何で開発・実行されていますか。

[shiketa]

\# 二回ですか…。わたしの環境では1000回は軽く繰り返せましたけれどね。

[You]

jimbeさん
vscodeを使用しております。

[jimbe]

vscode なら"再帰だから止まる"的なことは無さそうですね。
何が起きているのか今はちょっと想像が付かないです。
とりあえず momon-ga さんや私の回答のコードではどうなるでしょうか。

Answer 1

再帰呼び出しよりも普通のループを使う方がよいかと思います。

なるべく元のコードを残していますが、既回答にある通り、Scannerはつど生成しない方がよいです。

/** Main */
public class Main {
    public static void main(String[] args) {
        Question q = new Question();
//        q.print();
        q.input();
    }
}

/** Question */
import java.util.Arrays;
import java.util.Scanner;

public class Question {
    String answer;

    String str = "トランザクション処理プログラムが，データベース更新の途中で異常終了した場合，ロールバック処理によってデータベースを復元する。このとき使用する情報はどれか。";
    String A = "A：最新のスナップショット情報";
    String B = "B：最新のバックアップファイル情報";
    String C = "C：ログファイルの更新後情報";
    String D = "D：ログファイルの更新前情報";

    String[] st = { "A", "B", "C", "D" };

    public void print() {
        System.out.println(str);
        System.out.println(A);
        System.out.println(B);
        System.out.println(C);
        System.out.println(D);
        System.out.println("選択肢を入力してください。");
//        input();
    }

    public void input() {
        Scanner scanner = new Scanner(System.in);
        String answer;
        do {
	    	print();
	    	System.out.println("選択肢の中から答えを入力してください。");
	    	answer = scanner.nextLine().toUpperCase();
        } while(checkAnswer(answer, st) == false);
    }

    public static boolean checkAnswer(String answer, String[] st) { 
    	return Arrays.asList(st).contains(answer); // if文使わなくてもよい
//        if (Arrays.asList(st).contains(answer)) {
//            return true;
//        } else {
//            return false;
//        }
    }
}

Answer 2

環境による部分があるようですが、とりあえず。
Main はくっつけました。

再帰(printからinput を呼び, inputからprintを呼ぶ)はしないでください。
Scanner は何度も作らないようにしてください。

import java.util.Scanner;

public class Q358316 {
  public static void main(String[] args) {
    Scanner scanner = new Scanner(System.in);

    Q358316 q = new Q358316();
    for(String a=null; a==null; ) {
      q.print();
      a = q.input(scanner);
    }
  }

  String question =
      "トランザクション処理プログラムが，データベース更新の途中で異常終了した場合，ロールバック処理によってデータベースを復元する。このとき使用する情報はどれか。";
  String answers[] = {
      "最新のスナップショット情報",
      "最新のバックアップファイル情報",
      "ログファイルの更新後情報",
      "ログファイルの更新前情報",
  };

  public void print() {
    System.out.println(question);
    for(int i=0; i<answers.length; i++) {
      System.out.println(('A'+i)+"："+answers[i]);
    }
  }

  public String input(Scanner scanner) {
    System.out.println("選択肢を入力してください。");
    String answer = scanner.nextLine().toUpperCase();

    if (!checkAnswer(answer)) {
      System.out.println("選択肢の中から答えを入力してください。");
      return null;
    }
    return answer;
  }

  public boolean checkAnswer(String answer) {
    if(answer == null || answer.length() != 1) return false;
    int i = answer.charAt(0) - 'A';
    return 0 <= i && i < answers.length;
  }
}

Answer 3

追記。

trueが帰って来た時はinput()メソッドは終了します。print()メソッドも実行されません。
問題の解決にはなってませんよね。

ああ。たしかに。

該当しないfを入力した場合、プログラムが停止してprint();が実行されなかった。

あなたのソースをままコピペしてコンパイルして実行してみたら、プログラムは停止せずprint();が実行されました。現象そのものが発生しないので、わたしには解決策もありませんね。失礼しました。

>java Main
トランザクション処理プログラムが，データベース更新の途中で異常終了した場合，ロールバック処理によってデータベースを復元する。このとき使用
する情報はどれか。
A：最新のスナップショット情報
B：最新のバックアップファイル情報
C：ログファイルの更新後情報
D：ログファイルの更新前情報
選択肢を入力してください。
f
選択肢の中から答えを入力してください。
トランザクション処理プログラムが，データベース更新の途中で異常終了した場合，ロールバック処理によってデータベースを復元する。このとき使用
する情報はどれか。
A：最新のスナップショット情報
B：最新のバックアップファイル情報
C：ログファイルの更新後情報
D：ログファイルの更新前情報
選択肢を入力してください。

---

        if (checkAnswer(answer, st) == false) {
            System.out.println("選択肢の中から答えを入力してください。");
            print();
        }

trueが返ってきたとき、どうなりますか。 input()メソッドが終了するのではないですか？input()メソッドが終了すると、それを呼び出しているprint()メソッドはどうなりますか。

---

ちなみに。print()メソッドからinput()を呼び出し、input()メソッドからprint()をまた呼び出しているので、死ぬほど繰り返すとOutOfMemoryErrorが発生することでしょう。

Comments

[You]

trueが帰って来た時はinput()メソッドは終了します。
print()メソッドも実行されません。

問題の解決にはなってませんよね。

[xebme]

問題の切り分けのために、vscodeをやめて、Macのターミナルを直接開いて、javaを実行してみてはいかがでしょう。正常に実行できれば、vscodeの問題のようです。そうでなければ、javaのバージョンを変えてみるとか、 ... 。