質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.50%

  • Java

    15845questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

共通部分と独自処理が含まれる場合の継承[java]

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 2,863

hanayamaKaoru

score 28

前提・実現したいこと

前回、うまく質問できなったので再度質問します。
インターフェースをimplementsすると実装は共通部分が多いのでコピペを行うことになり、保守性が下がる。

そのため
実装が
①共通
②独自
③共通
と独自の処理が共通処理で囲まれている場合
以下のコードのように
①の共通部分を行うfirstRun()
③の共通部分を行うLastRun()
を定義したクラス(ControlDamage)を継承し独自処理をfirstRun()とLastRun()の間に記述することにしました
この解決策はナンセンスでしょうか?

*DecreaseDamageByBerryとIncreaseDamageByBerryでは独自処理を行うことでパラメータが変化しLastRun()の結果が変わる。

該当のソースコード

public abstract class Ability {
    protected ArrayList<Object>  params= new ArrayList<Object>();

    protected Ability(ArrayList<Object> params) {
        this.params = params;
    }

    public abstract void run();

}
public abstract class ControlDamage extends Ability {
    protected Magnification magnification;
    protected int i =0;
    ControlDamage(ArrayList<Object> params) {
        super(params);
        // TODO 自動生成されたコンストラクター・スタブ
    }

    protected void firstRun(){ 
        /*共通部分前半処理*/

    }
 
    protected void lastRun(){
        /*共通部分後半処理*/
    }

}
public class DecreaseDamageByBerry extends ControlDamage {

    DecreaseDamageByBerry(ArrayList<Object> params) {
        super(params);

    }

    @Override
    public void run() {
        // TODO 自動生成されたメソッド・スタブ
        super.firstRun();/*共通部分前半*/

        /*共通ではない部分を処理*/

        super.lastRun();  //共通部分後半


    }


}
public class IncreaseDamageByBerry extends ControlDamage {

    DecreaseDamageByBerry(ArrayList<Object> params) {
        super(params);

    }

    @Override
    public void run() {
        // TODO 自動生成されたメソッド・スタブ
        super.firstRun();/*共通部分前半*/

        /*共通ではない部分を処理*/

        super.lastRun();  //共通部分後半


    }


}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

この解決策はナンセンス

ということはありません。

この実装方法は "テンプレートメソッド・パターン" と呼ばれるデザインパターンの1つです。
https://ja.wikipedia.org/wiki/Template_Method_%E3%83%91%E3%82%BF%E3%83%BC%E3%83%B3

ただし、ご提示のソースコードは以下の部分が冗長に感じられます。

  1. AbilityクラスとControlDamageクラスは1つにまとめることが可能。
  2. run メソッドを abstract メソッドとして定義する必要はない。

私なら、以下のように定義・実装します。

import java.util.List;

public abstract class Ability {
    protected List<Object>  params;

    protected Magnification magnification;
    protected int i = 0;

    protected Ability(List<Object> params) {
        this.params = params;
    }

    public void run() {
        firstRun();/*共通部分前半*/

        /*共通ではない部分を処理*/
        originalRun();

        lastRun();  //共通部分後半
    }

    private void firstRun(){ 
        /*共通部分前半処理*/
    }

    private void lastRun(){
        /*共通部分後半処理*/
    }

    protected abstract void originalRun();
}
import java.util.List;

public class DecreaseDamageByBerry extends Ability {

    DecreaseDamageByBerry(List<Object> params) {
        super(params);
    }

    @Override
    protected void originalRun() {
        /*共通ではない部分を処理*/
    }
}
import java.util.List;

public class IncreaseDamageByBerry extends Ability {

    IncreaseDamageByBerry(List<Object> params) {
        super(params);
    }

    @Override
    protected void originalRun() {
        /*共通ではない部分を処理*/
    }
}


run(), firstRun(), lastRun() メソッドをAbilityクラスで実装し、「共通でない部分を処理」する originalRun() メソッドだけを abstract として定義すれば、Abilityクラスのサブクラスはこのメソッドだけを実装すればよいことになります。


ちなみに、Abilityクラスの全てのサブクラスに

run メソッドを呼び出すと
  ①firstRun()originalRun()lastRun()
を順番に実行する

ことを強制したいのであれば、run メソッドに final 修飾子を追加してやればよいです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+2

Java8 では default メソッド が利用可能になっています。

public interface A {
    default void b(String s) { System.out.println(s); }
}

とかけます。

以前は

public interface A {
    public void b(String s);
}

public abstract class B implements A  {
    public void b(String s) { System.out.println(s); }
}

と abstract クラスを挟むことで対応されていました。

詳しくは java Collection Framework のクラス群が後者の方法をつかっているので見てみるといいです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • Java

    15845questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。