Replies: 7 comments 48 replies
-
|
check はなんか d.py においてコマンドの実行可否を決める関数のことをそう呼ぶので、それで適切な命名だと思う |
Beta Was this translation helpful? Give feedback.
-
|
納得 |
Beta Was this translation helpful? Give feedback.
-
|
だとしたらモジュール名"checks"とかにするのが良いんかな |
Beta Was this translation helpful? Give feedback.
-
|
ソース見たところ、d.py の check として使うのではなくプレーンな関数として普通に呼んでる場面があるね。 こういうことするんだったらやっぱもっと丁寧な名前の方がいいかなーって気がする。 「d.py の check だけどついでにプレーンな使い方もする」ではなく「プレーンな関数だけど d.py の check としても使う」って考え方の方がいいと思うんよね。 |
Beta Was this translation helpful? Give feedback.
-
|
あ、取り上げてたか https://github.com/Huyu2239/ExpandBot/discussions/32#discussioncomment-828562 |
Beta Was this translation helpful? Give feedback.
-
|
staticmethodになってないのでアレな感じになってる感がありますね（selfが入るはずのところに代入しちゃっている） |
Beta Was this translation helpful? Give feedback.
-
|
それもアレなんだけど、そもそも staticmethod を使いたい場面ってそんな多くなくて、今回も間違いなくその場面ではないと思うんだよね |
Beta Was this translation helpful? Give feedback.
-
|
それはそう |
Beta Was this translation helpful? Give feedback.
-
|
各処理の check を一箇所に集めようっていうのがそもそも悪手っぽい。 |
Beta Was this translation helpful? Give feedback.
-
|
condition だよなあ |
Beta Was this translation helpful? Give feedback.
-
まって，ここなんなん？ なんで d.py の check の機能使ってないの？ slash commands と組み合わせられないとかなの？ |
Beta Was this translation helpful? Give feedback.
-
|
https://discordpy.readthedocs.io/ja/latest/ext/commands/api.html?highlight=check#discord.ext.commands.check |
Beta Was this translation helpful? Give feedback.
-
|
それです．check って名付けたからには当然知ってると思ってた． |
Beta Was this translation helpful? Give feedback.
-
|
展開するかどうか確認するときに設定を確認するから check ってつけちゃってました |
Beta Was this translation helpful? Give feedback.
-
|
そもそもこれぱっと見で check の集合に見えないんだよね。処理読めばちゃんと check なのかな。少なくとも命名は明らかに check っぽくないのだけど |
Beta Was this translation helpful? Give feedback.
-
|
コマンドと一対一で定義された関数を切り出してここでまとめておくっていう発想がなかったもんで池永さんの言いたいことが分かってなかった |
Beta Was this translation helpful? Give feedback.
-
|
そうそう。温度感の違いの源泉が、この関数名の解釈の違いにあったっぽいよね。 この関数名の付け方どうするかが実は一番の争点っぽい。 |
Beta Was this translation helpful? Give feedback.
-
|
本来は check は条件を端的に表した名前を付けてコマンドとは切り離して定義するのが最も良いと思う。っていうかわたしにはそもそも「特定の関数用の check を定義する」って発想がなかった。そういうことするなら無名関数使う |
Beta Was this translation helpful? Give feedback.
-
|
結局どっち？ @Huyu2239 |
Beta Was this translation helpful? Give feedback.
-
こっちの認識で合ってます |
Beta Was this translation helpful? Give feedback.
-
|
https://github.com/Huyu2239/ExpandBot/discussions/53#discussioncomment-829281 ここまで理解したところで分かってきたんだけど、 https://github.com/Huyu2239/ExpandBot/blob/f28edd6de2954a6bbb2a4a37cff8b61103f80353/lib/check.py#L31 これは d.py の check（＝実行するかしないかの判定）たりえるんだけど、 https://github.com/Huyu2239/ExpandBot/blob/f28edd6de2954a6bbb2a4a37cff8b61103f80353/lib/check.py#L45 これは匿名化するかしないかであって、d.py の check たりえないね。 |
Beta Was this translation helpful? Give feedback.
-
|
俺が書いた is_allowed_to_expand(posted_message) # mute を調べるやつ
is_allowed_to_be_quoted(quoted_message) # hidden と allow を調べるやつ
is_anonymous(quoted_message) # anonymity を調べるやつこのへん。 どう名付けるかなー。 |
Beta Was this translation helpful? Give feedback.
-
|
設定クラス群は設定の仕方の都合で分けつつ、Resolver はそれらの設定を元にどういう結論が出るかの種類で分ける（= |
Beta Was this translation helpful? Give feedback.
-
@aiotter ちなみにだけどサーバー設定もあって、設定の優先順位だの組み合わせだの色々考慮するみたいよ。 |
Beta Was this translation helpful? Give feedback.
-
|
ほなら個人設定とサーバー設定はクラス分けれそう |
Beta Was this translation helpful? Give feedback.
-
|
Mute（＝引用元ではなく投稿先のコンテキストを元にするやつ）だと、サーバー、チャンネルカテゴリ、チャンネル、投稿者、があるっぽい。 |
Beta Was this translation helpful? Give feedback.
-
|
ロールごとの設定もいずれ追加したいので、各種設定ごとに分けます |
Beta Was this translation helpful? Give feedback.
-
|
設定内容をどう取り扱うかシリーズなのに、embed_type や embed_color の設定取得だけ別扱いなのも俺的にはしっくりこないな。 |
Beta Was this translation helpful? Give feedback.
-
|
単一責任原則なんかも意識してるかなー。 |
Beta Was this translation helpful? Give feedback.
-
|
単一責任原則のもっと分かりやすい例がある記事ないかな。 |
Beta Was this translation helpful? Give feedback.
-
|
https://github.com/Huyu2239/ExpandBot/discussions/25#discussioncomment-830988
これも関連が深いね。「変更理由が Discord API の変更しかない部分を分けたい」と考えられる。 |
Beta Was this translation helpful? Give feedback.
-
|
「チェック処理関係をまとめる」って発想だとねー。「チェック処理」って一口に言っても色々ありえるから。関連の浅い処理を雑多に集めちゃう可能性あるよね。 https://qiita.com/ue_sho/items/59845b4891e12dfb3def |
Beta Was this translation helpful? Give feedback.
-
|
わかりやすかったです |
Beta Was this translation helpful? Give feedback.
-
https://github.com/Huyu2239/ExpandBot/blob/main/lib/check.py
class にメソッドしかなかったら class 作る必要ないと思います。関数だけトップレベルに書けばよさそう。あと「check」は大雑把すぎる命名なので「validator」とかにするかなあ
Beta Was this translation helpful? Give feedback.
All reactions