配列とメソッドについての質問です。

Question

配列の値の平均を計算するメソッドを持つプログラム(サイコロを３回ふり、でた目の和を１００００回分、配列に保存したもの)を作りたいのですが、42行目の”total = total+t”のところで”二項演算子'+'のオペランド型が不正です”というエラーが出てしまいます。
”total”は"int"で"t"は"int[]"なので同じ式で表すことができないのはわかるのですが、それを一体どうすれば解決することができるのかわかりません。
どのように変えたらうまくいくのでしょうか。
よろしくお願いします。

import java.util.Random;
public class abc {
    public static void main(String[] args) {
        Random rand = new Random();
        int a = 0, b=0, c=0, n;
        int countTable[] = new int[19];
	int repeatMax =1000000;
        
        for(int i=0; i<repeatMax; i++){
	    a = rand.nextInt(6)+1;
	    b = rand.nextInt(6)+1;
	    c = rand.nextInt(6)+1;
	    countTable[a+b+c]++; 
        }
	
	averageCount(countTable);	
    }

    public static double averageCount(int t[]) {
	int total=0;
	double ave=0;
	total = total+t;
	ave = total/1000000;
	System.out.println("Average is"+ave);
    }
    
}

Answer 1

修正案としては下記のよう形ではいかがでしょうか。

• クラス名は大文字で始めることが推奨されます。そのため、下記ではクラス名をAbcとしています。
• averageCountメソッドはdoubleを返却するように定義されていますが値を返しません。そのため、下記ではvoidとしています。
• 配列の中の数を参照する場合は、配列の要素の数だけループして処理する等の対応が必要になります。

import java.util.Random;

public class Abc {

    public static void main(String[] args) {
        Random rand = new Random();
        int a = 0, b = 0, c = 0;
        int countTable[] = new int[19];
        int repeatMax = 1000000;

        for(int i = 0; i < repeatMax; i++){
            a = rand.nextInt(6) + 1 ;
            b = rand.nextInt(6) + 1;
            c = rand.nextInt(6) + 1;
            countTable[a+b+c]++; 
        }

        averageCount(countTable);    
    }

    public static void averageCount(int t[]) {
        int total = 0;
        double ave = 0;
        for (int i = 3; i < t.length; i++) {
            total += t[i] * i;
        }
        ave = (double)total / (double)1000000;
        System.out.println("Average is " + ave);
    }
}

Comments

[frdhy]

とてもわかりやすかったです！
ありがとうございましたm(__)m

Answer 2

for(i = 3; i < t.length; i++) {
 total += t[i];
}

じゃなくてですか？

Comments

[frdhy]

for文を使わなきゃいけなかったんですね...
ありがとうございます！！

Answer 3

import java.util.Random;
public class abc {

    static int repeatMax = 1000000;

    public static void main(String[] args) {
        Random rand = new Random();
        int a=0, b=0, c=0, n;
        int countTable[] = new int[repeatmax];

        for(int i=0; i<repeatMax; i++){
            a = rand.nextInt(6)+1;
            b = rand.nextInt(6)+1;
            c = rand.nextInt(6)+1;
            countTable[i] = a + b + c;
        }

        averageCount(countTable);    
    }

    public static double averageCount(int t[]) {
        int total=0;
        double ave=0;
        for (int i=0; i< repeatMax; i++){
          total += t[i];
        }
        ave = total/repeatMax;
        System.out.println("Average is"+ave);
    }
}

こんなもんでどうでしょうか。質問の意図とあっているか不安ですが。。。

Comments

[LouiS0616]

aveをそのように取ると値が切り捨てられてしまいますよ。
あと、countTableの要素数は19のままで良いかと思います。

[frdhy]

参考になりました！
ありがとうございます！！

Answer 4

for文を使って、tの要素を一個ずつ取り出して足せばいいです。

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

あるいは、

for(int elem: t) {
    total += elem;
}

Comments

[frdhy]

２つ方法があるんですね...
いまいち上のやりかたわからなかったんですけど、LouiS0616さんの説明で理解することができました！
ありがとうございます！！