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

1 名前：デフォルトの名無しさん [04/02/19 05:59] var unko; 616 名前：デフォルトの名無しさん mailto:sage [2008/10/11(土) 22:35:28 ] それでもコメント書かない主義ってのはダメだな 「function**は○○用の関数」とか一言書いておくだけでかなり違うのに 617 名前：デフォルトの名無しさん mailto:sage [2008/10/12(日) 00:50:10 ] 他人に保守させるとかじゃなければOKかな 618 名前：デフォルトの名無しさん mailto:sage [2008/10/13(月) 07:41:36 ] コメントをコピペ修正忘れするくらいなら、コメントを書くことよりも、まともな関数名を考えることに注力して欲しい。 駄目な奴は何をやっても駄目だからしかたないが。 619 名前：デフォルトの名無しさん mailto:sage [2008/11/08(土) 20:52:10 ] >>580 string-> "S"tring　か?w ケツ蹴っ飛ばしたれ。w 620 名前：デフォルトの名無しさん [2008/11/09(日) 19:23:34 ] >>619 つっこみどころは arg1, arg2 だろ(´д｀;) 621 名前：デフォルトの名無しさん mailto:sage [2008/11/09(日) 19:38:32 ] どうやっても値が返らないってところに突っ込むべきじゃ。 622 名前：デフォルトの名無しさん mailto:sage [2008/11/09(日) 22:20:26 ] 内部状態更新するんじゃね？ 623 名前：デフォルトの名無しさん mailto:sage [2008/11/09(日) 22:30:41 ] create_string といういかにも値返しそうな名前なのに procedure というところ？ 624 名前：デフォルトの名無しさん mailto:sage [2008/11/10(月) 09:55:07 ] 突っ込みどころが多い、とｗ しかし単語の先頭を大文字にした上でアンダースコアでつないでるのを見るとイラっとくるな。 625 名前：デフォルトの名無しさん mailto:sage [2008/11/11(火) 07:18:47 ] C#ならstring小文字もメソッド名先頭大文字も普通。 アンダースコアはないが、他にそういう慣習の言語もあるのかも知れん。 言語を明示しないと突っ込みづらい。 626 名前：デフォルトの名無しさん mailto:sage [2008/11/11(火) 13:47:02 ] そういやAdaは予約語以外は大文字で始めて単語の切れ目は_入れるんだったな。 procedure Create_String_Procedure(Arg1, Arg2: in String) is : begin : end Create_String_Procedure; 580を冗長に書き直すと、こんなんだったはず。 まぁコンパイラが大文字小文字を区別しないから、小文字で書いても全く問題ないんだけど。 627 名前：デフォルトの名無しさん [2008/11/26(水) 02:03:59 ] >>580 で、こたえは？ 628 名前：デフォルトの名無しさん mailto:sage [2008/11/28(金) 18:38:24 ] >>537のってだめなの？ old new num はたまに使うんだけど 629 名前：デフォルトの名無しさん mailto:sage [2008/11/28(金) 18:56:44 ] >>628 newはしばしば予約されている。 oldやnumも、単体で使うのは有り得ない。 変数名の一部として使う分にはまぁいいと思うけどね。 >537は教条主義に陥っている気がするな。 630 名前：デフォルトの名無しさん mailto:sage [2008/11/28(金) 19:54:52 ] >>629 ああいや単体では使わんよ。 ただ変数名に含める事は結構あるんじゃないかと。 631 名前：デフォルトの名無しさん mailto:sage [2008/11/29(土) 12:16:02 ] 多くの場合に、 old, new よりは previous, current, next のほうがより明確だと思う。 対象領域に old, new の概念があり、それを表すならば、もちろん old, new がふさわしいだろう。 632 名前：デフォルトの名無しさん mailto:sage [2008/11/29(土) 18:05:50 ] なるほど、確かにpreviousとnextの方がいいかも。 633 名前：デフォルトの名無しさん mailto:sage [2008/11/29(土) 21:22:32 ] >537の理屈じゃ、oldTypeとかnewTypeでもダメそうだもんなw 634 名前：デフォルトの名無しさん mailto:sage [2008/12/01(月) 22:03:48 ] >>537のは、結局"故にすべてのプログラムはクソ"というｵﾁなんじゃない？ 635 名前：デフォルトの名無しさん mailto:sage [2008/12/02(火) 01:09:20 ] >>633 普通にダメだと思うぞそんな名前 636 名前：デフォルトの名無しさん mailto:sage [2008/12/02(火) 09:58:51 ] newが入ってるとこみると名前の一部だよなそれ？ 637 名前：デフォルトの名無しさん mailto:sage [2008/12/05(金) 09:57:06 ] >627 CamelでC#と勝手に予想して>621が正解。 最近C#書く機会が多いから俺もそうオモタ。 変化球でメンバ更新用だとしてもメソッド名で（ ´,_ゝ｀）できる。 638 名前：デフォルトの名無しさん mailto:sage [2008/12/05(金) 10:20:28 ] それは名前に突っ込んでるのか仕様につっこんでるのか 639 名前：デフォルトの名無しさん mailto:sage [2008/12/05(金) 19:57:42 ] stringはstd::stringなのか？ 640 名前：デフォルトの名無しさん [2009/01/08(木) 02:50:44 ] year2009 641 名前：デフォルトの名無しさん mailto:sage [2009/01/08(木) 12:03:10 ] bool eroikkaizutu = true; 642 名前：デフォルトの名無しさん mailto:sage [2009/01/08(木) 21:59:30 ] >>616 コメントが正しければね。 /* DB接続 */ int connect() /* DB接続 */ int disconnect() /* DB接続 */ int select() みたいなのがあると悲しくなるよ。 643 名前：デフォルトの名無しさん mailto:sage [2009/01/09(金) 00:52:00 ] アニメとか漫画のキャラの名前を変数に使うなよ！！！ 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) ] / [ 携帯版 ]

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