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

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

ただいまの
回答率

90.49%

  • Java

    14055questions

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

配列の平均値を出すメソッドの作成 JAVA

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 11K+

R.lawliet

score 44

//* クラスの出席番号順にString型配列nameに名前が、
//     * int型配列scoreにテストの点数が格納されています。
//     *
//     * 出席番号1 大悟 74点
//     * 出席番号2 琴音 70点
//     * 出席番号3 勇輝 88点
//     *  ・
//     *
//     * このクラスの平均点を算出するメソッドを作成。


public class MethodAdd07 {
        public static void main(String[] args) {
         String[] name = {"大悟", "琴音","勇輝","葵","綾乃","和樹","凛","愛花","結愛","孝太郎"};

         int[] score = {74,70,88,82,96,64,80,61,72,79};

↓ここから自分
        ★double ave =average(74,70,88,82,96,64,80,61,72,79);★

        System.out.println("平均は" + ave + "です");

    }

    ★private static double average(int i, int j, int k, int l, int m, int n, int o, int p, int q, int r){★

        ★int ave = (i + j + k + l + m + n + o + p + q + r)/ 10 ;★

        return ave;
}
}


それぞれの★で囲んだ3行を上手く最適化したいのですが、
double ave =average(score[]);
これを用いて効率化する方法をどなたか教えて頂けませんか?

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

checkベストアンサー

+2

プログラマー歴1年でjavaをほとんどもうかいてませんが
なんとなくjavaを書きたくなったのでためしに最適化しました。
もっといい方法があるかもしれません。

public class Main {

    public static void main(String[] args)
    {
        String[] name = {"大悟", "琴音","勇輝","葵","綾乃","和樹","凛","愛花","結愛","孝太郎"};
        int[] score = {74,70,88,82,96,64,80,61,72,79};

        System.out.println("平均は" + average(score) + "です");
    }

    private static double average(int[] score)
    {
               
        double ave = 0.0;

        for(int i:score)
        {
            ave += i;
        }

        return ave / score.length;
    }
}
学校の課題だと思いますががんばってくださいね。

4/09
コメントでお二人に指摘していただいたところを改善してみました。
私も勉強になりました。
あと変数の名前をaveとしていましたが
よく考えたらsumのほうが適切でしたOTL

改善したコード
public class Main {

    public static void main(String[] args)
    {
        String[] name = {"大悟", "琴音","勇輝","葵","綾乃","和樹","凛","愛花","結愛","孝太郎"};
        int[] score = {74,70,88,82,96,64,80,61,72,79};

        System.out.println("平均は" + average(score) + "です");
    }

    private static double average(int... score)//修正箇所1
    {
            double sum = 0.0;

            for(int i:score)
            {
                sum += i;
            }

            return score.length != 0 ? sum / score.length : sum ;//修正箇所2

    }
}


投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/04/08 10:37

    正にこういう無駄の無いコードを求めていました!
    ありがとう御座います。
    初めて1週間の見習いです。

    もうしばらく書いていないとの事ですが、こんなにスラスラ出てくるのですね!
    はやくそのレベルに達したいものです。頑張ります!!

    キャンセル

  • 2015/04/08 22:19

    > ave / score.length

    score.length が 0 の場合の対処をしておいたほうが良いかもしれません。

    score = []
    average(score)
    として、よばれる可能性があるからです。(対象の学生が 0 人というケース)

    キャンセル

+1

基本的には先の方の回答でほぼ全てですが,より汎用的にするには,
    private static double average(int[] score)
この部分を,
    private static double average(int... score)
こう書く方法もあります.(可変長引数といいます)
こうすることで,引数にint配列だけでなく,
    average(74, 70, 88, 82, 96, 64, 80, 61, 72, 79)
直接intを列挙することもできます.
scoreは内部的に配列として扱われるので,内部の処理を変更する必要はありません.

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/04/09 17:30

    これは今後も大活躍しそうなコードですね!
    知識が増えました、ありがとうございます♪

    キャンセル

+1

標準APIで平均を求めるものがあるので追求するとそこに行き着いてしまいます。

int[] score = {74,70,88,82,96,64,80,61,72,79};
double result = IntStream.of(score).average().getAsDouble();
System.out.println(result);

Java8 のStream APIというもので、マルチコアCPUでの処理効率がよい点もポイントです。
しかし、こうなると自分でプログラムを考えるというよりもAPIを知っているかどうかという話になってしまって趣旨が違ってきてしまいますね。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/04/09 17:26

    知識として新たな視野が開けました。ありがとうございます♪

    キャンセル

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

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

関連した質問

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

  • Java

    14055questions

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