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

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

var unko;

**デフォルトの名無しさん** · 05/01/30 02:10:31

いち、にぃ、はんがりあん♪

**デフォルトの名無しさん** · 05/02/24 22:05:55

>>1
assert(BrainCheck(">1"));

アサートが通らん気がする。

**デフォルトの名無しさん** · 05/03/17 04:11:39

var chan;

**デフォルトの名無しさん** · 05/03/18 15:12:19

double ore;

ko_hai=ore;

**デフォルトの名無しさん** · 05/03/19 00:44:54

以前の職場で

int _KONO_HENSU_KESUNA_

ってのがあったな。

**デフォルトの名無しさん** · 2005/03/21(月) 18:39:56

int _HAYAKU_IENI_KAERITAI_

**デフォルトの名無しさん** · 2005/03/23(水) 16:47:09

hogeFunc();
hogeFunc2();
hogeFunc3();
hogeFunc4();

最初は hogeFunc 内で条件分岐書きまくって頑張ってたようだが
そのうち無理が来て hogeFunc2() を作成。
2 を作ったら歯止めが効かなくなったのか
あっと言う間に 4 まで作成。
関数ヘッダがコピペされたままで
4番までコメントが一緒。違いが分からない。
2とか3とかつけられても困る。意味のある名前足せよ。

**デフォルトの名無しさん** · 2005/03/23(水) 16:58:03

>>118
2に込められた意味をきちんと理解してくれよ。2は1の次。
で1はどこ行った？ってなるわけだ。

**デフォルトの名無しさん** · 2005/05/25(水) 07:09:20

転職した先輩から引き継いだソースコードを読んでたら、sinjiやらasuka
といったアニメキャラの名前っぽい変数名が沢山出てきてちょっとひいた。

**デフォルトの名無しさん** · 2005/05/25(水) 08:48:13

おいらは

hoge

とか出てきたらブチまわしたくなる衝動がある

**デフォルトの名無しさん** · 2005/05/25(水) 10:00:13

HogeHoge。

**デフォルトの名無しさん** · 2005/05/25(水) 10:03:58

