継承元を抽象メソッドにするか、仮想メソッドにするか意見をください

Question

前提・実現したいこと

Unityでゲームを作っているのですが、キャラのクラスで、抽象メソッドを使用するか、仮想メソッドにするか悩む部分が出てきました。
なるべくみんながミスなくコーディングでき、依存も少なくするには、どちらがいいか、意見を頂けますでしょうか。

コード例

Awakeから呼ばれるOnAwakeメソッドについて

抽象メソッドを使用した場合

class Base{
    void Awake{
        // 共通処理
        // ...
        OnAwake();
    }
    protected abstruct void OnAwake();
}
class Abc : Base{
    protected override void OnAwake(){
        // 個別処理
        // ...
    }
}

◆メリット
継承元コードを追わなくても、OnAwakeの存在を意識できる
◆デメリット
今後OnAwakeを使用しない形になった場合に、全継承先を修正する必要がある

仮想メソッドを使用した場合

class Base{
    void Awake{
        // 共通処理
        // ...
        OnAwake();
    }
    protected virtual void OnAwake(){
    }
}
class Abc : Base{
    protected override void OnAwake(){
        // 個別処理
        // ...
    }
}

メリット
継承先クラスは必要でなければOnAwakeを書かなくて済むので、
今後OnAwakeを使用しない形になった場合などに、修正が最小限になる
デメリット
継承元をちゃんと読まないと、OnAwakeの存在に気づけない

補足情報

単純にAwakeを継承する案もありましたが、base.Awakeの呼び出し漏れが怖いのでやらない方がいいかと考えました。また、実をいうと、個人的には仮想関数を使用する形の方がいいと考えていたのですが、「継承元に空メソッドを置くのが悪！」という話を聞いたので、怖くなって質問させていただきました。

以上になります。よろしくお願いします。

Answer 1

ちゃんとis-a関係が成立しているクラス関係なら、子孫全部共通した振る舞いがあり、場合によって継承先で追加の振る舞いを行いたい場合はvirtualにするし、子孫同士で同じ振る舞いをしたくない場合はabstractにします。

メリット・デメリットについて処理の共通化と修正工数の観点から考えているみたいですが、それは設計意図を無視して実装上の都合での選択になりがちなので、その処理の本来の意味を考えて選択すれば良いんじゃないでしょうか。

また、実をいうと、個人的には仮想関数を使用する形の方がいいと考えていたのですが、「継承元に空メソッドを置くのが悪！」という話を聞いたので、怖くなって質問させていただきました。

子孫に共通の処理がないのに継承だけを目的としたvirtualメソッドには何の利点もないので、そういう場合はabstractにしましょう。

Comments

[JrMogura]

ご回答ありがとうございます！

> 子孫全部共通した振る舞いがあり、場合によって継承先で追加の振る舞いを行いたい場合はvirtual

今回はこちらに該当するので、OnAwakeではなく、Awakeをvirtualにするのが妥当かなあと思います

> それは設計意図を無視して実装上の都合での選択になりがちなので、その処理の本来の意味を考えて選択すれば良いんじゃないでしょうか。

はっとさせられました。たしかに設計意図通りじゃないと読みにくいコードになりそうですね。アドバイスありがとうございます！！

Answer 2

やりたいことによってケースバイケースだと思いますが、.NetのライブラリではOnXxxxはvirtualで定義されているのでそれに従うのが指針の一つになるのではないでしょうか。
Control.OnClick(EventArgs) メソッド (System.Windows.Forms) | Microsoft Docs