コマンド引数が整数でなかった場合に、エラーメッセージに該当引数を表示したい
退会済みユーザー
総合スコア0

Question

Javaファイルをcmdで実行させます。
引数を2つ用意し、そのどちらかでも整数でない場合、
エラーメッセージとして「整数を入力してください：a」のように、「a」の部分に該当引数を表示させます。
引数は2つ渡しているとして、どちらか1つだけ整数でない場合は、上記のように1文でよいですが、2つとも整数でない場合は、上記文を2行表示します。
下記コードのようにtry-catchで例外に飛ばした時の、該当引数の取得の仕方を教えてください。
またはほかの方法でないと出来ないのであれば、その方法を教えてください。
よろしくお願いいたします。

Kadai1.java
```java
public class Kadai1 {
	public static void main(String[] args) {
		int num1 = 0;
		int num2 = 0;
		try {
			num1 = Integer.parseInt(args[0]);
			num2 = Integer.parseInt(args[1]);
			
		} catch (ArrayIndexOutOfBoundsException e) {
			/* 引数存在チェック */
			System.out.println("コマンドライン引数の数は２つにしてください。");
		} catch (NumberFormatException e) {
			/* 引数の整数チェック */
			// 下記メッセージのaの部分に該当引数を表示したい
			System.out.println("整数を入力してください：" + a);
		}
	}
}

```

Accepted Answer

dodox86さんのアドバイスを元に下記コード作成、エラー無く実行しました。
```java
public class Kadai1 {
	public static void main(String[] args) {
		int num1 = 0;
		int num2 = 0;
		int check = 0;
		final int size = 2;
		int leng = args.length;
		if (leng != size) {
			System.out.println("コマンドライン引数の数は２つにしてください。");
			System.exit(0);
		}
		/* args[0]の整数チェック */
		try {
			num1 = Integer.parseInt(args[0]);
		} catch (NumberFormatException e) {
			/* 引数の整数チェック */
			System.out.println("整数を入力してください：" + args[0]);
			check = 1;
		} finally {
			try {
				/* args[1]の整数チェック */
				num2 = Integer.parseInt(args[1]);
				if (check == 1) {
					System.exit(0);
				}
				int sum = num1 + num2;
				System.out.println(args[0] + "と" + args[1] + "の和は" + sum + "です。");
			} catch (NumberFormatException e) {
				/* 引数の整数チェック */
				System.out.println("整数を入力してください：" + args[1]);
			}
		}
	}
}

```

Answer

エラーチェックと本体処理を分離したほうが見やすくありませんか？
```Java
pubic class Kadai1 {
    public static void main(String[] args) {
		int ef = 0;  // error flags
        int num1 = 0;
        int num2 = 0;
		if (args.length != 2) ef |= 1;
		else {
			try {
				num1 = Integer.parseInt(args[0]);
			} catch (NumberFormatException e) { ef |= 2; }
			try {
				num2 = Integer.parseInt(args[1]);
			} catch (NumberFormatException e) { ef |= 4; }
		}
        if ((ef & 1) != 0)
			System.out.println("コマンドライン引数の数は２つにしてください。");
		if ((ef & 2) != 0)
			System.out.println("整数を入力してください：" + args[0]);
		if ((ef & 4) != 0)
			System.out.println("整数を入力してください：" + args[1]);
		if (ef != 0) return;

        int sum = num1 + num2;
        System.out.println(args[0] + "と" + args[1] + "の和は" + sum + "です。");
    }
}
```
**追記**
ビット演算になじみがなければ、
```Java
pubic class Kadai1 {
    public static void main(String[] args) {
		boolean error = false;
        int num1 = 0;
        int num2 = 0;
		if (args.length != 2) {
			System.out.println("コマンドライン引数の数は２つにしてください。");
			return;
		}
		try {
			num1 = Integer.parseInt(args[0]);
		} catch (NumberFormatException e) {
			System.out.println("整数を入力してください：" + args[0]);
			error = true;
		}
		try {
			num2 = Integer.parseInt(args[1]);
		} catch (NumberFormatException e) {
			System.out.println("整数を入力してください：" + args[1]);
			error = true;
		}
		if (error) return;

        int sum = num1 + num2;
        System.out.println(args[0] + "と" + args[1] + "の和は" + sum + "です。");
    }
}
```

Answer

１つのtry～catchでは１つの例外しか補足できません。
したがって

> 2つとも整数でない場合は、上記文を2行表示します。

は、実現できません。

- 例外が起こらないように整数チェックを行う
- ２回分の例外補足をする(愚直に２回書くか、関数化するか）

のいずれかのようにコードを変更する必要があります。

```hava
public static void main(String[] args) {
	Integer num1;
	Integer num2;

    try {
    	num1 = toInteger(args[0]);
    	num2 = toInteger(args[1]);

        int sum = num1 + num2; // どちらかが整数でないなら、NullPointer
        System.out.println(args[0] + "と" + args[1] + "の和は" + sum + "です。");

    } catch (ArrayIndexOutOfBoundsException e) {
        /* 引数存在チェック */
        System.out.println("コマンドライン引数の数は２つにしてください。");
    } catch (NullPointerException e) {
    	// エラーメッセージは表示済みなので何もしない
    }

}

private static Integer toInteger(String a) {
	try {
		return Integer.parseInt(a);
	} catch (NumberFormatException nfe) {
		System.out.println("整数を入力してください：" + a);
		return null;
	}
}
```

Answer

うん?
普通に例外が飛んできた時の、NumberFormatException をcatchしているところに書けばいいのでは。

もし例外だけど、取得して処理したいとかなら、メソッド化すればいいかと。

メソッドの書き方はご存知ですよね?
わからないなら調べましょう。

```Java
    // クラス内だとして
    
    static public int[] inputEx( String[] args ){
        int[] res = new int[2];
        try{
            // todo: ここで整数に変換
        }catch( NumberFormatException e ){
            // todo: ここで標準入力によって取得する
        }
    return res;
    }
```

とかみたいにして、これで取得する。とか。

できればstaticじゃない方がいいのかもしれませんが。

Answer

- argsの配列のサイズが２未満だった場合。＞エラーを表示して終了
- args[0]が数値表現かをチェック＞偽の場合エラー表示
- args[1]が数値表現かをチェック＞偽の場合エラー表示
- 上記いずれかがエラーの場合、そのまま終了
- なにか本番処理？

こういうコードで行きましょう

関連した質問