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

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

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

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

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

**デフォルトの名無しさん** · 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のいう感じで、過去分詞のつもり。
原則、命名で過去形なんか使わん。
自動他動は気にしなかったけど、そこはどっちでもよくない？

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

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

いやexpiredは形容詞だと思うけど、どっちにしろ
現時点までのある時点で「期限切れ」になってないものに使うのは変だと思う。

**デフォルトの名無しさん** · 2019/09/05(木) 06:29:17.88

>>734
英語として正しいことが目的じゃなくて、わかりやすい変数名でしょ
文法的に正しくても、一般的でない単語や言い回しだと読む人が混乱するんじゃね？
まあ、誰が読むのかって定義は必要かも知れんが

つまり、YukoukaDate,MukoukaDateにしろ

**デフォルトの名無しさん** · 2019/09/05(木) 08:59:34.49

開き直るならyukoukabi、mukoukabiもありだと思うよ
ただし和英混合ありなしとヘボン式なのか何なのかは厳格に
それで一貫してるシステムは割と開発しやすかった

**デフォルトの名無しさん** · 2019/09/05(木) 22:54:21.19

>>736
「過去分詞」って知ってる？ｗ

**デフォルトの名無しさん** · 2019/09/05(木) 23:39:58.60

過去分詞の形容詞的用法って懐かしいよな
ちなみに未来完了形でwill have expiredという活用になることもある
いずれにしても話者の視点からは「失効した日」という意味になるから
おかしいと思うのは同感だ

**デフォルトの名無しさん** · 2019/09/05(木) 23:44:45.61

おまえらexpireで盛り上がりすぎ。
質問者いつ出てくるんだよ。

**デフォルトの名無しさん** · 2019/09/05(木) 23:58:09.84

思い…出した！
obsolete dateだわ
対義語はeffective date
我ながら見事な回答と言うしかないが、日本人エンジニアからはインテリぶってると思われる諸刃の剣
素人にはおすすめできない

**デフォルトの名無しさん** · 2019/09/06(金) 01:43:27.95

>>739
知ってるよ。
っていうか辞書引いてそれ言ってるのか

>>740の言う通りというか、いやちょっと違って、
もともと形容詞的な用法から派生したんだと思うが、今では独立した形容詞と見なされている単語だよ

expiredを過去分詞として使っているとしても（もちろん受動じゃなくて完了の意味で）
どっちにしろ現時点で過去になってない期日を表す言葉としては不適切だ

もう質問者さんどうでもいいみたいだけど、
こういう面倒な問題をスキップするためにも、>>729に書いたみたいに
可能なら期間を表す型を定義しちゃったほうがシンプルだね

**デフォルトの名無しさん** · 2019/09/06(金) 01:45:26.86

>>729じゃなくて>>728だった

**デフォルトの名無しさん** · 2019/09/06(金) 01:48:50.02

意味もなく頭に星の名前を付ける

**デフォルトの名無しさん** · 2019/09/06(金) 11:58:03.50

URIが与えられて、そのURIが表すリソースの種別を判定するのですが、
例えば、Aリソース、Bリソース、Cリソースなど。
各種別ごとに下のようにメソッド用意するのもあれなんで、
bool isAResourceUri(Uri uri);
bool isBResourceUri(Uri uri);
bool isCResourceUri(Uri uri);

一つにまとめて
解析結果を表すクラス parseUri(Uri uri); //

みたいな感じにしたいです。解析結果を表すクラスのクラス名をお願いします。
parseUriの名前も変えた方がいいならお願いします。

**デフォルトの名無しさん** · 2019/09/06(金) 12:39:37.74

ResourceType

**デフォルトの名無しさん** · 2019/09/06(金) 16:03:43.23

parseはあかん
ResourceTypeOf(URI uri);

**デフォルトの名無しさん** · 2019/09/06(金) 18:14:38.78

>>746
与えられるのが文字列ならparseでいいけど既にUri型になってるならparseは違和感ある
UriクラスにgetResourceType()みたいなメソッド追加すればいいんじゃね？

