合計、平均の出しかた

Question

class Art{
	public static void main(String[] args){

		//argを初期化
		String arg = "";
		//sumを初期化
		int sum = 0;

		for(int i = 0; i < args.length; i++){
			arg = args[i];
		}

		int num = Integer.parseInt(arg);

		double heikin = 0.0;

		//平均を算出
		heikin = sum/num;

		System.out.println("平均は「" + heikin + "」です");
		System.out.println("合計は「" + num + "」です");


	}
}

コマンドライン引数から受け取った値を合計をして平均をだすプログラムを作ったつもりなのですが、合計のところで何と何を合計したらいいのか、など、分からなくなってしまいました。
どなたか教えていただけませんか？

↓訂正をして、表示が出来るようになりました。
ですが、まだ答えが違う答えになります。

class Art02{
	public static void main(String[] args){

		//ループ中に取り出した文字列を、数値に変換して処理するための変数
		int sum = 0;
		//数値の合計入れるための変数
		int sum2 = 0;

		for(int i = 0; i < args.length; i++){

			//lengthから受け取ったargをint型に変換

			sum2 = sum + (Integer.parseInt(args[i]));
		}





		//数値の合計と配列数から求めた平均値を保持するための変

		//平均を算出
		double average=(double)sum/args.length;

		System.out.println("平均は「" + average + "」です");
		System.out.println("合計は「" + sum2 + "」です");


	}
}

Answer 1

質問文にコードをベースに書いてみました。

class Art02 {
    public static void main(String[] args) {
        if (args.length == 0) {
            return;
        }

        int sum = 0;
        for (int i = 0; i < args.length; i++) {
            // 受け取ったargsをintに変換してから、足し算していく
            sum += (Integer.parseInt(args[i]));
        }

        System.out.println("平均は「" + (double) sum / args.length + "」です");
        System.out.println("合計は「" + sum + "」です");
    }
}

Comments

[Tutti]

回答していただきありがとうございます！
あの、もしよろしければ、なぜ if 文で 0 のときは値を返すようにしているのかおしえていただけませんか？

[katoy]

0 で割り算をするとExceptionが発生してしまいます。
入力データ数が 0 個だったときにどうするかが質問文からは不明だったので、
ここでは、なにもしないでプログラムを終了 (return )しています。
値は返していません。main() から return するのでプログラムが終了するだけです。

[katoy]

本当はエラー処理はいろいろ検討をする必要があります。
入力した文字列が数字でなかった場合とか、足し算しているあいだに整数で表せる値の範囲を超えてしまった場合とか, ....

[Tutti]

なるほど！確かに0のときはエラーが発生しますからその処理を行わなければなりませんね。そのことに気がつきませんでした。
このような場合、returnで返してあげればエラー対処しなくても、いいのですか？
エラーを受け取ったほうがよいのでしょうか？

[Tutti]

すみません。問題文の記入が抜けていました。
これの問題文は、コマンドライン引数で複数の値を受け取り、合計して、その平均を出しなさいという問題です。
なので、今回は0を受け取ったときのエラーの対処は書かなくても大丈夫でした。

[Tutti]

それと、今回の問題、おかげさまで解くことが出来ました！！回答を参考にさせて頂きました。本当にありがとうございます！！助かりました。
このとき方を忘れないようにします！

Answer 2

変数名を簡略化しすぎで逆に何をする変数か分からなくなった、って感じですかね？

変数を宣言しているところで、「argを初期化」といったコメントを書いていますが、
ここは、変数の役割をコメントとして書いたほうが良いと思います。
argを初期化していることぐらい、コード見れば分かりますからね。

今回の場合なら、

//ループ中に、配列から文字列を1つ取り出して処理するための変数
String arg;

//ループ中に取り出した文字列を、数値に変換して処理するための変数
int num;

//数値の合計を行うための変数
int sum = 0;

//数値の合計と配列数から求めた平均値を保持するための変数
double heikin；

みたいな感じですかね？

Comments

[Tutti]

詳しく書いていただき、ありがとうございます！今後から、コメントの書き方はこのようにして書きます。

[abs123]

他人からみて、分かりやすいコメントというのを、早いうちから意識すると後が楽です。
分かりやすいコメント書いていると、徹夜明け等で倒れた後とかに、
いったい何を書いていたんだっけ？となることを防ぐことが出来ます。

[Tutti]

そんなことがあるんですね。。。教えていただきありがとうございます！気をつけます。

Answer 3

平均は合計/個数
で個数はargsの長さと同じ

int num=0;
//合計を計算
for(int i=0;i<args.length;i++){
  num+= Integer.parseInt(args[i]);
}
//平均を計算
double average=(double)num/args.length;

Java8ならラムダでもっときれいにかけそう・・
驚くべきことに一行でかけました=ラムダの出番はなかった

System.out.println(Stream.of(args).mapToInt(Integer::parseInt).average().getAsDouble());

###追記
0+args[i]を代入してますよ・・

Comments

[Tutti]

この処理を一文で書くことが出来るんですね！そのことに驚きです。。。
もっと沢山練習問題などを解いて、理解できるようにします。

Answer 4

こんな感じですか？

public class Art {

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ
        //sumを初期化
        int sum = 0;

        for(int i = 0; i < args.length; i++){
            sum = sum +Integer.parseInt(args[i]);
        }

        double heikin = 0.0;

        //平均を算出
        heikin = sum/args.length;

        System.out.println("平均は「" + heikin + "」です");
        System.out.println("合計は「" + sum + "」です");
	}

}

Comments

[tignear]

double(float)=int/intとすると答えの小数点以下が切り捨てられます

[Tutti]

それは困りますね！
小数点を残すにはdoubbleに直す必要がありますか？

[tignear]

どちらかの値をキャストしておけば大丈夫です
double heikin=(double)sum/args.length;
みたいな感じです

[Tutti]

ありがとうございます！！！！！

Answer 5

コマンドライン引数で受け取った値を合計するんじゃないですか？

Comments

[Tutti]

そうなんですが、、、
コマンドライン引数で受け取った値はnumに入れているので、そこでも悩んでいます。

[Zuishin]

ループ終わってから入れてもしょうがないと思います。

[Tutti]

おっしゃると通りです。

[Zuishin]

先生はそこを自分で考えて欲しかったんだと思います。答えが必要なら学生に頼まず自分で出せるでしょうから。