try-catchの処理で何故か無限ループになる

Question

じゃんけんをするプログラムのゲーム回数を入力してもらう部分です。

数字以外の「aaa」と入力すると、
ゲーム回数を入力してください:エラーです。

ゲーム回数を入力してください:エラーです。

ゲーム回数を入力してください:エラーです。

と無限ループします。

catch処理のあとループでまたtryの中に入ると思っていましたがうまくいきません。

自然数を入力するまで何度も入力させるようにしたいのですがどうしたらいいでしょうか。

public void play() {

        int rounds;

        Scanner scan = new Scanner( System.in);
        while ( true) {
            System.out.print( "ゲーム回数を入力してください:");
            try {
                rounds = scan.nextInt();
                while ( rounds<0) {
                    System.out.println( "エラーです。\n");
                    System.out.print( "ゲーム回数を入力してください:");
                    rounds = scan.nextInt();
                }
                break;
            }
            catch ( Exception e) {
                System.out.println( "エラーです。\n");
            }
        }

Comments

[退会済みユーザー]

無限ループはとまりませんよ。どう見ても有限です。

Answer 1

nextInt()は整数と見做せる部分を読み込んでくれるメソッドですが、"aaa"などとすると最初の文字'a'からしてもはや数字('0'～'9')でも符号でもないので一文字も入力が消費されないうちに例外が発生します。その状態でいくらnextInt()を繰り返しても例外しかおきないのです。

nextIntを使うのを止めましょう。

next()を呼び、その結果をInteger#parseIntで整数値へ変換するように直して試してみたらよいと思います。

nextIntは入力ミスを想定しなくてよい場合か、例外がおきたら処理を中断してよい場合に使いましょう。

Comments

[退会済みユーザー]

理解しました、ありがとうございます！

[退会済みユーザー]

hasNextIntでチェックできるよ。（笑）
※　クラス設計上の推奨策（例外コストガガガ）

[KSwordOfHaste]

そーなんですが、本要件では整数が読めるまで入力を継続しなければならないので、hasNextIntがfalseになったときにnext()でskipしなければならず、最初からnext()を用いてparseInt()した方が素直かなぁと感じたのです。

Answer 2

おそらく、

           rounds = scan.nextInt();

この関数の中でコンソールの入力、数値変換等をやってると思いますが、ここの処理が例外の発動で中途半端な状態になってるんじゃないでしょうか。
この関数のナカミがわかりませんが、そこらへんチェックしてみましょう

---

ああ、ScannerってJavaの標準関数（クラス？）だったのねｗ

Comments

[KSwordOfHaste]

失礼しました。かぶりました。
---
って、自作メソッドと思われてたんですね...いずれにせよ結論は同様のところへいくつくはずですね。

[y_waiwai]

いやー、不勉強なもので。。

[退会済みユーザー]

いやー、大規模アップデート内の些事（java tiger）での実装クラスですから把握してない人が多い気もする。

Answer 3

import java.util.Scanner;

public class readInteger {

    public static void main(String[] args) {
        // play();
        // playA();
        // playB();
        playC();
    }

    public static void play() {
        int rounds;
        Scanner scan = new Scanner(System.in);
        while (true) {
            System.out.print("ゲーム回数を入力してください:");
            try {
                rounds = scan.nextInt();
                while (rounds < 0) {
                    System.out.println("エラーです。\n");
                    System.out.print("ゲーム回数を入力してください:");
                    rounds = scan.nextInt();
                }
                break;
            } catch (Exception e) {
                System.out.println("エラーです。\n");
            }
        }
        System.out.println("回数:" + rounds);
    }

    public static void playA() {
        // "a b" と入力すると 2 回 エラーメッセージが出る
        int rounds;
        Scanner scan = new Scanner(System.in);
        while (true) {
            try {
                System.out.print("ゲーム回数を入力してください:");
                rounds = scan.nextInt();
                break;
            } catch (Exception e) {
                System.out.println("エラーです。\n");
                // 入力バッファのクリア
                scan.next();
            }
        }
        System.out.println("回数:" + rounds);
    }

    public static void playB() {
        // " 1" と入力すると、エラーメッセージが出る
        int rounds;
        Scanner scan = new Scanner(System.in);
        while (true) {
            try {
                System.out.print("ゲーム回数を入力してください:");
                rounds = Integer.parseInt(scan.nextLine());
                break;
            } catch (Exception e) {
                System.out.println("エラーです。\n");
            }
        }
        System.out.println("回数:" + rounds);
    }

    public static void playC() {
        // " 1" と入力すると、1 と認識する
        int rounds;
        Scanner scan = new Scanner(System.in);
        while (true) {
            try {
                System.out.print("ゲーム回数を入力してください:");
                rounds = Integer.parseInt(scan.nextLine().trim());
                break;
            } catch (Exception e) {
                System.out.println("エラーです。\n");
            }
        }
        System.out.println("回数:" + rounds);
    }
}

play() は質問文にあるコードです。
"a" と入力すると、エラーメッセージ表示が無限ループしてしまいます。

playA() は、無限ループしないように scan.next() 呼び出しを追加しました。また while を１ つに減らしました。
"a b" と入力すると、 2 回 エラーメッセージが表示されます
。
playB() は "a b" と入力しても、1回だけのエラーメッセージとなるようにしたものです。(行単位で入力を処理するようにしました)
" 3" と入力すると、エラーメッセージが表示されます。

playC() は、 " 3" を 3 と認識するようにしたものです。 (先頭の空白を取り除くようにしました)

参考情報

• 例外処理でリトライが無限ループする
  https://ameblo.jp/slont-code/entry-11886344684.html

Comments

[退会済みユーザー]

数字以外を3文字入力したから3回ループしてる「有限」のループにみせかけるとは質問者は（ｗ・

Answer 4

一番外側のwhile(true)を外せばいいと思います

Comments

[enoatu]

すいません、読み間違えていました。