java 得点表。アクセス関数を使用して数値を呼び出すプログラムを作りたい

Question

課題なのですが、
2つのオブジェクトを使って
名前を入力したときにアクセス関数によって
得点と偏差値が表示されるプログラムを作りたいのですが、
モデルを教えていただけないでしょうか？

1つは各人の名前と偏差値が書き込まれたオブジェクト、もう1つはメインのプログラムでmain()が書かれたものにしたいです。

途中まで書いたのですが、そこからの関数の使い方がイマイチわかりません。
条件としては名前.アクセス関数と得点.アクセス関数を別々にする。
privateを使うというものですが、
初心者なので、容量を得ません。
途中まで書けたのでそこからの持っていきかたのモデル
を教えていただけませんでしょうか。
宜しくお願いいたします。

package objects;

public class Scores {

	public static void main(String[] args) {

		Score[] scores={ new Score("A",70),
						 new Score("B",71),
						 new Score("C",72),
						 new Score("D",73)
		};
		System.out.println(scores[0].name+","+scores[0].score);
		
		
	}

}

package objects;

public class Score {
	private String name;
	private int score;
	private double standardScore;

	public Score(String n,int s){
		this.name = n;
		this.score= s;			
	}
	
	public String name(){return this.name;}
	public int score(){return this.score;}
	public double standard(){return this.standardScore;}
	
	public void setName(String s){this.name=s;}
	public void setScore(int n){this.score=n;}
	 
	public void standardScore(double average,double standardscore){
		this.standardScore=((this.score-average)*10)/standardscore)+50;
	}
	public void display(){
		System.out.println(name+"\\t"+score);
	}
				
	
	
	
	}

補足ですが、objectsというパッケージにScoresとScoreというクラスを設け、Scoresはmainで、各点数と名前（ここではA,B,C,D）を入力し、Scoreで偏差値の計算を行うプログラムをつくる。
そしてScoresでScoreの計算に基づいて名前や得点のインスタンスにアクセス関数を使用して
(名前,点数,偏差値)がSystem.out.printlnで表示されるプログラムをつくりたい。ということです。
よろしくお願いいたします。

Comments

[swordone]

名前.アクセス関数とか得点.アクセス関数ってなんでしょう？名前とか得点のインスタンスに対してアクセス関数を使えということでしょうか？

Answer 1

およそ授業でやらないような書き方をしてみました。

import java.util.HashMap;
import java.util.Map;
import java.util.NoSuchElementException;
import java.util.Objects;
import java.util.Optional;

public class Q51375 {

	public static void main(String[] args) {
		ScoreRecord record = new ScoreRecord();
		record.put("A", 70);
		record.put("B", 71);
		record.put("C", 72);
		record.put("D", 73);
		for (String s : new String[] { "A", "B", "C", "D", "E" }) {
			try {
				record.display(s);
			} catch (NoSuchElementException e) {
				System.out.println(e.getMessage());
			}
		}
	}

}

class ScoreRecord {
	private Map<String, Integer> map = new HashMap<>();
	private double sum = 0;
	private double sqSum = 0;

	public void put(String name, int score) {
		int prevScore = Optional.ofNullable(map.put(Objects.requireNonNull(name, "name must not null."), score)).orElse(0);
		sum += score - prevScore;
		sqSum += score * score - prevScore * prevScore;
	}

	public void display(String name) {
		int score = Optional.ofNullable(map.get(name)).orElseThrow(() -> new NoSuchElementException(name + "なんて人いないよ"));
		System.out.println(String.format("%1$s %2$d %3$2.2f", name, score, dev(score)));
	}

	private double dev(int score) {
		return (score - average()) * 10 / stdevp() + 50;
	}

	private double stdevp() {
		return Math.sqrt(sqSum / map.size() - average() * average());
	}

	private double average() {
		return sum / map.size();
	}
}

Comments

[M.M.O-05]

