配列のプログラムについて

Question

配列を使って最大値・最小値・平均値を求めるプログラムを作りたいのですが、うまく出力結果が出ません。(コンパイルは成功)

import java.util.Scanner;
public class MinMaxAve{
    public static void main(String[] args){
	System.out.println("数字を入力してください");
	Scanner kbd=new Scanner(System.in);
	int[] data=new int[1000];
	int min,max;//(左から)最小値・最大値
	double ave;//平均値
	int n=0;
	while(kbd.hasNext()){//hasNext()はリストに要素がなくなるまでtrueを返す
	    data[n]=kbd.nextInt();
	    n++;
	}
	if(n==0){
	    System.out.println("データがありません");
	}else{
	    min=data[0];
	    for(int i=1;i<n;i++){
		if(data[i]<min){
		    min=data[i];
		}
		System.out.printf("min=%d\n",min);
	    }
	}
	if(n==0){
	    System.out.println("データがありません");
	}else{
	    max=data[0];
	    for(int x=1;x>n;x++){
		if(data[x]>max){
		    max=data[x];
		}
		System.out.printf("max=%d\n",max);
	    }
	}
	if(n==0){
	    System.out.println("データがありません");
	}else{
	    int sum=0;//入力した要素(数字)の合計
	    for(int y=0;y<data.length;y++){
		sum=data[y];
	    }
	    ave=sum/data.length;
	    System.out.printf("ave=%d\n",ave);
	}
    }
}

実行結果
数字を入力してください
10
20
15
5
min=10
min=10
min=5
ave=Exception in thread "main" java.util.IllegalFormatConversionException: d != java.lang.Double
at java.util.Formatter$FormatSpecifier.failConversion(Formatter.java:4302)
at java.util.Formatter$FormatSpecifier.printInteger(Formatter.java:2793)
at java.util.Formatter$FormatSpecifier.print(Formatter.java:2747)
at java.util.Formatter.format(Formatter.java:2520)
at java.io.PrintStream.format(PrintStream.java:970)
at java.io.PrintStream.printf(PrintStream.java:871)
at MinMaxAve.main(MinMaxAve.java:46)

もう最後の方はよく分からず、if文を3回繰り返してしまいました。

どこをどう直せば良いのでしょうか。
よろしくお願いします。

Answer 1

ちなみに今はこんなことができたり。

import java.util.IntSummaryStatistics;
import java.util.Scanner;

public class MinMaxAve {

	public static void main(String[] args) {
		System.out.println("数字を入力してください");
		Scanner sc = new Scanner(System.in);
		IntSummaryStatistics statistics = new IntSummaryStatistics();
		while (sc.hasNextInt()) {
			statistics.accept(sc.nextInt());
		}
		if (statistics.getCount() == 0L) {
			System.out.println("データがありません");
			return;
		}
		System.out.println("max = " + statistics.getMax());
		System.out.println("min = " + statistics.getMin());
		System.out.println("ave = " + statistics.getAverage());
	}

}

Comments

[退会済みユーザー]

お答えいただきありがとうございます。
...初心者の私にはちょっと難しいワードがたくさんあってクラクラしちゃいます^^;

[swordone]

ワードは難しいですが、IntSummaryStatisticsが内部でやってることはLouiS0616さんのコードと大差ありません。

Answer 2

エラーをしっかり読みましょう。フォーマット変換が不正だと言っています。
IllegalFormatConversionException 難しい英語じゃないはずです。

問題の行はここです。

System.out.printf("ave=%d\n",ave);

書式指定子が違います。
そもそも、こう書けばよいと思うのですが。

System.out.println("ave=" + ave);

---

もう最後の方はよく分からず、if文を3回繰り返してしまいました。

難しく考えすぎなんじゃないですか？全部まとめて処理すればよいです。

if(n == 0){
    System.out.println("データがありません");
} 
else {
    int sum, min, max;
    double ave;
    sum = min = max = data[0];

    for(int i = 1; i < n; i++) {
        if(data[i] < min){
            min = data[i];
        }
        else if(data[i] > max) {
            max = data[i];
        }

        sum += data[i];
    }

    ave = 1.0 * sum / n;
    System.out.println("min=" + min);
    System.out.println("max=" + max);
    System.out.println("ave=" + ave);
}

動作確認はしていません。

Comments

[退会済みユーザー]

ご指摘ありがとうございます。
先ほどのバグ(？)は解消されたのですが、出力結果に数字が表示されなくなりました。(min=max=ave= と表示されます)
よろしければまた教えていただけないでしょうか。

[LouiS0616]

修正後のコードを載せてください。

[LouiS0616]

失礼しました、回答のコードに間違いがありました。
取り急ぎ修正します。... 修正しました。

[退会済みユーザー]

正しく出力されました！
お手数をおかけして申し訳ございませんでした...
本当にありがとうございました。

[swordone]

これsumにdata[0]が加算されていないのでは？
int sum = data[0];
のように初期化するべきでは。

[LouiS0616]

> swordone様
おっしゃる通りです。コードを修正しました。