**デフォルトの名無しさん** · 2019/09/06(金) 19:08:30.01

そうですか、型はUriですね。もちろん、文字列にして内部でUriにしてもいいですが。
で、そのUriのパスのフォームからリソースタイプを判定するのですが。
Uriはビルトインクラスなので、メソッド追加できません。拡張メソッドみたいのもありませんし。

**デフォルトの名無しさん** · 2019/09/06(金) 19:13:06.97

後、解析結果なんですが1つのリソースタイプだけじゃなく複数の情報が返されるのです。
たから、現状クラス名がUriParsedResultとかにしてて..

後parse以外にanalyzeとか思いつきましたがなんか大袈裟というか

**デフォルトの名無しさん** · 2019/09/06(金) 19:15:06.10

最初のbool isAResourceUri
で、返される情報は１つだけみたいな誤解を与えてしまいましたすみません

**デフォルトの名無しさん** · 2019/09/06(金) 19:59:09.78

web全然知らんけど、要するに実際にファイルをwebから拾って中身を見て解析するわけじゃなく、
単純にパスの拡張子を見るだけってこと？

そうならそこを強調しないとまずいよねたぶん。知らんけど。
適当な名前空間に収まってる前提で
ResourceType PathParser.GetType(Uri uri)

**デフォルトの名無しさん** · 2019/09/06(金) 20:24:08.80

そうですね。実際にWebから中身拾ってくるわけじゃありません。例えば
http://hoge.com/users/[userId]
このUriならuserを表すuri
http://hoge.com/tweets/[tweetId]
このUriならtweetをあらわす

これを解析してどのリソースを表すかと、例えばuserを表すuriなら抜き出したuserIdも返す感じです

**デフォルトの名無しさん** · 2019/09/06(金) 23:54:53.62

>>750
> Uriはビルトインクラスなので、メソッド追加できません。
言語がわからんけど（Javaのfinal classとかC#のsealed classとかで）派生できないようにされてるの？

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

>>754
これ、そもそもURIはリソースを指しているんじゃなくて、
アプリケーション（？）の種類とそのパラメータを表してるんじゃないの？

その意味じゃ全然URIじゃないねw

**デフォルトの名無しさん** · 2019/09/07(土) 02:36:02.55

extractResourceType

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

ResourceUri ResourceUri.of(Uri uri)

ResourceUriはtypeやidなどを持つ

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

>>756
RESTのURIじゃないのかな

**デフォルトの名無しさん** · 2019/09/07(土) 17:18:57.36

getURIInfo

これなら、いろいろふんわりやから気にならんやろ。ｗ

**デフォルトの名無しさん** · 2019/09/07(土) 17:42:43.45

getSomethingAbout(uri)

**デフォルトの名無しさん** · 2019/09/07(土) 21:13:20.29

>>759
RESTのUriはあんな感じですよね。
>>760
ふんわりしてていいかもですね。

**デフォルトの名無しさん** · 2019/09/14(土) 19:11:22.43

A, ファイルをバイナリとして開いて検索
B, 起動中のプロセスのメモリを検索

検索部分は共通した処理だから同じクラスでやりたい
この場合はクラス名何がいいかな？
MemorySearchとかだとBだけになるし
BinarySearchだとメモリもバイナリでいいの？ってならないかな？

**デフォルトの名無しさん** · 2019/09/14(土) 20:18:51.13

>>763
class BinarySearch {
public BinarySearch(String FileName){ … }
public BinarySearch(Pid ProcessId){ … }
…
}
みたいな感じでいいんじゃね？

**デフォルトの名無しさん** · 2019/09/14(土) 20:53:18.08

Binary searchは「二分探索」って意味になるからそれは避けたほうがよいと思う

**デフォルトの名無しさん** · 2019/09/14(土) 21:20:28.76

>>763
検索対象のデータはストリームみたいに抽象化するんだろうから、
「何から」検索するかではなく「何を」検索するかに注目して
名前を付けた方がいいんじゃない？

だとするとざっくり検索って言われてもな感じ。
これだとFinderとかSeekerとかぐらいしか言いようがないよね

