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

1 ：12/01/22

クラス名、変数名のつけ方に悩んだら書き込むスレです。
命名規則や設計の善し悪しについて議論するのは基本的に禁止。
前スレ
クラス名・変数名に迷ったら書き込むスレ。Part20
http://toro.2ch.net/test/read.cgi/tech/1311942721/

2 ：12/01/22

>>1乙
> 前スレファイル名議論
住所の最後が名前、
パスの最後がファイル名、みたいなもんだな。
できるだけ簡潔なものを選ぶ余地がある。

3 ：12/01/22

1おつ
>>前1000
極論ではなく一般論
それもコーディングだけではなく自然言語も含めたより広い世界での一般論
極論してるのはそっちだよ

4 ：12/01/22

>>3
馬鹿は来るな

5 ：12/01/22

>>4
じゃあお前が帰れよ

6 ：12/01/22

>>5
だからお前が失せろ

7 ：12/01/22

>>6
だから馬鹿はお前なんだからお前が消えねーとだめだろ
説明しないとわからんほど馬鹿なのか

8 ：12/01/22

>>7
さっさと失せろ馬鹿

9 ：12/01/22

>>8
>>4

10 ：12/01/22

>>9
>>4

11 ：12/01/22

小学校かここは

12 ：12/01/22

前スレ>>992
それ同じ事を言い方を変えただけ。
「その機能や概念を示す」のと「機能を読み取れること」のどこが違うんだ。
それなりに複雑な機能をもったクラスやメソッドの機能を短い名前で完全に表現
できないのは当たり前で、仔細な点は捨象して大枠の機能を抽象した名前を付けるに決まっている。
馬鹿じゃないのか。

13 ：12/01/22

>>12
>「その機能や概念を示す」のと「機能を読み取れること」のどこが違うんだ。
例を出して言えば「オイラー方程式」が前者で「理想流体の運動法則」が後者だ
どちらも同じ物を示しているが前者は機能や概念を隠蔽した名称だ
ぜんぜん違うだろ？
そして議論で使うのはどちらが自然かといえば前者だ
後者をいちいち使うのはあまりにもクドい

14 ：12/01/22

前スレ >>990
> だったらutf8だとかeucだとか文字エンコードも併記すべきな気がするぞ…
併記すべきかどうかは場合による
複数の文字コードを扱っているのであれば、併記しなければ分からない
扱っている文字コードがOSや環境のデフォルトだけであり、
あるファイルフォーマット内の文字列部分を処理しているなどなら、
文字エンコードを併記する必要性は感じない

15 ：12/01/22

>それなりに複雑な機能をもったクラスやメソッドの機能を短い名前で完全に表現
>できないのは当たり前で、仔細な点は捨象して大枠の機能を抽象した名前を付けるに決まっている。
あと一応言っておくが俺はクラス名やメソッド名を短く省略しろとは言っていない
狭い範囲、スコープに限定された文脈の中では短いものを使えと言っているのだ
>馬鹿じゃないのか。
それを読み取れなかったお前がBAKAです

16 ：12/01/22

>>15
まだいたのか
消えろ馬鹿

17 ：12/01/22

>>16
>>16

18 ：12/01/22

>>17
>>16

19 ：12/01/22

論破されると罵倒するしかできないってなんか可哀想だな
神様ももうすこしIQ平等に分けてあげればよかったのに罪なことするぜ

20 ：12/01/22

>>13
なるほど俺の誤解みたいだね。っていうかまさかそこまで馬鹿とは思わなかったw
要するに、
String.GetLength()
などという機能をそのまま表現した名前はよくないから、
String.func001()
String.YamadaTarou001()
のように単なる識別子以上のものではない名前を付けろという訳だ。
関数名と機能の対応を参照するバインダーがないと読めないコードが存在するって
都市伝説、あれは本当だったんだなw
こんなガチ馬鹿がどうしてこんなスレにいるんだろう。

