[java]同一クラスを継承した複数のインスタンスのArrayList作成方法

Question

前提・実現したいこと

[java]

ある１つのクラスを継承した２つのクラスから生成したインスタンスを同じArrayListに入れたい。

具体的には、Humanクラスを継承したHeroクラスとWariiorクラスのインスタンスを１つのArrayListに入れたい。

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

ArrayListに入れる際の型の指定の仕方がわからず、Humanに指定すると全てHumanクラスのhitPoint,attackPointになってしまう。

それぞれHero,Warriorの値になるようにしたい。

該当のソースコード

コードは以下の通りです。

public class Sample {
	public static void main(String args[]){
		ArrayList<Human> members = new ArrayList<>();
		members.add(new Hero("勇者"));
		members.add(new Warrior("戦士"));

		for (Human member : members) {
			System.out.println("名前:" + member.getName() + ",HP:" member.getHitPoint() + ",攻撃力:" member.getAttackPoint());
		}
	}
}

public class Human {
	final int MAX_HITPOINT = 100;
	private String name;
	private int hitPoint = 100;
	private int attackPoint = 20;

	public Human(String name) {
		this.name = name;
	}

	// getter略
}

public class Hero extends Human {
	final int MAX_HITPOINT = 150;
	private int hitPoint = this.MAX_HITPOINT;
	private int attackPoint = 30;
	
	public Hero(String name) {
		super(name);
	}

	// getter略

}

public class Warrior extends Human {
	final int MAX_HITPOINT = 120;
	private int hitPoint = this.MAX_HITPOINT;
	private int attackPoint = 50;
	
	public Warrior(String name) {
		super(name);
	}

	// getter略

}

Answer 1

主な解決法は2通りになると思います。
0. getterを適切にオーバーライドする
getterが示されていないのでわかりませんが、getHitPoint,getAttackPointメソッドのオーバーライドが適切にされていない可能性があります。
正しくオーバーライドすることで、望み通りの表示になるはずです。

1. Humanクラスにコンストラクタを追加し、hitPoint,attackPointの初期化を適切に行う。

そもそもの問題として、HumanクラスとHero,Warriorクラスが別々に同じパラメータを持っている状態は好ましいとは言えません。
コンストラクタでそれらのパラメータを一緒に初期化するようにすればいいと思います。

Comments

[kyo._.]

ありがとうございます。
getterのオーバーライドが適切にできていませんでした。
最初の自分の仮説が間違っていたようです。