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

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

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

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

前スレ
クラス名・変数名に迷ったら書き込むスレ。Part19
http://hibari.2ch.net/test/read.cgi/tech/1293201229/
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は「子オブジェクトを作れ」
普通はこういう風になる。

0478デフォルトの名無しさん2011/12/04(日) 19:24:07.91
#define オブジェクト object
#define は .
#define の .
#define 、 .
#define 正常である isValid
#define 名前を取得する getName
#define 生成せよ create

オブジェクト は 正常である()
オブジェクト の 名前を取得する()
オブジェクト 、 生成せよ()

日本語は形式が柔軟だから便利だね
0479デフォルトの名無しさん2011/12/04(日) 19:50:30.00
それは、C言語限定？
0480デフォルトの名無しさん2011/12/04(日) 19:58:50.10
Java や C、C# なんかで a.b と書くときがあるけど、
a や b の名前を使って「全体として」英語の文として読もうとか、
あるいは「全体として」英語の文になるように a や b の名前を決めようとか、
そんな馬鹿なことに注力して名前を決めるものじゃない

何らかの工夫をしなければ、対象となるオブジェクトの識別子より
関数を表す識別が必ず左側に来る言語だって普通にある

名前は特定言語の文法にできるだけ依存しないように、
その名前を参照する側から見た役割や機能、意味などを元に決めるべき

全体として英文のように読める（解釈できる）かどうかは些末な問題だ
0481デフォルトの名無しさん2011/12/04(日) 20:01:57.48
>>480
と言ってはみても、それを理解できる猛者はこのスレにはいないだろうね
0482デフォルトの名無しさん2011/12/04(日) 20:13:42.29
ObjectIsValid(Type object);
GetValueFromObject(Type object);
DoSomethingWithObject(Type object);

関数名だけで英文が完成するようにする
引数名は関数名である英文に現れる単語を示すように名付ける
これがもっとも読みやすい命名法
0483デフォルトの名無しさん2011/12/04(日) 20:23:32.12
>>482
荒すな
0484デフォルトの名無しさん2011/12/04(日) 20:25:30.47
>>483
嵐じゃないです
0485デフォルトの名無しさん2011/12/04(日) 20:33:19.92
>>480-481
むしろそこは誰も異論がないと思うんだけど....
0486デフォルトの名無しさん2011/12/04(日) 20:35:59.14
英文として読めるどころかバシバシ省略形で書いていいんだけどね
どうせドキュメント読まないなんてことはあり得ないんだから長い名前書くだけ無駄
衝突さえ回避できればC標準のような適当さで十二分なんだよ
0487デフォルトの名無しさん2011/12/04(日) 20:46:51.23
お前ら揚げ足が取りたいだけじゃないか。
なぜなら俺のレスには、レスしてないし。きれいに避けて通ってるｗ

ちなみにどこまでも英文に近くしたいのなら、
SmalltalkかObjective C でも使っとけって話だ
0488デフォルトの名無しさん2011/12/04(日) 21:44:36.69
お前のレスってどれだよ
0489デフォルトの名無しさん2011/12/04(日) 23:20:27.31
>>487
どこまでも英文に近くしたいならCOBOLじゃね?
0490デフォルトの名無しさん2011/12/04(日) 23:26:42.85
>>489
適当に「COBOL サンプル」などでググると分かるけど、
COBOL って思ったほど英文っぽくないんだわ
0491デフォルトの名無しさん2011/12/04(日) 23:34:48.54
COBOLは確かにひとつひとつの文の書式は英文臭いけど
抽象化し辛いせいで、結局はその辺の言語以上に英文から離れてくのよねw
ホントに英文っぽくしようとすると命令ごと大量に作らないといけなくて
可能な限りでそうしようとした結果が膨大な単語が並んだ予約語表
しかもそうまでしてもやっぱり英文には程遠いという…
0492デフォルトの名無しさん2011/12/05(月) 00:45:11.47
>>477
×CreateChildは「子オブジェクトを作れ」
○CreateChildは「子オブジェクトを作る」
0493デフォルトの名無しさん2011/12/05(月) 01:02:38.30
>>467
たいていの場合は叙述で解釈できるよ
0494デフォルトの名無しさん2011/12/05(月) 07:08:01.76
要するに何とでも都合が良いように解釈できる＆してきたわけだよ
0495デフォルトの名無しさん2011/12/05(月) 08:23:41.73
よし関数型言語での命名法でも考えようぜ
0496デフォルトの名無しさん2011/12/07(水) 10:33:29.74
read() -> doSomething() -> write() の順に実行される
テンプレートパターンのJavaクラスを名前を考えてください。

IOエラーなど生じやすい危険からBridgeScope, CriticalSection
みたいのがいいかなと思うのですがどうでしょう？

abstract class ??? {
onRead(){}
onReadError(){}
onSomething(){}
onWrite(){}
onWriteError(){}
}
0497デフォルトの名無しさん2011/12/07(水) 10:44:52.89
Converter
04984962011/12/07(水) 11:39:48.92
＞Converter
onSomething()がリソースの変換とは限らず
振る舞い(ビヘイビア)でも通じるのが良いです。
DTOとしての自身の変数（フィールド）に
マッピングすることが主な目的なので。

class ??? {
__@WebSession
__@Read
__@Write
__Integer id; // Read and Write

__@WebCookie
__@Read
__String login_name; // Read only

__@File(path="/log.txt")
__@Write
__String log; // Write only
__
__@Override
__void onSomething(){
____Service service = new Model();
____log = service.service(this);
__}
}

04994962011/12/07(水) 12:43:08.73
onSomething()もしくはonError()で
Webページのforwardを行わせるので

Resolverにしてみようと思います。
0500デフォルトの名無しさん2011/12/11(日) 17:50:04.58
C++のソートなどに使う比較関数なんだけど
bool compare(X a, X b)
{
a.hoge > b.hoge;
}
これの名前はどうする
hoge_of_a_is_greater_than_hoge_of_bってすると長くて間抜けな気がするし変数名が入ってヤナ感じ
0501デフォルトの名無しさん2011/12/11(日) 19:02:20.62
Xのインスタンスメソッドとして実装すれば何も考えなくても....

っていうか、例えば
bool GT(X a, X b)

ってシグネチャの関数があったときに、これを(b > a)を返す関数だと解釈する人はまずいないと思うけど....
とくに説明がなければ、常識的に言って(a > b)が返ってくるんだろうなと解釈するでしょ。
0502デフォルトの名無しさん2011/12/11(日) 19:54:23.18
bool PriceGreaterThan(X a, X b)
{
　return a.price > b.price;
}

bool HogeGreaterThan(X a, X b)
{
　a.hoge > b.hoge;
}
■ このスレッドは過去ログ倉庫に格納されています