コマンド引数が整数でなかった場合に、エラーメッセージに該当引数を表示したい

Question

Javaファイルをcmdで実行させます。
引数を2つ用意し、そのどちらかでも整数でない場合、
エラーメッセージとして「整数を入力してください：a」のように、「a」の部分に該当引数を表示させます。
引数は2つ渡しているとして、どちらか1つだけ整数でない場合は、上記のように1文でよいですが、2つとも整数でない場合は、上記文を2行表示します。
下記コードのようにtry-catchで例外に飛ばした時の、該当引数の取得の仕方を教えてください。
またはほかの方法でないと出来ないのであれば、その方法を教えてください。
よろしくお願いいたします。

Kadai1.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);
		}
	}
}

Comments

[y_waiwai]

提示のコードではどういう不具合があるんでしょうか

[退会済みユーザー]

不具合はありません。
説明に書いた仕様を実現したいための質問です

[y_waiwai]

説明に書いてある仕様を実現できていないところがあるんでしょ？
そこはどこでしょうか。

[dodox86]

思いっきり"Kadai1"(課題1)クラスとなっているので、回答に躊躇しますね。

> try-catchで例外に飛ばした時の、該当引数の取得の仕方を教えてください。
> またはほかの方法でないと出来ないのであれば、その方法を教えてください。

力技でやれば引数毎にtry～catchすればできるでしょうし、そうではなく、引数分ループして処理すれば自然に2行になるのではないですか。

ちなみに今のコードでは、3つ以上引数をセットするとOKになってしまいますが、これは良いのでしょうか。細かいことを書くと、"コマンドライン引数の数は２つにしてください。"と言うメッセージに合致しません。

[退会済みユーザー]

確かに課題クラスと書いてあったら解答も難しいかもしれないですね。
力技の方法がなんとなく流れが思い浮かべたので、まずそれで作ってみようと思います。
コメントありがとうございます。

また、3つ以上の引数に関しましては仕様の確認漏れと私の単純なミスですので修正します。
ご指摘ありがとうございます。

[dodox86]

力技でもループでも、args[n]の添え字(n)が変数であれば、何番目の引数でNGだったかも分かるはずです。（複雑に考える必要はなくて、プログラムの組み方の問題です）

[退会済みユーザー]

何度もすみません。。
変数nを用いると、例えば1つ目の引数でNGだった場合、2つ目の引数のチェックが通らないかと思うのですが。。。
実は今、力技の方法を書いていて気が付いたのですが、
try1つ目の引数チェック→finally→try2つ目の引数チェック
としたとき、1つ目がNGで2つ目がOKの場合の処理がうまくいかないことに気づき止まっておりました。

両方のチェックが通り（引数は2つのみ渡したと仮定します）、2つともOKだった場合のみ処理続行という方法がどうしても分かりません。
整数チェックはNumberFormatExceptionでという条件があるのでtry-catchしか方法はないかと思っているのですが、方法はあるのでしょうか？
それとも複雑に考えすぎなのでしょうか。。。

[dodox86]

> 両方のチェックが通り（引数は2つのみ渡したと仮定します）、2つともOKだった場合のみ処理続行という方法がどうしても分かりません。

こういうのはプログラムの組み方で様々な方法が取れます。例えばチェックOKの回数を保存する変数を用意し、全てのチェック後、それが2回を
示していたら処理続行であるとか、1回でもNGを検出するのでよければ、NGを示すフラグ変数を使うとか。ループやtry～catchの書き方、プログラムの構造如何で、そのような変数すら必要でなくなるかもしれません。既に回答もいただいているので、そちらも参照されるのが良いと思います。

[退会済みユーザー]

ご丁寧にアドバイス有難うございます。
チェック回数の変数は思いつきませんでした。
確かにもっと柔軟に考えればいろいろな方法が取れそうですね。
もう少し考えてみます。
有難うございました。

Answer 1

エラーチェックと本体処理を分離したほうが見やすくありませんか？

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 + "です。");
    }
}

追記
ビット演算になじみがなければ、

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 2

１つのtry～catchでは１つの例外しか補足できません。
したがって

2つとも整数でない場合は、上記文を2行表示します。

は、実現できません。

• 例外が起こらないように整数チェックを行う
• ２回分の例外補足をする(愚直に２回書くか、関数化するか）

のいずれかのようにコードを変更する必要があります。

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 3

dodox86さんのアドバイスを元に下記コード作成、エラー無く実行しました。

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 4

うん?
普通に例外が飛んできた時の、NumberFormatException をcatchしているところに書けばいいのでは。

もし例外だけど、取得して処理したいとかなら、メソッド化すればいいかと。

メソッドの書き方はご存知ですよね?
わからないなら調べましょう。

    // クラス内だとして
    
    static public int[] inputEx( String[] args ){
        int[] res = new int[2];
        try{
            // todo: ここで整数に変換
        }catch( NumberFormatException e ){
            // todo: ここで標準入力によって取得する
        }
    return res;
    }

とかみたいにして、これで取得する。とか。

できればstaticじゃない方がいいのかもしれませんが。

Answer 5

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

こういうコードで行きましょう