javaの継承について

Question

前提・実現したいこと

PersonクラスからEmployeeクラスへの継承を行いたいのですが、
Employeeのコンストラクタの引数を「String name, int age, int employeeID」ではなく、
「Person person, int employeeID」としたいと考えています。

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

super（）の書き方が分からず、詰まっています。

該当のソースコード

 class Person {
	 private String name;
	 private int age;
	

	 public Person(String name, int age) {	        
	        this.name = name;
	        this.age = age;
 	}

	 public String getName() {
	       return name;
	 }

	 public void setName(String name) {
	        this.name = name;
	 }

	 public Integer getAge() {
	        return age;
	 }

	 public void setAge(int age) {
	        this.age = age;
	 }
}

public class Employee extends Person {
     private Person person;
　　 private int employeeID;

     public Employee(Person person, int employeeID) {
            super(person.getName, person.getAge);
            this.employeeID = employeeID;
    }

     public int getEmployeeID() {
            return this.employeeID;
  }
  
     public Person getPerson() {
            return this.person;
     }
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[y_waiwai]

提示のコードではどういう不具合があるんでしょうか

Answer 1

Person を継承するのでしたら、 Person の name, age 両フィールドとも protected にし、 Employee コンストラクタでは直接 person.name, person.age と指定する方法もあります。
また、継承しているのですから Employee オブジェクト自体が Person オブジェクトですので getPerson は不要です。

それとこの場合 コンストラクタパラメータの Person オブジェクトと生成された Employee オブジェクトは同じ Person を示すことになりますので、 equals 等のメソッドをオーバーライドされたほうが良いかと思います。　

何となく課題っぽい内容ですので Person を弄るとか等ダメなのかもしれませんが。

Answer 2

継承の意味を理解していますか？

Personクラスは、name と age の 2つのフィールドを持っています。
Employeeクラスは、Personクラスを継承しているので、
name、age、person、emploieeID の 4つのフィールドを持っています。

super(person.getName, person.getAge); を
super(person.getName(), person.getAge()); にすると、
コンパイルエラーは無くなります。

Employeeのコンストラクタで、
name、age、emploieeID の 3つのフィールドは初期化されますが、
person は初期化がないので null です。

getPersonメソッドで this.person を返してもらっても、それは null です。

これでいいんですか？

Comments

[taka0145]

super(person.getName(), person.getAge());は
personではなく、nameとageを表しているためnullになるということでしょうか？

理解力がなく申し訳ありません。

[dodox86]

@質問者 taka0145さん

kazuma-sさんは
> public class Employee extends Person {
> private Person person;

に対して、Employeeクラスのコンストラクタ等でフィールドpersonを初期化する部分が無いけどいいんですか？　と聞かれています。現状のコードでpublic Person getPerson() を呼び出せば、何が問題かわかるはずです。
しかしながら EmployeeクラスはそもそもPersonクラスを継承しているのだから、自身はPersonクラスでもあり、フィールドpersonを用意する意味は無いはずなのです。それを補足しているのが、別の回答者であるy_waiwaiさんの回答です。

[taka0145]

理解力不足でした。
再度検討してみます。

Answer 3

もひとつ、

     public Person getPerson() {
            return (Person)this;
     }

Answer 4

super（）の書き方が分からず、詰まっています。

そう難しく考えることはなく、普通にgetterメソッドを呼び出せばよいです。提示のコードでは()が抜けていますね。

public Employee(Person person, int employeeID) {
    // NG super(person.getName, person.getAge);
    super(person.getName(), person.getAge());
    this.employeeID = employeeID;
}