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

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

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

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

前スレ
クラス名・変数名に迷ったら書き込むスレ。Part19
http://hibari.2ch.net/test/read.cgi/tech/1293201229/
0205デフォルトの名無しさん2011/10/29(土) 16:15:50.91
アンダーバーのプレフィクスを使うと総合開発環境の入力候補でprivateメンバを後ろに追いやる事ができるのが便利なんだよ

0206デフォルトの名無しさん2011/10/29(土) 16:24:59.02
>>205
現状それしか方法がない、仕方がないとは言え、
やはり解決方法としてはダサいの一言に尽きますなぁ

入力候補の並び替え機能は何で付けないんだろ
0207デフォルトの名無しさん2011/10/29(土) 17:07:54.16
privateメンバを後ろに追いやりたいという動機そのものが理解できないけどね俺は。
0208デフォルトの名無しさん2011/10/29(土) 18:06:11.19
privateとかpublicを判断してソートしてくれりゃいいのにね
intelliなんだから
0209デフォルトの名無しさん2011/10/29(土) 18:13:09.03
むしろ、そんな必要性を感じるのなら、きっとコードの書き方が間違ってるんだよ。
0210デフォルトの名無しさん2011/10/29(土) 19:26:38.46
>>209
一覧の表示方法とコードの書き方にどのような繋がりがあるのか、
もっとしっかりちゃっかりまったり説明してくれ
0211デフォルトの名無しさん2011/10/29(土) 19:41:55.54
当たり前のことだが、アクセスレベルとはクラスの外側からのメンバーの見え方を制御するもの。

つまり視点をクラスの内部に置いている時（そのクラスを書いている時）には
呼び出そうとしているメソッドのアクセスレベルなんか気にしても意味ない。

っていうか、メソッド一覧がアクセスレベルで分類なんかされたら
かえってありがた迷惑でしかない。
0212デフォルトの名無しさん2011/10/29(土) 20:02:37.84
>>211
外からアクセスしねーのに一覧に表示されたら邪魔だろ
そんな簡単なことも分かんねーのかモンキー
0213デフォルトの名無しさん2011/10/29(土) 20:06:43.73
intelliって言ってるからMSのVSの事を言ってるんだと思うが、
元々MSのインテリセンスは現在のコンテキストから見えるメンバしか表示されませんよ。

何意味不明なこと言ってるんだろうね。
0214デフォルトの名無しさん2011/10/29(土) 20:13:42.29
などと意味不明の供述をしており動機は未だ不明
0215デフォルトの名無しさん2011/10/29(土) 20:54:54.39
>>213
病院行ってこい
0216デフォルトの名無しさん2011/10/29(土) 22:42:29.85
（え、protected/privateでも表示されるよな・・・）
0217デフォルトの名無しさん2011/10/29(土) 22:53:50.02
213っておかしなこと言っているか？
その通りだと思ったが
protected/privateが現在のコンテキストから見えれば表示される
0218デフォルトの名無しさん2011/10/29(土) 22:57:07.69
C#は出ないけどVC++はコンテキスト関係なく全部表示されなかったか
0219デフォルトの名無しさん2011/10/30(日) 05:57:32.71
変数名にクラス名サフィックス付いてるのよく見るけど良いのか悪いのか分からない
熟語になるし一目で型は分かるけど補完に使えない
0220デフォルトの名無しさん2011/10/30(日) 10:30:28.54
>>218
またVC#贔屓か
0221デフォルトの名無しさん2011/10/30(日) 11:44:24.83
↑アホ
0222デフォルトの名無しさん2011/10/30(日) 11:49:43.31
贔屓どころかC++のインテリセンスの方が普通に考えてコストかかってるだろう
C#のように完璧は無理だから間違って出るはずのものが出ないよりは全部出たほうがマシって判断なんじゃね
0223デフォルトの名無しさん2011/10/30(日) 12:02:25.34
いいから黙ってろザコ供
0224デフォルトの名無しさん2011/11/02(水) 12:49:40.72
何か処理する前と後に呼び出されるコールバック関数があるんだけど
Hoge(preHoge , ???Hoge);
???って何が入るの？
0225デフォルトの名無しさん2011/11/02(水) 13:01:25.04
preの反対はpost
0226デフォルトの名無しさん2011/11/02(水) 13:53:23.93
はい次の患者さんどうぞ
0227デフォルトの名無しさん2011/11/02(水) 17:42:51.33
callableかな
0228デフォルトの名無しさん2011/11/02(水) 19:05:14.63
HogeHeader
HogeFooter

