MyBot が多すぎるデータを保持している

[aiotter]

Cog に分けているということは，各機能を MyBot 自体に負わせずに別のファイルに切り出したいって話だと思うんだよ．
それなのに，例えば
https://github.com/Huyu2239/ExpandBot/blob/b49b41a79691d2078565a848102e828f2ca8ee82/main.py#L43-L49
ここに関しては，/cogs/mute.py でしか使わないデータが main.py:MyBot に定義されてる．
これはおかしくて， /cogs/mute.py の中で実装してあるべきだと思う．
このあたりの問題が複数積もった結果 #34 のような心配が生まれているのではないかな．

もし mute.py のなかで定義する方法がイメージできないのなら PR 出すから言って欲しい．

[Huyu2239]

https://github.com/Huyu2239/ExpandBot/blob/b49b41a79691d2078565a848102e828f2ca8ee82/cogs/on_message.py#L67
mute_dataはここでも使ってます

[aiotter]

いやそういう小手先の話じゃなくて，mute.py に関係する実装が on_message.py にもまたがってるっていうのが良くないと私は思う．

[aiotter]

@ikngtty と @kirisaki はどう思う？

[ikngtty]

コグのノリが未だによく分かんないから、ひとまずその辺は考慮しないで話すけど、ミュートに関するロジックは全部 Mute クラスみたいなところに置いて、mute_data は Mute クラスの中にカプセル化し、on_message() からは Mute クラスの提供する機能（not 変数）しか呼ばない、ってするのが良いと思う。

[Huyu2239]

なるほど
check.pyの関数をそれぞれ(mute.pyとかset.pyとか)に分けて、on_messageから呼び出すのはどうです？
#32 も解決できそう

[ikngtty]

コグ専属ではないプレーンな部品が増える感じかな。
見てみないとなんともだけど、なんとなく良い方向性のこと言ってる気はする。

[ikngtty]

このあたりの問題が複数積もった結果 #34 のような心配が生まれているのではないかな．

同意。
パーツに分けようとして分けきれてないんよね。データ（mute_data）まで含めて一つのパーツとして分けないと分けきれたことにならない。
この辺はクラスというものをちゃんと使えてるかにかかってくるな。クラスってデータ（＝メンバ変数）と処理（＝メソッド）をひとまとめにするものなんだけど、慣れてないと「データはこのクラスにまとめる」「処理はこのクラスにまとめる」みたいな作り方してしまいがちなんだよね。