javaでの入力チェック

Question

入力された値が 0or1以外　ならもう一度値を繰り返すという処理を作成しています。

入力はScannerクラスを使用しており、int型を変数に受け取るようにしています。
もし入力された値が数字以外ならエラーを出し、もう一度入力する箇所に戻るという仕様にしたいです。

コードはこのようにしているのですが、正しく動作しません。
例えばdを入力すると、「入力エラーです」が表示されてループはされません。
どのように改善せるべきでしょうか。

int flg=0;
Scanner scan=new Scanner(System.in);
do {
	System.out.println("現金の場合は1,ICの場合は0を入力してください");
	try {
		flg = scan.nextInt();
	}catch(InputMismatchException e) {
		System.out.println("入力エラーです");
	}
　}while(flg>1||flg<0);

Answer 1

nextIntで例外が生じたときflgの値は書き換えられません。
この場合、値は初期値の0のままであり、ループから脱出します。初期値を適切に変更して下さい。

コメントを受けて

一点見落としていました。

nextIntに失敗した場合、バッファに食い残しができます。
その食い残しを何度も何度もnextIntで読もうとしてしまうので、無限ループします。

catch節で一行読んで捨てておけば良いでしょう。

try {
    flg = scan.nextInt();
}catch(InputMismatchException e) {
    System.out.println("入力エラーです");    // ここは標準エラー出力がいいかも
    scan.nextLine();
}

Comments

[monatos]

この書き方だとエラーが発生した時はdo-whileに戻らずに外に出てしまっていると思うのですが、それへの対処を教えていただきたいです。

[LouiS0616]

回答に書いたとおりです。「初期値を適切に変更して下さい」。

[monatos]

int flg=-1;に変更して実行してみました。
そうすると無限ループになり、値を入力するところにたどり着きませんでした。

現金の場合は1,ICの場合は0を入力してください
入力エラーです
現金の場合は1,ICの場合は0を入力してください
入力エラーです
現金の場合は1,ICの場合は0を入力してください
...
上記のような出力結果です。

[LouiS0616]

それはちょっと想定外でした。手元で試してみるので少々お待ち下さい。

[monatos]

お手数おかけします。
ありがとうございます。

[LouiS0616]

原因と対処法を回答に追記しました。

[monatos]

ありがとうございます。解決することができました。
自分ではたどり着けない改善法でした。。
これからはバッファについても考えるようにしてみます。