初歩的な継承に関して。

Question

java学習初学者です。

現在サブクラスであるDogクラスに、スーパークラスであるAnimalクラスを継承して、
実行結果に「①ポチは走りました　②ポチは食事しました」と出力したいのですが、
結果としては、「①nullは走りました　②nullは食事しました」と出力されます。

自分なりに試行錯誤してみましたがどこが間違っているのかが分かりません。
初歩的な内容で申し訳ありませんが、何卒お力添えの程よろしくお願いいたします。

**[①Main クラス]**


public class Main {
	public static void main(String[] args) {
		// commonパッケージのCatクラスをインスンタンス化
		Dog dog = new Dog();

		// Dog型インスタンスのnameフィールドに ポチ を代入
		dog.name = "ポチ";

		// Dog型インスタンスのrunメソッドを実行
		// ※ ポチは走りました。が表示されれば正解
		dog.run();

		// Dog型インスタンスのeatメソッドを実行してください
		// ※ ポチは食事をしました。が表示されれば正解
		dog.eat();

	}
}


**[②Dog クラス]**


package common;

// 同一パッケージのAnimalクラスを継承
public class Dog extends Animal{

	public String name ; 
}


**[③Animal クラス]**


package common;

public class Animal {
	
	public String name;


	public void run() {
		System.out.println(this.name + "は走りました。");
	}

	
	public void eat() {
		System.out.println(this.name + "は食事をしました。");
	}


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

	public String getName() {
		return this.name;
	}

}

Answer 1

Dogクラスのnameフィールドと、Animalクラスのそれが共存しているのが問題です。
Dogクラスの実装は次のように空で充分でしょう。

public Dog extends Animal {
}

Dogクラスは本当に必要なのか、nameフィールドはpublicで良いのか、etc...
いくつか不思議な点がありますが、教材の狙いが分からないので取り扱わないことにします。

Comments

[beginner_7]

迅速なご回答ありがとうございます！
ご回答内容を踏まえ、Dogクラスからフィールドを消し、nameの修飾子をprivateに、Mainの方でdog.setName("ポチ");　としたところうまくいきました。本当にありがとうございました！

参考までにお伺いさせていただきたいのですが、Animalクラスのフィールドnameの修飾子がprivateでなければならない理由はなんでしょうか？
アクセサをしようしているからでしょうか？

何卒よろしくお願いいたします。

[LouiS0616]

> Animalクラスのフィールドnameの修飾子がprivateでなければならない理由はなんでしょうか？
アクセサをしようしているからでしょうか？

主従が逆転しています。
x アクセサを使用するからフィールドをprivateにする
o フィールドをprivateにするからアクセサが使用される

オブジェクト指向プログラミングの原則の一つにカプセル化があります。
Javaの立場を踏まえてざっくり言うと、『フィールドの値を意識せずにオブジェクトを適切に操作できる』ようにするのがカプセル化です。
その観点では、フィールドを外部から好き放題いじれるのは好ましくないのです。

最近関連する議論がありました。参考にどうぞ。
https://teratail.com/questions/214364

[LouiS0616]

動物の名前を変更することを想定しないのであれば、setNameは用意せずコンストラクタで名前を受け取るようにすれば良いです。

[beginner_7]

詳しくありがとうございます！
無事に解決致しました。