【Java】頑固な入力待ちをしたい

Question

まだ入門書を読破していないレベルですが、モヤモヤするので質問させてください。

やりたいことは、次のようなコードで
0か1が入力されるまでひたすらdoを回したいだけなんです。

```java
import java.util.Scanner;
import java.util.InputMismatchException;

class IntBlnRensyuu {
  public static void main(String[] args) {
    try(Scanner stdIn = new Scanner(System.in);) {

      int x;
      do {
        System.out.print("0=true／1=false：");
        x = stdIn.nextInt();
      } while (x < 0 || x > 1);

      boolean a;
      if (x == 0) {
        a = true;
      } else {
        a = false;
      }
    System.out.println("論理型変数に" + a + "を代入しました");

    } catch (InputMismatchException e) {
      System.out.println("不正な入力値です");
      }
  }
}

```

しかしながら、これだと-1や2の入力ではループしますが
『a』や『1.5』を入力すると「不正な入力値です」を表示して終了してしまいます。
不正な入力があっても尚ループさせるには、どう修正するものなのでしょうか。

入門書の序盤はどれも、こういうのを気にしたら負けのようですが
私は気になるのでよろしくお願いします。

Accepted Answer

**catchがwhileの外にある**のが原因です。例外が起これば何があろうと(対応する例外がある)catchに飛ぶため、whileから外れてしまいます。ループを続けるためには、**whileの中でcatchする**必要があります。ただし、今使っているtry-with-resourcesでcatchすると、catchする場合標準入力が閉じられてしまうため、別のtryを入れます。 ```java int x = -1; // あらかじめwhileの継続条件を満たす値を入れておくと安全 do { try { System.out.print("0=true／1=false："); x = stdIn.nextInt(); } catch (InputMismatchException e) { System.out.println("不正な入力値です"); } } while (x < 0 || x > 1); ``` ### 無限ループの原因(コメントを受けて) > 試してみると、 > 「0=true／1=false：不正な入力値です」の永久ループになってしまいました・・自分でも試してみると、aなどの不正入力をすると無限ループ発生。無限ループしないように工夫してスタックトレースを2回出してみるも、 Integer.parseInt()のNumberFormatExceprionと違って Scanner#nextInt()のInputMismatchExceptionは入力が何かはスタックトレースに乗らないらしいそこで[Scanner#nextInt](https://docs.oracle.com/javase/jp/8/docs/api/java/util/Scanner.html#nextInt--)の説明を読むとこんな文言が。 > 入力の次のトークンをintとしてスキャンします。次に示す方法で、次のトークンを有効なint値に変換できない場合、このメソッドはInputMismatchExceptionをスローします。**変換に成功すると、スキャナは一致した入力の先に進みます。** この太字部分は、裏を返せば「変換できない場合スキャナの位置は動かない」ということではないか。となるとどういうことが起こるか。例えば"a"を入れた場合、 0. nextInt()で数値変換できないため例外発生。 0. スキャナの位置は動かないため、まだ入力には"a"が残った状態になる。 0. ループによりこの状態で再びnextInt()が呼ばれるため、また"a"を変換しようとして例外発生。これが永遠に繰り返されるわけだ。これを防ぐためには、例外発生時に変換できなかった文字を飛ばす必要がある。 ```java do { try { System.out.print("0=true／1=false："); x = stdIn.nextInt(); } catch (InputMismatchException e) { System.out.println("不正な入力値です"); stdIn.next(); // 変換できなかった文字列を飛ばす } } while (x < 0 || x > 1); ``` ただこれだと不格好なので、nextInt()を使わず文字列で読み込んでparseInt()という方法も考えられる。 ```java do { try { System.out.print("0=true／1=false："); x = Integer.parseInt(stdIn.next()); } catch (NumberFormatException e) { System.out.println("不正な入力値です"); } } while (x < 0 || x > 1); ``` --- ちなみに ```java boolean a; if (x == 0) { a = true; } else { a = false; } ``` ここのコードは次の1行で片づけられます。 ```java boolean a = (x == 0); ```

Answer

Ctrl-D の入力にも対処してみました。

```java
import java.util.Scanner;

class IntBlnRensyuu {
    public static void main(String[] args) {
        Scanner stdIn = new Scanner(System.in);

        // 0, 1, または Ctrl-D が有力されるまで ループする。
        // ループを抜けたときに、x には次のように設定されている。
        //  0 (0 が入力されたとき)
        //  1  (1 が入力されたとき)
        //  -1 (Dtrl-D が入力されたとき)
        int x;
        while (true) {
            System.out.print("0=true／1=false：");

            // CTRL-D が入力されたときの処理
            if (!stdIn.hasNext()) {
                x = -1;
                break;
            }
            String line = stdIn.nextLine().trim();
            if (line.equals("0") || line.equals("1")) {
                x = Integer.parseInt(line);
                break;
            }

            System.out.println("不正な入力値です");
        }

        if (x == -1) {
            System.out.println("入力が終了されました。");
        } else {
            boolean a = (x == 0);
            System.out.println("論理型変数 a に" + a + "を代入しました");
        }
    }
}
```

Answer

参考までに、himejiy3さんのアプローチとは他の方法を紹介します。
```Java
import java.util.Scanner;
import java.io.IOException;

class Main {
    static int input_1_or_0(String message, Scanner sc) throws IOException {
        while(true) {
            System.out.print(message);
            String input = sc.nextLine();
            
            if(input.equals("0")) return 0;
            if(input.equals("1")) return 1; // return Integer.valueOf(input); も可
            
            System.out.println("論理型変数に" + input + "を代入しました");
        }
    }

    public static void main(String[] args) {
        try(Scanner sc = new Scanner(System.in)) {
            System.out.println(
                input_1_or_0("0=true／1=false： ", sc)
            );
        }
        catch(IOException e) {
            e.printStackTrace();
        }
    }
}
```

実行例 [Wandbox](https://wandbox.org/permlink/56oZXPg358bOAMzo)
こういうときは文字列のまま扱った方が楽ですね。

無限ループの原因(コメントを受けて)

関連した質問