再利用しやすいクラス設計というものが分かりません。

Question

前提

現在、グローバルホットキーを設定できるクラスを、wxPythonを用いて作っております。
「修飾キー・ホットキー・キー押下時のイベント処理」
を1つのセットとして捉えて、それを作成するアプリごとに、外部から好きなだけ、複数のキーを設定できるようなクラスを考えています。

まだプログラミングを始めたばかりで慣れておらず、ネットでよく拝見する「読みやすい・再利用しやすい・修正しやすい」といったクラスの作り方というものが、自分は分かっておりません。
設計に関する本なども読みましたが、全てJavaで書かれており、あまり理解できませんでした。

宜しくお願い致します。

実現したいこと

• main()の中で実行する処理を定義できるようにしたい(TODOコメント2箇所の部分です。)
• もしくは 違う方法でのより良い書き方を知りたい

該当のソースコード

#! /usr/bin/env python3.9
import wx


class GlobalHotkey(wx.Frame):
    def __init__(self, modifier_keys: any, hotkeys: any) -> None:
        self.__modifier_keys: any = modifier_keys
        self.__hotkeys: any = hotkeys

        self.__root = wx.App(False)  # MEMO:移動禁止(wx.Appを作成してからsuperでインスタンス化する必要があるため)
        super(GlobalHotkey, self).__init__(None)

    def bind_hotkey(self, modifier_keys: any, hotkey: any) -> None:
        """ホットキーを登録"""
        id_hotkey = wx.NewIdRef(count=1)
        self.RegisterHotKey(id_hotkey, modifier_keys, hotkey)

        # HotkeyのEventHandlerをbind
        self.Bind(
            wx.EVT_HOTKEY, self.ShowMessage, id=id_hotkey
        )  # TODO: ここにホットキーごとの処理を定義し、呼び出し元からappendするようにしたい

    def start_thread(self) -> None:
        # インスタンス変数に渡された全てのホットキーをループ処理でバインド
        for (mk, hk) in zip(self.__modifier_keys, self.__hotkeys):
            self.bind_hotkey(mk, hk)

        GlobalHotkey(self.__modifier_keys, self.__hotkeys)
        self.__root.MainLoop()

    def stop_thread(self) -> None:
        self.__root.ExitMainLoop()

    # -------------------------------------------------------------------------
    # ホットキー押下時のイベント
    # -------------------------------------------------------------------------
    def ShowMessage(self, event) -> None:
        dialog = wx.MessageDialog(self, "ホットキーが押下されました", "イベントメッセージ")
        dialog.ShowModal()
        dialog.Destroy()


class HotKeyList:
    CTRL = wx.MOD_CONTROL
    SHIFT = wx.MOD_SHIFT
    ALT = wx.MOD_ALT

    def ASKKEY(self, keystr) -> str:
        """文字列リテラルをasciiコードに変換"""
        return ord(keystr)


if __name__ == "__main__":
    modifier_keys_list: list = []
    hotkeys_list: list = []

    hk = HotKeyList()
    modifier_keys_list.append(hk.CTRL + hk.ALT)
    hotkeys_list.append(hk.ASKKEY("B"))
    # TODO: ここにホットキーごとの処理を定義し、クラスへ渡せるようにしたい

    g = GlobalHotkey(modifier_keys_list, hotkeys_list)
    g.start_thread()

試したこと

自分なりにコードを書いてみましたが、
修飾キー・ホットキーに関しては、リストとループを使い
クラス外部(ex.main)からクラスへ定義することができました。
しかし、キー押下時のイベント処理をどうすればいいか分からず、そこで躓いてしまいました。(まさか、関数の処理自体をクラスへ渡す？なんてことをしたりするんでしょうか)
継承→オーバーライドなども調べてみましたが、有効な活用法が浮かびませんでした。

補足情報（FW/ツールのバージョンなど）

Windows11 64bit
python3.9

Answer 1

まずは設計が重要。「クリーンアーキテクチャ」や「DDD」を調べてみるといいですよ。
実装はデザインパターンが参考になるかと思います。今回のはコマンドパターンですね。
コマンドを事前登録しておいて、選ばれたコマンドを実行するパターン。
コマンドは、キーの組み合わせを辞書キーにして、コマンドを辞書値にしておけばいいかと思います。
(shift: bool, control: bool, alt: bool, キー: str) のタプルデータを辞書キーにすればいいかと思います。

Comments

[RedKai871]

大変参考になりました、クリーンアーキテクチャなどについても調べてみます。
ありがとうございました。