ご回答ありがとうございます。
今回の目標としては、objectsというパッケージに
ScoreとScoresという2つのクラスが含まれているプログラム
を作りたいのですが、お答え頂いた回答の方法ですとエラーが出てきてしまうのですが
どうしたらよろしいでしょうか。
Scoreには各点数と名前を、Scoresにはそれを計算し、
System.out.prinlnで
(名前,点数,偏差値）という形で結果が表示される
プログラムだとどうなりますか。
すいませんが、宜しくお願いいたします。

[swordone]

さすがに課題丸ごと請け負いはしませんよ。持って行き方を聞かれたのでそれを説明する概念的な意味で書いたコードなので。
エラーがでてるのはおそらくJava7以下だからではないでしょうか。

Answer 2

条件としては名前.アクセス関数と得点.アクセス関数を別々にする。

privateを使うというものですが、

書かれたプログラムでは、 score と name はクラス Score の private 変数なので、そのままでは外からアクセスできません。

 System.out.println(scores[0].name+","+scores[0].score);

は Score の private 変数に直接アクセスしようとしているので、コンパイルエラーになると思います。
main からはアクセス関数を通して、 score[0].score() のようにアクセスするように修正する必要があります。
値取得用のアクセス関数が変数名と同じなのは若干気持ちが悪いので、getScore, getStandard とかにされたほうが良いでしょう。

名前を入力したときにアクセス関数によって

得点と偏差値が表示されるプログラムを作りたいのですが、

まず、全員のデータが揃ったところで、平均と標準偏差を計算する必要があります。これは、 main の中でループで Score を巡回して実現することになるでしょう。

次に各人の偏差値を計算して設定する必要があります。これも main の中でループを回して全員の standardScore を呼び出す必要があるでしょう。

最後に名前の入力を受け付けて、得点と偏差値を印刷するプログラムを作る必要があります。

1. 標準入力から名前を読み込む
2. 読み込まれた名前と一致する Score を探す。（ループを回して探す）
3. 見つかった Score の display メソッドを呼び出して印刷する

書かれた display() メソッドでは偏差値を印刷していないので、偏差値を印刷するように修正する必要があるでしょう。

読み込まれた名前と一致する Score を探すためにハッシュテーブルを使うほうが一般的です。現在は、Score のインスタンスを配列に格納されていますが、 Hashｔable に格納しておけば、簡単に早く探せると思います。件数が増えてくると、配列を前から探す方法では件数に比例して遅くなりますが、ハッシュテーブルにしておけば、遅くなりません。

Comments

[M.M.O-05]

ご回答ありがとうございます。
main()のオブジェクトについて
```lang-java
package objects;

public class Scores {

public static void main(String[] args) {

Score[] scores={ new Score("A",70),
new Score("B",71),
new Score("C",72),
new Score("D",73)
};
System.out.println(scores[0].name()+","+scores[0].score());

}

}
```
となったのですが、
実行した時に
Aの値しか出なかったのですが、どうすればよいでしょうか。
あと偏差値をどうアクセスさせればよろしいでしょうか。

[mit0223]

scores[0] しか印刷していないので、当然、A しか表示されません。Bは score[1], Cは score[2], D は score[3] に入ってます。

目標を達成するためには、私の回答に従って、かなりコーディングする必要があります。
私の回答の中で「ループを回して」となっているところは、for でループを回してすべての要素を巡回してください。
たとえば、すべての要素を印刷してみる（あくまでテスト目的で目標にはふくまれていない）のなら

for(int i=0; i< scores.length; i++){
System.out.println(scores[i].name()+","+scores[i].score());
}

などとしてください。偏差値も印刷するなら、　scores[i].standard() を追加してください。

目標達成までは、まだ、かなり道のりは長そうです。

Answer 3

学生さんだとしたら授業はちゃんと聞いた方がいいですよ
社会に出ると、お金もあって勉強もしたくても時間がなかったりするので、学生の時間は本当に貴重です。

まず、privateメンバにする意味は分かりますか？
仮にnameに「最大10文字まで」という制限を付けたい場合、

private String name;
public String setName(name){
    if(name.length <= 10) {
        this.name=s;
    } else {
        throw new IllegalArgumentException('nameは10文字まで');
    }
}

とかして、その制限に沿った値しか受け付けないという事が出来るわけですが
public にしてしまうと

scores[0].name = "abcdefg hijklmn";

とか書いても、通ってしまうんですね。
なので、privateにして、アクセサメソッド(setName() getName())を使うわけです。

で、質問文の中のソースコードでは、以下のようにしてname等を出力しようとしていますが

System.out.println(scores[0].name+","+scores[0].score);

これは、アクセサメソッドを介さずに、直接参照してる形ですよね。
なので、これもアクセサメソッドを利用するようにします。

System.out.println(scores[0].name()+","+scores[0].score());

メンバとアクセサメソッドが同名だと、このように混乱も招くので、「setName」「getName」等、わかりやすい名前にしておくと良いです。

同じようにして全員のscoreを取得したら、それを計算してaverageを算出します。
同様に、standardScoreも計算します。

その後、ご自身で作られたstandardScore()の引数に与えればよいのでは。

Comments

[kaputaros]

付け足しで、

> System.out.println(scores[0].name+","+scores[0].score);

これだと、Scoreクラスのname、scoreフィールドは private なので、
通常クラスの外からはアクセスできないはずです。
なので、コンパイルエラーになるんじゃないかと。