while 構文について

Question

以下のソースコードで「100　200　end　300 400」とコマンドラインを打ち込み実行すると処理結果が
「コマンドラインで指定された引数は100　200
合計は300です。」となるようにソースコードをかきたいのですが
「コマンドラインで指定された引数は
合計は0です。」
となってしまいます。自分の考えでは合計値300を算出した後にループの終了条件に到達して
終了すると思って組んだのですが上記の結果にならない理由について教えていただきたいです。
また理由とともに解決例も出していただけると幸いです。

(追記)whileループから脱出するための条件として"end"との比較が必要です。

	int sum = 0;
	if(args.length==0){
		System.out.println("コマンドラインで、整数値を一つ以上引き渡してください。");
	}

	System.out.print("コマンドラインで指定された引数は");

	
	while(args.equals("end")) {
		for(int i=0;i<args.length;i++){
			sum = sum+(Integer.parseInt(args[i]));
			System.out.print((Integer.parseInt(args[i]))+" ");
		}
	}
	
	
	System.out.println("");
	System.out.println("合計は"+ sum +"です。");

Comments

[Zuishin]

おそらく「コマンドライン」という言葉の意味が間違っているために回答者に通じていません。

[Zuishin]

コマンドラインで指定するのであれば、end は不要です。

Answer 1

while(args.equals("end")) {

これのせいです。ループ内に入れません。

追記

while(args.equals("end")) {
for(int i=0;i<args.length;i++){
sum = sum+(Integer.parseInt(args[i]));
System.out.print((Integer.parseInt(args[i]))+" ");
}
}

これを次のように。

for(int i=0;i<args.length;i++){
if (args[i].equals("end")) break;
sum = sum+(Integer.parseInt(args[i]));
System.out.print((Integer.parseInt(args[i]))+" ");
}

追記

for を while に変えるのは簡単です。

for (int i = 0; i < 10; i++) {
処理
}

↓

int i = 0;
while (i < 10) {
処理
i++;
}

Comments

[Zuishin]

多分、提出しても不正解。

[momon-ga]

もう１回、合計がゼロのままですって質問が出る気がする。

[Zuishin]

絶対標準入力だと思うんですよね。コマンドライン引数だったら while を使う必要無いし、カウンタ使うから for の方がいいし、実行した後であのメッセージが出るのも不自然。
あと入力例に java コマンドや end が無いのも腑に落ちない。
私が思うに、end が入力されるまで空白区切りの入力を処理し続けるのが課題じゃないかと。

[momon-ga]

コマンドラインって、そうゆうことですか。
なるほど。たしかに、それならwhile使う意味がありますね。
でも、argsなんて名前にするかな・・・

[Zuishin]

args は質問者が「コマンドライン」でググって勝手にこれを使うと思ったんじゃないかと思います。

[kaputaros]

> Zuishinさん、momon-gaさん
コマンドライン引数、どこ調べても
「mainメソッドに渡す引数」ってありますよ？
ただ、for文とwhile文の書き換え をテーマにした講義だったかもしれません。
決めつけはよくないと思いますよ。
そこをドウコウ言ってもしかたないですし、ここで議論することではないと思いますが。

[Zuishin]

コマンドライン引数はその通りですが、コマンドライン引数とは質問のどこにも書いていませんね。

[Zuishin]

いや、書いてるか。まあこれで質問者が満足してるんだからいいんですが、たぶん質問を読み違ってるんじゃないかなって話です。確認しようにも返事をしないからどうしようもない。

Answer 2

たぶん、これじゃないと思うけど。
whileループから脱出するための条件として"end"との比較をする例

boolean TДT = true;
http://teratail.com/questions/263544
while(TДT) {
	for(int i=0;i<args.length;i++){
		if(args[i].equals("end")) break http;
		sum = sum+(Integer.parseInt(args[i]));
		System.out.print((Integer.parseInt(args[i]))+" ");
	}
}

無限ループには注意が必要。

Comments

[Zuishin]

http:// にそんな使い方が。

[momon-ga]

なんと、https://でも大丈夫でーーす。

Answer 3

while(args.equals("end"))は不要かと思います。

for(int i=0;i<args.length;i++)でargs全体をループするようになっていますので、さらに外側からwhileをかける必要がありません。

そして、args.equals("end")は絶対に成立しない条件なので、whileの中身は一度も実行されないことになります。

Comments

[maisumakun]

（追記）

forの中でbreakさせるのが手っ取り早いです（どうしてもwhileで書かなければいけない事情はあるのでしょうか）。

[Wisteria315]

コードにwhile文を用いる指定がされています。

[maisumakun]

forは機械的にwhileに書き直せます。

[momon-ga]

コードにwhile文を用いる指定がされています。は、わかりましたが、
String[]と、Stringを比較するので、絶対にtrueにならないのは、わかりますよね？
whleにせよ、forにせよ配列の要素にアクセスすれば解決では？

Answer 4

whileの条件文というのは、成立しているときにナカミを実行するもの、です。
その条件文、成立してるんでしょうか