クラス名・変数名に迷ったら書き込むスレ。Part28 [無断転載禁止]©2ch.net

**ネミ子** · 2017/05/07(日) 18:01:52.03

クラス名、変数名のつけ方に悩んだら書き込むスレです。

命名規則や設計の善し悪しについて議論するのは基本的に禁止。

前スレ
クラス名・変数名に迷ったら書き込むスレ。Part27
http://echo.2ch.net/test/read.cgi/tech/1476572490/

**デフォルトの名無しさん** · 2019/03/21(木) 07:20:00.18

短くしたいなら漠然とした一般的な部分は削除候補になるので
max_parallel

**デフォルトの名無しさん** · 2019/03/21(木) 08:10:15.75

>>633
RunLimit

**デフォルトの名無しさん** · 2019/03/21(木) 09:29:35.49

外部からこの処理やってねって頼まれる場合なら
AcceptLimit？

**デフォルトの名無しさん** · 2019/03/21(木) 09:57:23.13

>>635
これ
calcXXXみたいなもん

**デフォルトの名無しさん** · 2019/03/21(木) 22:47:37.75

>>636
parallelはちょっと読みすぎやろ。

**デフォルトの名無しさん** · 2019/03/22(金) 00:02:52.39

え？proc_maxも最大プロセス数みたいな感じじゃないの
バッチ処理やバッファリングのサイズ上限みたいなイメージってこと？
むしろRunLimitあたり「魔法のランプは3回まで」みたいな回数限度で、同時であるニュアンスが落ちすぎだと思ってたわ
まあいくつか挙がった内から近いものがあれば選べばいいとは思うが

**デフォルトの名無しさん** · 2019/05/26(日) 22:08:46.49

ボタンをｎ回押すと実行されるイベントがあります。
（回数ｎは別途指定します）
ｎ回押されたとき、その時刻を設定ファイルに記録するのですが
そこに使うキーとなる文字列をお願いします

foo: 1970-01-01T09:00:00
みたいな感じです

**デフォルトの名無しさん** · 2019/05/26(日) 22:36:33.72

button_pushed
じゃアカンの？

**デフォルトの名無しさん** · 2019/05/26(日) 22:43:11.01

>>642
> foo: 1970-01-01T09:00:00
> みたいな感じです
それでなんの問題があるんだ？
あとスレ違いだし

**デフォルトの名無しさん** · 2019/05/26(日) 22:43:36.79

ありがとう
特定回数という意味はあえて入れないということかしら

**デフォルトの名無しさん** · 2019/05/26(日) 22:56:58.41

>>645
ボタンn回押すのが何かのコマンドか何かであるなら、それを名前に使った方がいいね。

そういうのじゃなく、本当に純粋にボタンがn回押されたタイミングを記録したいだけだとすると
なかなか適切なのが思いつかんなあ

**デフォルトの名無しさん** · 2019/05/26(日) 23:12:23.37

>>646
具体的なイベントもコールバックで指定されるんで
なにかが起こるとしか言えない感じ

**デフォルトの名無しさん** · 2019/05/26(日) 23:27:10.28

>>647
そういう漠然としたものに名前を与えろっていうのも無理筋な気がするw
まあOnEveryNthClickとか？

**デフォルトの名無しさん** · 2019/05/26(日) 23:45:04.89

あーなるほど
ｎ回ってのをそのまま名前にしちゃうという手があったか
いちいち特定回数みたいな日本語に直して考えてたわ

**デフォルトの名無しさん** · 2019/06/22(土) 18:33:43.34

Goldfish.AzureGoldfishIsMany() //アジュール色の金魚が多いときに真
のように、形容詞＋クラス名と同じ名詞＋ is ＋形容詞という命名の関数があったとして
クラスと関数でGoldfishがダブってるわけだけどどうしてる？
たとえば関数名のほうを消して Goldfish.AzureIsMany() にできるか？みたいな

**デフォルトの名無しさん** · 2019/06/22(土) 19:02:05.23

AzureColorIsManyじゃダメなん？
あと多いってのがアバウトじゃね？いいの？

**デフォルトの名無しさん** · 2019/06/22(土) 19:33:14.17

このダブってしまうケースに合うとしばらく思考停止してしまう

