Java 子クラスを介して親クラスのメソッドにアクセスする処理について

Question

###質問
Java
継承関係にあるクラスの扱いについて

下記のように、子クラスに個別の処理を、
親クラスに共通の処理を実装した際に、メインのクラスから子クラスのmethodAを介して、
親クラスのmethodAにアクセスするのと、親クラスのmethodAのアクセス修飾子を"public"にし、
メインクラスから直接アクセスするのだと、どちらが正しい(良い)のでしょうか？

子クラスでオーバーライドしているわけでも、オーバロードしているわけでもないので、
わざわざ子クラスを介しているのは無駄なのかと考えているのですが、いかがでしょうか。

###ソースコード

public class SuperClass {
    protected void methodA(int aParam) {
        // 何かしらの処理
        methodB();
        // 何かしらの処理
    }
    private void methodB() {
        // 何かしらの処理
    }
}

public class SubClassA extends SuperClass {
    public void methodA(int aParam) {
        super.methodA(aParam);
    }

    // 個別の処理
}

public class SubClassB extends SuperClass {
    public void methodA(int aParam) {
        super.methodA(aParam);
    }
    
    // 個別の処理
}

public class MainClassA {
    public static void main(String[] args) {
        SubClassA sub = new SubClassA();
        sub.methodA(1);
    }
}

public class MainClassB {
    public static void main(String[] args) {
        SubClassB sub = new SubClassB();
        sub.methodA(2);
    }
}

Answer 1

子クラスの methodA が全部親クラスの methodA に処理を委譲してしまうなら、もはや methodA を protected とする意味はないので、親クラスのメソッドを public として子クラス側の override メソッドを削除するべきです。

処理を他のクラスに委譲したいだけなら、継承すらも止めるべきです。

Comments

[Hanbe.T]

返答ありがとうございます。

>処理を他のクラスに委譲したいだけなら、継承すらも止めるべき
親と子で違う処理をさせたいから、継承し、オーバーライドなり、オーバーロードをするのですね。

早速、以降の開発で参考にさせていただきます。

Answer 2

そもそもこれらのクラスがすべて同じパッケージに属していれば、protectedメンバにもアクセスできるので継承してpublicにしなくてもアクセスできます。なのでmethodAに関してはオーバーライドする必要がありません。
したがって、私ならこうします。

public class SuperClass {
    protected void methodA(int aParam) {
        // 何かしらの処理
        methodB();
        // 何かしらの処理
    }
    private void methodB() {
        // 何かしらの処理
    }
}

public class SubClassA extends SuperClass {

    // 個別の処理
}

public class SubClassB extends SuperClass {

    // 個別の処理
}

public class MainClassA {
    public static void main(String[] args) {
        SubClassA sub = new SubClassA();
        sub.methodA(1);
    }
}

public class MainClassB {
    public static void main(String[] args) {
        SubClassB sub = new SubClassB();
        sub.methodA(2);
    }
}

Comments

[Hanbe.T]

早速の返答ありがとうございます。
また、ソースコードまで書いていただき、ありがとうございます。

申し訳ありません。パッケージに関する情報を失念していました。
メインクラスとスーパクラス及びサブクラスは別パッケージになります。

やはり、子クラスに特別処理がないので、親クラスに直接アクセスするべきなのですね。