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

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

ただいまの
回答率

88.93%

偏差値を正しく求めたい。

解決済

回答 5

投稿 編集

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

前提・実現したいこと

コマンドライン引数から成績を処理するコードをつくっているのですが、標準偏差がうまく求められません。本来ならsd = 18.85となるはずなのですが…
解決方法を教えてください。お願いします。
ちなみにコマンドライン引数は以下の通りです。
「Anne 100 Glbert 99 Diana 60 Emily 89 Bryce 79 Jane 74 Kilmeny 42 Marigold 64」

該当のソースコード

package hello;

public class Seiseki2 {

    public static void main(String[] args) {

        int i = 0;
        double sum = 0;
        double average = 0;
        double sd = 0;

        String names[] = new String[args.length/2];
        double score[][] = new double[2][args.length/2];

        for(i = 0; i < args.length; i += 2 ) {
            names[i/2] = args[i];
            score[0][i/2] = Double.parseDouble(args[i + 1]); 
            sum += score[0][i/2];

        }for(i = 0; i < score[0].length; i ++ ) {
            average = sum / 8.0;
            double ssum = 0;
            ssum += sqr(score[0][i/2] - average);
            sd = Math.sqrt(ssum / (args.length/2));
            double hensachi = 50 + ((score[0][i/2] - average)/sd )* 10;
            score[1][i] = hensachi; 

        } System.out.println("Average = " + average);
          System.out.println("Standard Deviation = " + sd);
        for(i = 0; i < 16; i += 2 ) {

            double scoreX = score[0][i/2];
            String s = new java.text.DecimalFormat("0.#").format(scoreX );
            double scoreY = score[1][i/2];
            String ss = new java.text.DecimalFormat("0.#").format(scoreY);

            System.out.print("    " + names[i/2] + "    ");
            System.out.print(s + "    ");
            System.out.println(ss);

        }

    }

    private static double sqr(double d) {
        return d * d;
    }


}

結果は以下の通りになります。
Average = 75.875
Standard Deviation = 4.640388251536718
Anne    100    78.3
Glbert    99    78.3
Diana    60    78.3
Emily    89    78.3
Bryce    79    21.7
Jane    74    21.7
Kilmeny    42    78.3
Marigold    64    78.3

試したこと

18行目から21行目の式を変えてみたりしましたが、うまくいきません。

補足情報

最終的な形は以下の通りにしたいと思っています。

Average = 75.9
Standard Deviation = 18.85
Anne    100    62.8
Glbert    99    62.3
Diana    60    41.6
Emily    89    57.0
Bryce    79    51.7
Jane    74    49.0
Kilmeny    42    32.0
Marigold    64    43.7

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • CarpeDiemnosiru

    2020/07/09 10:54

    今電卓で計算してみたのですが、18.85099…となりました。

    キャンセル

  • asahina1979

    2020/07/09 11:06

    すまん、確かにまともに計算したら 18.85 (四捨五入)だった

    とりあえずインデントを合わせましょう

    キャンセル

  • CarpeDiemnosiru

    2020/07/09 11:25

    インデント合わせました、ありがとうございます。

    キャンセル

回答 5

+2

ここで使っているiは?

        for (i = 0; i < score[0].length; i++) {
            double average = sum / 8.0;
            double ssum = 0;
            ssum += sqr(score[0][i] - average);
            double sd = Math.sqrt(ssum / 8);

            for (i = 0; i < score[0].length; i++) { // ←ここ
                double hensachi = 50 + ((score[0][i] - average) / sd) * 10;
                score[1][i] = hensachi;
            }
            System.out.println("Average = " + average);
            System.out.println("Standard Deviation = " + sd);

            for (i = 0; i < 16; i += 2) {// ←ここ

                double scoreX = score[0][i];
                String s = new java.text.DecimalFormat("0.#").format(scoreX);
                double scoreY = score[1][i];
                String ss = new java.text.DecimalFormat("0.#").format(scoreY);

                System.out.print("    " + names[i] + "    ");
                System.out.print(s + "    ");
                System.out.println(ss);
            }

        }


インデントちゃんとしないと・・・

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/07/09 11:23

    インデント揃えました。ありがとうございます。「i」の使い方が間違っているということでしょうか?

    キャンセル

  • 2020/07/09 11:32

    for(i = 0; i < score[0].length; i ++ )のループの中で
    for(i = 0; i < score[0].length; i++)とかfor(i = 0; i < 16; i += 2 )での最初期化?

    キャンセル

  • 2020/07/09 12:10

    二重ループがあるということですね、直します。

    キャンセル

  • 2020/07/09 14:13

    何かごっちゃにしてませんか?
    合計を算出するのと、平均を算出するのは別のことです。
    ・・・機能毎に、分けてみましょう・・・

    キャンセル

checkベストアンサー

+1

ozwkさんの回答にもありますが、配列を2倍、用意しているので、期待と異なる結果となります

double score[][] = new double[2][args.length];

for(i = 0; i < args.length; i += 2 ) {
        score[0][i] = Double.parseDouble(args[i + 1]); 
} 

// ↑ のように格納した場合 scoreは以下のようになる。
// score[0] は、{100,0,99,0,60,0,89,0,79,0,74,0,42,0,64,0}