**デフォルトの名無しさん** · 2019/06/22(土) 20:06:21.99

このケースだからそう思うのかもしれないけど、
IsManyを判定できる関数がこのクラスに居るのが違和感
Goldfishの集まりを持っているクラスが他にあって(水槽とか)
FishTank.ContainsManyAzureGoldfishes()
みたいな感じで定義したい
Goldfishクラスには
Goldfish.IsAzure()
関数があればよいはず

まあ単に X.IsAandB() みたいな命名でよいケースもあるような気もする

>>650
「多い」の実装がどうであるかは呼び出し側は気にしたくないんだろう多分

**デフォルトの名無しさん** · 2019/06/22(土) 21:43:09.00

金魚が「Goldfish」なことに違和感。ｗ
直訳？
誰や、んな訳語をつくったんは！

**デフォルトの名無しさん** · 2019/06/22(土) 22:34:20.22

>>650
まあ既に指摘されてるけど、集まりを評価する関数を要素のクラスに持たせるのが
そもそも変だねw

何か事情があるにしても、HasManyAzureとかIsRateOfAzureHighでいいんじゃないか。
むしろいちいちGoldfishと断る意味が分からん

**デフォルトの名無しさん** · 2019/06/22(土) 23:38:09.91

>>653
なるほど、同じ文意になる文章がないかと考えてみるのは良さそう

>>651>>655
形容詞（この場合はAzure）に対してmanyと言って良いのか悩んだんだ
多いのはあくまでも金魚の方だから

**デフォルトの名無しさん** · 2019/06/23(日) 00:50:33.60

>>656
azureは他の色と同様名詞の用法もあるみたいだよ
辞書的には不可算名詞みたいだけど、たぶん他の色と同じように場合によってはs付けても大丈夫だと思う

**デフォルトの名無しさん** · 2019/07/17(水) 15:09:38.38

Dataを継承して値の変更を検出できるようにしたDataのクラス名

**デフォルトの名無しさん** · 2019/07/17(水) 19:07:28.81

>>658
Data_notifiyable

**デフォルトの名無しさん** · 2019/07/18(木) 01:58:50.63

>>658
変更検出の方法に基づいて考えたい気がする。
変更時に通知が出るなら >659 とかでいいけど、原本を控えておいて比較によって検出するような形だとまた違ってきそう。

**デフォルトの名無しさん** · 2019/07/18(木) 12:59:08.35

Observerとかでええんちゃう

**デフォルトの名無しさん** · 2019/07/18(木) 20:10:35.28

>>660
検出するのがオブジェクト自身なら、その方法をクラス名に含めないほうがええやろ。
実装の詳細はできるだけ隠すべき。

>>661
それは、Dataオブジェクトの外側の何かやろ。

**デフォルトの名無しさん** · 2019/07/19(金) 12:28:07.16

変更を検知する方法を外部に提供するのか
自分で検出して自分でなにかするのかによって違うわな

**デフォルトの名無しさん** · 2019/07/20(土) 00:09:53.37

ChangeDetectableData

**デフォルトの名無しさん** · 2019/07/20(土) 09:39:17.20

DetectableなDataをChangeするになってしまわないか？

**デフォルトの名無しさん** · 2019/07/20(土) 10:21:23.54

そもそも値の違いを検出できるクラスを作ること自体が設計としてまずいんじゃねえの

**デフォルトの名無しさん** · 2019/07/20(土) 11:37:02.08

そこの議論は一応スレチなので…
Angular詳しくないけどこういう枠組みはあるもよう
https://qiita.com/lacolaco/items/523d96ddbfe55c4e6949

で、実質は Change-Detectable な Data かも知れないけど、
周りからは DataChangeDetector とでも呼ばれるべきという考え方はあるかもしれない

**デフォルトの名無しさん** · 2019/07/20(土) 14:56:22.84

もう素直に DataWithChangeDetect でよくね？

**デフォルトの名無しさん** · 2019/07/20(土) 18:43:27.60

それならDataWithChangeDetectionかな
スレチで我慢してたけどDataとChangeDetectorを同じクラスにするのは微妙だから、名前にWithとか入って微妙感が出るのは自然な成り行きと思う

