try-catchの使い方について

Question

### 前提・実現したいこと

try-catch文で「０から２の数字の判定」、「文字列かどうかの判定」を行いたいです。
「０から２」以外が入力された場合、エラーメッセージを表示し最初の選択肢に戻るを実行させたいです。

発生している問題・エラーメッセージ

「０から２の数字の判定」はうまくいくのですが、「文字列かどうかの判定」をしたときにエラーメッセージが表示されるとともに、選択の余地なくif文の処理が勝手に実行されてしまいます。

該当のソースコード

package JankenGame;

import java.util.InputMismatchException;
import java.util.Scanner;

//Playerクラス
class Player extends TotalPlayer {
	Scanner scanner = new Scanner(System.in);
	private int myHand;

	/**
	 * Playerの手を入力するメソッド
	 * 
	 * @return 整数値の標準入力
	 */
	int getHand() {
		return this.myHand;
	}

	void getJudgeHand() {
		while (true) {
			try {
				System.out.println("コマンド？ 0:グー 1:パー 2:チョキ");
				this.myHand = scanner.nextInt();
				if (this.myHand <= 2) {
					break;
				}
			} catch (InputMismatchException e) {
				System.out.println("整数値ではありません");
				break;
			}
			System.out.println("０から２で入力してください");
		}
	}

}

試したこと

breakを消したり、他の位置に移動させてみたりしましたが解決策がわかりません。
デバック機能も使用してみたのですが、原因はつかめませんでした。
不勉強で申し訳ありませんが、アドバイスなど頂けたらと思います。
よろしくお願いします。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

＞０から２の数字の判定
これはtry-catchの役目ではないのでは？
Exceptionでもなんでもないです。

[momo1122]

ご指摘ありがとうございます。

>これはtry-catchの役目ではないのでは？
Exceptionでもなんでもないです。
ご回答ありがとうございます。質問させて頂きたいのですが、そうなると「０から２以外の数字の判定」はif文で判定を行えばいいのでしょうか？

Answer 1

Scanner の hasNextInt を使って数値であるかどうかチェックできます。

    void getJudgeHand() {
        while (true) {
            System.out.println("コマンド？ 0:グー 1:パー 2:チョキ");
			if (scanner.hasNextInt()) {
                myHand = scanner.nextInt();
                if (myHand >= 0 && myHand <= 2) break;
			}
			else {
				scanner.next();
                System.out.println("整数値ではありません");
            }
            System.out.println("0から2で入力してください");
        }
    }

Comments

[momo1122]

kazuma-sさん
いつもご回答頂きありがとうございます。
こういった書き方もあるのですね。大変参考になります。
ありがとうございます????‍♀️！！

Answer 2

もう解決しているようですので、参考のために回答させていただきます。
たかが0から2の数値チェックで例外を起こすのはあまりよろしくないかと思います。
今回程度のプログラムなら何も問題はないのですが、例外（Exception）の発生というのは、
時間がかかったり、メモリーを多く消費したり、大変コストがかかる処理になるからです。
なので、例えば以下のように事前にチェックをする方がいいかと思います。

void getJudgeHand() {
    while (true) {
        System.out.println("０から２で入力してください。");
        System.out.println("コマンド？ 0:グー 1:パー 2:チョキ");
        if (!scanner.hasNext("[0-2０-２]")) {
            scanner.next();
            System.err.println("入力が正しくありません。");
            continue;
        }
        this.myHand = scanner.nextInt();
        System.out.println(this.myHand + "が入力されました。");
    }
}

Comments

[momo1122]

root_jpさん
ご丁寧にご教示いただきありがとうございます。
こういった処理の書き方もあるのですね。必ず例外処理を行わないといけないと勘違いしていました。
大変勉強になります、いただいたコードを実行してみて勉強させて頂きます。ありがとうございます。

Answer 3

以下でよいかと。

void getJudgeHand() {
	while (true) {
		try {
			System.out.println("コマンド？ 0:グー 1:パー 2:チョキ");

			this.myHand = scanner.nextInt();
			if (this.myHand <= 2 && this.myHand >= 0) { // 2以下 かつ、0以上ならbreak
				break;
			}
		} catch (InputMismatchException e) {
			System.out.println("整数値ではありません");
			scanner.next();	// 入力を捨てる
		}
		System.out.println("０から２で入力してください");
	}
}

Comments

[momo1122]

momon-gaさん
ご回答ありがとうございます。
ご教示いただいたコードで解決することができました。
１つ質問をさせて頂きたいのですが、「入力を捨てる」とはどういった仕組みなのでしょうか？
よろしくお願い致します。

[momon-ga]

String tmp = scanner.next();
とすれば、わかるでしょうか。
ただ、tmp自体は使い道がないため、ただscanner.next();を呼び出しています。

[momo1122]

ご回答いただきありがとうございます。
ただ呼び出されるということについて少しが理解ができないので、自分自身でいろいろ検索してみます。
ご丁寧にご説明してくださり、ありがとうございました！！