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

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

ただいまの
回答率

90.52%

  • C#

    7108questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Visual Studio

    1831questions

    Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

2つの配列の相関係数、ユークリッド距離の計算

解決済

回答 1

投稿 編集

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

meroy25

score 5

前提・実現したいこと

何度も質問してしまって申し訳ありません。
前回質問したプログラムのソースコードを、ご回答をもとに改良しました。
30人分の国語と英語の試験結果をもとに、各教科の平均点・最低点・最高点を計算し、さらに2つの教科の点数の相関係数、ユークリッド距離を求めるプログラムです。
各メソッドに計算式を当てはめることはできました。

発生している問題・エラーメッセージ

foreach文を使って、
Corrメソッド(相関係数)では相関係数を導くために必要な値を
Distメソッド(ユークリッド距離)では偏差平方和を求めたいと思っています。
そのためには、入力した2つの配列(要素数30)を読み込まなければなりません。
しかし、それぞれforeachの下部に以下のようなエラーが出てしまいます。
正常に計算させるためにはどのような文にすればよいでしょうか。
ご助言いただけると幸いです。

'int''int[]'に変換できません。

該当のソースコード

public class ExamResult
    {

        int[] points;
        public int Sum { get; private set; }

        public ExamResult(int[] p)
        {
            points = new int[p.Length];

            for (int i = 0; i < p.Length; i++)
            {
                points[i] = p[i];
            }
            points = p;

            foreach (var point in points)
            {
                Sum += point;
            }
        }

        public double Avg()
        {
            if (points.Length==0)
            {
                return 0;
            }
            return Sum / (double)points.Length;
        }


        public int Min()
        {
            return points.Min();
        }

        public int Max()
        {
            return points.Max();
        }

        public double Stdv()
        {
            double sum2 = 0;
            double mean = Sum/points.Length;
            foreach (int i in points)
            {
                sum2 += Math.Pow(i-mean, 2);
            }
            double variance = sum2/points.Length;
            return Math.Sqrt(variance);
        }
        public double Corr(ExamResult x)
        {
            if (points.Length <= 2) throw new ArgumentException("points.Length <= 2");
            int  sumx, sumy, sumxx, sumxy, sumyy;
            sumx = sumy = sumxx = sumxy = sumyy = 0;
            foreach (int[] d in points)
            {
                sumx += d[0];
                sumy += d[1];
                sumxx += d[0] * d[0];
                sumxy += d[0] * d[1];
                sumyy += d[1] * d[1];
            }
            var r = Math.Sqrt((points.Length * sumxx - sumx * sumx) * (points.Length * sumyy - sumy * sumy));
            return (points.Length * sumxy - sumx * sumy) / (r != 0 ? r : 1);//相関係数
        }

        public double Dist(ExamResult x)
        {
            int sumx, sumy;
            sumx = sumy = 0;
            foreach(int[] d in points)
            {
                sumx += d[0];
                sumy += d[1];

            }
            double devsq =Math.Pow(sumx-sumy,2);//偏差平方和
            return Math.Sqrt(devsq);//ユークリッド距離
        }



    }



    class ExamResultTest
    {
     public static void Main()
    {
        ExamResult English = new ExamResult(new int[] { 80, 35, 45, 36, 78, 89, 67, 58, 37, 58, 53, 59, 50, 47, 58, 37, 19, 97, 86, 90, 10, 79, 37, 87, 45, 67, 68, 69, 65, 100 });
        ExamResult Japanese = new ExamResult(new int[] { 32, 55, 38, 25, 78, 80, 70, 48, 50, 42, 78, 63, 51, 40, 41, 67, 35, 86, 89, 75, 53, 45, 60, 30, 67, 43, 86, 73, 65, 90 });
        Console.WriteLine("英語の平均点は{0},最低点は{1},最高点は{2}",English.Avg(),English.Min(),English.Max());
        Console.WriteLine("英語の標準偏差は{0}", English.Stdv());
        Console.WriteLine("国語の平均点は{0},最低点は{1},最高点は{2}",Japanese.Avg(), Japanese.Min(), Japanese.Max());
        Console.WriteLine("国語の標準偏差は{0}", Japanese.Stdv());
        Console.WriteLine("英語と国語の相関係数は{0}",English.Corr(Japanese));
        Console.WriteLine("英語と国語のユークリッド距離は{0}", English.Dist(Japanese));


     }
    }

試したこと

CorrメソッドとDistメソッドのint[]部分をintに変更してみましたが、そうするとforeachの中の文が成立しませんでした。

補足情報(言語/FW/ツール等のバージョンなど)

Visual Studio2015です。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+2

アドバイス

public double Corr(ExamResult x) // <- 引数がコード本体のどこにも出てこない = おかしい
{
    if (points.Length <= 2) throw new ArgumentException("points.Length <= 2");
    int  sumx, sumy, sumxx, sumxy, sumyy;
    sumx = sumy = sumxx = sumxy = sumyy = 0;
    foreach (int[] d in points) 
// <- pointに入っているものは何と定義しましたか? とある**1科目**の点数リストですよね?
// 自分でそう定義しておいて、ここでなぜ2科目入っていると思ったんですか?
    {
        sumx += d[0];
        sumy += d[1];
        sumxx += d[0] * d[0];
        sumxy += d[0] * d[1];
        sumyy += d[1] * d[1];
    }
    var r = Math.Sqrt((points.Length * sumxx - sumx * sumx) * (points.Length * sumyy - sumy * sumy));
    return (points.Length * sumxy - sumx * sumy) / (r != 0 ? r : 1);
}

English.Corr(Japanese)で全然Japanese使ってませんね。
おかしいですね。
直しましょう。

能書きはいいから答えはよ

なるべく元のコードの雰囲気を残すと

public double Dist(ExamResult x)
{
    if (points.Length <= 2) throw new ArgumentException("points.Length <= 2");
    var ls = points.Zip(x.points,(a,b)=>new[]{a,b});

    int sumx, sumy;
    sumx = sumy = 0;
    foreach(int[] d in ls) 
    {
        sumx += d[0];
        sumy += d[1];

    }
    double devsq =Math.Pow(sumx-sumy,2);//ところでこれがユークリッド距離なのかは疑問
    return Math.Sqrt(devsq);
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/06 17:02

    早速参考にさせていただきました。
    エラーも出ずに計算することができました。
    ありがとうございます!

    キャンセル

  • 2017/02/06 17:18

    ところでこのMainにあるデータの場合
    ユークリッド距離はいくつになって欲しいんですか?

    キャンセル

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

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

関連した質問

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

  • C#

    7108questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Visual Studio

    1831questions

    Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。