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

**デフォルトの名無しさん** · NG

var unko;

**デフォルトの名無しさん** · 2008/10/13(月) 07:41:36

コメントをコピペ修正忘れするくらいなら、コメントを書くことよりも、まともな関数名を考えることに注力して欲しい。
駄目な奴は何をやっても駄目だからしかたないが。

**デフォルトの名無しさん** · 2008/11/08(土) 20:52:10

>>580
string-> "S"tring　か?w
ケツ蹴っ飛ばしたれ。w

**デフォルトの名無しさん** · 2008/11/09(日) 19:23:34

>>619
つっこみどころは arg1, arg2 だろ(´д｀;)

**デフォルトの名無しさん** · 2008/11/09(日) 19:38:32

どうやっても値が返らないってところに突っ込むべきじゃ。

**デフォルトの名無しさん** · 2008/11/09(日) 22:20:26

内部状態更新するんじゃね？

**デフォルトの名無しさん** · 2008/11/09(日) 22:30:41

create_string といういかにも値返しそうな名前なのに
procedure というところ？

**デフォルトの名無しさん** · 2008/11/10(月) 09:55:07

突っ込みどころが多い、とｗ
しかし単語の先頭を大文字にした上でアンダースコアでつないでるのを見るとイラっとくるな。

**デフォルトの名無しさん** · 2008/11/11(火) 07:18:47

C#ならstring小文字もメソッド名先頭大文字も普通。
アンダースコアはないが、他にそういう慣習の言語もあるのかも知れん。
言語を明示しないと突っ込みづらい。

**デフォルトの名無しさん** · 2008/11/11(火) 13:47:02

そういやAdaは予約語以外は大文字で始めて単語の切れ目は_入れるんだったな。
procedure Create_String_Procedure(Arg1, Arg2: in String) is
:
begin
:
end Create_String_Procedure;
580を冗長に書き直すと、こんなんだったはず。

まぁコンパイラが大文字小文字を区別しないから、小文字で書いても全く問題ないんだけど。

**デフォルトの名無しさん** · 2008/11/26(水) 02:03:59

>>580
で、こたえは？

**デフォルトの名無しさん** · 2008/11/28(金) 18:38:24

>>537のってだめなの？
old new num はたまに使うんだけど

**デフォルトの名無しさん** · 2008/11/28(金) 18:56:44

>>628
newはしばしば予約されている。
oldやnumも、単体で使うのは有り得ない。
変数名の一部として使う分にはまぁいいと思うけどね。
>537は教条主義に陥っている気がするな。

**デフォルトの名無しさん** · 2008/11/28(金) 19:54:52

>>629
ああいや単体では使わんよ。
ただ変数名に含める事は結構あるんじゃないかと。

**デフォルトの名無しさん** · 2008/11/29(土) 12:16:02

多くの場合に、 old, new よりは previous, current, next のほうがより明確だと思う。
対象領域に old, new の概念があり、それを表すならば、もちろん old, new がふさわしいだろう。

**デフォルトの名無しさん** · 2008/11/29(土) 18:05:50

なるほど、確かにpreviousとnextの方がいいかも。

**デフォルトの名無しさん** · 2008/11/29(土) 21:22:32

>537の理屈じゃ、oldTypeとかnewTypeでもダメそうだもんなw

**デフォルトの名無しさん** · 2008/12/01(月) 22:03:48

>>537のは、結局"故にすべてのプログラムはクソ"というｵﾁなんじゃない？

**デフォルトの名無しさん** · 2008/12/02(火) 01:09:20

>>633
普通にダメだと思うぞそんな名前

**デフォルトの名無しさん** · 2008/12/02(火) 09:58:51

newが入ってるとこみると名前の一部だよなそれ？

**デフォルトの名無しさん** · 2008/12/05(金) 09:57:06

>627
CamelでC#と勝手に予想して>621が正解。
最近C#書く機会が多いから俺もそうオモタ。
変化球でメンバ更新用だとしてもメソッド名で（ ´,_ゝ｀）できる。

**デフォルトの名無しさん** · 2008/12/05(金) 10:20:28

それは名前に突っ込んでるのか仕様につっこんでるのか

**デフォルトの名無しさん** · 2008/12/05(金) 19:57:42

stringはstd::stringなのか？

**デフォルトの名無しさん** · 2009/01/08(木) 02:50:44

year2009

**デフォルトの名無しさん** · 2009/01/08(木) 12:03:10

bool eroikkaizutu = true;

**デフォルトの名無しさん** · 2009/01/08(木) 21:59:30

>>616
コメントが正しければね。

/* DB接続 */
int connect()

/* DB接続 */
int disconnect()

/* DB接続 */
int select()

みたいなのがあると悲しくなるよ。

**デフォルトの名無しさん** · 2009/01/09(金) 00:52:00

アニメとか漫画のキャラの名前を変数に使うなよ！！！

**デフォルトの名無しさん** · 2009/01/09(金) 02:04:22

bool dame;

…

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

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

**デフォルトの名無しさん** · 2009/01/11(日) 00:49:09

>>644
つまりソース書いたやつがdame

**デフォルトの名無しさん** · 2009/01/11(日) 09:17:42

次いってみよう！

**デフォルトの名無しさん** · 2009/01/12(月) 01:38:03

int NullPo;

**デフォルトの名無しさん** · 2009/01/28(水) 15:22:48

if (だめだこりゃ) 次いってみよう;

**デフォルトの名無しさん** · 2009/01/28(水) 15:40:31

カセット！

**デフォルトの名無しさん** · 2009/01/28(水) 18:51:40

while ("終わりが無いのが終わり　それがゴールド・エクスペリエンス・レクイエム") {

　break;
}

**デフォルトの名無しさん** · 2009/01/29(木) 00:55:28

>>650
breakすんな

**デフォルトの名無しさん** · 2009/01/31(土) 00:28:52

変数名や関数名にまともな名前付けれない奴が良いプログラミングなんてできるわけがないに激しく同意。
うちの部署にもいるが、「変数名なんて何でもいい。考えている時間が無駄だ。」とか言い出す始末。

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

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

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

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

**デフォルトの名無しさん** · 2009/01/31(土) 09:15:55

その程度の職場にその程度のプログラマ

**デフォルトの名無しさん** · 2009/01/31(土) 12:49:29

そんな名前をつけられる位なら、逆に30年前の大型汎用機の開発のような「連番＋台帳管理」の方がマシだな。
（すくなくとも記録は確実に残るし）

**デフォルトの名無しさん** · 2009/01/31(土) 15:34:20

グローバル変数クラスなんていうシロモノがある時点でオワットルな

**デフォルトの名無しさん** · 2009/01/31(土) 17:49:37

>>業務処理クラス→GYM
COBOLer警報が鳴り止まない。

**デフォルトの名無しさん** · 2009/02/08(日) 03:30:35

普通
private int m_hoge;
コンストラクタ(int hoge){this.m_hoge = hoge;}
て、外出ししない変数にプレフィックスとか（必要なら）つけると思うけど

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

**デフォルトの名無しさん** · 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) {}

**デフォルトの名無しさん** · 2009/02/08(日) 05:15:16

スコット・メイヤー先生やハーブ・サッター先生の教えに従えば
メンバ変数が公開されることはないので
メンバ変数にはテキトーな名前をつけてもいいんじゃないだろうか…

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

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

**デフォルトの名無しさん** · 2009/02/08(日) 07:19:00

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

**デフォルトの名無しさん** · 2009/02/08(日) 10:57:36

>>658
private int hoge;
からしてC++じゃなくてjavaだろ、常識的に考えて。

**658** · 2009/02/08(日) 12:01:07

>>661
ほんとだ。ごめん。 m_hoge 見て C++ だと思い込んだ。

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

**デフォルトの名無しさん** · 2009/02/08(日) 12:28:59

ぱっと見で把握できないような大きめのクラスは必ずpimplになるから
実質this->付いてるのと同じだな、自分の場合。

**デフォルトの名無しさん** · 2009/02/08(日) 12:58:18

利用者側に変な装飾付けない方がいいのは同意。
装飾付けるなら内部の方がいい。

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

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

**デフォルトの名無しさん** · 2009/02/08(日) 14:41:37

とりあえずメンバ変数宣言をコメント化すればすぐに分かったのにね。

**デフォルトの名無しさん** · 2009/02/08(日) 14:50:58

そういう問題じゃないだろ・・・

**デフォルトの名無しさん** · 2009/02/08(日) 17:08:49

命名規則スレが別になかったっけ？

**デフォルトの名無しさん** · 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*)

**657** · 2009/02/12(木) 00:40:10

>>658
>>661
すみません、ほんとすみません、>>657はC#のつもりです(；´Д｀)
べつにJavaと変わらんけど

**デフォルトの名無しさん** · 2009/03/20(金) 03:19:07

hogeの次に使う変数ってpiyo？

**デフォルトの名無しさん** · 2009/03/20(金) 03:21:48

fuga

**デフォルトの名無しさん** · 2009/03/20(金) 03:28:59

huhun

**デフォルトの名無しさん** · 2009/03/20(金) 03:38:01

mogyo

**デフォルトの名無しさん** · 2009/03/20(金) 04:15:47

zamasuzamasu

**デフォルトの名無しさん** · 2009/03/25(水) 00:49:23

so_degansu

**デフォルトの名無しさん** · 2009/03/25(水) 10:24:54

hunga_()

**デフォルトの名無しさん** · 2009/03/25(水) 10:37:40

maji
meni
haji
mena
saiyo

**デフォルトの名無しさん** · 2009/03/27(金) 07:30:51

kuukiyome

**デフォルトの名無しさん** · 2009/03/27(金) 08:39:12

ore_tsueeeeeeee_max

**デフォルトの名無しさん** · 2009/03/27(金) 21:47:09

kintama_kick

**デフォルトの名無しさん** · 2009/03/28(土) 13:22:51

int k;
int kk;
int kkk;
UINT kkkk;
char* a;
char* as;
char asdf;

みたいなものすごく投げやりなのは見たことあるw

**デフォルトの名無しさん** · 2009/04/07(火) 00:58:03

//UPD start 200x/12/24 型を変更
//String strSummary = "0";
long strSummary = 0L;
//UPD end 200x/12/24 型を変更

ﾊﾝｶﾞﾘｱﾝ万歳ｗ

**デフォルトの名無しさん** · 2009/04/07(火) 10:14:31

>>682
それどう見てもstrSummary使ってるとこ全部直さなきゃならんだろ。
変数名をそのままにする意味あんのか。

...って聞くだけ野暮ってもんか。

**デフォルトの名無しさん** · 2009/04/07(火) 12:25:14

それ見ると機械的なリファクタリングのありがたみが指に沁み渡る

**デフォルトの名無しさん** · 2009/04/29(水) 22:12:50

