トップページ⇒tech
1001コメント266KB

クラス名・変数名に迷ったら書き込むスレ。Part20

■ このスレッドは過去ログ倉庫に格納されています
0001デフォルトの名無しさん2011/07/29(金) 21:32:01.00
クラス名、変数名のつけ方に悩んだら書き込むスレです。

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

前スレ
クラス名・変数名に迷ったら書き込むスレ。Part19
http://hibari.2ch.net/test/read.cgi/tech/1293201229/
0377デフォルトの名無しさん2011/11/15(火) 15:25:55.17
10進数⇔64進数の簡単なプログラム書いてるのだけど、64進数って英語で何ていうの？
0378デフォルトの名無しさん2011/11/15(火) 15:46:07.36
radix 64
base 64
0379デフォルトの名無しさん2011/11/15(火) 16:04:54.64
>>378
Base64やRadix-64ではなくて、
単に10進数と64進数を相互変換するだけの簡単なクラスです。
すみません。
0380デフォルトの名無しさん2011/11/15(火) 16:14:36.79
質問の仕方が悪かった。

10進数と64進数を相互変換するだけの簡単なクラスを作ったのだけど、
Base64だとMIMEのBase64と混同してしまうので、何か適当な名前をお願いします！
0381デフォルトの名無しさん2011/11/15(火) 16:32:20.37
基数変換(Radix Conversion)
からRadixConverterでいいんじゃないの？
0382デフォルトの名無しさん2011/11/15(火) 16:34:21.47
RadixConverter
03833802011/11/15(火) 17:05:32.05
>>381
>>382
ありがとう。

BaseConverterやRadixConverterだと、汎用性のある基数変換みたいな名前だけど、
作ったのは10進⇔64進という決め打ちで汎用性の無いものだったので、もっとズバッとした名前がいいかなぁと思って。
でも64っていう数字を入れると、Base64やRadix64になって、MIMEのそれと混同してしまうかなと。

Radix64Converterでいいかな。
コード書いてる時間より名前考えてる時間の方が長い。。。
グダグダとスマン。
0384デフォルトの名無しさん2011/11/15(火) 17:12:39.88
そんな心配は名前に盛り込まないで
コメントに一言入れておけばいい
0385デフォルトの名無しさん2011/11/17(木) 10:26:18.57
「はみ出し」って英語でなんて言えばいいですか？(名詞で)
枠の中に書いた文字が枠からはみ出ているとか、そういう状態を指す言葉。

辞書を引くとprotrusionという単語が見つかったけど、「はみ出てる」と
言うより「飛び出てる」「ポコッと突き出てる」という感じがしてしまう
のでもっと適切な単語があれば教えてください。
0386デフォルトの名無しさん2011/11/17(木) 10:48:44.06
1行目無視して、2行目の状況を表す単語ということでよければoverflow
0387デフォルトの名無しさん2011/11/17(木) 11:30:15.60
どうもありがとうございます。

説明がうまくなくてすみません。
「字数が多すぎて記入欄から何文字かはみ出した」という場合には
確かにoverflow がぴったりですね。

>>385で考えていたのは、郵便番号のような1文字1マスの領域が
ある上に書いた文字が枠よりでかかったりズレてたりして、
上下左右にてんでにはみ出している（最悪、枠から完全に外れてる）
という状況なのでoverflowだと違うかなと。
0388デフォルトの名無しさん2011/11/17(木) 13:14:36.14
overflowをヒントに考えて、spillにしました。
0389デフォルトの名無しさん2011/11/18(金) 11:49:00.56
整数のアプリケーション定義のエラーコードを（動的ライブラリを使って）文字列に変換するクラス（関数オブジェクトクラス）にふさわしい名前はなに？
0390デフォルトの名無しさん2011/11/18(金) 12:49:50.58
ErrorDescriber
0391デフォルトの名無しさん2011/11/18(金) 23:00:12.22
>>389
エラーコードからエラーメッセージに変換するクラス？
0392デフォルトの名無しさん2011/11/19(土) 06:56:46.02
class IProcessor {
　virtual void Process(Value);
};

class Xxx {
　void Process(KeyValuePair element) {
　　processors.Find(element.Key).Process(element.Value);
　}

　List<IProcessor> processors;
}

こんな感じのクラス（Xxx）を命名してくれ
俺的には ProcessorSummary なんてどうかと思うんだが
0393デフォルトの名無しさん2011/11/19(土) 08:52:22.26
>>392
KVPDispatcher
0394デフォルトの名無しさん2011/11/19(土) 15:05:53.29
>>392
コマンドパターンみたいなものだからCommander？

