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

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

var unko;

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

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

**デフォルトの名無しさん** · 2005/12/16(金) 10:50:19

>>212
下請けっつーか孫請け玄孫請けだろ。

**202** ◆cofYHAha3E · 2005/12/16(金) 10:54:50

>>209
いっとくけどすげーですよ？

class kyoutuuhensuu{
　　int pos;
　　

　　
}

**202** ◆cofYHAha3E · 2005/12/16(金) 11:03:22

途中でおしちゃった・・・

グローバル変数的に使ってるクラスがあるんですが、

class CommonClass{
　　/* ～snip～ */
　　public int pos;
　　
　　public void setPos(int val){
　　　　pos=val;
　　}
　　
　　public int getPos(){
　　　　return pos;
　　}
　　/* ～snip～ */
}

みたいになってて、実際のコードからposを取得するときは
setPos getGos を使わずに CommonClass.pos で取得してたり。。。
もう、なんか嫌がらせとしか思えないｗ
C#初心者と言うよりもPG初心者だったんだろうなぁ・・・
俺も昔はこんなコード書いてたかも、と思うとさらにガクブルです。

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

もともとposはprivateだったのにpublicに書き換えた可能性あるな。
おそらくデスマーチが始まって最初に設計した香具師と違うアフォが
動員されたと。

**デフォルトの名無しさん** · 2005/12/16(金) 22:41:34

>>215
C#のプロパティの書き方ってそんな感じだっけ？

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

get{};set{};

**デフォルトの名無しさん** · 2005/12/17(土) 00:23:05

つーか、そんなんリファクタリング楽勝じゃん。

そんなんで
>いっとくけどすげーですよ？
って、お前もスゲーよ。脳味噌の足りなさが。

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

> グローバル変数的に使ってるクラスがあるんですが、
これがいちばんすげーよ

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

>>220
実は C++ の標準入出力だというオチだったらどうする ?

**202** ◆cofYHAha3E · 2005/12/17(土) 13:46:17

C#です。カキコした部分は>>218のやり方で書き直しました。

>>219
その部分は楽勝なんですが、他にもグローバル変数的に使っている変数が
50以上あるし、コメントも最小限、
グローバル変数的に使ってる4次元位のジャグ配列も、
使う機能毎に要素の中身が違ったりすると、
もう何が何だか・・・ってあんばいです。

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

そんなん大した事ないし。
書いた方はたしかにレベルが低いが、
それをさっと直せないで何ヶ月も掛けてるのも最高にレベル低い。

第一 >>202の時点でスレ違いだろドアホウが。

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

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

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

int death;
float hell;
double fuckyou;

これでGO

**デフォルトの名無しさん** · 2006/01/02(月) 15:07:57

const y=2006

**デフォルトの名無しさん** · 2006/01/03(火) 04:51:40

やたら長い変数は痛い?
英語で端的な説明になってるけど、略語になってない。
しかも、単語間はアンダースコアでつなげてある。
数個のグローバル変数がこういう名前なんだけど単語5つ以上だと結構長い。

**デフォルトの名無しさん** · 2006/01/03(火) 04:56:00

グローバル変数は長くて醜くて入力しにくい名前のほうが良い。

**デフォルトの名無しさん** · 2006/01/03(火) 08:25:14

なんで？

**デフォルトの名無しさん** · 2006/01/03(火) 08:35:46

>>229 軽い気持ちで使われちゃ困るからだろ。

**デフォルトの名無しさん** · 2006/01/03(火) 12:00:16

>>228
でも覚えやすい方がいいな
最低なのは短いのに忘れやすい名前

しかもよく使う言葉だとよく被るんだよなあ

**デフォルトの名無しさん** · 2006/01/03(火) 12:48:58

>>231
グローバル変数に flag なんて名前を使ってる馬鹿がいた。
おまけにその馬鹿、ローカル変数でも flag とか使ってるし。orz

**デフォルトの名無しさん** · 2006/01/03(火) 13:43:16

variable nanteguro-barunimoro-karunimotukatteruyatunanntemitakotoaruze

**デフォルトの名無しさん** · 2006/01/03(火) 23:28:50

nanteの跡でCtrl+Space打てば気楽に使えちゃうね、と思う俺は
VS色に染められてんでしょうか

**デフォルトの名無しさん** · 2006/01/13(金) 05:35:50

単に長いっていうヲチではなくて（ｒｙ

**デフォルトの名無しさん** · 2006/02/03(金) 21:52:28

UNKNOWNのスペルミスでUNKOWN

**デフォルトの名無しさん** · 2006/02/07(火) 03:56:34

LINKがRINK

**デフォルトの名無しさん** · 2006/02/07(火) 08:33:56

２項演算子の間空けない奴多し。

**デフォルトの名無しさん** · 2006/02/08(水) 02:19:32

foo . barだとかfoz -> barだとか書く香具師は殺したくなりますが何か。
あー違うか。二項演算子の間だから foo < < bar ってことか? それじゃコンパイルできねぇべさ。