>>665
changeには名詞と動詞があるからそうも読めるけど、change detectionという一般名詞があるから誤読されにくい
クラス名が現れるべき文脈でメソッド名と間違われることなんて実際少ないから、混同への配慮よりも俺なら英語として自然であることを選ぶ

**デフォルトの名無しさん** · 2019/07/20(土) 21:10:11.95

質問者もやる気なさそうだから真面目に考えるだけ無駄だと思うよw

だいたい、こんな命名をそのクラスが必要になる背景も、
「検出」の具体的な意味も示さずにやれとか言う時点で何をかでしょ

**デフォルトの名無しさん** · 2019/07/20(土) 22:33:32.11

普通の実装ならDetectableを継承したDataになるはず

**デフォルトの名無しさん** · 2019/07/20(土) 22:42:52.16

普通ではないね
むしろ勘違いオブジェクト指向っぽいw
その多態、本当に必要か？
いらないでしょw

**デフォルトの名無しさん** · 2019/07/21(日) 15:05:16.00

全ての品番と型式が対になったリスト
型はC#のDataTableです

**デフォルトの名無しさん** · 2019/07/21(日) 16:59:41.32

>>673
品番はキーだろうから普通にModelsとかModelNumberとかでいいんじゃね

**デフォルトの名無しさん** · 2019/07/21(日) 17:18:39.47

>>672
モデルビュー的なことをするならそんなにおかしくもない。
「多態」が目的じゃないから。

**デフォルトの名無しさん** · 2019/07/21(日) 18:11:15.08

>>675
どう考えても多態そのものだと思うけど、
それはともかく特殊な条件付けたら何でも正当化できるよw

もとの質問にそんな条件なんか付いてないのは自明でしょ

**デフォルトの名無しさん** · 2019/07/21(日) 18:22:06.33

>>671-672はイチャモンつけたいだけだからスルーでいいよ
そのために
> 命名規則や設計の善し悪しについて議論するのは基本的に禁止。
の注記あるんだし

**デフォルトの名無しさん** · 2019/07/21(日) 18:28:01.59

最初に言いだした>>666俺だわすまん

**デフォルトの名無しさん** · 2019/07/21(日) 20:14:42.06

>>677
これもういうの何度目？
この手の馬鹿は自分もそのお仲間である自覚が皆無のおめでたい馬鹿

**デフォルトの名無しさん** · 2019/07/21(日) 20:20:53.26

ついでに言えば、こんな過疎ってるスレ（もう今ではこの板の大半のスレがそうだが）で
「スレ違いガー」なんてそれこそ言い掛かりでほとんど無意味。

お前こそ他人にイチャモン付けたいだけだろ馬鹿w
今時2chにいるなんてもういい歳こいてるんだろうから、もうちょっと合理的かつ柔軟に考えたらどうなのかね
派生的な話題を許容しても誰も困らないだろう。

**デフォルトの名無しさん** · 2019/07/21(日) 20:38:37.46

混乱の元
そんなに気に入らないならまずは次スレを立てる時にテンプレから外しな

**デフォルトの名無しさん** · 2019/07/21(日) 21:02:34.97

新参か？
実際に設計の善し悪し含めた話OKの命名スレが立っても、そのスレは全く活用できなかったという
実績が確実に残ってるのよね

そこんとこ分からない程度のオツムなら、設計の善し悪しなんか話すオツムなんか当然ないんだからやめとけ

**デフォルトの名無しさん** · 2019/07/21(日) 21:21:18.61

そりゃ名前聞きたいだけなのに生半可な知識の奴に設計ガーとか言われても困るだけ
バカな言い合いで賑わうぐらいなら過疎ってた方がまだマシだよ

**デフォルトの名無しさん** · 2019/07/21(日) 21:51:52.82

>>674
ありがとうございます

**デフォルトの名無しさん** · 2019/07/21(日) 23:43:53.38

>>682
意味が分からない。
自分で自分が何をっているのか理解しているのか

>>683
こういう馬鹿、言ってる傍から自分で自分の言ってることに矛盾してる自覚もないのな。

**デフォルトの名無しさん** · 2019/07/21(日) 23:54:23.20

