複数名分の生徒を管理する生徒情報クラスの平均値を求める

Question

複数名分の生徒情報（名前、ID）を管理し平均年齢を求めるプログラムを書いています。以下のように書いてエラーはないのですが、求めた平均年齢がNaNとなってしまいます。何が間違っているのでしょうか。

public class Main {

public static void main(String[] args) { 
	StudentInfo student = new StudentInfo(); 
	student.addStudents(new Student("Taro Sato","111",21)); 
	student.addStudents(new Student("Akari Kato","112",20)); 
	student.addStudents(new Student("Taizo Hayashi","113",21)); 
	student.addStudents(new Student("Hikari Watanabe","114",22)); 
	
	double average = StudentInfo.getAverage(); 
	System.out.println("平均年齢:"+ average); 
	
}

}

public class Student {

 String name ="";  //名前    
 String studentID ="";  //学生ID    
 int age ;   //年齢 
 public Object id;

public Student(String name, String id, int age) {
this.name = name;
this.studentID = id;
this.age = age;
}

public String getName(){
return name;
}

public String getId(){
return studentID;
}
public int getage(){
return age;
}
}

public class StudentInfo {

public static int MAX_COUNT=100; //取り扱えるStudentの最大数;
static Student[] students = new Student[MAX_COUNT]; //管理するStudent
static int studentCount =0;  //配列に格納されたStudentの数
    
private Object id = " ";
private String name = null;
private int age;

//・引数 ：student ／Student 追加する Student 
//・戻り値：追加できた時は true, 失敗したときはfalse を返す． 
//・目的 ：StudentInfo 内の配列に Student を追加する．引数の値が null である場合，
//         配列で扱 える最大数を超えた場合や，既に同じ情報を持つ Student が登録されている場合は， 
//         登録されずにfalse を返す． 

public boolean addStudents(Student student){
    //引数にStudent以外のクラスが指定 された時はfalseを返す
        if(student instanceof Student != true )
        	return false;
        students[studentCount] = student;
    //配列で扱える最大数を超えた場合
        if(studentCount > MAX_COUNT || student == null){
        	return false;
        }                            
    //配列の要素を一つ一つ同じである かどうかを確認し，全て同じであれ ばtrueを返す
        Student target = (Student)student;
        if(this.name != target.name || this.id != target.id || this.age != target.age)
        	return true;
        	studentCount ++;
        	return true;
}

//・引数 ：なし 
//・戻り値：配列に登録されている Studentの平均値 
//・目的 ：StudentInfo の配列に登録されている Student の平均年齢を算出して返す．
//         登録され ている Student がない場合は0 を返す． 

public static double getAverage(){
    double average = 0.0;
        for(int i=0; i < studentCount; i++)
            average += students[i].getage();
        return average/studentCount;
}

}

Comments

[momon-ga]

それぞれのメソッドのやりたいことで、目的のどの部分がわかりますか？
たとえば、getAverageで登録されてるStudentがない場合に０を返す部分は書けますか？

[TaWa]

public boolean addStudents(Student student) の「配列で扱 える最大数を超えた場合や，既に同じ情報を持つ Student が登録されている場合は， 登録されずにfalse を返す。」の実現方法がわかりません。

[momon-ga]

返信ありがとうございます。内容は質問を修正してください。

[javahack]

"java 配列"で検索して配列の使い方を勉強してはいかがでしょうか。

Answer 1

修正依頼から１か月以上も放置されたものが編集されるとは思いませんでした。

public boolean addStudents(Student student) の「配列で扱 える最大数を超えた場合や，既に同じ情報を持つ Student が登録されている場合は， 登録されずにfalse を返す。」の実現方法がわかりません。

返信ありがとうございます。内容は質問を修正してください。

で、当初の質問とは変わったようなので、

エラーはないのですが、求めた平均年齢がNaNとなってしまいます。何が間違っているのでしょうか。

について回答します。

結論からいうと、studentCountが0から更新されないため、average /0 が表示されています。

System.out.println(0.0 / 0);

とやると、上記と同様にNaN(Not A Number)が表示されます。

Comments

[TaWa]

ありがとうございます。かなり前進しました。まだ重複チェックが解決できてはいませんが、それ以外は動くようになりました。

Answer 2

独学でやっていて、調べてもやり方が一向にわからない

どのように学習を進めているかは質問からわかりませんが、ネットで調べるだけで進めているとしたら今すぐ中止し、

• 「Java入門」などとタイトルのあるプログラミング入門書
• paiza、progateなどオンライン学習サイト

を利用して基本のキから体系的にプログラミングを学習していってください。
お金がないなら図書館でも書籍は借りることができます。