・・・最高にダサいな
0229デフォルトの名無しさん2011/11/02(水) 22:47:39.87
ディスパッチャーとコントローラーってどう違うの・・・？
0230デフォルトの名無しさん2011/11/02(水) 22:57:48.15
何もかも違うだろう
何が同じだと思ったんだ
0231デフォルトの名無しさん2011/11/02(水) 22:58:14.64
ディスパッチャーは何かを適切なところへ送り届ける役目。
コントローラーは何かを管理・制御する役目。
0232デフォルトの名無しさん2011/11/02(水) 23:41:37.27
むしろコントローラーといったらディスパッチされる側になることが多いんじゃないか
0233デフォルトの名無しさん2011/11/03(木) 01:20:55.79
>>231
ありがとう
ということは今までコントローラーとつけてたのはディスパッチャーに
マネージャーとつけてたのはコントローラーにしないとなあ
0234デフォルトの名無しさん2011/11/03(木) 08:29:06.18
コントローラーは入力を適切な形に変換して対象のオブジェクトに届けるんだよ
0235デフォルトの名無しさん2011/11/03(木) 09:04:19.79
例として、2Dのシューティングゲームを想像してほしいのだけれど、
自機、敵、弾丸、アイテムなど、プレイ中に出現する「動くもの」全般の基底クラスの名前をお願いしたい。

これらは、自身の座標、移動量、生存フラグ、出現してからの時間といった変数の他、
自身の更新メソッド＆描画メソッドなどを持っている。

0236デフォルトの名無しさん2011/11/03(木) 09:14:42.80
class MovableEntity : public GameEntity ;
class DynamicEntiry : public GameEntity ;
0237デフォルトの名無しさん2011/11/03(木) 11:11:46.66
class Hoge
{
public:
SomeResourceHandle Get(void) { return hRes ; }
private:
SomeResourceHandle hRes ;
};

リソースの所有権がHogeにあって、Getで取得した生ハンドルを外部で保持するのはまずい
ということを利用者にわかりやすく伝えるメソッド名はなにがよいか
0238デフォルトの名無しさん2011/11/03(木) 11:31:57.01
>>237
こんな答えは望んじゃいないだろうが、俺なら外に出さなくてすむようにする。
そしたら、少なくともそういう難しいメソッド名（や、使いにくいルールの押し付け）で悩まなくてすむ。
0239デフォルトの名無しさん2011/11/03(木) 11:35:13.60
>>237
俺なら、そういう「適切な利用方法」に関する情報は名前には入れない
名前には「それは何か」という情報だけを入れる

外部で保持されてはまずい生ハンドルがどうしてGetで取得できちゃうの？
0240デフォルトの名無しさん2011/11/03(木) 11:44:32.43
そういう情報はコメントで書くもんだ
0241デフォルトの名無しさん2011/11/03(木) 11:49:45.20
DangerousGetHandle()
0242デフォルトの名無しさん2011/11/03(木) 12:21:54.03
>>236
ありがとう MovableEntityにしてみる
0243デフォルトの名無しさん2011/11/03(木) 12:33:41.70
>>237
.NETだと下層のAPIをラップしたものなど、動作に関して使用者が責任を
持つ必要がある安全でない型やメンバーはUnsafeプレフィックスが付く

