refactor: Charactorのタイプを型で表現する

[chun37]

voicevox_engine/voicevox_engine/metas/MetasStore.py

Line 31 in 18ce0a2

class Character:

talkかsingかの判別が結構な頻度で行われているので、メンバではなく型で持っても良さそう
Talkable や Singable などの mixin 方式でも、BaseCharactor を継承した TalkableCharactor や SingableCharactor を作る方式でも良さそう

[tarepan]

提案ありがとうございます！

メンバではなく型で持っても良さそう

👍️
一般論として、種別判定が多い場合にクラスレベルで分離させるという方向性はありと考えます。

（私の記憶の範囲では）Character はそこそこ色々なところで取り回されていると理解しています。
なのでメリット・デメリットも色々ありそうです。
そのため、導入の是非は draft PR を見て判断するのが望ましいと考えます。

もし @chun37 さんがよろしければ、実際にリファクタリングを行っていただき draft PR として出していただけるととても助かります！

その他コメント

リファクタリング issue 作成時は、issue テンプレート「Feature Request（機能要望・改善提案）」の利用が推奨です。

[chun37]

承知しました。
実装してみます。

[github-actions]

本 Issue は直近 30 日間で活動がありません。今後の方針について VOICEVOX チームによる再検討がおこなわれる予定です。

[Hiroshiba]

@chun37 その後いかがでしょうか･･･！ 👀

Characterの取り回しは結構いろいろやっているので単純に置き換えるのは難しいかもしれないので、１回実装してみる @tarepan さんの案が良さそうなのかなと思っています！

[chun37]

忙しくなって忘却してました…
今年のうちに出します

[github-actions]

本 Issue は直近 30 日間で活動がありません。今後の方針について VOICEVOX チームによる再検討がおこなわれる予定です。