　　　　　　　　　ﾉﾉﾉﾉ　　　　＿　＿　　　　　.'　　，　．.　　 ∧＿∧
　　　　　　　　(ﾟ∈ﾟ　)_　- ― ＝￣　￣`:,　.∴　'　　　　　（　　　　）
　　　　　　　　　ヽ-''￣　　　　＿＿――＝',　・,‘　ｒ⌒>　 _／／
　　　　　　　　／　　,,-―　￣￣　　￣"'"　. 　　’ |　ｙ'⌒　　⌒i
　　　　　　　/　　　ノ＼＼　　　　　　　　　　　 .　　|　　／　　ノ｜
　　　　　/　　 /　　＼＼　　　　　　　　　　　, ー' 　／´ヾ_ﾉ
　　　　　　ﾚ　　ノ　　　　　ヽ_つ　　　　　　　　／　,　　ノ
　　　　　　/　 /　　　　　　　　　　　　　　　　.／　／／
　　　　　/　 /|　　　　　　　　　　　　　　　／　／　,'
　　　　（　（　､　　　　　　　　　　　　　／　／| 　|
　　　　　|　 |､　＼　　　　　　　　　　!､＿／ / 　〉
　　　.　　|　/　＼　⌒l　　　　　　　　　　　　|_／
　　　　　　|　|　　　）　/
　　　　　ノ　）　　し'
　　　　(＿／　　　　　　　　　　　　　-=＝≡≡≡＝==-

**デフォルトの名無しさん** · 2005/05/25(水) 10:11:03

hoge??

**デフォルトの名無しさん** · 2005/05/25(水) 10:15:46

　　　 _, ,_ 　パーン
　（　゜д゜）
　　　⊂彡☆＝＝＝＝＝＝ /|
　　　　　　＿＿　　　　　　　|/
　　　　　ヽ| l　l│
　　　　　　┷┷┷

**デフォルトの名無しさん** · 2005/05/26(木) 15:53:59

>>120
おまいの先輩はｴｳﾞｧｦﾀか？

**デフォルトの名無しさん** · 2005/08/10(水) 02:21:38

Dim 初号機

**デフォルトの名無しさん** · 2005/08/10(水) 09:29:16

declare @nifty

**デフォルトの名無しさん** · 2005/08/10(水) 12:03:18

hage

**デフォルトの名無しさん** · 2005/08/10(水) 17:22:06

for (int nesuto;
for (int fukasugi;
　for (int mousiran;
　for (int atotanomu;
とかなら実際に見た事がある。

**デフォルトの名無しさん** · 2005/10/09(日) 18:47:34

ﾜﾛｽ

**デフォルトの名無しさん** · 2005/10/09(日) 22:19:14

int i, ii, iii, iii, iiiiiiii, iiiiii

**デフォルトの名無しさん** · 2005/10/09(日) 22:20:16

tmp的な変数で、
int a, s, d, f

何かと思ったら、キーボードで左から順番

**デフォルトの名無しさん** · 2005/10/12(水) 14:28:08

int a, b, c, d, e, f, g, h, i;
int aa, bb, cc, dd, ee, ff, gg, hh, ii;
char AA, BB, CC, DD, EE, FF, GG, HH, II;
3目ならべ(マルバツ)のソースより。実話。

**デフォルトの名無しさん** · 2005/12/01(木) 05:05:46

ﾜﾛﾀｗ

**デフォルトの名無しさん** · 2005/12/01(木) 10:33:37

*pr = lna() + px; みたいなのがあったな。
略号だらけで意味がわからん。

**デフォルトの名無しさん** · 2005/12/01(木) 11:59:13

ローマ字でも漢字でも変な省略でもまあ許すけど
変数にtempとかブール関数にcheckとか、「意味のある名前付けろ」の
意味を分かってない名前は勘弁してほしいんだけどほんとに多い。
たとえば後で使う販売店名称を保存するのに
The_temporary_variable_used_later
なんて英語の変数だったら HBT_MSO のほうが「まだまし」だと思う。

**デフォルトの名無しさん** · 2005/12/01(木) 17:04:36

TOMOYO Linux

**デフォルトの名無しさん** · 2005/12/01(木) 22:49:23

>>137
後者はどう言う命名の仕方なんだ？

**デフォルトの名無しさん** · 2005/12/02(金) 01:24:11

>>136
ローカル変数だと、長すぎる変数名は邪魔なだけだからある程度の略語はしょうがない。
しかし推測できないのや、長い関数だと駄目だが。

**デフォルトの名無しさん** · 2005/12/02(金) 01:59:00

変数名ではないが、某企業のライブラリの実装ガイドの
サンプルコードに

System.out.println("ｷﾀ----(ﾟ∀ﾟ)----!!!!");

orz

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

>>137
後のほうで使うデータ保存する変数を、使われる場所がすぐにわからない
ような前の方に宣言すること自体、痛いのでは。

**137** · 2005/12/02(金) 15:20:31

>>142
言語仕様上、前の方でしか宣言できない場合、どうするのかと。

# C99以前のCとか

「The_temporary_variable_used_later」が英語なのはわかるが、
「HBT_MSO」がどう言う命名規則でこうなったのか、漏れには見当がつかない。

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

HanBaiTen_MeiSyOu

**デフォルトの名無しさん** · 2005/12/02(金) 15:58:58

>>139
>>143
いやHBT_MSOは>>22から拝借したの。販売店名称をローマ字にして適当に縮めたんじゃない？
ちょっとたとえがまずかったですか。つまり
「変更前の販売店名称を、後で使うために仮保存する」んだったら
「変更前の販売店名称」だと少しでも分かる名前にしてくれ、と言いたかった。
「後で(直後でもね)使うために(仮)保存する」のはあらゆる変数がそうなんだから。

int retValue = GetItemCount(); if (retValue ...
じゃなくて
int itemCount = GetItemCount(); if (itemCount ...
がいいな、と。

**137** · 2005/12/02(金) 22:24:41

>>145
言いたい事はよくわかった。
が、例えで出した名称は>>22を拝借せず、
大抵の人がすぐ判る名前にすべきだったね。

# 販売店名称なら、安直に「ShopName」とかでいいんｼﾞｬﾏｲｶ？

**デフォルトの名無しさん** · 2005/12/02(金) 23:06:34

「HBT_MSO というのも相当ひどい変数名だが、それでも The_temporary_variable_used_later よりはマシだ」

てなニュアンスだとおもうんだ

**デフォルトの名無しさん** · 2005/12/02(金) 23:15:17

int cunt;

**デフォルトの名無しさん** · 2005/12/02(金) 23:50:53

double speeed; // (km/h)

**デフォルトの名無しさん** · 2005/12/02(金) 23:55:10

int WindowChenge(int flg)

**デフォルトの名無しさん** · 2005/12/03(土) 16:17:18

>>147
どちらも相当ひどいが、糞コードを押しつけられて何をやっているか解析しろと
言われた場合、全コードに渡って前者のような意味不明の略号変数が散乱して
るものよりも、後者の方がまだマシだと思うな

**デフォルトの名無しさん** · 2005/12/03(土) 16:20:40

>>151
派閥味噌

**デフォルトの名無しさん** · 2005/12/03(土) 17:14:53

>>151
どっちがマシとか決めるのが無意味なレベルだと思うけどな

**デフォルトの名無しさん** · 2005/12/03(土) 20:15:38

toriaezu_001
toriaezu_002
toriaezu_003
toriaezu_004
　　・
　　・
　　・
toriaezu_138

ヽ(`Д´)ﾉソース見る気失せた