他人をバカにしたいだけのヤツは、天に召されていただきたいものだ。

**デフォルトの名無しさん** · 2019/07/21(日) 23:57:14.14

>>672みたいなレスは

-　誰かの参考になる可能性がある
-　2chの現状を考えれば誰の迷惑にもならない

のに対して>>677みたいのは

-　誰の役にもたなない
-　それどころか一部の人間の気分とスレッドの雰囲気を悪くする

どちらが悪質で罪深いか言うまでもない。
毎度のことだけど、2chに限らずネットではこの程度のことに考えも及ばないバカに限って
本人は自分が正しいと思い込んでる

なぜならこういう馬鹿は内省的になるのが怖くて仕方がなく、
内省的な自分を抑圧する手段の一つとして他人に言い掛かりをつけている自分に無自覚だからだ

**デフォルトの名無しさん** · 2019/07/22(月) 00:00:52.53

そんでこれが一番重要なことだが、このスレのテンプレにもあるような
ローカルルールは、結局その手の馬鹿に体のいい言い訳を与えているだけ。

「他人を批判したい」という卑しいだけの動機を抑圧して「正しいこと」に変える欺瞞のね

**デフォルトの名無しさん** · 2019/07/22(月) 00:26:05.35

こーゆーのずっと繰り返してpart28なのよ
んでご希望の設計込みのスレ立てて分離したけど、そっちに居着かないのよ
言い出しっぺもね

どうしても納得いかないならもう一回スレ立てたら？
このスレのルールを変えようとすることだけが目的の馬鹿がまた来たなあとしか思わんね
どうせまた消えるに1億ペリカ

**デフォルトの名無しさん** · 2019/07/22(月) 01:04:48.79

>>687
> -　誰かの参考になる可能性がある
ない。
他人のやりかたを「勘違いオブジェクト指向」呼ばわりするようなクソ発言が参考になんかなるか。

> -　2chの現状を考えれば誰の迷惑にもならない
実際に迷惑なんだよ！

**デフォルトの名無しさん** · 2019/07/22(月) 04:51:07.44

>>687
> 本人は自分が正しいと思い込んでる
ブーメランすぎるw

**デフォルトの名無しさん** · 2019/07/22(月) 08:57:03.38

>>682
(どうせ使われないだろうけど)って皮肉がわからん新参か？

**デフォルトの名無しさん** · 2019/07/22(月) 11:07:52.75

>どうせ使われないだろうけど
スレ内ヒット>>692

>どうせ
スレ内ヒット>>488>>689>692

>使われない
スレ内ヒット>>102>>103>>692

>だろうけど
スレ内ヒット>>375>>574>>692

どの言葉も話の流れにヒットしないんで>>692が創造主になってるんだが
皮肉の元が真面目に分からん

**デフォルトの名無しさん** · 2019/07/22(月) 11:25:07.67

混乱の元
そんなに気に入らないならまずは次スレを立てる時にテンプレから外しな(どうせ使われないだろうけど)

**デフォルトの名無しさん** · 2019/08/05(月) 16:15:13.02

1つだけ有効な値が入っていてそれ以外は0になる配列ってなんて命名すればいいだろうか？
例えば、
{0, 0, 1, 0, 0, 0, 0} これはOK
{0, 0, 0, 0, 0, 0, 1} これもOK
{0, 0, 0, 1, 1, 0, 0} これはNG

**デフォルトの名無しさん** · 2019/08/05(月) 16:43:27.68

それがナニを意味するか、それを名前に入れないと意味ないのでは？
それがナニを意味するかはあんた以外誰も分からないのでは？

**デフォルトの名無しさん** · 2019/08/05(月) 18:58:37.11

>>695
array_valid_only1

それっぽい単語を並べるしかないのでは？

**デフォルトの名無しさん** · 2019/08/05(月) 19:12:10.92

既に有効が1つある状態で別の場所が有効になるケースはあるの？
{0, 0, 1, 0, 0, 0, 0} これが
{0, 1, 0, 0, 0, 0, 0} これになるケース
それとも最初はゼロフィルで1度有効の場所が決まれば不変？

**デフォルトの名無しさん** · 2019/08/05(月) 19:28:02.73