21 ：12/01/22

>>20
>>15
ちょっと前のレスも読めない馬鹿がなんでいまだに生きてるんだろう

22 ：12/01/22

>>21
よ馬鹿

23 ：12/01/22

>>22
>>19

24 ：12/01/22

>>23
>>19

25 ：12/01/22

>>20
それはクラスというひとつの大きな単位の中のメソッド名ですよね
>>15 で彼は
> あと一応言っておくが俺はクラス名やメソッド名を短く省略しろとは言っていない
> 狭い範囲、スコープに限定された文脈の中では短いものを使えと言っているのだ
と言っているので、それは反論の方向としては間違っていると思うのですが
たとえば、
void func () {
＊＊＊ = String.GetLength()
場合の＊＊＊の部分などの場合の事だと思います

26 ：12/01/22

>>21
間違いを指摘されると「俺の真意はお前の理解と違う」ってか。
みっともない奴だな。
前スレの>>992は俺が書いた>>982に対する反論なんだから、そんな言い訳が通用する訳がない。
どんなクルクルパーだよ。

27 ：12/01/22

それから、メソッドやブロックのスコープ内でも明示的な名前を使え、
なんて言ってる奴が一人でも居たか？
そんな奴いねえよ一人も。
誰と戦ってるつもりなんだろうなまったく。

28 ：12/01/22

http://awabi.2ch.net/test/read.cgi/poverty/1327050821/3

29 ：12/01/22

スコープが広域であれば、略さず完全な名前を付ける
　superHogeCountWithPiyo
　pugeraDescription
スコープの狭さに応じて単語を省く
例えばローカル関数であれば、
　countWithPiyo もしくは count
　pugeraDesc または desc
できる限り略語を使わない
　×superHogeCountWithPiyo → shCountWithPiyo
　×pugeraDescription → pDesc, pdesc
そもそも短い単語を略す必要はない
　×cnt ○count
　×idx ○index

30 ：12/01/22

○　c
○　i

31 ：12/01/22

・・・とは書いてみたものの、狭域スコープでは hogeCount を nHoges みたいに略すことがある
単語を略すのであれば、number (of) → num みたいな中途半端な略し方ではムラが生じてしまう（numでは生じないだろうが）ので、一文字 number (of) → nまで略す
その単語の略語にムラが生じることはないだろうと自分で確信が持てるのであれば、使うことはある
例えば、Description は desc 以外に略語を持たないと判断できるし、Information は info 以外の略語を持たないので、狭域スコープではこれを使う
Temporary は temp と tmp が存在するので、略したいときは 一文字「t」まで略する

32 ：12/01/22

>>30
ごく短い、例えば3行以下の関数ローカルスコープであれば一文字でもかまわない
またはループカウンタとしてのcやiは、自分では許可している

33 ：12/01/22

> ごく短い、例えば3行以下の関数ローカルスコープであれば一文字でもかまわない

34 ：12/01/22

変数名をゴッチャリ飾って区別しないといけないようなのは、
既にメソッドが大きすぎるか、メンバ変数の場合はそれらが多すぎる。
fooBarBazList = 
appleBeerList = 
codeDoorList = 
みたいに併記しているような場合、うまくいくと
list = 
これで住むような可能性がある。

35 ：12/01/22

>>34
うむ。コーディングの技術がしっかりしてればほとんどの識別子は最低限まで縮めても問題は起こらないよね
というか問題が起こらないように作れると良い設計になってることが多い

36 ：12/01/22

いやいや大きいとか多いとかじゃなくて対象コードブロックの中に機能や状態を持ち込み過ぎ
struct A{int x,y;}a;のaじゃなく
後から読み解く奴の身になってx,yのどちらか必要な方だけを取り出して書けよと

37 ：12/01/22

現実問題、英語で言ったら形容詞抜きの"list"だの"count"だので十分説明の機能を果たすような
クラスはまずあり得ない。
皆無とは言わないが、むしろ例外的。
そういう特殊な前提がないと成立しないようなアホみたいな議論はするだけ時間の無駄。

38 ：12/01/22

>>35
丁寧なコメントが消臭剤でしか無いような場合があるのと同様に、
説明的な変数が消臭剤でしか無いような場合があるな。
早い段階ですべきは、元の悪臭漂うクソ構造を改めること。

39 ：12/01/22

就活中
(p)http://livedoor.blogimg.jp/jin115/imgs/3/1/31a6f8e6.jpg
就職後
(p)http://livedoor.blogimg.jp/jin115/imgs/2/b/2b790359.jpg
街の人（やらせ業者）募集中です

40 ：12/01/22

その画像独り歩きしてるけど
２枚の画像は１年間開いてるからね
その娘は就活うまく行ってそのインタビュー場所の近くの会社に勤めてるって事

41 ：12/01/22

後ろに居る男も横断歩道の向こう側にいる人も
インタビュワーも停まってる車も天候も変わってないな

42 ：12/01/22

偶然ってあるもんなんだな

43 ：12/01/22

Google Code Searchがなくなったのは痛すぎるな。
とりあえずつけたクラス名やメソッド名が外人的にあり得るか？否かをしらべるのに便利やった。

44 ：12/01/22

ええーーっ！！　ちょっと見ない間に無くなったのか！

45 ：12/01/22

google code

46 ：12/01/22

>>30
count_hogeもcHogeもほとんど見易さ変わらないだろ

47 ：12/01/22

ch

48 ：12/01/23

>>47
もはやなんだかわからんだろそれ

49 ：12/01/23

決まってんだろ
チャンネルだよ

50 ：12/01/23

入門書とか読むと、よく「チャネル」って書いてあるよね
その発音の方がより本物に近いのは分かるが、なんか気取ってて嫌

51 ：12/01/23

俺はにちゃねらー

52 ：12/01/23

お前らはアニモー。

53 ：12/01/23

>>50の中では「JISに合わせること＝気取っている」なのか

54 ：12/01/23

　前スレの、chrとかcharの974です。
　まさか論争に発展するとは見通し甘かった。
　私が書いているのはゲームの為の周辺ツールで、個人作業です。
　ですから厳密な命名規則が欲しいって程でもないんです。
＃報告
　一度フルスペルで書いたのですが、書き進める内に用途が広がって
キャラクターの意味に相応しくなくなったので一切なくなってしまいました。
　計画性のないプログラムは、これだから・・・！

55 ：12/01/23

>>53
違う
JISに合わせる、合わせないは全く関係ない
チャネルという言い方が気取っているように感じるだけ
>>53の中ではたった一例を挙げて、それがたまたまJISに合ってただけで
「JISに合わせること＝気取っている」と解釈するのか

56 ：12/01/23

>>54
論争になったらなったで、それを話題に盛り上がる訳で、面白いものなのだよ・・・

57 ：12/01/24

hashtable = {
　"foo.name": "value1",
　"foo.text": "value2",
　"bar.name": "value3",
　"bar.text": "value4"
}
みたいなデータから
{ "foo.name": "value1", "foo.text": "value2" }
だけをとりだす関数の名前を考えて下さい。
prefix = "foo."
print func(hashtable, prefix) #=> { "foo.name": "value1", "foo.text": "value2" }
今の候補
・get
・matched
・retrieve
お願いします。

58 ：12/01/24

find

59 ：12/01/24

find_by_key_prefix

60 ：12/01/24

複数候補だからenumかfindfirst→findnext

61 ：12/01/24

>>57
PickUpPrefixMatches

62 ：12/01/24

filterByPrefix

63 ：12/01/25

ウィンドウが画面の外に出ないように防ぐ処理
をお願いします

64 ：12/01/25

関数名？引数や返り値は何？

65 ：12/01/25

関数名です
戻り値なし、引数は画面のRECTとウィンドウのRECTです

66 ：12/01/25

その関数の副作用は何？

67 ：12/01/25

「ウィンドウが画面の外に出ないように防ぐ処理」じゃなくて
「（ウィンドウが画面の外に出ないように防ぎつつ）ウィンドウ移動する処理」なのでは？

68 ：12/01/25

>>57
Perlっぽいけどfilterできんじゃねーの？

69 ：12/01/25

要するにそういう細かいことをまとめて行う処理です

70 ：12/01/25

ちっとも細かくないやん 基本処理にしか見えん

71 ：12/01/25

ではそろそろ
ウィンドウが画面の外に出ないように防ぐ処理
をお願いします

72 ：12/01/25

MoveWindowInScreen

73 ：12/01/25

avoid-tilesaw

74 ：12/01/25

void MoveWindowInsideScreen(
　const RECT & windowRect,
　const RECT & screenRect);

75 ：12/01/25

void MoveWindowInsideScreen(
　const RECT & windowRect,
　const RECT & screenRect,
　RECT * pOutput);

76 ：12/01/25

>>71
消えろゴミ

77 ：12/01/25

なんでMOVEなんだよｗ

78 ：12/01/25

void ウィンドウが画面の外に出ないように防ぐ処理(
矩形 const & ウィンドウ,
矩形 const & スクリーン,
矩形 & 結果) ;
いい加減英語識別子から脱却しろ
いちいち名前付なんてくだらない作業で悩む必要がなくなって捗るぞ

79 ：12/01/25

preventとかcropとかかなぁ？captureはちと違うか。

80 ：12/01/25

confine

81 ：12/01/25

「ウィンドウが画面の外に出ないように防ぐ処理」って何だかよく分からないな。
この手のことをする場合、例えばWindowsなら普通は、
(1) 移動中にデスクトップの上下左右いずれかの隅に達したら、その方向にはそれ以上
移動できないようにする。
(2) 移動完了（マウスアップとか）時、ウィンドウがデスクトップの上下左右のいずれかの隅を
超えていたらデスクトップの範囲内に戻す。
のどちらかじゃいかと思うんだが、ニュアンス的にどっちとも取れないな。

82 ：12/01/25

>>71
KeepInsideDisplay()

83 ：12/01/26

ProcessingPreventedSoThatAWindowMeyNotComeOutOfAScreen

84 ：12/01/26

略してMoveWindow

85 ：12/01/26

日本語の段階で既におかしいのにそれを英語にしても無駄

86 ：12/01/29

2つのリスト（配列でもいいですが）a=[a1, a2, a3, ・・・]　b=[b1, b2, b3, ・・・] を
c=[a1, b1, a2, b2, a3, b3, ・・・] の様に1つのリストに合成するのは interleave ですが、
この逆方向の様にリスト c をリスト a とリスト b に分ける関数の名前は何が良いでしょうか
単純にリスト c のある要素を境に2つに分けるなら split や break が適切だと思いますが・・・
とりあえず今のところ、トランプカードを配る様に見えるので deal と名付けていますが、
もっと適切な名前はありますか？

87 ：12/01/29

どういう基準で分けるかぐらい書けば？

88 ：12/01/29

>>87
いや、だから interleave とは逆なんですけど・・・
c=[a1, b1, a2, b2, a3, b3, ・・・] を
a=[a1, a2, a3, ・・・] と b=[b1, b2, b3, ・・・] に分ける

89 ：12/01/29

>>87
言葉で説明すると
リストcの n/2 番目の要素をリストaのn番目の要素に
リストcの (n/2)+1 番目の要素をリストbのn番目の要素に入れる
そんな分け方です

90 ：12/01/29

すいません、間違ってました
リストcの 2n 番目の要素をリストaのn番目の要素に
リストcの 2n+1 番目の要素をリストbのn番目の要素に入れる
でした

91 ：12/01/29

>>86
distribute なり divide とかでいいんじゃね？

92 ：12/01/29

Cutでいいよ

93 ：12/01/29

プロジェクト名ってどうやって決めてますか？
公開するとなるとさらに慎重になりますよね

94 ：12/01/29

>>86
interleave ⇔ deinterleave
で良いのでは？
単に1つ(or n)おきに取り出すだけならdecimateもありだと思う。

95 ：12/01/29

uninterliveか、 >>94 が挙げている deinterleave か、迷った。
>>94
decimateはdec-に「10ごと」という意味があるのと、
その10ごとの1を「処刑する」という意味があるのがよろしくない気がする。

96 ：12/01/29

decimateは普通のプログラマはあまり使わないかも知れないが、間引きを表す術語みたいなものなんですよ。

97 ：12/01/30

>>93
状況次第だけど、ソフトウェアの名前そのまま、もしくは逆に一般名詞にすることが多いかなあ。

98 ：12/01/30

あえて全然関係ないコードネームにする
そうするとカフェで仲間と話し合うときにハッカーっぽさがましてカッコイイ気分になれる

99 ：12/01/30

一周を[0, 1)で正規化した角度のことをなんて名付けるべきかな
（0.5なら弧度法でのpiに、1.0なら弧度法での2piに相当する）

100read 1read

1read 100read

TOP カテ一覧 スレ一覧 2ch元 削除依頼 ▲

【Google】Androidアプリ作成part13 (294)
メガデモを語る fr-08 (603)
人工知能は感情を獲得する事ができると思いますか? (480)
IS<インフィニット・ストラトス>総合 (491)
datファイルを共有するP2Pソフト o2on 17dat (286)
BASICの宿題はお前にまかせた (550)
--log9.info------------------
【Lycee】リセ質問スレ その32【講習会in2ch】 (294)
【海外記事】遊戯王　4訳目【翻訳】 (549)
イナズマイレブンGO TCG 3ゴール目 (223)
【遊戯王】パーミッションスレ Part11 (829)
【DB】ミラクルバトルカードダス【OP】 Part2 (803)
★モンコレ限定　オラがデックは世界一！ 8/8/8 (749)
【紅葵】遊戯王の霊使い達って萌えるよな2【雅鉄】 (784)
【遊戯王】おジャマで相手をお邪魔するスレ3 (726)
【MTG】土地破壊デッキで天下取ろう【ランデス】 (205)
【モンコレ】コラボレーション総合スレ (539)
MTGアンチスレ 18 (254)
【遊戯王】　獣族・獣戦士族総合スレ (328)
バトルスピリッツのオリジナルカード【バトスピ】 (467)
同人カードゲーム　system[DA,BA] (499)
【MTG】MTG Wikiについて語るスレ5【ギャザリング】 (956)
ｽﾜﾛﾌｽｷｰな感じでｽﾀｰﾀﾞｽﾄﾄﾞﾗｺﾞﾝを作りたい (102)
--log55.com------------------
【医療】アトピー性皮膚炎で第3の抗炎症外用薬が登場
【速報】東京の各所で商品の買い占め、買い溜めが始まる　小池都知事の会見受け　スーパーに大行列　もはやパニック ★46
【東京】渋谷に若者！18歳男性「服を買いに行ったり、ぶらぶら街を探索しました」　 批判的に報道　海外メディア
【ＷＨＯ】テドロス「各国はこれ以上貴重な時間を無駄にすべきでない」 「行動すべき時期は実際、１カ月余りまたは２カ月前だった」
【外出自粛】「落ち着いた購買行動を」農水省が消費者に呼びかけへ
【速報】安倍晋三「東京は新型コロナ蔓延のおそれが高い」 ★4
【歴史】硫黄島の戦いから75年、２万8000人が戦死…遺骨収集が課題
【コロナ】1月からずっと警告を発していたのは高須院長や保守系の人々。リベラル系はデマだと言っていた。実はイギリスでも同じ★2