if xxx is False

[ikngtty]

https://github.com/Huyu2239/ExpandBot/blob/c448dee6b9aa976d06a6081bd9dce1b78f5d36ae/cogs/on_message.py#L42-L43

msg_is_allowed = await self.bot.Check.allow(self.bot, message, msg)
if not msg_is_allowed:

とかの方がリーダブルやね。

cf. http://www.dcom-web.co.jp/technology/pgpoint1/

[ikngtty]

check って関数名が終わってる理由の一つもこれ

http://www.dcom-web.co.jp/technology/pgpoint1/

true と false のどちらがダウンロード可能なのか
変数名から推測できません。

[Huyu2239]

なるほど
変数名はそちらのほうがいいですね
checkの命名も悩んでます

[ikngtty]

checkの命名も悩んでます

これ、どうロジックを整理すべきかも含めてちょっと俺も考え中。細かいところまで中身把握できるまではちょっと保留。

[ikngtty]

変数名はそちらのほうがいいですね

ちなみに関数名も同様。
cf. http://tech.innovation.co.jp/2018/08/30/naming-of-functions-that-return-boolean.html