「１つだけ有効な値がある配列」ではなく
「ある処理に対して有効な配列」という方向での命名はどう？

**デフォルトの名無しさん** · 2019/08/05(月) 21:18:13.38

>>695
論理回路がらみでone-hotっていう用語があるよ。ググって。

**デフォルトの名無しさん** · 2019/08/05(月) 21:45:25.18

意味ないってそんなの
定数1に名前を付けろと言ってるのと同じだから

だから、1がナニを意味するのか分かんなきゃ名前なんか付けようがないだろうにw

**デフォルトの名無しさん** · 2019/08/05(月) 23:37:04.95

具体的な用途がないと型に名前を付けられないというのも乱暴な話だな
QueueやTupleのような単純なデータ構造だってそれが役に立つなら型と名前が与えられるべき
そういう場合、用途は変数名が表せば充分

**デフォルトの名無しさん** · 2019/08/06(火) 00:17:40.47

何その意味不明な意見

**デフォルトの名無しさん** · 2019/08/06(火) 00:22:42.23

この人、QueueやTupleって名前が何の意味も表してないと思ってるのかな

**デフォルトの名無しさん** · 2019/08/06(火) 00:32:01.73

表してるに決まってるだろ
逆に聞くが質問者がいう配列は何も表してないと思うのか？

**デフォルトの名無しさん** · 2019/08/06(火) 00:35:14.30

だから何でそうなるのw
何かを表しているはずだと思ってるからそれを言ってくださいと言ってるんでしょw
もう大丈夫かマジw

**デフォルトの名無しさん** · 2019/08/06(火) 00:38:42.21

だからね、「一週間の日数を表す定数の名前を付けてくれ」って質問はありだけど、
何を意味してるのか伏せたまま「定数7に名前を付けてくれ」って質問に意味があるとか
思うわけ？

あるわけないじゃんww

**デフォルトの名無しさん** · 2019/08/06(火) 00:50:02.95

少なくとも、あれは、one-hot encoding という立派な名前のついた状態表現方法。
ただの数字と一緒にするのは無知。

**デフォルトの名無しさん** · 2019/08/06(火) 01:02:24.48

特定の問題を解く上で便利なデータ構造があるなら、それが695の説明ほぼそのままの抽象度の高いものであってもいいだろ
自然物や既知の概念に縛られる必要はない
データの特性が明かされている以上、定数7の例示は過度の単純化をしていて相応しいとは思えない
抽象性の高いままに案を提示してくれている人々に対して「意味ない」とは俺は思わん

**デフォルトの名無しさん** · 2019/08/06(火) 08:53:55.01

695です

このデータ構造は、OneHot〇〇って命名するのが適切そうなのでそうします
ありがとうございます

>> 698
そのケースはあります

**デフォルトの名無しさん** · 2019/08/06(火) 10:53:14.30

>>708
だから特定の問題領域で何と呼ばれるか、そんなの何の意味もない
彼の問題領域でそれが何を意味しているか、それが重要。
当たり前でしょ

そもそも配列がベクトルか、行列の列や行を表しているのか、数列か、
あるいは数学的な意味は持たないただの集合か、状況によって
配列そのものがまったく違った意味を持つ

もちろん抽象度が高い配列の特定の状態（例えば全部の要素が規定値とか）が
汎用的な意味を持つ場合もあるだろうし持たない場合もあるだろうが、
普通に考えればこんなのか後者に決まってる

>>709
抽象度が高いのが悪いとか誰も言ってないの。
重要なのは何を意味しているのかだと何度言わせるの
大丈夫かマジで

**デフォルトの名無しさん** · 2019/08/06(火) 15:23:01.65

>>708 で終了だろ
何をグダグダ言ってるんだよw

**デフォルトの名無しさん** · 2019/08/06(火) 20:33:38.37

>>711
名前の提案ができないのなら、黙ってればいいんだよ。
できない言い訳をするスレじゃないから。

**デフォルトの名無しさん** · 2019/08/06(火) 21:44:11.23

勤勉な無能という奴よ

**デフォルトの名無しさん** · 2019/08/07(水) 20:46:09.76

抽象的と言うか
外から見た役割で命名しろというのは大前提も大前提だろ

