【汚物】痛い変数名・関数名【破廉恥】at TECH
【汚物】痛い変数名・関数名【破廉恥】 - 暇つぶし2ch657:デフォルトの名無しさん
09/02/08 03:30:35
普通
private int m_hoge;
コンストラクタ(int hoge){this.m_hoge = hoge;}
て、外出ししない変数にプレフィックスとか(必要なら)つけると思うけど

private int hoge;
コンストラクタ(int hogeArg){this.hoge = hogeArg;}
て書く人がいる。

658:デフォルトの名無しさん
09/02/08 03:49:06
>>657
> コンストラクタ(int hoge){this.m_hoge = hoge;}

普通は初期化ですむところで代入なんか使わない。
 コンストラクタ(int hoge) m_hoge(hoge) {}

> コンストラクタ(int hogeArg){this.hoge = hogeArg;}

利用者側に見せる名前に、実装側の都合で変な装飾がつくのは良くないね。
ここも初期化を使っていればこれでいい。
 コンストラクタ(int hoge) hoge(hoge) {}

659:デフォルトの名無しさん
09/02/08 05:15:16
スコット・メイヤー先生やハーブ・サッター先生の教えに従えば
メンバ変数が公開されることはないので
メンバ変数にはテキトーな名前をつけてもいいんじゃないだろうか…

と思ったが、コードは書かれるよりも読まれることの方がずっと多いと
アンドリュー・ハント先生が言っていたのを思い出した。

やっぱりわかりやすく書かないとダメだ…

660:デフォルトの名無しさん
09/02/08 07:19:00
メンバ変数のプリフィックスにアンダースコアを付けるのは
グローバルスコープでないのだから良いと思っていたが、
マクロというスコープを越えてやって来る邪悪なる存在に襲われ、
次回のプロジェクトではアンスコはサフィックスにすることにした。

661:デフォルトの名無しさん
09/02/08 10:57:36
>>658
private int hoge;
からしてC++じゃなくてjavaだろ、常識的に考えて。

662:658
09/02/08 12:01:07
>>661
ほんとだ。ごめん。 m_hoge 見て C++ だと思い込んだ。

Java にしても this.hoge = hoge でいいから、やっぱり
利用者側に(略)変な装飾がつくのは良くないね。

663:デフォルトの名無しさん
09/02/08 12:28:59
ぱっと見で把握できないような大きめのクラスは必ずpimplになるから
実質this->付いてるのと同じだな、自分の場合。

664:デフォルトの名無しさん
09/02/08 12:58:18
利用者側に変な装飾付けない方がいいのは同意。
装飾付けるなら内部の方がいい。

メンバ変数に装飾付けないプログラム見たけど、
どれがメンバ変数なのか分かり辛くて大変だった。
しかも、メンバ変数と同名のローカル変数作ってたり・・・。
最後まで読み進めると、その同名のローカル変数しか使ってなくて、
メンバ変数使って無いじゃん! とか。

this.hoge 派は this. で装飾付けた気になってんだろうけど、
それ知らない奴がそのコードいじると this. 付けなかったりする事もあるから
独りよがりなコードだね。

665:デフォルトの名無しさん
09/02/08 14:41:37
とりあえずメンバ変数宣言をコメント化すればすぐに分かったのにね。

666:デフォルトの名無しさん
09/02/08 14:50:58
そういう問題じゃないだろ・・・

667:デフォルトの名無しさん
09/02/08 17:08:49
命名規則スレが別になかったっけ?

668:デフォルトの名無しさん
09/02/10 23:08:27
dpkg-1.14.20ubuntu6/lib/dpkg.h
>void ohshit(const char *fmt, ...)
>void ohshitv(const char *fmt, va_list al)
>void ohshite(const char *fmt, ...)
>void ohshitvb(struct varbuf*)

669:657
09/02/12 00:40:10
>>658
>>661
すみません、ほんとすみません、>>657はC#のつもりです(;´Д`)
べつにJavaと変わらんけど

670:デフォルトの名無しさん
09/03/20 03:19:07
hogeの次に使う変数ってpiyo?

671:デフォルトの名無しさん
09/03/20 03:21:48
fuga


672:デフォルトの名無しさん
09/03/20 03:28:59
huhun

673:デフォルトの名無しさん
09/03/20 03:38:01
mogyo

674:デフォルトの名無しさん
09/03/20 04:15:47
zamasuzamasu

675:デフォルトの名無しさん
09/03/25 00:49:23
so_degansu

676:デフォルトの名無しさん
09/03/25 10:24:54
hunga_()

677:デフォルトの名無しさん
09/03/25 10:37:40
maji
meni
haji
mena
saiyo


最新レス表示
レスジャンプ
類似スレ一覧
スレッドの検索
話題のニュース
おまかせリスト
オプション
しおりを挟む
スレッドに書込
スレッドの一覧
暇つぶし2ch