**デフォルトの名無しさん** · 2019/09/14(土) 22:13:28.42

>>765
ああ、言われてみればそうだな

**デフォルトの名無しさん** · 2019/09/14(土) 22:16:17.66

Seeker
Searcher
SearchEngine

**デフォルトの名無しさん** · 2019/09/15(日) 00:06:01.47

BinDataSearch

**デフォルトの名無しさん** · 2019/09/15(日) 10:33:36.53

ありがとう

A, バイナリエディタの検索部分
B, メモリエディタの検索部分
検索部分だけのプロジェクトを作ってAとBに使い回すなら
両方で違和感の無い名前がいいよなってのが経緯

「何を」と言われると
　A, 修正箇所
　B, 値の変動に応じて絞り込み検索してアドレスを特定
こうなるから同じにせずに分けた方がいいのかなと思えてきた
でもやってる事は同じだから共通化させたいけどこういう時は分けるもんなんかな？

**デフォルトの名無しさん** · 2019/09/15(日) 11:24:36.57

>>770
よほど特殊な機能を持ってるなら別だけど単なる検索機能だけならコード量もたいしたことないだろうから俺なら別々にしておくと思う

**デフォルトの名無しさん** · 2019/09/15(日) 12:41:40.84

共通部分をis-aなりhas-aなりで共通化することに問題はないと思う

**デフォルトの名無しさん** · 2019/09/15(日) 12:48:53.00

簡単な名前でいいならBinaryDataSearcher
Dataみたいな語は情報量が少ないから削られがちだけど、この位置のDataを無造作に省略すると修飾先が変わっておかしなことになる
もう少し真面目に考えると、バイナリかテキストかという初歩的な分類用語よりも、エンディアンとか解釈した結果のバイトストリームを扱ってるならそういう名前の方が適切
ほかには多少語弊が生じるけど、お好みでProcessとかProgramとかExecutableあたりの語を使ってもいいかも

**デフォルトの名無しさん** · 2019/09/15(日) 13:13:26.96

>>770
要するに、

(1) ストリームの中から
(2) 指定されたバイト列に完全一致する場所を探す

ってことね。部分一致とか、パターンマッチと、CRC的な符号が一致するとか、そういうのじゃないわけだ。

じゃあ「バイト列の探し屋」か？
ByteSequenceFinderとかByteArraySeeker みたいな感じ？

**デフォルトの名無しさん** · 2019/09/15(日) 14:39:03.93

Cで言えばstrstrのバイナリ版GNU拡張のmemmemだよね。
ファイルが対象だとしてもmmapすれば同じだし。

**デフォルトの名無しさん** · 2019/09/15(日) 15:50:14.45

もっと単純BytesFinderでもよかったか

**デフォルトの名無しさん** · 2019/09/15(日) 16:44:36.00

そこまで抽象化するなら検索アルゴリズムの実装になるから検索アルゴリズムの名前でいいんじゃ

**デフォルトの名無しさん** · 2019/09/16(月) 08:48:03.32

ありがとう

そう、完全一致だけで部分とかパターンとかは無し
この流れをヒントにして考えてみるよ

ありがとう

**デフォルトの名無しさん** · 2019/10/21(月) 11:26:02.37

テキスト検索で、検索ボックスに入力する文字列と検索対象の文字列(本文)はどう名付けるのが無難ですか？

**デフォルトの名無しさん** · 2019/10/21(月) 12:13:57.58

>>779
前者はkeyword？
後者は置かれる文脈次第だろうね。シンプルにtextで通じるならそれでいいと思う

他の候補は
target, targetText, scannedText, searchee

ググってみたがsearcheeって言葉は正式な英語じゃない（当たり前か）
でも意味は分かると思う

**デフォルトの名無しさん** · 2019/10/21(月) 13:48:15.35

box, search という単語を変数名に入れた方がいいと思う
同じような変数が何種類か出てくることになると思うので

**デフォルトの名無しさん** · 2019/10/21(月) 14:26:40.55

それはあるある

**デフォルトの名無しさん** · 2019/10/21(月) 15:02:23.84

検索文字列はsearchStringがよく使われる印象
boxみたいにUIの形状に基づく変数名は本当にそれが妥当なのかよく考えてから付けたい

**デフォルトの名無しさん** · 2019/10/21(月) 16:25:59.50

よくあるのは、needle, haystack

**デフォルトの名無しさん** · 2019/10/21(月) 17:44:13.34

UI要素につける名前なのか
UIから受け取った検索文字列と検索対象文字列を一時的に格納するローカル変数につける名前なのか
それを渡す検索メソッドの引数につける名前なのか

コンテキストによって適切な名前は変わってくるから
それを共有しないと時間の無駄

**デフォルトの名無しさん** · 2019/10/21(月) 20:46:03.97

ムダだと思うなら黙っとけ！

**デフォルトの名無しさん** · 2019/10/21(月) 21:00:10.44

普通に考えれば>>779の人が検索ボックスと言ってるのは
その方が話が早いからでUI要素の名前を聞いてるわけではないと思うよ

英語のkeywordに相当する日本語って意外とないから。
検索文字列とか言っても見つけ出したい文字列じゃなくてスキャンされる方の
文字列を指してると誤解される可能性がある。

**デフォルトの名無しさん** · 2019/11/13(水) 09:16:45.77

例えば、Twitterのツイートがあります。ツイートが他のツイートへの返信の場合、
どのツイートへの返信かを表すIN_REPLAY_TO_TWEET_IDみたいのがあります。
で、この逆の関係を表現したいのです。
あるツイートがあって、このツイートへ返信してるツイートのIDsを表す場合
なんて名前がいいでしょうか？

**デフォルトの名無しさん** · 2019/11/13(水) 11:02:28.02

replies

**デフォルトの名無しさん** · 2019/11/13(水) 11:18:20.64

TWEET_FROM_REPLAY_ID

**デフォルトの名無しさん** · 2019/11/13(水) 11:18:40.12

この手のってコード書いてると訳分からんようになるんだよなー

**デフォルトの名無しさん** · 2019/11/13(水) 11:20:16.90

連投すまん
上2レスをまとめて書いたらNGワードで蹴られたから分けてみたら通った
ERROR: Rock54: Warning:NG～ってやつ
何じゃそら

**デフォルトの名無しさん** · 2019/11/13(水) 13:41:02.27

なんでreplayなの？

**デフォルトの名無しさん** · 2019/11/13(水) 14:16:26.15

788のにそのまんま返してしまったけど、replyだったねｗ

**デフォルトの名無しさん** · 2019/11/13(水) 15:07:27.24

>>789で終わってたw
逆にこれ以外に何があるんだw

ついでに、IN_REPLAY_TO_TWEET_IDなんて意味不明だから
素直にparentとかした方がいいんじゃないの？

**デフォルトの名無しさん** · 2019/11/13(水) 17:41:41.87

in reply to tweet id は返信先のツイートのIDって意味じゃないですかね？確かTwitterAPI のJSONはこういう名前のフィールド持ってたから外人の人が命名しただろうから、盲目的に信じてたが、おかしいのか？

**デフォルトの名無しさん** · 2019/11/13(水) 17:45:10.84

Tweetクラスに、InReplyToプロパティとRepliesプロパティ追加しとげばいいか。どっちがどっちになりそうだけど仕方ないのか

**デフォルトの名無しさん** · 2019/11/13(水) 17:47:41.19

in_reply_to_tweet_idってのはTwitter APIの仕様だから変えるのはいささか難しいだろうな
リプライオブジェクトを格納するrepliesと、リプライIDだけを格納するreplyIDsを名前で区別したい場合はある
名前に対称性を持たせるならreply_tweet_ids

**デフォルトの名無しさん** · 2019/11/13(水) 17:48:07.09

つか今見ると俺がreplayだったのか。すまん、typoしてました

**デフォルトの名無しさん** · 2019/11/13(水) 18:03:10.32

>>795のparentというアイデアを借りて、
単にParentとChildren。これじゃ何の親子関係か分かりづらい?のでReplyParentとReplyChildrenじゃくどい？

