文字列の数値かそれ以外かの判断

Question

いつもお世話になっております。

受け取ったString文字列が数値かそうでないか、を判断するコードを作る課題があったので、自分なりに作りました。
こちらです↓

/*
コマンドライン引数より取得した文字列が
数字であれば「true」を、
文字であれば「false」を返すプログラムを作成せよ。
*/

package no211_Test_22;

public class Test14 {
	public static void main(String[] args) {
		String str = args[0];
		int i = 0;
		int len = str.length();
		char ch = str.charAt(i);
		while (i < len){
			boolean result = Character.isDigit(ch);
			if(i == len){
				System.out.println(true);
			} else {
				System.out.println(result);
				break;
			}
			i++;
		}
	}
}

Character.isDigit()ではString文字列を扱えないので
charAt(i)を用いてループを回し１つずつ確認させる、というやり方です。

先生に提出したところ、
処理が面倒くさいのでもっと簡単なのにしろ、と言われたのですが、
どうしても上手くまとめる方法が思いつきません。

どなたか知恵をお貸し下さい。
よろしくお願いします。

↓以下修正後

public class Test14 {
	public static void main(String[] args) {
		try {
			int number = Integer.parseInt(args[0]);

			System.out.println(true);

		} catch (NumberFormatException e) {
			System.out.println(false);
		}
	}
}

こんな感じになりました。
とても短くて見やすいと実感。
いかに自分がダメな事をしていたか。。。

ループの回し方にも問題がたくさんありましたが、
これらの点にもご指摘頂いた皆様、ありがとうございました。

一応これで解決済みにしたいと思います。
何か問題があれば解決済みですがコメント頂けるとありがたいです。

PS
numberという変数を出すだけだして使えていないのですが、これはしょうがないのでしょうか？
うまく纏める術を伝授して頂けると幸いです。

Answer 1

修正後のコードですが，数字を使うことが目的のコードではないので，numberを用意する必要が無いですね．
parseIntはint型の数値を返しますが，それを収める変数が無いだけで問題ありません．
もちろん変数を用意していても何ら問題ありません(Eclipseを使うと「変数numberの値は使われていません」と怒られますが)．

Comments

[R.lawliet]

なるほど、書かなくてよろしいのですね！
スッキリしました。
ありがとうございます。

Answer 2

        while (i < len){
            boolean result = Character.isDigit(ch);
            if(i == len){
                System.out.println(true);
            } else {
                System.out.println(result);
                break;
            }
            i++;
        }

これだと今度は1文字目しかチェックできないのでは・・・・？
whileでi<lenという条件なので，iを操作せずに直後にあるi==lenという条件は絶対に成立しません．

Comments

[R.lawliet]

言われてみればそうですね、うぅん、頭の悪い自分がもどかしい。
i+1 == len
サイレント修正で条件成立かな？と思うのですがこの場合の問題点はありますか？
while (i < len){
boolean result = Character.isDigit(ch);
if( i+1 == len){
System.out.println(true);
} else {
System.out.println(result);
break;
}
i++;
}

[swordone]

1文字目でbreakされて抜けてしまうので話になりません．
kutsulogさんのアルゴリズムをもう一度よく見て考えてください

[R.lawliet]

回答ありがとう御座います。
for eachを使ってなかったですね、もうちょっと練りなおしてみます。

Answer 3

このコード，本当はif文のtrueにはresultが入るべきなのでは？
たぶん自分の中で(resultがtrueなら・・・)ってお考えかと思いますが，
この書き方ではhtsignさんの言うとおり，最後のelse文に入り得ません(いわゆるデッドコード)．

あなたの考え方に沿うなら，kutsulogさんの手法が一番自然ですが，
個人的にはipipip0129さんの手法を取りたいですね．
Integer#parseInt(String)は，引数が数値変換できなければNumberFormatExceptionを投げるので，
それをcatchした時falseとすれば要件は満たせます．

Comments

[R.lawliet]

デッドコード確認致しました。
質問のコードを上書きしてみたので再度見て頂けるとありがたいです。

Integer#parseInt(String)の意見が多いので取り扱ってみようと思います。
ご丁寧にありがとうございます。

Answer 4

受け取ったString文字列をparseIntして例外処理をすればいいんじゃない?
参考になればいいなと思います。

Comments

[R.lawliet]

ループしか頭に無かった僕への打開策になりました。
感謝致します！

Answer 5

問題文とは関係ないですが、

if (true) {
} else {
}

では絶対に elseが実行されないと思いますがそれはいいのですか？
&&演算子で true をつなげる意味も分かりません。

Comments

[R.lawliet]

まだまだ知識が足りないのでまともな人から見ると滅茶苦茶なコードになっている事と思います（汗）
１つずつ無駄なモノを省いて行こうと思います。

ちなみに、変なモノが一杯くっついているみたいですが、
現在の確認ではelseは動いております。
ご指摘ありがとうございます。

[htsign]

そうなのですか、無粋なコメントだったようですね。
すみませんでした。

[R.lawliet]

いえ、最終的には全部まともにしたいので質問とは別の問題点でもご指摘頂けるのはありがたいです。

Answer 6

学校の課題のようなので考え方だけ書きます
1.for文を使って添え字を0から最後までループ(for-each形式を使ってもいいです)
2.ループの中ではisDigitがfalseの場合だけreturnして、trueを返すのはループを抜けた後
この辺が先生の言いたいところだと思います

(先生の想定を超えたいなら難易度が上がりますが「正規表現」を使う方法もあります。)

Comments

[R.lawliet]

ご配慮ありがとうございます。
答えを教えてもらうのは卑怯だと思うので、ありがたいです。

最初の質問の時に書けば良かったのですが、
一文字ずつ見る方法ではないモノを使ってみてください、と言われました。
なのでkutsulogさんの提示して頂いた考え方は僕のコードの修正にはぴったりですが、
求められているものはちょっと違うのかもしれません。。。

正規表現、調べてみます！