ただ、その内部処理を書いてるときとか
実装べったりな命名やら、類似処理で使える汎用的なプリフィクス/サフィックスやらが欲しいときも当然あるさ

**デフォルトの名無しさん** · 2019/08/19(月) 09:15:28.15

質問者は
OneHotXXX
↑こういうのの「OneHot」という言葉が欲しかったんでしょうに。

**デフォルトの名無しさん** · 2019/08/19(月) 09:47:08.91

そんなのなら素直にValidってやった方いい
意味不明より抽象的な方がミスリードの心配がなく情報量も多いから

**デフォルトの名無しさん** · 2019/08/19(月) 09:58:34.23

>>716
いきなりどうした？
既に質問者（と思われる人）は
> このデータ構造は、OneHot〇〇って命名するのが適切そうなのでそうします
> ありがとうございます
ってレスしてるよ

**デフォルトの名無しさん** · 2019/08/29(木) 01:46:53.52

Analysisが長いのでAnalにしようと思います

**デフォルトの名無しさん** · 2019/08/29(木) 03:45:05.82

pencilが長いのでpenisにしようと思います

**デフォルトの名無しさん** · 2019/08/29(木) 07:54:05.23

penisが長いのでinsertしようと思います

**デフォルトの名無しさん** · 2019/08/29(木) 09:51:10.58

insertまでできるんだったらmagnumを使おうと思います

**デフォルトの名無しさん** · 2019/09/04(水) 05:26:03.52

商品であれば取扱開始日と終了日まで、アカウントであれば有効化日と無効化日を表す変数名で悩んでます。
今はとりあえずstartdateとenddateにしてるんですが、どんな名前にするのがいいでしょうか？
validateddateとinvalidateddateのほうがいいですか？

**デフォルトの名無しさん** · 2019/09/04(水) 07:08:03.51

>>723
regist_date、cancel_dateとか

**デフォルトの名無しさん** · 2019/09/04(水) 09:29:15.85

registなんて英語はない

**デフォルトの名無しさん** · 2019/09/04(水) 10:00:04.36

Active, Disable？
取扱開始日と有効化日、終了日と無効化日を同じ変数名で？
それなら商品の方が微妙になるか

**デフォルトの名無しさん** · 2019/09/04(水) 10:06:44.94

期間っていう構造体かクラスのメンバーじゃないの？

**デフォルトの名無しさん** · 2019/09/04(水) 10:28:30.97

>>723
いつでもそうできるわけじゃないだろうけど、開始日/終了日を一組にしてしまえば、

DealingSpan.First
DealingSpan.Last

とかできるはず。

**デフォルトの名無しさん** · 2019/09/04(水) 12:46:50.07

下手に微妙な名前をつけるよりstartdate, enddateで一般化したほうがいろいろ具合がいいという側面もある
個人的にはbegindateの方が好き
型が日付なのか日時なのかわからない名前は嫌

**デフォルトの名無しさん** · 2019/09/04(水) 14:03:19.01

beginning dateだな

**デフォルトの名無しさん** · 2019/09/04(水) 14:12:20.29

次に読んだ時に、何が初まるの？とはないだろうか？

**デフォルトの名無しさん** · 2019/09/04(水) 19:40:52.08

>>723
終わりのほうはexpired一択。
始まりはなんだろ？対応させるならsubscribedかな？

ただの開始と終了じゃなく、なんか有効期間的な範囲だったら。

**デフォルトの名無しさん** · 2019/09/04(水) 19:46:59.42

>>732
現時点で過去とは限らないのでそういうのはどうかと思うよw

**デフォルトの名無しさん** · 2019/09/04(水) 21:15:39.20

難しいな
他動詞なら過去分詞を受動態として無造作に使えるけど、自動詞では、時制が過去かどうかはともかく、完了相が前に出るということだろうか
expire dateでは文法的におかしいし、expiringも進行相だし、素直にexpiryかexpiration dateしかないのか

**デフォルトの名無しさん** · 2019/09/04(水) 23:09:38.87

>>733
>>734のいう感じで、過去分詞のつもり。
原則、命名で過去形なんか使わん。
自動他動は気にしなかったけど、そこはどっちでもよくない？

英語は言うほど詳しくないんだけども。ｗ