// 以下のように全要素でssumを求めると余計な0が含まれるので、期待と異なる結果になる
for(i = 0; i < score[0].length; i ++ ) {
        double ssum = 0;
        ssum += sqr(score[0][i] - average);
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

インデントが汚いです。直しましょう。
インデントをきちんと整理してみると2重ループ構造になってしまっているのがわかると思います。


多少無駄にループが回ろうがわかりやすさ優先で書きましょう:

int main(String[] args){
    // コマンドライン引数を読み込むためのループ

    // 平均値を計算するためのループ

    // 標準偏差を計算するためのループ

    // 偏差値を計算するためのループ

    // 表示するためのループ
}

scorenameを人数の倍用意して1つ置きにデータを入れてますね?
コマンドライン引数が(名前,点数)の組であるためにi+=2とした都合できっと工夫したのでしょうが、
その後の展開が非常にわかりづらくなる=ミスりやすいので1つ置きに入れるのはやめましょう。
(実際、標準偏差を求めるときにミスってます)
namescorei/2で代入すれいいです:

String names[] = new String[args.length/2];
double score[][] = new double[2][args.length/2];

for(i = 0; i < args.length; i += 2 ) {
    names[i/2] = args[i];
    score[0][i/2] = Double.parseDouble(args[i + 1]); 
    sum += score[0][i/2];
}

ループの外で使わないのにループの変数iをループ外で宣言するのはヤメましょう。
なんか使うのかな?と読むとき不安になります。

- int i = 0;
- for(i = 0; ...)
+ for(int i = 0; ...)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/07/09 12:57

    ループとループの間でやって下さい

    キャンセル

  • 2020/07/09 14:16

    何度もすみません、ループの間でできました!
    すべてi/2に変えてみたのですが、また違う値が出てしまいます。

    キャンセル

  • 2020/07/09 15:42

    その話はコマンドライン引数からnameとscore作るとき だけ の話です

    キャンセル

0

コード中に 8 で割るところがありますが、そういう固定値ではないはずです。
args.length / 2 の値にしないといけないでしょう。

点数と偏差値を 2次元配列の [0] と [1] で区別しているから面倒になっています。
次のようにしましょう。

    public static void main(String[] args) {

        int n = args.length / 2;
        String names[] = new String[n];
        double score[] = new double[n];
        double deval[] = new double[n];

        double sum = 0;
        for (int i = 0, j = 0; i < n; i++) {
            names[i] = args[j++];
            score[i] = Double.parseDouble(args[j++]); 
            sum += score[i];
        }

        // ここで、平均、標準偏差、偏差値を計算

        System.out.printf("Average = %.1f\n", average);
        System.out.printf("Standard Deviation = %.2f\n", sd);   

        for (int i = 0; i < n; i++)
            System.out.printf("%-9s %3.0f %6.1f\n",
                names[i], score[i], deval[i]);
    }


追記
もう解決済みですか?
質問のコードをどんどん修正すると、回答の指摘が何のことかわからなくなるので
修正コードは追記されるほうが良いと思います。

現在のコードにも average = sum / 8.0; のように 8 が残っています。
また、コードの中に /2 がたくさんあります。
最初に int n = args.length / 2; として、その n を使うとコードが見やすくなります。
また、小数点以下の文字数を指定するのには printf を使うと簡単です。

package hello;

public class Seiseki2 {

    public static void main(String[] args) {

        int n = args.length / 2;
        String names[] = new String[n];
        double score[][] = new double[2][n];

        double sum = 0, j = 0;
        for (int i = 0; i < n; i++) {
            names[i] = args[j++];
            score[0][i] = Double.parseDouble(args[j++]); 
            sum += score[0][i];
        }
        double average = sum / n;

        double ssum = 0;
        for (int i = 0; i < n; i++)
            ssum += sqr(score[0][i] - average);

        double sd = Math.sqrt(ssum / n);

        for (int i = 0; i < n; i++)
            score[1][i] = 50 + 10 * (score[0][i] - average) / sd;

        System.out.printf("Average = %.1f\n", average);
        System.out.printf("Standard Deviation = %.2f\n", sd);   

        for (int i = 0; i < n; i++)
            System.out.printf("%-9s %3.0f %6.1f\n",
                names[i], score[0][i], score[1][i]);
    }

    private static double sqr(double d) { return d * d; }
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/07/09 14:09

    args.length/2、訂正しました。
    テキストの方で点数と偏差値は二次元配列に格納しろとの指示がありまして。

    キャンセル

0

とりあえず。サンプル
ただ、 引数からではなく標準入力からの登録に変えてあるので イメージとして理解してください。

import java.util.*;

public class Main {
    public static void main(String[] args) throws Exception {
        try (Scanner s = new Scanner(System.in)) {
            int n = s.nextInt();

            String[] a = new String[n];
            double[] b = new double[n];
            double   c = 0D;
            double   d = 0D;
            double[] e = new double[n];
            double[] f = new double[n];
            double   g = 0D;
            double   h = 0D;
            double   i = 0D;

            for (int z = 0; z < n; z++) {
                a[z]  = s.next();
                b[z]  = s.nextInt();
                c    += b[z];
            }
            d = c / n;
            for (int z = 0; z < n; z++) {
                e[z] = b[z] - d;
                f[z] = Math.pow(e[z], 2);
                g += f[z];
            }
            h = g / n ;
            i = Math.sqrt(h) ;

            System.out.printf("%-20s:%4.0f%n", "Total", c);
            System.out.printf("%-20s:%7.2f%n", "Average", d);
            System.out.printf("%-20s:%7.2f%n", "Standard Div", i);

            for (int z = 0; z < n; z++) {
                System.out.printf("%-20s:%4.0f ( %4.1f )%n", a[z], b[z], 50 + ((b[z] - d) / i ) * 10);
            }
        }
    }
}

結果

Total               : 607
Average             :  75.88
Standard Div        :  18.85
Anne                : 100 ( 62.8 )
Glbert              :  99 ( 62.3 )
Diana               :  60 ( 41.6 )
Emily               :  89 ( 57.0 )
Bryce               :  79 ( 51.7 )
Jane                :  74 ( 49.0 )
Kilmeny             :  42 ( 32.0 )
Marigold            :  64 ( 43.7 )

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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