abstract クラスがフィールドを持つのは正しいか

Question

###前提・実現したいこと
こんばんは、毎度お世話になっております。自分のコードをリファクタリング中に疑問に思ったことがあったので質問させてください。
題名の通り、abstract クラスがフィールド（public static final でない）を持つのは正しいかが疑問です。
abstractに関してまだ、こうやって使うんだみたいな感覚がわかってないので、なにとぞご教授よろしくお願いいたします。

###自分が疑問に思っている部分のコード
キャラクターを管理する抽象クラスAbstractCharacter

// import 省略
abstract public class AbstractCharacter 
{
	private int hp; // このようにabstract クラスがフィールドを持ってよいか、疑問に思いました。
	private int mp;
	private Image image;

	private Image readImage(URL url){
		// 画像読み込み省略
		return 画像
	}

	protected void addHp(int hp){
		this.hp += hp;
	}

	protected void addMp(int mp){
		this.mp += mp;
	}

	public AbstractCharacter(int hp, int mp, URL url){
		this.hp = hp;
		this.mp = mp;
		this.image = this.readImage(url);
	}

	abstract public void attack(敵);

// その他の処理は省略
}

AbstractCharacterを継承したWizardクラス

// import 省略
public class Wizard extends AbstractCharacter
{
	public Wizard(){
		super(1500, 5000, Wizard.class.getResource("Wizard.png"));
	}

	@Override
	public void attack(敵){
		敵.addHp(-200);
		super.addMp(-50);
	}
// その他の処理は省略
}

###フィールドを持たせないように上のコードを直したら

キャラクターを管理する抽象クラスAbstractCharacter

// import 省略
abstract public class AbstractCharacter 
{
	// private int hp; // フィールドもない
	// private int mp;
	// private Image image;

	// コンストラクタはない
	// public AbstractCharacter(int hp, int mp, URL url){}

	abstract public void attack(敵);

// その他の処理(abstract なメソッド)は省略
}

AbstractCharacterを継承したWizardクラス

// import 省略
public class Wizard extends AbstractCharacter
{
	private int hp; // 代わりに派生クラス側でフィールドを全て記述
	private int mp;
	private Image image;

	private Image readImage(URL url){
		// 画像読み込み省略
		return 画像
	}

	public Wizard(int hp, int mp, URL url){
		this.hp = hp;
		this.mp = mp;
		this.image = this.readImage(url);
	}

	@Override
	public void attack(敵){
		敵.hp -= 200;
		this.hp -= 50;
	}
// その他の処理は省略
}

###なぜ疑問に思ったか

1. こういった記述のコードを見たことがないから
2. php を教わった先生に、「抽象クラスに実装があってもあまりうれしくない」と言っていたことを思い出したから

Comments

[Stripe]

とりあえず、abstractクラスになっていませんが？

[chankane]

すみませんミスです。修正します

[Stripe]

abstractと書いただけでは、abstractクラスとしての意味がないですよ。

[chankane]

それはコメント文に書いた通り省略しています。省略しないほうがよろしかったですか？お返事次第で追加いたします。よろしくお願いしますm(._.)m

[Stripe]

あなたが提示したコードは、ただ単にクラス継承を表すものでしかありません。これは今回の質問とどんな関係があるんですか？あなたが何を疑問に思っているのか、このコードからは読み取れません。

[chankane]

abstractなメソッドとコメント文と直した場合こうなるという例を追加致しました。まだ要望があるようでしたら、情報修正致します_(._.)_

Answer 1

別に非抽象メソッドをもっていても問題ないと思います。抽象クラスの性質の一つ「派生クラスの共通の親」として運用するうえで、共通メソッドやフィールドを書いておくというのはありえるはずです。ただ、抽象メソッドを一つでも持つのであればそれは抽象クラスでなければなりません。

抽象クラスの最たる性質は「直接インスタンス化できない」ことです。派生クラスに、抽象メソッドを具象化することを要求します。そして、派生先が抽象メソッドの存在しない具象クラスとなれば、これに対してインスタンス化が可能となります。

要するに、「派生前提の中間的な共通部品」とかの意味合いがあります。

もちろん、すべてのフィールドをなくしてメソッドをabstructにしても問題はありません。ただし、それをするならばInterfaceで充分です。しかもjavaの場合は、Intefaceは多重実装可能（classの多重継承は不可能）なので、Intefaceの方が適切です。

Comments

[chankane]

おはようございます。「派生前提の中間的な共通部品」とかの意味合いですか。
とてもわかりやすいです。つまり、abstract クラスがフィールドを持つのはありということですね。
そちらの方が便利なので、今後共通する部分はどんどん基底クラスに書いていこうとおもいます。ご回答ありがとうございました。

Answer 2

こういった記述のコードを見たことがないから

普通にありますよ。

「TemplateMethodパターン」で検索してみてください。

抽象クラスにメインの実装があります。切り替えたい箇所を抽象メソッドで記述します。
共通で使うメンバ変数があれば抽象クラスに実装します。

「抽象クラスに実装があってもあまりうれしくない」

そういう場合はInterfaceを使えばいいのかなと思います。
Interfaceを使えば実装は持ちません。

Comments

[chankane]

template method パターンですか。初知りです。ググってみましたが普通に使われているみたいですね。「こういった記述のコードを見たことがないから」なんていって勉強不足だなと思いました。
今回の例では、Wizardの他にも、Knight、Soldier、Archer... とドンドン増えそうなのでその template method パターンとしておくととても管理が楽になりいいと思います。なんかこうゆうのって無意識で使ってしまっていることもけっこうありますね。
ご回答ありがとうございました。