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

1 名前：デフォルトの名無しさん [04/02/19 05:59] var unko; 490 名前：デフォルトの名無しさん mailto:sage [2008/02/09(土) 12:25:49 ] iとjが区別しやすいフォント使えよ。 ……と言って済ませたいところだが、実は俺もi,jは避けて、 for (int r = 0; (略)) // Row Number 　　for (int f = 0; (略)) // Field Number とかやっているのだ。 491 名前：デフォルトの名無しさん mailto:sage [2008/02/09(土) 17:16:07 ] >>490 すなおに row, field にしとけよ。 492 名前：デフォルトの名無しさん mailto:sage [2008/02/09(土) 17:20:24 ] >>490 ＞for (int r = 0; (略)) // Row Number 少なくとも、本当にこう書いているならスレタイ通りの汚物だな。 493 名前：デフォルトの名無しさん mailto:sage [2008/02/10(日) 03:30:46 ] >>489 よし、洗脳完了 っていうか、実際オススメ >>490 フォントはConsolas かHG行書体 494 名前：デフォルトの名無しさん mailto:sage [2008/02/11(月) 11:46:49 ] Dim bcd ' 部コード Dim kcd ' 課コード Dim scd ' 社員コード これならまだ分かるんだ。　　むしろ統一されていれば覚えやすい気もする Dim TTL どうも合計値（ToTaL）らしい。 Time To Liveかと思ったよ。 COBOLer が ASP を触り始めた現場(´･ω･) 495 名前：デフォルトの名無しさん mailto:sage [2008/02/11(月) 22:28:46 ] Transister Transister Logicかと思いそう 496 名前：デフォルトの名無しさん mailto:sage [2008/02/11(月) 23:59:16 ] OTLの亜種かと思った 497 名前：デフォルトの名無しさん [2008/02/12(火) 05:19:57 ] つーか、おまいら変数名にプリフィクス付けないの？ 498 名前：デフォルトの名無しさん mailto:sage [2008/02/12(火) 08:36:16 ] ポインタ以外はつけない 499 名前：デフォルトの名無しさん mailto:sage [2008/02/12(火) 11:09:28 ] なんでポインタだけ特別なんだよ？ 500 名前：デフォルトの名無しさん [2008/02/12(火) 17:20:49 ] >>497 　宣言見れば解るし、名前がまともなら察しがつく。クラスメンバーとローカル変数程度なら、ローカル変数の宣言確認するだけで済む。そもそも宣言確認できない規模で変数が点在する設計は糞。 　因みに、昨今のOOライブラリならプリフィクスはタブー。 501 名前：デフォルトの名無しさん mailto:sage [2008/02/12(火) 23:41:04 ] 別に接頭辞=ハンガリアンでもないだろうが 502 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 00:35:41 ] 個人用である程度ウンコプログラム作るならハンガリアンも全然あり 503 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 01:29:47 ] コーディング規約も定められてない個人用なら ハンガリアンなぞよけい使わんわ 504 名前：デフォルトの名無しさん [2008/02/13(水) 03:15:49 ] ハンガリアンなしでどうやって型の間違いを知るの？ たとえば、 a=b+c ってあって、a、cが整数、bが文字列であっても 実行するまで気づかないの？ 505 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 03:23:57 ] そもそも適切な名前であれば型情報をその名前に含まなくても型の誤認識をしないのが一般の人 506 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 05:00:53 ] ジェネリックプログラミングなら可能な限り型を限定しないのが普通 型名を名前に含めるにしてもtraitsとかcontainerとか抽象化する 507 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 13:45:17 ] ハンガリアンは型の識別よりもプログラマの質の識別ができて便利 508 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 14:30:25 ] >>504 その場合は a と b と c の定義や代入のとこを見ないとなかなか気づかない 一応断っておくが a や b や c という1文字アルファベットの変数名だという前提の話だからな >>506 list とかいう名前だとちょっと困る 509 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 15:23:30 ] >>504 だから宣言を見える範囲に納めろと…。 スクリプトなら BookName とか SerialNumber とか BackgroundImage とかきっちり名前付ければ間違え要がないだろ 510 名前：デフォルトの名無しさん [2008/02/13(水) 16:11:06 ] なんとかtmpとか、なんとかworkという変数名がいちばん痛いな。 511 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 16:36:03 ] C++やDelならまず問題ない。 512 名前：デフォルトの名無しさん mailto:sage [2008/02/13(水) 21:34:26 ] >>504 言語によるね。Javaだったらまず問題にならない。 513 名前：デフォルトの名無しさん mailto:sage [2008/02/14(木) 00:56:52 ] >>510 普通に使うだろ 変数aの値と変数bの値を入れ替えたい時とか いちいち意味のある変数名なんて考えねーよｗ 514 名前：デフォルトの名無しさん mailto:sage [2008/02/14(木) 02:59:17 ] >>510の言ってることは分かる気がする。 なんとか、の後に続いてるのもポイントだな。 変数の置換みたいな、まさにtempでいいような状況より、 もうちょっと複雑なことをしてるときにも、 そういう意味の無い名前ばっかり付けてコードを分からなくする奴がいる。 一時的な値を保存する変数が複数になると目も当てられない。 そういう奴に共通する特徴として、ありふれた処理を 独特のアルゴリズムで実装する、というのがあって、 そこに暗号的な変数名が加わると･･･ 515 名前：デフォルトの名無しさん mailto:sage [2008/02/14(木) 20:35:55 ] まだ一緒のソースいじってないからいいけど、 bRetとかiRetとか書かない奴、頭死んでるんじゃね？ 516 名前：デフォルトの名無しさん mailto:sage [2008/02/14(木) 21:12:31 ] >>515 お前が寂しい奴だということはわかった 517 名前：515 mailto:sage [2008/02/14(木) 21:21:19 ] 俺の頭が死んでたｗｗｗ逆だ逆ｗｗｗｗ bRetとかしか書かない奴は死ね。 518 名前：デフォルトの名無しさん mailto:sage [2008/02/15(金) 21:25:59 ] 【放送】報道ステーション・古舘伊知郎氏が謝罪　「アダルトチルドレン」言葉の意味を知らずに発言し、関係者を傷つける★３ mamono.2ch.net/test/read.cgi/newsplus/1203060700/ アダルトチルドレンも相当糞なネーミングだな 519 名前：デフォルトの名無しさん mailto:sage [2008/02/15(金) 21:57:54 ] コイズミチルドレン 520 名前：デフォルトの名無しさん mailto:sage [2008/02/18(月) 01:52:08 ] ファーストチルドレン 521 名前：デフォルトの名無しさん mailto:sage [2008/02/21(木) 22:47:34 ] 絶対可憐チルドレン 522 名前：デフォルトの名無しさん [2008/02/27(水) 06:17:18 ] ミスターチルドレン 523 名前：デフォルトの名無しさん mailto:sage [2008/02/28(木) 04:48:12 ] 自分はまだ学生なんだけど隣の席の奴がカウンタ i ii iii iiiiっていうふうにしててすごい見にくい 524 名前：デフォルトの名無しさん mailto:sage [2008/02/28(木) 07:36:38 ] ループが４重になってる時点で基本的におかしい。 525 名前：デフォルトの名無しさん mailto:sage [2008/02/28(木) 10:26:43 ] >>524 for ループ1個ごとに変数名変えてるとか？ いずれにしろ、変数名以前の問題で痛いけど。 526 名前：デフォルトの名無しさん mailto:sage [2008/02/28(木) 21:31:48 ] i ii iii iv にすればいいのにと思ったｗ 527 名前：デフォルトの名無しさん mailto:sage [2008/02/29(金) 02:41:39 ] ゲシュタルト崩壊するからやめてやめて 528 名前：デフォルトの名無しさん mailto:sage [2008/02/29(金) 13:42:34 ] xivとかみたくない。 529 名前：デフォルトの名無しさん [2008/02/29(金) 16:11:36 ] >>102 Blue-rayは「青い光」で、固有名詞じゃないからアメリカで商標登録できなかったってのが原因だぜ。 530 名前：デフォルトの名無しさん [2008/03/01(土) 17:29:42 ] 頭.clear(); 531 名前：デフォルトの名無しさん mailto:sage [2008/03/11(火) 13:00:35 ] ハゲたのか 532 名前：デフォルトの名無しさん mailto:sage [2008/03/18(火) 23:01:47 ] ExcelVBA始めたばっかの頃 Dim 魔貫光殺法 as hoge if ピッコロ大魔王 hogehoge then 魔貫光殺法 end if とかやってたら取引先に書式を渡したボケがいて 「ピッコロ大魔王って何ですか？」って真面目に 尋ねられた事がある。 例え自分のためだけに作ろうとも、 自分のPCの中だけにあっても、油断はできない。 一度作ったものはどこへ誰を（何を）経由して漏れるか分からない。 いい勉強になった。 533 名前：デフォルトの名無しさん mailto:sage [2008/03/20(木) 14:59:12 ] 漏れるって段階で会社としてダメだな 534 名前：デフォルトの名無しさん [2008/05/02(金) 13:05:18 ] まぁ、そんなこともあるさｗ 535 名前：デフォルトの名無しさん mailto:sage [2008/05/04(日) 02:02:44 ] 漏れる事が悪いかとか何とかってのは、ここではどうでもいいがな アルファベット1文字または2文字 + 数字 で数字がものによってはabcd...になることがあった まあネタにする程面白くはないが...大層読みづらかった 536 名前：デフォルトの名無しさん mailto:sage [2008/05/04(日) 09:41:24 ] 意味不明 537 名前：デフォルトの名無しさん mailto:sage [2008/05/10(土) 18:46:16 ] 変数名にこれが付いてたら大体クソ data arg num my temp object flag old new ret ptr sign of by 538 名前：デフォルトの名無しさん [2008/06/16(月) 16:37:24 ] 〜オレ用関数の命名規則〜Version.0.0.1 主旨：関数を副作用の強弱でレベル分けする。副作用の少ない関数を、＜純粋度が高い関数＞と呼ぶことにする。 　　　適用する命名規則は低い方に合わせる。自分の呼び出す関数の中で、もっとも低い方に合わせた規則を適用する（main関数は例外とする） 　　　 ＜純粋度が高い関数＞ 純粋度4　全て小文字で、最初の文字をアンダースコアにする　（例：atoi()→_atoi()、sin()→_sin()、strlen()→_strlen()） 　　　「ローカル変数のみを使う。」「ポインタでない引数を使う（もちろん、使わなくても良い）。」 　　　「ポインタの引数を使い逆参照も行うが、中身は書き換えない。」のいずれかのみに該当 純粋度３　単語の先頭が小文字の名前にする　（例：strcat()→strcat()、memmove()→memmove()） 　　　「ポインタ型の引数を受け取り、逆参照して操作する（要素がポインタでない１次配列と構造体まで）。」に該当 　　　※純粋度4の関数Aがローカル変数を純粋度３の関数Bにアドレス渡しする場合は、関数Aは純粋度４に留まって良いとする。 　　　　例えば、関数Aのローカルな配列をソート関数Bに渡す場合、B。 純粋度２　単語の先頭が大文字の名前にする　（例：malloc()→MAlloc()、free()→Free()、rand()→Rand()（←たぶん静的変数を使ってるから）） 　　　「構造体が持つ、ポインタ型のメンバ変数を逆参照し読み書きする。（二重、三重ポインタの使用、リスト操作など）」 　　　「malloc、freeやそれに類似したAPIなどを使う。」 　　　「静的変数の読み書きを行う。」のいずれかに該当 純粋度１　単語の先頭が大文字の名前で　最後にアンダースコアをつける　（例：fopen()→FOpen_()、fprintf()→FPrintF_()） 　　　「グローバル変数を読み書きする。」 　　　「画面への出力や、キーボードからの入力、ネットワーク通信、ファイルを開く、保存、などを行う（主記憶領域以外との通信）」 　　　のどちらかに該当 ＜純粋度が低い関数＞ これから使うつもり('A`;)。変じゃないだろうか。 539 名前：デフォルトの名無しさん [2008/06/16(月) 16:39:25 ] ↑誤爆したorzマジでごめん 540 名前：デフォルトの名無しさん mailto:sage [2008/06/16(月) 19:53:55 ] ＞全て小文字で、最初の文字をアンダースコアにする グローバルな識別子として処理系に予約されています。 絶対に使用しないでください。 541 名前：デフォルトの名無しさん mailto:sage [2008/06/16(月) 21:55:50 ] >>540 最初のアンダースコアを使うのは止めておくことにします。長文の投稿失礼しました。 542 名前：デフォルトの名無しさん [2008/06/17(火) 07:00:34 ] 誤爆なの？ｗ 543 名前：デフォルトの名無しさん [2008/06/17(火) 22:36:11 ] hoge を使う奴はたいてい初心者 544 名前：543 [2008/06/17(火) 23:07:59 ] プロの俺はpneumonoultramicroscopicsilicovolcanoconiosisを使う 545 名前：デフォルトの名無しさん mailto:sage [2008/06/17(火) 23:10:08 ] 珪性塵肺症か。 546 名前：デフォルトの名無しさん mailto:sage [2008/06/17(火) 23:17:08 ] なげえプレースホルダーだなｗ 547 名前：デフォルトの名無しさん mailto:sage [2008/06/17(火) 23:34:31 ] 初心者の女PGでmegu1 megu2 megu3って自分の名前を付ける奴がいた Class Megu1{ meguArg01 = ...; meguArg02 = ...; meguFunc(){...} } って感じで。もう何やってんのか全く分からん 548 名前：デフォルトの名無しさん mailto:sage [2008/06/17(火) 23:46:30 ] >>547 中身をじろじろ見ないでくれという 初級プログラマからの心の叫び。 549 名前：デフォルトの名無しさん [2008/06/19(木) 04:55:47 ] >>547 かわいいなｗ 550 名前：デフォルトの名無しさん mailto:sage [2008/06/19(木) 09:23:29 ] >>540 C/C++に関してはされてないな 551 名前：デフォルトの名無しさん mailto:sage [2008/06/19(木) 11:22:47 ] うむ、C/C++ に関しては、アンダースコア二つで始まるのがメーカー独自拡張だな。 552 名前：デフォルトの名無しさん mailto:sage [2008/06/19(木) 11:40:06 ] >>550-551 いや、アンダースコア始まりで小文字が続くものもグローバルスコープで予約されてるよ。 553 名前：デフォルトの名無しさん mailto:sage [2008/06/19(木) 11:47:22 ] みんながあまりに安易に多用しすぎて混乱しそうだったから予約したんだろうな 554 名前：デフォルトの名無しさん mailto:sage [2008/06/19(木) 18:43:31 ] どうでもいいけど言語によって名前のつけ方のセオリーって全然違うんじゃないの 555 名前：デフォルトの名無しさん [2008/06/20(金) 01:28:19 ] >>554 ひまわり とか？ｗ 556 名前：デフォルトの名無しさん mailto:sage [2008/06/20(金) 07:25:03 ] >>552 VC++ の独自の関数とかそうだしね。 _stricmp とか。 C++ だとローカルスコープでは予約されてないけど やっぱりグローバルスコープでは予約されている。 557 名前：デフォルトの名無しさん mailto:sage [2008/06/20(金) 08:05:50 ] >>547 俺もやってた時期があったｗ どうせバグ報告が来るなら誰が書いたかすぐ解った方が良い、と本気で思ってた。 二週間と続かなかったが。 558 名前：デフォルトの名無しさん mailto:sage [2008/06/20(金) 12:33:09 ] >>551 ＞アンダースコア二つ C++では、識別子に含めることが禁止されている。 先頭だけじゃなく途中でもダメ。 559 名前：デフォルトの名無しさん mailto:sage [2008/06/20(金) 20:02:32 ] 処理系が使用する分には問題ない 560 名前：デフォルトの名無しさん [2008/06/21(土) 03:07:46 ] ハマショーが好きな上司が、クラス名に SurrenderLikeSnow って付けてた。 「悲しみは雪のように」って言おうとしたんだろうが、たぶんSadnessの間違いだと思う 561 名前：デフォルトの名無しさん mailto:sage [2008/06/21(土) 13:25:26 ] 英訳を頑張っちゃった系のソースは読むのがツライ。 その上スペルミスもあると、もう何がなんだかわからなくなる。 562 名前：デフォルトの名無しさん mailto:sage [2008/06/21(土) 13:32:12 ] スペルミス大量でインデントもぐちゃぐちゃなソースがあるけど 意外と読めるもんだな・・・。 イライラはするが。 563 名前：デフォルトの名無しさん mailto:sage [2008/06/21(土) 16:28:26 ] ローカル変数ならまだいいけどDBのテーブル名、カラム名でスペルミスは勘弁してほしい。 使ってる方が恥ずかしくなってくるぜ。 564 名前：デフォルトの名無しさん mailto:sage [2008/06/21(土) 21:59:08 ] なんにせよuwariteよりはマシ。 565 名前：デフォルトの名無しさん mailto:sage [2008/07/14(月) 00:08:28 ] 変数名のセンスが無いやつほど 無駄な一時変数を好む傾向にある。 566 名前：デフォルトの名無しさん mailto:sage [2008/07/14(月) 09:32:27 ] 名前ではないけど、ヘッダファイル(当然いくつかのファイルで include されている)で static 変数を定義して、値が変わらないと悩んでいた人がいた。そら変わらんわなあ。 567 名前：デフォルトの名無しさん mailto:sage [2008/07/14(月) 09:52:44 ] それは、多重定義のエラーが出てstaticをつけたらエラーが消えたからそのまま使ってるんだろうｗ 568 名前：デフォルトの名無しさん mailto:sage [2008/07/15(火) 00:52:58 ] >>566 ヘッダファイルに変数を書く時点で既に変。 （つか、普通はextern 宣言だけだろ？） 569 名前：デフォルトの名無しさん mailto:sage [2008/07/15(火) 01:39:59 ] >>567 あるある 570 名前：デフォルトの名無しさん mailto:sage [2008/07/16(水) 21:44:13 ] >>566-569 それ（static無）やってる職場に以前いたんだが、(当然ながら)全く同じ定義のせいかエラーが出ずにリンク出来てた。 動作も正常。むしろエラーになってくれる方がありがたかったな。という思い出。 571 名前：デフォルトの名無しさん mailto:sage [2008/07/17(木) 01:33:32 ] >>568も変だけどな。 572 名前：デフォルトの名無しさん mailto:sage [2008/07/17(木) 20:12:07 ] 変じゃないだろ ヘッダでメモリ確保しちゃダメなのは常識 573 名前：デフォルトの名無しさん [2008/08/22(金) 14:10:24 ] >>572 ｋｗｓｋ 574 名前：デフォルトの名無しさん [2008/08/22(金) 22:19:51 ] 送信済みデータ保存関数 saveSendedData 575 名前：デフォルトの名無しさん mailto:sage [2008/08/22(金) 22:34:56 ] >>574 読み込み済みデータでreadedDataってのもあったぞ。 576 名前：デフォルトの名無しさん mailto:sage [2008/08/23(土) 00:03:00 ] >>573 ヘッダファイルとして分離する目的は何でしょう？ 複数のモジュールから、同一の定義を参照する事が目的。 ゆえに変数定義など、唯一でなければならないモノをヘッダに書いてはならない。 そもそも、static変数をヘッダで定義して あちこちのモジュールでインクルードせにゃならんってのは、 構造的に間違ってる。　うつくしくない。 577 名前：デフォルトの名無しさん [2008/08/23(土) 21:21:27 ] >>547 getmegu Insert_megu open_megu (man, co) せくはら 578 名前：デフォルトの名無しさん [2008/08/23(土) 21:37:33 ] VB質問スレから Dim strFikeNamae これは秀逸だと思ったので 今までfnameやfilenameにしていたけど これからはfnamaeやfilenamaeにしようと思った。 579 名前：デフォルトの名無しさん mailto:sage [2008/08/23(土) 22:42:06 ] 一瞬単なるタイポかと思わせておいて「なまえ」とは･･ 580 名前：デフォルトの名無しさん mailto:sage [2008/08/23(土) 23:10:21 ] public void Create_String_Procedure( string arg1, string arg2 ){ ・・・・・ } PCをけっ飛ばしたくなった。 581 名前：デフォルトの名無しさん mailto:sage [2008/08/23(土) 23:51:29 ] -aeで終わるとラテン語の複数形みたいに見えるな ラテン語でnameはnomenだが。 582 名前：デフォルトの名無しさん mailto:sage [2008/08/24(日) 00:11:50 ] aglicora, aqua 583 名前：デフォルトの名無しさん mailto:sage ウソダ [2008/08/24(日) 00:32:55 ] programa 584 名前：デフォルトの名無しさん mailto:sage [2008/08/24(日) 00:34:34 ] COM次第だな 悪のCOMをインストールするのだ 585 名前：デフォルトの名無しさん mailto:sage [2008/08/25(月) 21:31:48 ] >>578 Fikeってなんかカッッコイイ 586 名前：デフォルトの名無しさん mailto:sage [2008/08/26(火) 00:11:43 ] >>580 それだけだと何が悪いのか分からん 587 名前：デフォルトの名無しさん mailto:sage [2008/08/28(木) 00:08:42 ] >>580 C#か？キャメルケースなのにアンダーバーでくくられているのが気になるのか、 副作用がある作りにしているのが気になるのか、 argうんちゃっらっていう意味もへったくれもない変数が気になるのか、 Procedureとかわざわざ名前をつけているのが気になるのかハッキリ汁。 あとお前、この業界にいるには 気が短すぎるのとコミュニケーション能力が低すぎると思われるから 他人のこと言う資格ないと思うぞ。 588 名前：デフォルトの名無しさん mailto:sage [2008/08/28(木) 01:39:19 ] まースレタイ通り、変数名・関数名を言ってんだろう。 予想："どんな"StringをCreateするProcedureなのか分からない。ってことじゃないか？ 589 名前：デフォルトの名無しさん mailto:sage [2008/08/28(木) 09:57:27 ] dataやarg1, arg2みたいな名前が適切な場合もあるよね。 GTKでは任意のデータを受け付ける(void *) dataっていう引数を 引き回していたような記憶。 Lispでも高階関数を定義するときにarg1, arg2みたいな引数名を使ったりする。 （この場合は可変長のargsとかが良いのかもしれんが。） 590 名前：デフォルトの名無しさん mailto:sage [2008/08/28(木) 19:53:39 ] 自分だったらl,r使うかなあ 591 名前：デフォルトの名無しさん mailto:sage [2008/08/28(木) 22:00:25 ] lhs rhs 592 名前：デフォルトの名無しさん mailto:sage [2008/08/28(木) 23:46:27 ] >>591 それ、3文字略語はマナー悪いよなぁとも思うし、 もはや専門用語化してて略語じゃないと思える気もするし。 593 名前：デフォルトの名無しさん mailto:sage [2008/08/29(金) 06:28:31 ] 3つ以上あったらどうするの 594 名前：デフォルトの名無しさん mailto:sage [2008/08/29(金) 08:48:05 ] まあ、lhs, rhs は2項演算に対して使う用語だし。 595 名前：デフォルトの名無しさん mailto:sage [2008/08/30(土) 06:52:15 ] >>594 ソートに使用する比較関数の変数とかね。 >>593 3つの引数が等価な計算とか無理やりな例以外見たことない。 596 名前：デフォルトの名無しさん mailto:sage [2008/08/30(土) 08:49:39 ] >>580の引数がたまたま2つだったから二項演算の話になってるのかな ちょっと理解に苦しむ 597 名前：デフォルトの名無しさん mailto:sage [2008/08/31(日) 04:47:26 ] 正直、Effective C++ではじめてrhsとlhsが何の略なのか知った。 598 名前：デフォルトの名無しさん mailto:sage [2008/08/31(日) 06:12:26 ] ロハスとロハスだろ。 599 名前：デフォルトの名無しさん mailto:sage [2008/09/01(月) 13:40:01 ] Perlだと比較関数で使う仮変数は$a, $bだね。 sort { $b <=> $a } @values みたいに指定すると降順ソートができる。安直だけど分かりやすい。 600 名前：デフォルトの名無しさん mailto:sage [2008/09/03(水) 18:46:51 ] でも、$a, $bを特殊変数に指定してしまったのはかなり汚い。 601 名前：デフォルトの名無しさん [2008/09/04(木) 22:35:23 ] int debu1 ; int debu2 ; int debu3 ; 同僚でこんな変数を作っているやつがいた。 「デブデブデブっておまいデブ専なのか？」って聞いたら debuはdebugの略だと言っていた。 602 名前：デフォルトの名無しさん mailto:sage [2008/09/05(金) 05:57:30 ] でぶまで書いたんならデバッグまで書けよでぶ 603 名前：デフォルトの名無しさん mailto:sage [2008/09/07(日) 15:59:30 ] >>600 どんな変数名がいいと思う？ 604 名前：デフォルトの名無しさん [2008/09/07(日) 22:53:26 ] >>580は普通頭に来るだろう。 これを許すなら、文字列を２個受け取って文字列を返すメソッドは 全部同じ名前で1,2,3...が付くだけとかで書かれてしまう。 もちろんこんな名前を付ける人にはクラス名やコメントやドキュメントは期待できない。 605 名前：デフォルトの名無しさん mailto:sage [2008/09/07(日) 23:09:04 ] 引数自体に意味を持たせず、ただ順序だけが重要なのかも。 いずれにしろ、>>580自身が説明しないとダメでしょう。 606 名前：デフォルトの名無しさん mailto:sage [2008/09/07(日) 23:10:09 ] >>601 それぜったい狙ってやってると思う 607 名前：デフォルトの名無しさん mailto:sage [2008/09/08(月) 10:47:00 ] 昔火の付いたプロジェクト手伝ってたら、 int型のflagつう変数があって0がオンで1がオフに使われてた。 コメントもなんも無し。 608 名前：デフォルトの名無しさん mailto:sage [2008/09/09(火) 13:24:58 ] 火のつくプロジェクトはまずい人材か従順に言うことを聞くだけの人材しかいない。 人月にあらず。そこにいるのは何もできないか、まずいことしかやらない人々なのだ。 609 名前：デフォルトの名無しさん mailto:sage [2008/09/09(火) 22:17:16 ] 管理層も馬鹿じゃないから、火がついても経営的な危機に陥らないような プロジェクトにしか無能な奴は集めないけどな。板違いｽﾏｿ 610 名前：デフォルトの名無しさん [2008/09/28(日) 21:06:12 ] IT技術者は免許制にしてほしいよね 611 名前：デフォルトの名無しさん mailto:sage [2008/09/28(日) 21:22:06 ] いや、GPL使うを免許にすべき 612 名前：デフォルトの名無しさん mailto:sage [2008/10/08(水) 20:01:42 ] externってあんまり使わないほうがいいんかね 昔、学校のグラフィックスの授業でOpenGLを使った課題を発表し合ったとき、 gnuplotの3次元グラフみたいなものを動的に表示させるプログラムを書いた人がいて、すげーと思って ソース見たら『void function001(void)...function002..003....〜この後数十個の関数が続く』っていう中身になってて当時の俺だと 「次元が違う人の書くプログラムは読むのも難しいなー」と思ったんだけど、あれってやっぱりダメだったんだな・・・コメント書かない主義だったし 613 名前：デフォルトの名無しさん [2008/10/08(水) 21:37:45 ] それ逆コンパイラか難読化ツールを作ったのが発表だったんじゃね？ 614 名前：デフォルトの名無しさん mailto:sage [2008/10/08(水) 22:19:02 ] 逆コンパイラで何かぱくってきたの方に1票。 615 名前：デフォルトの名無しさん mailto:sage [2008/10/09(木) 13:12:50 ] >>612 最適化か効率化かが極まった特例としてはアリかも。 キー入力応答やレンダリングとかの処理がいっぱい 必要な場合は、関数の名前を考える時間もバカに ならないんで。 C/C++には無名関数もないしね。 処理系によっては、いちいちコールバック関数が 必須だったりするんでなおさら。 見習ってはいかんと思うけど、アタマから否定しない 度量もあっていい。 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