しかし、余計なお世話とは思うけど、KeyValuePairの存在理由が謎。
わざわざ使いにくくしてるだけのような....
0395デフォルトの名無しさん2011/11/19(土) 15:06:24.81
>>392
コマンドパターンみたいなものだからCommander？

しかし、余計なお世話とは思うけど、KeyValuePairの存在理由が謎。
わざわざ使いにくくしてるだけのような....
0396デフォルトの名無しさん2011/11/19(土) 15:10:49.15
>>392
コマンドパターンみたいなものだからCommander？

しかし、余計なお世話とは思うけど、KeyValuePairの存在理由が謎。
わざわざ使いにくくしてるだけのような....
0397デフォルトの名無しさん2011/11/19(土) 15:12:11.59
>>392
コマンドパターンみたいなものだからCommander？

しかし、余計なお世話とは思うけど、KeyValuePairの存在理由が謎。
わざわざ使いにくくしてるだけのような....
0398デフォルトの名無しさん2011/11/19(土) 15:15:56.49
>>392
Dictionary使えばいいのに
0399デフォルトの名無しさん2011/11/19(土) 15:53:53.97
Map<ErrorCode, Inspector> map;
map.get(err).inspect(err);
0400デフォルトの名無しさん2011/11/19(土) 22:13:41.32
…
04013922011/11/20(日) 02:16:54.12
>>393-398
ん？もしかしてC#関連のクラスと勘違いしていないか？
例として出している名前、型名は構造を想像してもらうためのダミーだぞ

>>399
確かに、processors を List 型と書き示したのがまずかったな。型は Map と記した方が正しい

class Xxx {
　void Process(Element data) {
　　processors.Find(data.Type).Process(data);
　}

　bool AddProcessor(IProcessor newProcessor) {
　　return processors.Insert(
　　　newProcessor.TargetDataType, newProcessor);
　}

　Map<DataType, IProcessor> processors;
}
0402デフォルトの名無しさん2011/11/20(日) 02:47:37.09
ん？別にC#じゃなくてもいいけど
C#で想像しちゃまずいのか？
C#にあるのと別の働きをするもの？
04033922011/11/20(日) 10:16:20.97
>>402
同じ
基になるコレクションは連想配列（Dictionary）でなくてもいいけど
0404デフォルトの名無しさん2011/11/20(日) 10:53:58.05
>>401
お前さ、人に もの聞くのに上から目線か。オナニーでもしてろ
0405デフォルトの名無しさん2011/11/20(日) 13:07:56.16
どっちにしろ、無理にElementなんて型を導入するのが意味不明に思えるな。
わざわざ使いづらくしてるだけじゃん。

Element.TypeでMapからIProcessorを引っ張ってくる意味も分からない。
それなら最初からElemetにTypeではなくIProcessorを持たせれば済む。
何がしたいんだろう。
0406デフォルトの名無しさん2011/11/20(日) 16:39:43.12
>>405
ご法度
04073922011/11/21(月) 04:12:25.43
>>405
もう少し具体的に書いてみたが、どうだろうか

class Xxx {
　ContactCache DetectContacts(Body bodyA, Body bodyB) {
　　ShapeTypePair stp(bodyA.ShapeType, bodyB.ShapeType)
　　return narrowPhases.Find(stp).DetectContacts(bodyA, bodyB);
　}

　bool TryAddNarrowPhase(ShapeTypePair stp, NarrowPhase newNarrowPhase) {
　　return narrowPhases.TryAdd(stp, newNarrowPhase);
　}

　Map<ShapeTypePair, NarrowPhase> narrowPhases;
}
0408デフォルトの名無しさん2011/11/27(日) 15:52:38.23
a.bとかの . てなんて呼べばいいだろう？
separatorとかdelimiterだとaとbが全く別のデータみたいな言い方だし、なんかないかな？
0409デフォルトの名無しさん2011/11/27(日) 16:06:15.01
そもそも a.b ってナニ
0410デフォルトの名無しさん2011/11/27(日) 16:15:01.88
プロパティへのアクセスとか。setting.name
0411デフォルトの名無しさん2011/11/27(日) 16:38:05.75
ドット演算子
メンバアクセス演算子
0412デフォルトの名無しさん2011/11/27(日) 17:06:29.83
まあ、なんとかoperatorだろうな。
0413デフォルトの名無しさん2011/11/27(日) 17:18:12.26
dot operator
member access operator
0414デフォルトの名無しさん2011/11/27(日) 17:39:19.80
言語の本に載ってるでしょ
0415デフォルトの名無しさん2011/11/30(水) 01:48:24.40
〜も含めてってなんて言えばいい？search(word, なんとかBody)みたいな感じで、通常は
タイトルを検索するんだけど、search(word, true) の場合は本文も検索対象に含めるみたいな。
0416デフォルトの名無しさん2011/11/30(水) 07:29:19.45
関数名なのか変数名なのか、マクロ名なのか列挙型の名前なのか・・・
さっぱりだから、including とか with とかで良いだろとしか言いようが無い
0417デフォルトの名無しさん2011/11/30(水) 09:30:43.88
数字を吐き出す時に前回出力から大きく外れない乱数を吐き出す関数があるんだ。
システム的なものもあって、
出力を安定させる乱数
みたいな名前を付けたいんだが、
ちょっと中ニ病ちっくでかっこいい関数名付けてくれないか
0418デフォルトの名無しさん2011/11/30(水) 10:04:36.00
先物だからデリバティブだな
0419デフォルトの名無しさん2011/11/30(水) 10:50:50.19
randomでおｋ
クライアントコードは分布なんて意識したくない
0420デフォルトの名無しさん2011/11/30(水) 12:08:04.00
>>417
attractorField(); //特に意味はない
0421デフォルトの名無しさん2011/11/30(水) 12:41:14.23
>>417
静寂せし微乱の数 ();
0422デフォルトの名無しさん2011/11/30(水) 12:46:23.53
>>417
GetFluctuatingValue (揺らいでる値を取得)
0423デフォルトの名無しさん2011/11/30(水) 12:51:43.65
世界の終わりにキミが届けるランダム値をおれはそっと抱きしめる()
0424デフォルトの名無しさん2011/11/30(水) 13:48:47.01
/** 極限原点 */
int ParticleConvergence();
0425デフォルトの名無しさん2011/11/30(水) 15:27:59.64
黄昏を乱す小さきモノ();
0426デフォルトの名無しさん2011/11/30(水) 16:57:19.66
中二病でいいならMyRandom
0427デフォルトの名無しさん2011/12/01(木) 12:27:00.04
GetNearChaos_DeadEnd() なんてどうだ？
0428デフォルトの名無しさん2011/12/01(木) 14:24:11.68
thenEnemyDie();
//相手は死ぬ
0429デフォルトの名無しさん2011/12/01(木) 15:59:06.89
ゲームでメッセージとかアイテムリストとか表示したりメニュー開いたり閉じたりとかスコア表示したりするクラスってなんていうの？
ユーザーインターフェースとかかな？
0430デフォルトの名無しさん2011/12/01(木) 16:05:28.58
うん
0431デフォルトの名無しさん2011/12/01(木) 17:19:52.73
除外する ID を指定するパラメータ ignoreId があるんですが、
“ignoreId で除外したIDが持つ子も除外“するパラメータ名でいいのないでしょうか？
paramname = true で子も除外、false で指定 ID のみ除外、みたいな感じです。
よろしくお願いします。
0432デフォルトの名無しさん2011/12/01(木) 17:27:01.94
ignoreChild
0433デフォルトの名無しさん2011/12/01(木) 17:28:06.80
ignoresDescendants
0434デフォルトの名無しさん2011/12/01(木) 17:32:04.79
>>431
関数名は何？
0435デフォルトの名無しさん2011/12/01(木) 19:21:32.37
列挙型にして SpecifiedOnly、WithChildren じゃダメ？
04364322011/12/01(木) 19:25:00.99
レス遅くなってすみません。
>>434
getList にしてますが getTree でもいいかなと思ってます。
04374312011/12/01(木) 19:25:34.29
>>436 は 432 になってますけど >>431 の間違いです。
04384312011/12/01(木) 19:41:46.11
>>432,433,435
検討してみて ignoresDescendants 頂くことにしました。
レス下さった皆さん、ありがとうございました。
0439デフォルトの名無しさん2011/12/02(金) 04:43:31.20
Lightness（輝度）が基準値を超えているときは、暗くする、
基準値以下なら、明るくする関数の名前をお願いします。

function xxx() {
return (lightness > 0.5) ? darken() : lighten();
}
0440デフォルトの名無しさん2011/12/02(金) 04:57:27.48
輝度はLuminanceだよ
モニタの輝度調整とかならBrightnessが一般的
04414392011/12/02(金) 05:02:45.88
>>440
失礼しました。
こちらに、輝度（Lightness）とあるので、混同してしまいました；