**デフォルトの名無しさん** · 2005/12/03(土) 21:22:39

int zero;
int one;
int SUM;
int RAND[700];

**デフォルトの名無しさん** · 2005/12/03(土) 23:54:11

>>143

>言語仕様上、前の方でしか宣言できない場合、どうするのかと。
># C99以前のCとか

関数分割しろよ。素人じゃあるまいし。
1関数をせいぜい20行程度に納めるようなコントロールって
誰でもやってんじゃないのか???

**デフォルトの名無しさん** · 2005/12/03(土) 23:55:45

そうそう。
変数宣言と実際に使うところの距離があるってことは
その関数が複雑になりすぎてるってこと。

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

今編集しているプログラム、main()だけで2000行。
取り敢えず関数分割前にローカル変数の整理を始めた。
全てmain()の先頭で宣言していたのだが、ここのブロック内に移動できるものはいどうしたわけだ。
…何故か変数が半分に減った挙句、処理速度が数%向上した……

**デフォルトの名無しさん** · 2005/12/04(日) 13:53:44

生保系システムのレビューを行ったら、変数名だけで100文字ってのがあった。

**デフォルトの名無しさん** · 2005/12/04(日) 15:31:48

>>159
レビューだけで良かったねｗ

**デフォルトの名無しさん** · 2005/12/05(月) 12:56:21

