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

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

ただいまの
回答率

90.54%

  • C#

    8844questions

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

  • Visual Studio

    2300questions

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

配列の値を各メソッドに読み込ませるには

解決済

回答 2

投稿

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

meroy25

score 5

前提・実現したいこと

たびたびの質問で失礼します。
前回質問し、一部解決したプログラムです。
30人分の国語と英語の試験結果をもとに、各教科の平均点・最低点・最高点を計算し、さらに2つの教科の点数の相関係数、ユークリッド距離を求めるものです。
各メソッドの中に計算式を入れるのですが、そのために点数の合計の値を使う必要があります。
その値をpublic ExamResult(int[] p)の部分から求めています。

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

これから各メソッドに計算式を入力するところですが、ExamResultTestクラスに書き込んだ配列の値をそれぞれのメソッドに読み込ませる良い方法が思いつきません。
また、public ExamResult(int[] p)の部分から計算した点数の合計の値も読み込めません。
配列の値、点数の合計を各メソッドに反映させ、計算式を作るにはどうすればよいでしょうか?
皆さんの助言をいただけると幸いです。

・値を返さないコードパスがあります。(Avg,Min,Max,Stdv,Corr,Dist下部)

該当のソースコード

using System;
using System.Linq;

 public class ExamResult
    {

        int[] point;

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

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

        }

        public double Avg()
        {

        } 


        public int Min(params int[] point)
        {

        }
        public int Max(params int[] point)
        {


        }
        public double Stdv()
        {


        }
        public double Corr(ExamResult x)
        {

        }

        public double Dist(ExamResult x)
        {

        }


    }



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


     }
    }

試したこと

Avg(平均値)のメソッドをpublic ExamResult(int[] p)の中に含め、p.Length(点数の配列の合計)を反映しようとしたのですが、コンテキストにp.Lengthはない、とエラーが出てしまいました。

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

Visual Studio2015です。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+3

どこの部分でつまっているのかよくわかりませんが、
やりたいことはわかったので、とりあえず一部書いてみました。

public class ExamResult
{
    //配列なので気持ち的に名前を複数形にしたい
    int[] points;

    //合計点
    //(クラス外から参照する必要がなければprivateなフィールドでも良い)
    public int Sum { get; private set; }

    public ExamResult(int[] p)
    {
        //※↓は配列をコピーしているだけなので
        // 意図的な理由がなければそのまま代入しても良い
        //point = new int[p.Length];
        //
        //for (int i = 0; i < p.Length; i++)
        //{
        //    point[i] = p[i];
        //}
        points = p;

        //合計点を計算しておく
        foreach (var point in points)
        {
            Sum += point;
        }
    }

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

        //doubleにキャストしないと小数点以下が落ちてしまう
        return Sum / (double)points.Length;
    }

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

}

気になった点

  • 「p.Length(点数の配列の合計)」と書かれていますが、(ご理解されてるとは思いますが)Lengthは配列の要素数(この場合は人数)であって、合計値ではないです。

  • MinとMaxの引数に (params int[] point) と書かれているのは何を意図したものでしょう?
    点数はコンストラクタ( public ExamResult(int[] p) )で受け取って保持しているのでもらう必要がありませんし、
    この params というのは「可変長引数」を表すものなので、意図して使う分には良いんですが、何か勘違いされているかなと思いました。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/06 15:53

    参考にさせていただきましたが、私の希望通りのプログラムになりました。
    ありがとうございます!

    キャンセル

+2

コンストラクタでpointに入れているので、これを各メソッドで使用すればいいのではないですか?

    public double Avg()
    {
        return point.Average();
    }


こんな感じでいいんですかね?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/06 15:53

    この方法は思いつきませんでした。
    ありがとうございます!

    キャンセル

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

  • C#

    8844questions

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

  • Visual Studio

    2300questions

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