この例だと、UnsafeGetHandle()かUnsafeHandle()あたり
後はドキュメントに取得したハンドルの有効期限などを記述すれば良い
0244デフォルトの名無しさん2011/11/03(木) 13:05:49.72
.NETだとほんとにヤバいのはDangerous
0245デフォルトの名無しさん2011/11/03(木) 15:50:41.16
.NETにある機能の名前ってよく質問に出るね
これくらいググれないものなのかね
0246デフォルトの名無しさん2011/11/03(木) 15:52:53.23
機能から名前って、意外にググるの難しいだろ
余計なものがいっぱい混ざってるし
だから、逆引き辞典の需要があるわけだし
0247デフォルトの名無しさん2011/11/03(木) 15:59:49.02
そうかな
まあ俺が上手いだけか
0248デフォルトの名無しさん2011/11/03(木) 16:39:21.37
そうだよ

だからコツを教えてあげなよ
0249デフォルトの名無しさん2011/11/03(木) 16:42:50.06
247 さんかっこいー
0250デフォルトの名無しさん2011/11/03(木) 20:21:03.48
getAllXXX
のXXXは単数形？複数形？
0251デフォルトの名無しさん2011/11/03(木) 20:28:39.76
複数
0252デフォルトの名無しさん2011/11/03(木) 20:46:55.47
それは、中学英語の範疇ではないのか
0253デフォルトの名無しさん2011/11/03(木) 20:48:43.60
単数にしてる人多いんだよね
0254デフォルトの名無しさん2011/11/03(木) 21:01:57.14
英語的にはallの場合は可算名詞であっても単数になる場合があったと思ったけどね。
プログラムでは常に複数形にするのが無難だとは思うけど
0255デフォルトの名無しさん2011/11/03(木) 21:26:41.46
あれ、念のため確認してみたら、all+可算名詞の場合は常に複数形が正しいみたいだな。

ただ可算名詞であっても、同じ名詞が抽象名詞として使われる場合単数形になる、
ということはあるみたいで、それとごっちゃになって覚えていたのかな。
0256デフォルトの名無しさん2011/11/03(木) 22:47:02.39
複数が前提だけどひとつしか値が返らない場合はどうするんですか？
別に複数形にしなくたってわかるから1byte節約して書かないほうがマシ
0257デフォルトの名無しさん2011/11/04(金) 01:20:06.06
自分の場合、配列など複数の要素を持つ変数は複数形で名付けてる。
そのルールに従うと、今回のメソッドも複数形になる。

が、そもそも自分はAllを省いて getXXXs() で良いと思う派。
0258デフォルトの名無しさん2011/11/04(金) 01:29:43.73
ああ、確かに何か特別な理由でもない限りAllなんて付けないのが普通だな。
0259デフォルトの名無しさん2011/11/04(金) 09:00:21.45
複数形にするだけだとデータ構造が見えないから困る
0260デフォルトの名無しさん2011/11/04(金) 09:10:25.08
そういうものはコメントやドキュメントに書けばいい
0261デフォルトの名無しさん2011/11/04(金) 09:20:56.56
ドキュメント読む手間をなるべく少なくするような命名を模索してんだろ
なんでもドキュメント読めばいいで終わらせてたら名前なんていらんわ
0262デフォルトの名無しさん2011/11/04(金) 09:25:01.92
データ構造は型情報って気もするなあ。
下手するとシステムハンガリアン（） と言われかねないし。
0263デフォルトの名無しさん2011/11/04(金) 09:36:47.14
〜sって「抽象的なコンテナ型である」って意味のハンガリアンみたいなもんだろ
「〜のよくわからない集合」って書くのと「〜のリスト」「〜のスタック」
と書くのどっちがましかという話
まともな神経をしてれば後者を選ぶだろう
コンテナの抽象化はパフォーマンスに直結する悪手だよ
0264デフォルトの名無しさん2011/11/04(金) 09:52:21.50
>>259
困ったことが一度としてないんだがｗ
0265デフォルトの名無しさん2011/11/04(金) 10:12:37.98
名前に情報を詰め込むには限度がある
「〜の集合」てのが最も重要でこれが基本
データ構造を付け足すかどうかはオプション