http://ja.wikipedia.org/wiki/HLS%E8%89%B2%E7%A9%BA%E9%96%93

ようするにHSL色空間のLのことです。
0442デフォルトの名無しさん2011/12/02(金) 05:22:16.31
アラヤダ。そんな事情があるともしらず、申し訳ないです。

で、本題のほうなんだけど、安易にadjustとか使いたくなるんだけど、そういうわけにもいかないのかなぁ。
AdjustLightness(threshold) とかで伝わりそうといえば伝わりそうなんだけど
04434392011/12/02(金) 05:44:14.48
>>442
adjustだと、増減値を自分で指定するという感じがしますね。

AdjustLightness(amount)のように・・・
0444デフォルトの名無しさん2011/12/02(金) 08:22:49.72
例によって clamp かなぁ、と思ったけど、指定範囲に収めるのとは
ちょっと違いますかねぇ。
0445デフォルトの名無しさん2011/12/02(金) 13:48:47.68
Approachは？
0446デフォルトの名無しさん2011/12/02(金) 16:16:35.88
converge
0447デフォルトの名無しさん2011/12/02(金) 17:31:03.07
明/暗順応（light/dark adaptation）の関係だな

applyLightnessAdaptation
0448デフォルトの名無しさん2011/12/02(金) 20:43:42.56
>>439
動詞じゃないのを許せるならLightStatとか。

余計なお世話だけど、そういう単純な制御だと、ネガティブフィードバックになってたら
条件が揃えば発振するよねそれw
0449デフォルトの名無しさん2011/12/03(土) 00:38:22.92
データベーステーブル名を複数形にするという規約があるもとで、
分類テーブルと、分類付テーブルの名前をどうしたらいいですか？
categoriesとcategorizationsとなってますが、おかしいですか？
0450デフォルトの名無しさん2011/12/03(土) 00:49:28.00
分類はclassじゃね？
classesとclassifications・・・なの？
04514492011/12/03(土) 00:56:14.70
>>450
classでもよさそうですね。

ing+sとかで動詞の複数形？にするのはおかしいですかね？
一応名詞を選んでいます。

英語難しいです・・・

categoriesは他のテーブルと関連付けるための情報をもつテーブルで、
categorizationsで、実際にcategoriesと他のテーブルを関連付けしている感じになります。
0452デフォルトの名無しさん2011/12/03(土) 01:06:43.84
categories
reations
0453デフォルトの名無しさん2011/12/03(土) 01:30:33.02
categoriesと、関連付けられる他のテーブルが例えばitemsだったりしたら、item_categoriesとか。
0454デフォルトの名無しさん2011/12/04(日) 10:22:13.00
複数形関連で便乗質問させてください。
「student」に対する「students」など、
単体クラスを複数管理するクラスをよく作っていたのですが、
「series」って、複数形も「series」で、
「series群」というクラスをどういう名前にすればよいのか困っています。

他の命名規則と合わせて「serieses」という造語を使っておくべきでしょうか？
(「"serieses"」でググるとそれなりには引っかかるのですが)
0455デフォルトの名無しさん2011/12/04(日) 10:30:45.63
文脈でわかるからただの複数形（この例ならseries）でいいよ
0456デフォルトの名無しさん2011/12/04(日) 10:51:29.95
利用側からは管理クラスしか見えないなら、単体クラスの方をリネームするのはどうだろうか。
Entityとか適当なサフィックス付けて。
0457デフォルトの名無しさん2011/12/04(日) 11:04:14.84
複数形に擦るだけではどんな集合なのかが見えない
その集合の性質をよく表したクラス名にするのが正解
04584542011/12/04(日) 11:56:00.39
>>455-457
seriesというクラスが存在して、それを複数管理するクラスなんです。
そして、この「複数のseries」を、関数内で使ったり、
他のクラスがメンバとして持つ、という構造です。

内部では普通の配列や連想配列で管理してますが、使う側には見えない部分で、
arrayやmapという単語をクラス名には付けたくないので、複数形を使っています。

なんか英語としては「series's」が正解なようなのですが、
これをクラス名にしようとするとどうなるのでしょうかね…。
0459デフォルトの名無しさん2011/12/04(日) 12:06:58.64
だからそれは何のために複数管理するのか
その目的がはっきりしていれば自ずと適切な名前も決まるでしょ
そして何の目的もなくただ集約してるだけならデータ構造の名前つけるぐらいしかないだろう
0460デフォルトの名無しさん2011/12/04(日) 12:27:14.00
コレクションクラスの名前を要素の型の複数形にするってあまりやらないような...
変数の名前なら複数形でいい場合が多いと思うけど。

