Presentationが_implではなくRPAに依存するようにする

[Apricot-S]

概要

各Presentationは現在_implの BrowserBase, MessageQueueClientBase, PresentationCreatorBase をメンバーに持っているが、できればRPAをメンバーに持つようにする ただし、RPAのプライベートメンバーへのアクセスを避けつつ外部には非公開にする必要がある(要設計)

なぜ必要か

• 現在の実装だと利用者はRPA.waitからしかPresentationBaseを受け取れず、自分で生成できない
• RPA.close()にてBrowserBase, MessageQueueClientBaseを終了してもPresentationからの参照が残っているため _implのクラスの__del__が呼ばれない

To Be

Presentationクラスの__init__に_implのクラスではなくRPAクラスを与える RPAはPresentationに各メンバーを公開する手段を持つ(フレームワーク利用者には非公開にする)

[Apricot-S]

PresentationBase が RPA のプライベートメンバにアクセスするプライベートプロパティを持っていればよさそう そうすれば既存のコードの変更も不要になるし、Ruff の警告も 1 箇所ずつの計 2 箇所で済む

[Apricot-S]

PresentationBase が RPA のプライベートメンバにアクセスするプライベートプロパティを持っていればよさそう そうすれば既存のコードの変更も不要になるし、Ruff の警告も 1 箇所ずつで済む

この場合の変更範囲

• PresentationBase の__init__ の実装および docstring
• PresentationBase にプライベートプロパティを追加(ここは警告が出るが無視する)
• ***Presentation の__init__ の実装および docstring
• PresentationCreator の引数

テストは変更不要のはず(要確認)

[Apricot-S]

変更した場合の影響範囲について

RPA と Presentation*** が循環参照になる RPA.wait() は本来別のクラスかトップレベルの関数になるべきかもしれない

とりあえず Presentation*** の import をwait() メソッド内に移動すればエラーは起きなくなる 本格的な変更は一旦後にする

[Apricot-S]

疑問

RPA クラスを経由すると browser と message_queue_client が None でないか毎回確認が必要となるが設計として正しいか? 一旦ローカル変数に受けるべき?

[Apricot-S]

https://github.com/Apricot-S/majsoulrpa/commit/aa21233e859fb4929000cf4edadcfb1ca7159bb1 にて対応