で最後InReplyToとReplies

皆さんはどれがお好みでしょうか？

**デフォルトの名無しさん** · 2019/11/14(木) 00:23:50.76

特定のツイートとリプライは1対Nの参照関係だけど親子関係ではないし
ドメイン用語にParentやChildは無いのでそういうのは入れないほうがベター
InReplyToとRepliesがいい

**デフォルトの名無しさん** · 2019/11/14(木) 00:29:00.53

いやいやいやw
フツーにツリー構造だからw

**デフォルトの名無しさん** · 2019/11/14(木) 09:41:50.47

循環するツリーってあるんですね(笑)

**デフォルトの名無しさん** · 2019/11/14(木) 11:24:59.93

循環てw
明日書かれるツイートに今日返信できるのかwww

**デフォルトの名無しさん** · 2019/11/14(木) 11:28:27.61

ついでに言うと、論理的には木構造は循環する可能性を必ずしも排除しないはず。
ツイートは循環なんかしないけどね

**デフォルトの名無しさん** · 2019/11/14(木) 13:15:57.33

>>802
ツイートとツイートの関係(=relationship)の話と
それをプログラムで扱う際にどういうデータ構造で表現するのかという話はレイヤーが違う
親子関係じゃないということとツリー構造で扱うということは両立しうる話

特定のデータ構造で扱うことを前提とした命名をすべき状況で
ツリー構造に依存した名前にしたければそうすればいい

**デフォルトの名無しさん** · 2019/11/14(木) 18:19:32.64

自己言及してるツイート見て来いよバカ

**デフォルトの名無しさん** · 2019/11/14(木) 18:48:28.45

ツリー構造であるかどうかを議論してもしょうがないと思うぞ
ツリーの関係性を有するデータの名前に漏れなくChildrenをつけるべきか？
これは常には成立しないよな

ツリー以前に、単なる1:Nの親子関係を有するデータにも同じことがいえる
雇用関係ならEmployerとEmployeesが妥当であって、EmploymentParentとEmploymentChildrenという命名は拙い

**デフォルトの名無しさん** · 2019/11/14(木) 22:24:40.93

>>806
なるほど、では君はファイルシステムで上位のディレクトリのことを親って言わないんだねきっと。
どういう問題領域のどういう対象であろうと、それが木構造にみなせる以上、親は親だ。
普通はそう考えると思うけど

**デフォルトの名無しさん** · 2019/11/15(金) 00:27:23.00

「金槌しか道具を持っていない人は、何もかも釘であるかのように取り扱う」by マズロー

**デフォルトの名無しさん** · 2019/11/15(金) 00:33:09.85

>>809
ふつうじゃねえよ。異常。
独立したものの参照関係がツリーのように見えているだけで、本質的には親子関係が成立するツリーじゃないぞ。

**デフォルトの名無しさん** · 2019/11/15(金) 00:49:32.81

>>811
それは君がツリーっていう言葉の語感に影響されているだけ。
だからUIみたいに「見える化」されているものだけを指すと勘違いしてるんだろう。
抽象的な思考が苦手なタイプによくある勘違いだ。

木構造は単に論理的な関係を表現しているだけだ。
っていうか自分でツイートとそれに対するリプの関係を図示してみろって。
それ、ツリーそのものだろアホか。

っていうかだから、直上のディレクトリを親と言うのか言わないのか、答えてみろって。
頭悪いにも程があるよほんと

**デフォルトの名無しさん** · 2019/11/15(金) 00:58:36.52

>>810
こいつも何か言ってるつもりなんだろうけど馬鹿だよねw
悪いけど「バカの一つ覚え」なのはそっちの方だ。

何もそれを表現するより具体的な表現や用語があるのなら
木構造だの親だの、そんな抽象的な言葉を使つ必要なんかない。

ディレクトリの例もそうだが、それがないから「親」と呼べばいいじゃないかと言ってる。
なのにこの手の御仁は「もっと具体的な言葉を使うべきだ！！！」というバカの一つ覚え。
笑えるね