>>158
世の中、上には上が居る。(w
# こう言う上は居て欲しくないがな。

>> ttp://www.pro.or.jp/~fuji/mybooks/cdiag/index.html より
最長不倒関数： 6344行

**デフォルトの名無しさん** · 2005/12/05(月) 12:59:25

>>154
そのインデックスが、『せめて』10進数である事を祈る。

**158** · 2005/12/05(月) 13:49:56

>>161
あーいや、ある意味翁の取り上げた関数を超えている。
なんせ、2000行の中にgotoで仕込まれたループが6つもあるんだ。
それぞれのループの中は200行ほどの同じような演算だと言うのに。
膨大なローカル変数だけでなくグローバル変数も使っているから
どこで分離して関数化するかが悩みどころなんだ。
#しかも、コードを見る限り大幅にコードを削減した旨のコメントがあるんだよね。

**デフォルトの名無しさん** · 2005/12/06(火) 12:56:57

Cプログラミング診断室自体の質はどうなんだろう・・・

**デフォルトの名無しさん** · 2005/12/06(火) 13:28:18

古いけど悪くはないと思うよ。

**デフォルトの名無しさん** · 2005/12/06(火) 14:29:45

マジか
　　　　ﾍ⌒ヽﾌ
　　　（　・ω・）　㌧
　　　/ ~つと）

**デフォルトの名無しさん** · 2005/12/06(火) 20:27:35

>>166
かわいいね、それ

**デフォルトの名無しさん** · 2005/12/06(火) 21:02:30

if ($kigen == '1') {
print '昭和';
} else if ($kigen == '2') {
print '平成';
}

「期限」なのかなあと思っていたら、こんなコメントがあった。
なんのことだか理解するのに苦労した。書いたやつは日本人だったのだろうか。

$kigen; //紀元節

**デフォルトの名無しさん** · 2005/12/06(火) 22:21:25

それは何か勘違いをしているのではないかな?
紀元節とは今の建国記念日のことなのだが。
まさか、天皇否定の三国人が紀元節なんて書くはずもないし。

**デフォルトの名無しさん** · 2005/12/07(水) 04:22:13

某王手のソースがこんなんでげんなりした

if (ほげほげ) {
　　if (ほげほげ) {
　　　　if (ほげほげ) {
　　　　　　if (ほげほげ) {
　　　　　　　　if (ほげほげ) {
　　　　　　　　　　if (ほげほげ) {

　　　　　　　　　　1000行くらい

　　　　　　　　　　}
　　　　　　　　}
　　　　　　}
　　　　}
　　}
}　

**デフォルトの名無しさん** · 2005/12/07(水) 06:42:56

>>170
その書き方だと、全ての if が同じ条件式のように見えてキモさ５割増。

**デフォルトの名無しさん** · 2005/12/07(水) 08:25:47

>>171
むしろ100倍

**デフォルトの名無しさん** · 2005/12/07(水) 09:02:14

public boolean isSyoriShimashita() { return syoriShimashita; }
ってのなら新人時代にやったことがある。

booleanのgetterにはisXXXって命名するのが普通だけど、
変数名をなんちゃらフラグみたいにすると、isなんちゃらフラグってのは変だよなー。

**デフォルトの名無しさん** · 2005/12/07(水) 11:23:34

今日こんなの目撃orz

GetAccessNameAndIDBeforePostForPurchaseProductsThankYou(引数6こくらい);

**デフォルトの名無しさん** · 2005/12/07(水) 12:39:34

さすがにそれだとそのプロジェクト特定できるような

最後のThankYouはなんだ？本当にありがとうございました。みたいな感じ？

**174** · 2005/12/07(水) 20:59:07

ネタにマジレスすんなアホ

**デフォルトの名無しさん** · 2005/12/07(水) 23:55:54

むしろ176の方がネタにマジレスに見える件。

**174** · 2005/12/08(木) 00:56:57

負け惜しみなんか言って恥ずかしくないのかな。

**デフォルトの名無しさん** · 2005/12/08(木) 03:47:09

おまいら、テストデータはどんなんにしてる？

**174** · 2005/12/08(木) 09:33:52

何でそんなこと教えなきゃならねんだよ、糞して寝ろ。

**デフォルトの名無しさん** · 2005/12/08(木) 12:22:43

あー、もしかして>>174が大まじめに書いたソースが
このスレで晒されちゃったとか？それで粘着してんの？

**174** · 2005/12/08(木) 20:03:49

やかましいわ　氏ね

**デフォルトの名無しさん** · 2005/12/08(木) 20:13:48

>>174
君を見てると昔の僕を見るようだ

**174** · 2005/12/08(木) 22:48:32

>>183
お前を見てると未来の旦那様を見ているようだ

**174** · 2005/12/08(木) 23:59:12

俺厨房なんですぐクビつって死んできます
本当にありがとうございました。

**デフォルトの名無しさん** · 2005/12/09(金) 00:16:43

久しぶりにID無し板の楽しさを味わった気がするよ。

**174** · 2005/12/09(金) 03:51:51

そんなおまいに空中元彌チョップ

**デフォルトの名無しさん** · 2005/12/09(金) 12:02:43

s/空中/宇宙/g

**デフォルトの名無しさん** · 2005/12/09(金) 12:29:00

グローバルにしたって1個しかマッチしないやんけ！

**デフォルトの名無しさん** · 2005/12/09(金) 14:06:15

汚物：グローバルにしたって1個しかマッチしないやんけ！
普通：グローバルにしたって1個しかマッチせえへんやんけ！

**デフォルトの名無しさん** · 2005/12/09(金) 14:25:10

拡張：グローバルにしたかて1個しかマッチせえへんやんけ！

**デフォルトの名無しさん** · 2005/12/09(金) 14:31:49

すげえ、俺の作る class より拡張性たけえや

**デフォルトの名無しさん** · 2005/12/10(土) 04:03:25

何このスレのＡＡ

**デフォルトの名無しさん** · 2005/12/10(土) 05:38:14

汚物：何このスレのＡＡ

**デフォルトの名無しさん** · 2005/12/10(土) 13:44:36

吐瀉物:汚物：何このスレのＡＡ

**デフォルトの名無しさん** · 2005/12/10(土) 15:11:17

排泄物:吐瀉物:汚物：何このスレのＡＡ

**デフォルトの名無しさん** · 2005/12/10(土) 22:59:02

何このスレ・・・・・・・？
　　　　　　　　　　　　/ヽ　　　　　　 /ヽ
　　　　　　　　　　　　/　ヽ　　　　　 /　ヽ
　　＿＿＿＿＿＿ /U　ヽ＿＿＿/　　ヽ
　　|　＿＿＿＿　／　　　U　　　　:::::::::::U:＼
　　| |　　　　　　 /／＿__　　　＼　　::::::::::::::|
　　| |　　　　　　 |　　|　　 |　　　　　U :::::::::::::|
　　| |　　　　　　.|U　|　　　|　　　　　　::::::U::::|
　　| |　　　　　　 |　├―-┤　U.....:::::::::::::::::::/
　　| |＿＿＿＿　ヽ　　　　　.....:::::::::::::::::::::::<
　 └＿＿＿／￣￣　　　　　　:::::::::::::::::::::::::|
　　|＼　　　 |　　　　　　　　　　　:::::::::::::::::::::::|
　　＼＼　＼＿＿＿　　　　　 ::::::

**デフォルトの名無しさん** · 2005/12/11(日) 07:13:25

ちょっと前まで良スレだったと思うんだ

**デフォルトの名無しさん** · 2005/12/11(日) 11:37:30

どこでボタンを掛け違えちゃったんだろうな

**デフォルトの名無しさん** · 2005/12/14(水) 09:13:39

ＣＬＳ３

**デフォルトの名無しさん** · 2005/12/14(水) 16:47:57

int tmp, tmp2, tmp3, ... tmp30;

痛い・・・

**デフォルトの名無しさん** · 2005/12/14(水) 18:46:36

public string[] strExcelColID = {
　　　　　　　　"", "A","B","C","D","E","F","G","H","I","J","K","L","M","N","O","P","Q","R","S","T","U","V","W","X","Y","Z",
　　　　　　　　"AA","AB","AC","AD","AE","AF","AG","AH","AI","AJ","AK","AL","AM","AN","AO","AP","AQ","AR","AS","AT","AU","AV","AW","AX","AY","AZ",
　　　　　　　　"BA","BB","BC","BD","BE","BF","BG","BH","BI","BJ","BK","BL","BM","BN","BO","BP","BQ","BR","BS","BT","BU","BV","BW","BX","BY","BZ",
　　　　　　　　"CA","CB","CC","CD","CE","CF","CG","CH","CI","CJ","CK","CL","CM","CN","CO","CP","CQ","CR","CS","CT","CU","CV","CW","CX","CY","CZ",
　　　　　　　　"DA","DB","DC","DD","DE","DF","DG","DH","DI","DJ","DK","DL","DM","DN","DO","DP","DQ","DR","DS","DT","DU","DV","DW","DX","DY","DZ",
　　　　　　　　"EA","EB","EC","ED","EE","EF","EG","EH","EI","EJ","EK","EL","EM","EN","EO","EP","EQ","ER","ES","ET","EU","EV","EW","EX","EY","EZ",
　　　　　　　　"FA","FB","FC","FD","FE","FF","FG","FH","FI","FJ","FK","FL","FM","FN","FO","FP","FQ","FR","FS","FT","FU","FV","FW","FX","FY","FZ",
　　　　　　　　"GA","GB","GC","GD","GE","GF","GG","GH","GI","GJ","GK","GL","GM","GN","GO","GP","GQ","GR","GS","GT","GU","GV","GW","GX","GY","GZ",
　　　　　　　　"HA","HB","HC","HD","HE","HF","HG","HH","HI","HJ","HK","HL","HM","HN","HO","HP","HQ","HR","HS","HT","HU","HV","HW","HX","HY","HZ",
　　　　　　　　"IA","IB","IC","ID","IE","IF","IG","IH","II","IJ","IK","IL","IM","IN","IO","IP","IQ","IR","IS","IT","IU","IV","IW","IX","IY","IZ",
　　　　　　　　"JA","JB","JC","JD","JE","JF","JG","JH","JI","JJ","JK","JL","JM","JN","JO","JP","JQ","JR","JS","JT","JU","JV","JW","JX","JY","JZ",
　　　　　　　　"KA","KB","KC","KD","KE","KF","KG","KH","KI","KJ","KK","KL","KM","KN","KO","KP","KQ","KR","KS","KT","KU","KV","KW","KX","KY","KZ"
　　　　　　　　};

**202** · 2005/12/14(水) 18:52:43

↑ごめんなさい・・・・でも本当なんです。
外注が作ったぷろぐらむなんです。Excelを使いたかったらしいんです。
他のclassでも同じ事を宣言してます。しっかりきっちりprivateで。

バグだらけ、と言うか完成度30%くらいで納品してきた物を、
リストラ対象だった前任者が受け入れテストをせずに客先に納品して辞めていきました。
完成度30%ですが、外注先からはこの件についてはこれ以上対応できない、と言うメールをもらいました。

以来何ヶ月もこんなコードをずっと直し続けてます・・・

**デフォルトの名無しさん** · 2005/12/14(水) 20:40:50

>>203
ソースもひどいが、受け入れテストもせずに納品出来てしまう管理体制も十分痛いな。

**デフォルトの名無しさん** · 2005/12/14(水) 21:42:07

結論：そんな会社やめちまえ

**174** · 2005/12/14(水) 22:23:04

またネタにマジレスされた。

**174** · 2005/12/15(木) 04:38:50

ここはいい釣り堀ですね。

**202** ◆cofYHAha3E · 2005/12/15(木) 13:03:33

ネタじゃ無いですよ・・・
ネタだったらどれほど肩の荷が下りることか。

**デフォルトの名無しさん** · 2005/12/15(木) 14:13:33

その仕事、私のところで引き受けてやろう。
#いや、その手の改修はお手の物で……_/￣|○

**208** ◆r5LpnmM55A · 2005/12/16(金) 00:29:03

またネタにマジレス。いい加減見抜けよ。

**デフォルトの名無しさん** · 2005/12/16(金) 05:17:07

結構よくある話だよ。
ネタとか言ってるやつは無職鉄板。

**デフォルトの名無しさん** · 2005/12/16(金) 05:46:51

大手メーカーのソフト開発の下請けで書いてる業者が
そういうコード書いてることが多い