[表示 : 全て 最新50 1-99 101- 201- 301- 401- 501- 601- 2chのread.cgiへ]
Update time : 05/09 15:07 / Filesize : 121 KB / Number-of Response : 678
[このスレッドの書き込みを削除する]
[+板 最近立ったスレ&熱いスレ一覧 : +板 最近立ったスレ/記者別一覧] [類似スレッド一覧]


↑キャッシュ検索、類似スレ動作を修正しました、ご迷惑をお掛けしました

【汚物】痛い変数名・関数名【破廉恥】



1 名前:デフォルトの名無しさん [04/02/19 05:59]
var unko;

644 名前:デフォルトの名無しさん mailto:sage [2009/01/09(金) 02:04:22 ]
bool dame;



if (!dame) {
//登録失敗
 …
}



どっちなんだよ。
dame は駄目だろ。
もういやだ。


645 名前:デフォルトの名無しさん [2009/01/11(日) 00:49:09 ]
>>644
つまりソース書いたやつがdame

646 名前:デフォルトの名無しさん mailto:sage [2009/01/11(日) 09:17:42 ]
次いってみよう!

647 名前:デフォルトの名無しさん mailto:sage [2009/01/12(月) 01:38:03 ]
int NullPo;

648 名前:デフォルトの名無しさん [2009/01/28(水) 15:22:48 ]
if (だめだこりゃ) 次いってみよう;

649 名前:デフォルトの名無しさん [2009/01/28(水) 15:40:31 ]
カセット!

650 名前:デフォルトの名無しさん mailto:sage [2009/01/28(水) 18:51:40 ]
while ("終わりが無いのが終わり それがゴールド・エクスペリエンス・レクイエム") {








 break;
}

651 名前:デフォルトの名無しさん mailto:sage [2009/01/29(木) 00:55:28 ]
>>650
breakすんな

652 名前:デフォルトの名無しさん mailto:sage [2009/01/31(土) 00:28:52 ]
変数名や関数名にまともな名前付けれない奴が良いプログラミングなんてできるわけがないに激しく同意。
うちの部署にもいるが、「変数名なんて何でもいい。考えている時間が無駄だ。」とか言い出す始末。

関数クラスのクラス名→F
グローバル変数クラス→G
業務処理クラス→GYM

絶対クラスの意味わかってねーだろコイツ。

FC.ToStr("あああ"); //文字列変換処理

なんか意味あんのかこのメソッドは。プロなのかコイツは。



653 名前:デフォルトの名無しさん mailto:sage [2009/01/31(土) 09:15:55 ]
その程度の職場にその程度のプログラマ

654 名前:デフォルトの名無しさん mailto:sage [2009/01/31(土) 12:49:29 ]
そんな名前をつけられる位なら、逆に30年前の大型汎用機の開発のような「連番+台帳管理」の方がマシだな。
(すくなくとも記録は確実に残るし)


655 名前:デフォルトの名無しさん mailto:sage [2009/01/31(土) 15:34:20 ]
グローバル変数クラスなんていうシロモノがある時点でオワットルな


656 名前:デフォルトの名無しさん mailto:sage [2009/01/31(土) 17:49:37 ]
>>業務処理クラス→GYM
COBOLer警報が鳴り止まない。

657 名前:デフォルトの名無しさん mailto:sage [2009/02/08(日) 03:30:35 ]
普通
private int m_hoge;
コンストラクタ(int hoge){this.m_hoge = hoge;}
て、外出ししない変数にプレフィックスとか(必要なら)つけると思うけど

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

658 名前:デフォルトの名無しさん mailto:sage [2009/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 名前:デフォルトの名無しさん mailto:sage [2009/02/08(日) 05:15:16 ]
スコット・メイヤー先生やハーブ・サッター先生の教えに従えば
メンバ変数が公開されることはないので
メンバ変数にはテキトーな名前をつけてもいいんじゃないだろうか…

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

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

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

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

662 名前:658 mailto:sage [2009/02/08(日) 12:01:07 ]
>>661
ほんとだ。ごめん。 m_hoge 見て C++ だと思い込んだ。

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



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

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

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

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

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

666 名前:デフォルトの名無しさん mailto:sage [2009/02/08(日) 14:50:58 ]
そういう問題じゃないだろ・・・

667 名前:デフォルトの名無しさん mailto:sage [2009/02/08(日) 17:08:49 ]
命名規則スレが別になかったっけ?

668 名前:デフォルトの名無しさん mailto:sage [2009/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 mailto:sage [2009/02/12(木) 00:40:10 ]
>>658
>>661
すみません、ほんとすみません、>>657はC#のつもりです(;´Д`)
べつにJavaと変わらんけど

670 名前:デフォルトの名無しさん [2009/03/20(金) 03:19:07 ]
hogeの次に使う変数ってpiyo?

671 名前:デフォルトの名無しさん mailto:sage [2009/03/20(金) 03:21:48 ]
fuga


672 名前:デフォルトの名無しさん mailto:sage [2009/03/20(金) 03:28:59 ]
huhun



673 名前:デフォルトの名無しさん mailto:sage [2009/03/20(金) 03:38:01 ]
mogyo

674 名前:デフォルトの名無しさん [2009/03/20(金) 04:15:47 ]
zamasuzamasu

675 名前:デフォルトの名無しさん mailto:sage [2009/03/25(水) 00:49:23 ]
so_degansu

676 名前:デフォルトの名無しさん mailto:sage [2009/03/25(水) 10:24:54 ]
hunga_()

677 名前:デフォルトの名無しさん mailto:sage [2009/03/25(水) 10:37:40 ]
maji
meni
haji
mena
saiyo






[ 新着レスの取得/表示 (agate) ] / [ 携帯版 ]

前100 次100 最新50 [ このスレをブックマーク! 携帯に送る ] 2chのread.cgiへ
[+板 最近立ったスレ&熱いスレ一覧 : +板 最近立ったスレ/記者別一覧]( ´∀`)<121KB

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef