質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

91.01%

  • Java

    12132questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • 配列

    453questions

    配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

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

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 272

frdhy

score 3

配列の値の平均を計算するメソッドを持つプログラム(サイコロを3回ふり、でた目の和を10000回分、配列に保存したもの)を作りたいのですが、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);
    }

}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

checkベストアンサー

+3

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

  • クラス名は大文字で始めることが推奨されます。そのため、下記ではクラス名を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);
    }
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/01 21:44

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

    キャンセル

+1

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

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

あるいは、

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/01 21:50

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

    キャンセル

+1

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);
    }
}


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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/01 21:25

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

    キャンセル

  • 2017/10/01 21:47

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

    キャンセル

+1

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

じゃなくてですか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/01 21:46

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 91.01%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Java

    12132questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • 配列

    453questions

    配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。