それが外からリストに見えるのなら〜List、連想配列に見えるなら〜Map、
単なる集合以外の意味を持たないなら〜Collection、みたいに機械的に命名すればいいだけでは？
0461デフォルトの名無しさん2011/12/04(日) 13:40:02.98
命名規則がなんかジャバっぽいよね
0462デフォルトの名無しさん2011/12/04(日) 13:50:29.90
CLASS object;
object.Do();
これっておかしくないか
objectは三人称単数なんだから
object.Does();
にするべきでは
0463デフォルトの名無しさん2011/12/04(日) 13:53:59.51
>>462
命令形のDoだろう。
0464デフォルトの名無しさん2011/12/04(日) 13:55:56.05
>>462
メソッドは叙述ではなく命令ですよ基本。
dog.Run()は「ワンコが走ってる」じゃなく、「ワンコ、走れ」。
0465デフォルトの名無しさん2011/12/04(日) 13:58:51.34
なるほど
ではobject.isValidやobject.GetValueはどう解釈すれば
object、正常であれという命令？
これでは正常化するメソッドに見えてしまいますよ
0466デフォルトの名無しさん2011/12/04(日) 14:04:06.46
|冫、)ｼﾞｰ
0467デフォルトの名無しさん2011/12/04(日) 14:06:15.19
>>465
>>464 は、メソッドは叙述ではなく命令ですよ「基本」と言ってる
基本じゃない場合もあるんだよ
（たいていの場合は命令で解釈できるから、基本と言っているんだと思う）

>>465 の場合は、一般的には「object は正しい」という述語として
解釈されてきた歴史がある（だから戻り値もたいていは論理値）

一律に「全て命令だ」という訳ではないんだよ
そうやって解釈されてきた歴史的な経緯は知らんが

基本はちゃんと押さえながら、かつ柔軟に解釈していこうよ
0468デフォルトの名無しさん2011/12/04(日) 14:13:52.34
耳学問なんだよね所詮は
0469デフォルトの名無しさん2011/12/04(日) 14:15:24.26
>>467
ちなみに、命令じゃなくて普通の動詞として解釈する人もいて、
こちらも別に間違いじゃない

window.createChild (・・・)
window が ・・・という属性の子ウィンドウを作る

命令と解釈するのは、たぶんオブジェクト指向以前の名（伝統）だろうね
0470デフォルトの名無しさん2011/12/04(日) 16:40:44.78
>>464
でたらめ言うなゴミクズ
0471デフォルトの名無しさん2011/12/04(日) 16:42:44.56
>>469
訂正

> たぶんオブジェクト指向以前の名（伝統）だろうね

たぶんオブジェクト指向以前の名残（伝統）だろうね
0472デフォルトの名無しさん2011/12/04(日) 17:52:00.61
>>462
お前は英語分かっていないな。三人称単数現在ってのは
ちゃんとニュアンス持ってるんだよ。
命令形の場合とは違う
0473デフォルトの名無しさん2011/12/04(日) 17:55:47.70
>>465
命令形は Be - だよ。
ちなみに get〜 だけはお前の言うとおり
おかしい。これはネイティブも（おかしいと）分かって
るけど、get にしてると思う。

>>469
２番目の意味では、
window.createsChildになる
0474デフォルトの名無しさん2011/12/04(日) 18:15:59.23
なんか変な流れになってるけど....

object.IsValidも、例えば、"tell whether the object is valid or not"
のような感じの省略表現だと考えることが出来る。

やはり基本は命令表現でしょう。

object.GetValueみたいな形はアクセスメソッド以外では今時あまり使わないと思うけど。
0475デフォルトの名無しさん2011/12/04(日) 18:19:32.76
ツマラン
0476デフォルトの名無しさん2011/12/04(日) 18:58:56.28
>>474
デタラメ撒き散らすな
黙っとけカス
0477デフォルトの名無しさん2011/12/04(日) 19:15:15.16
デタラメとか雑音まいてる馬鹿は英語できないんだろうな。
こういう馬鹿を説得する義理もないけど、だったら仮にメソッド名に日本語を
使うとしたらどうなるか考えてみろと。

IsValidは「正当か」、CreateChildは「子オブジェクトを作れ」
普通はこういう風になる。

■ このスレッドは過去ログ倉庫に格納されています