0266デフォルトの名無しさん2011/11/04(金) 10:16:33.14
データ構造ってリストやスタックみたいな簡単なものばかりじゃないしなぁ
0267デフォルトの名無しさん2011/11/04(金) 10:23:55.59
OOPで考えるとデータ構造はModelクラスとして扱えば十分だからなぁ。
下位の、より小さなコンテナの変数を、getterで外に出すような展開もべつに無いし。
0268デフォルトの名無しさん2011/11/04(金) 10:27:03.38
パフォーマンス気にするならなおさらドキュメント類は読むべきだな
名前だけで判断しようとしてるからコケる
0269デフォルトの名無しさん2011/11/04(金) 10:48:34.99
hogesってところまでしかわかっていない場合は
例えば検索するときにどのアルゴリズムを選択するかとかが判断しにくくなる
出社してそれほど時間が経過していなくて頭がすっきりしていて型が完全に頭に入った状況でのコーディングならhogesで問題はないが
何時間もディスプレイやキーボードと格闘して疲労の溜まったPGが5分前の記憶を覚えてるとは限らない
そのたびにhogesってどんな集合だっけとプロトタイプを確認する作業が追加される
さいしょからsortedHogeVectorと名前がついてれば３日徹夜した後であろうとも直ちにランダムアクセス可能、二分探索が有効とわかる
0270デフォルトの名無しさん2011/11/04(金) 10:52:39.18
SortedVector hoges;
でおｋ
0271デフォルトの名無しさん2011/11/04(金) 10:55:59.53
変数の型を忘れるほど「遠くに」書いてしまうようでは、
変数名を工夫したところで負担の小さいコードにはならない。
02722712011/11/04(金) 10:56:53.26
>>270
同意。
0273デフォルトの名無しさん2011/11/04(金) 11:27:20.40
動的型言語だったらどうするの？
0274デフォルトの名無しさん2011/11/04(金) 11:41:05.22
パフォーマンス気にして動的型付けねぇ
へぇ
0275デフォルトの名無しさん2011/11/04(金) 11:41:12.28
変数名に実装方法を現すような単語まで入れちゃうと、
やっぱ遅いから同じインターフェイスのあっちの実装にとかなったら、
変数名ぜんぶ変えなきゃいけなくなっちゃう。
0276デフォルトの名無しさん2011/11/04(金) 11:42:13.30
表す
0277デフォルトの名無しさん2011/11/04(金) 12:42:11.06
>>275
同意

>>269 なんか、パフォーマンス考えたら配列から
ツリー構造（順序付きキュー）に変わる可能性も大いにある

そうなったら、
何時間もディスプレイやキーボードと格闘して疲労の溜まったPGが
sortedHogeVector を全部変えるんだぞ