痛い変数名というより、マ板の方の「この会社辞めようと思(ry
向きのネタだな。

**デフォルトの名無しさん** · 2009/05/02(土) 18:20:58

引数に必ずNULLを指定しなければならない関数全て

**デフォルトの名無しさん** · 2009/06/02(火) 20:35:34

String kensakuKekkaStatus = "";

山本高広のケインコスギ芸か

**デフォルトの名無しさん** · 2009/07/04(土) 06:57:48

久々に熱い変数名の記事を見たわｗ

UKTKKNSHINF → 受付禁止情報
SKSI → 作成
HNKN → 変換
KKT → 確定
CHKN → 中間
DTM → Datetime
DTA → Data

ひどすぎるネーミング - idesaku blog
http://d.hatena.ne.jp/idesaku/20090703/1246593915

**デフォルトの名無しさん** · 2009/07/04(土) 16:11:25

RMJKRBINNKTNSKHNKNize

**デフォルトの名無しさん** · 2009/07/04(土) 21:47:17

　,j;;;;;j,. ---一､｀　―--‐､_ l;;;;;;
　｛;;;;;;ゝ T辷iﾌ i 　　f'辷jァ　 !i;;;;;　　ローカルなスコープではどんな痛い変数名も
　ヾ;;;ﾊ　　　　ﾉ　　　　　　　.::!lﾘ;;rﾞ　色なりIDEのリファクタリングなりで無問題
　　 `Z;i　　　〈.,_..,.　　　　　　ﾉ;;;;;;;;>
　　,;ぇﾊ、　､_,.ｰ-､_',. 　　,ｆﾞ: Y;;f.　　そんな風に思っていた時期が
　　 ~''戈ヽ　｀二´ 　　 r'´:::.　`! 　　俺にもありました

全然カプセル化されてねーよ!
スパゲティなコードの上にグローバル、ローカル同じ名前とかないよ!

**デフォルトの名無しさん** · 2009/09/03(木) 00:45:03

痛いコメントも多いよな

「？」とか「なぜか～なる」とか「とりあえず」とか
コメント入ったまま納品すんなｗ

**デフォルトの名無しさん** · 2009/09/03(木) 02:43:35

消すとリビジョン変わるからそのまま納品でいいんだよ！！！

**デフォルトの名無しさん** · 2009/09/12(土) 22:57:22

コードフリーズされる前に消しておくことだな。

**デフォルトの名無しさん** · 2009/10/04(日) 05:20:00

実際にあるから怖いね

**デフォルトの名無しさん** · 2009/10/04(日) 06:26:24

俺がリーダーの時、以前のプロジェクトでテキストボックスならTBox,チェックボックスなら
CBoxとかの省略形の接頭辞、接尾辞つけるルールを変えさせた。慣れないといちいち、変換表みるはめに
なり、一瞬止まるから。素直にフルスペルでつけさせた。XXXXTextBoxとか。

**デフォルトの名無しさん** · 2009/10/07(水) 08:26:23

変換表があるだけマシ

**デフォルトの名無しさん** · 2009/10/09(金) 09:42:24

今スレ開くと>>650-651がちょうど上の方にあって、
見る度に笑ってしまう。

確かに、while(1)の代わりにリテラル文字列入れとくという遊びは、
害のない(?)悪戯として使えるかもしれないけど。

**デフォルトの名無しさん** · 2009/10/09(金) 09:51:02

for (;"ever";) {

// これはいいかもしれんw
}

**デフォルトの名無しさん** · 2009/10/09(金) 10:47:31

for(;;)
{
}
の方がいい
括弧の中に何か入ってると身構えてしまう

**デフォルトの名無しさん** · 2009/10/11(日) 00:19:19

#define forever for(;;)

そんな時代もありました…

**デフォルトの名無しさん** · 2009/10/20(火) 14:27:14

int gero; // <Global> End Row Offset

**デフォルトの名無しさん** · 2009/11/10(火) 20:43:05

昔、X68000のXCでNASHI NASHI NASHIみたくNASHIがいっぱい入ってた記憶があるんだけど、
あれはやっぱ「ナシ」だったんだろうか

**デフォルトの名無しさん** · 2009/11/10(火) 22:43:30

fook って付けようとして fuck って付けてしまた

**デフォルトの名無しさん** · 2009/11/10(火) 22:57:05

もしかして hook

**デフォルトの名無しさん** · 2009/11/10(火) 23:03:00

hook ですた orz

**デフォルトの名無しさん** · 2009/11/11(水) 09:21:45

まさに痛い変数名…

**デフォルトの名無しさん** · 2009/11/11(水) 10:18:31

ネットゲーのfackの話のコピペ思いだしたｗ

**デフォルトの名無しさん** · 2009/12/01(火) 18:44:02

int unk

unknownの略かな？w

**デフォルトの名無しさん** · 2009/12/03(木) 04:17:05

ウ●コ？

**デフォルトの名無しさん** · 2010/02/01(月) 23:14:00

２０１０

**デフォルトの名無しさん** · 2010/04/02(金) 11:54:47

#define BUF1024 1024000

**デフォルトの名無しさん** · 2010/04/03(土) 19:07:41

>>14
カタコトの日本語みたいだなｗ

**デフォルトの名無しさん** · 2010/04/16(金) 12:05:25

huu　baa

**デフォルトの名無しさん** · 2010/07/04(日) 21:26:26

http://pc12.2ch.net/test/read.cgi/tech/1277016982/49

**デフォルトの名無しさん** · 2010/11/13(土) 17:11:34

めんちょう

**デフォルトの名無しさん** · 2010/11/22(月) 19:55:03

Dim ぬるり As Object = Nothing
ぬるり.ToString()

**デフォルトの名無しさん** · 2010/12/12(日) 14:41:15

char* han;

**デフォルトの名無しさん** · 2010/12/31(金) 15:15:53

int* age;