チェック処理の中でメッセージ送信とかしてる

[ikngtty]

https://github.com/Huyu2239/ExpandBot/blob/f28edd6de2954a6bbb2a4a37cff8b61103f80353/lib/check.py#L1-L10

コマンドとクエリは分けた方がいい。（ここでは、メッセージ送信はコマンドで、実行可否結果の取得がクエリ）

cf. https://bliki-ja.github.io/CommandQuerySeparation/

"Check" は名前が明らかにクエリなのに、中でメッセージ送信とかされるのはかなりビビる。悪い意味で開発者の予想を裏切る。
開発者の予想を裏切らないためには、とにかく関数の名前で示したこと以外のことはしないのがいい。

[ikngtty]

あーでもこれ d.py の check はこういう想定なんかなー。

[aiotter]

そういう想定じゃないよ． d.py の checks は単に可否だけを見て，否なら Error を raise する．

[ikngtty]

調べたけどほんまや

[ikngtty]

Error でやってくれれば、この関数はちゃんと「チェックした結果を返す」ということになるので Check の名前通りで自然だし、コマンドをせずにクエリに専念できてるし、テスタブルやね。

[Huyu2239]

そうします