Java 継承元のクラスをインスタンス化を防ぐためだけに抽象クラスにしていいのか

Question

現在swingでインベーダーゲームを作っています。

public abstract class Shot {
    protected static int x;
    protected static int y;
　　// 弾の画像
    protected Image image;

    
    public void draw(Graphics g){
        g.drawImage(image, x, y, null);
    }
    //その他メソッド,フィールド等省略してます
}
↓継承
public class PlayerShot extends Shot {
    //省略
}
public class EnemyShot extends Shot {
    //省略
}


↓パネルでの描画メソッド
public void draw (Shot shot,Graphics g) {
    shot.draw(g);
}

上記のような作りになっています。
最初Shotは抽象クラスではない普通のクラスだったのですが、メソッドの型としか使わないのにインスタンス化できるのはどうかと思い、インスタンス化を防ぐためだけにabstractをつけました。(抽象メソッドがない)

ただ、そのためだけにabstractをつけるのはイケてないなと感じ、他にやり方を調べると、privateコンストラクタでインスタンス化を防ぐ方法等が見つかりましたが、これだと継承先でもインスタンス化ができなくなるので、結局abstractをつけたままにしています。

そこでお聞きしたいのは
インスタンス化を防ぐためだけに抽象クラスにするのはやめたほうがいいのか？
他に回避方法があるのか？
まずこういう風になる設計がおかしいのか？
等です。

以上、よろしくお願いします

Comments

[umyu]

回答ではないので、こちらで。Shot クラスの変数xとyをstatic宣言しているため、PlayerShotとEnemyShotクラスで値が共有されています。

[michael-ilcsy]

指摘ありがとうございます。staticを外して作り直しました。

Answer 1

そもそも、 abstract class や　interface であっても具象クラスを使わずにインスタンスを作ること自体は厳密には防げないと思います。

public abstract class Hoge {}

Hoge h = new Hoge() {}

抽象クラスのコンストラクタを protected や修飾子なしにしつつ、具象クラスではファクトリメソッドを追加するなどでも良いのではないでしょうか。

public abstract class Hoge { 
  Hoge() {}
}

public class Fuga extends Hoge {
  Fuga() {}

  public static Fuga create() {
    return new Fuga();
  }
}

Comments

[mather]

インターフェースを継承クラスを作らずにインスタンス化する例では一番目にするのは Runnable かな。

[michael-ilcsy]

回答ありがとうございます！
匿名クラスは勉強しましたが、使うことが無かったのですっかり忘れていました。
ただ、今回は間違えてインスタンス化ができないようにするためで意図を持ってインスタンス化することは防がないので、またファクトリなども別の開発で使ってみたいと思います。

Answer 2

非staticメソッドがあるということは、継承クラスのインスタンス化は認めるのですよね？であれば、抽象クラスは妥当な選択だと思いますよ。

Comments

[michael-ilcsy]

回答ありがとうございます！
妥当な選択だとわかって安心しました。ありがとうございます！

Answer 3

変更不可能な定数としてのstatic フィールドを持つだけなら intaface にしてはどうでしょうか？
インスタンス化できないし、定数のカテゴリ別にinterfaceにしておくことで複数の組み合わせが可能です。

3.7. インターフェースで定数定義

機能はかなり限定されますが、defaultメソッドの実装も持つことも出来ます。
今更 Java8 を使ってみる ～その2 インターフェースの default 実装編～

Comments

[michael-ilcsy]

回答ありがとうございます！
元々クラスとして完成していたもの(メソッド、フィールド等)を抽象クラスにしているのですが、defaultメソッドを使ってインターフェースにしたほうがいいのでしょうか？

[euledge]

回答の中にも書いたのですがdefaultメソッドは状態を持たないメソッドしか作成できないので、用途によってはかなり厳しいかもしれません。既存クラスからの変更だとそぐわないことのほうが多いかもしれませんね。あまり役に立たない回答になってしまってすみません