まぁ、普通そういう探索アルゴリズムとかは、
コンテナを利用する側ではなく提供する側が実装すべきだけどな
まさに実装方法に依存するんだし
0278デフォルトの名無しさん2011/11/04(金) 12:43:47.07
フルボッコわろた
0279デフォルトの名無しさん2011/11/04(金) 15:07:25.73
コンテナ代わったらメソッドも変わるから書き換え作業しなきゃならないのは同じだろ
0280デフォルトの名無しさん2011/11/04(金) 15:50:57.20
ぶっちゃけ名前が原因で問題が出るぐらいにあちこちで変数に直接触ってる時点で負けと気がつくべきだろうね
メンバ関数やインラインのヘルパ関数を適切に使っていれば変数名なんて長くても三文字（気をきかせるなら一行コメント）もあれば十分と気がつくよ
0281デフォルトの名無しさん2011/11/04(金) 15:57:52.32
さらにアホが寄ってきた
0282デフォルトの名無しさん2011/11/04(金) 16:00:40.89
いや言ってることは正しいだろ。3文字で十分はネーヨって思うけど
0283デフォルトの名無しさん2011/11/04(金) 18:00:57.78
昔のベーシックは2文字だったよな
0284デフォルトの名無しさん2011/11/04(金) 18:01:53.94
あ、識別子として区別されるのが頭2文字で、
変数名自体は何文字でも良かったか
0285デフォルトの名無しさん2011/11/04(金) 18:25:16.40
たしかスコープも配列初期化も無かったろ
0286デフォルトの名無しさん2011/11/04(金) 23:37:36.00
>>269
お前はメモ帳でコーディングしてんのか？
0287デフォルトの名無しさん2011/11/04(金) 23:42:28.20
ったりメーだろバカ
0288デフォルトの名無しさん2011/11/06(日) 11:24:21.19
λが有るとまじで変数名いらないな
0289デフォルトの名無しさん2011/11/10(木) 17:19:22.13
商品が表示されるページが店舗別か全体表示かのフラグの名前で悩んでます。
isStore にしてるんですが、何か違う気がしてます。
〜別、という場合はどんな名前が良いでしょうか？
0290デフォルトの名無しさん2011/11/10(木) 18:13:31.63
>>289
〜別だとpartとかsectionとかsegmentとか
どれもフラグっぽくないな
store_sectionとかするとcobolっぽくなるな
0291デフォルトの名無しさん2011/11/10(木) 18:46:29.75
isStoreLocalView
0292デフォルトの名無しさん2011/11/10(木) 18:49:43.52
もうcatalogDisplayModeにしとけば？
0293デフォルトの名無しさん2011/11/10(木) 19:01:35.59
データベースだと、group by store とかになりそうだし、そこからヒントに出来ないものか
0294デフォルトの名無しさん2011/11/10(木) 22:21:25.52
店舗別に表示するかどうか
SetShowsByStore(value : bool) : void { showsByStore = value; }
ShowsByStore() : bool { return showsByStore; }
0295デフォルトの名無しさん2011/11/11(金) 00:29:16.13
なにそのデタラメ
0296デフォルトの名無しさん2011/11/11(金) 08:13:53.47
至るところで細かく

if( isStore ){
    〜
}

とかやってると思うとゾワッと来た
0297デフォルトの名無しさん2011/11/11(金) 08:37:57.01
主語がねえ
0298デフォルトの名無しさん2011/11/11(金) 11:02:00.51
言語は？Java？
0299デフォルトの名無しさん2011/11/11(金) 18:25:55.53
isHogeは疑問文じゃなくて Object is hoge だって主張を何処かで見た気がするんだけど、
これってOOの場合のみなんだろうか
0300デフォルトの名無しさん2011/11/11(金) 18:29:52.42
Cでもisdigitとかあるじゃん
0301デフォルトの名無しさん2011/11/11(金) 19:37:22.88
hoge is Valid object.
↓
hoge.isValidObject();
↓
hoge.isValid();
0302デフォルトの名無しさん2011/11/12(土) 04:33:37.34
>300は>297に対し？
0303デフォルトの名無しさん2011/11/12(土) 05:33:05.62
直前のレスに対してだろ・・・
0304デフォルトの名無しさん2011/11/12(土) 07:59:33.42
>>303
is〜の形式が、疑問文かどうかという問いに一切触れてなくね？
0305デフォルトの名無しさん2011/11/12(土) 10:00:33.43
XXX.isYYYとかisYYY(XXX)は疑問文じゃないよ
「XXXはYYYである」
という命題の真偽を返り値として返すということ
だって疑問文だとnotしたときにおかしくなるだろ？
■ このスレッドは過去ログ倉庫に格納されています