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

■ このスレッドは過去ログ倉庫に格納されています
1デフォルトの名無しさん
垢版 |
NGNG
var unko;
NGNG
Exception e;
ApplicationException ae;
IOException ioe;
SQLException sqle;
ClassNotFoundException cnfe;
↑と言う感じで書いてたら↓に直せと言ってきた
Exception exception;
ApplicationException applicationException;
IOException ioException;
SQLException sqlException;
ClassNotFoundException classNotFoundException;

これって一般的には常識ですか？
NGNG
>>74
全てPublicだからだろ。
NGNG
>>75
Javaでは常識です。
NGNG
ユーザ名：speed
管理パスワード：hitoe
一般パスワード：takako　eriko　hiroko
テストデータにて…
ところでみなさん
カラム名でなくテストデータにある程度自由な文字入れるのは自由ですよね？

79デフォルトの名無しさん
垢版 |
NGNG
>>78
公序良俗に反しなければｗ
NGNG
突然客先からテストデータを要求されてもいいように、「あの客うぜぇ」なんてコメント入れるのはやめましょう。
81デフォルトの名無しさん
垢版 |
NGNG
>>75
めんどくせ
全部 e でいいよ
NGNG
分かりきったローカルスコープに長い変数名使うのはイタイねぇ。
NGNG
Javaって最低
NGNG
比べてRuby最高！
85デフォルトの名無しさん
垢版 |
NGNG
ﾊﾟﾋｭｰｰﾝ！！！
NGNG
>>75
直せっていう奴が馬鹿
87デフォルトの名無しさん
垢版 |
NGNG
long chinko
short chinko
NGNG
long long ... // ｽｹﾞｰ

double ... // ｷﾓｲﾖｰ
signed ... // ﾅﾝｶﾂｲﾃﾙﾖｰ
NGNG
汚物でましたので報告まで。つ●
90デフォルトの名無しさん
垢版 |
NGNG
うん
91デフォルトの名無しさん
垢版 |
NGNG
NGNG
#define VALUE_ONE 1
93デフォルトの名無しさん
垢版 |
NGNG
#define VALUE_ICHI 1
94デフォルトの名無しさん
垢版 |
NGNG
ttp://www4.telge.kth.se/~d99_kme/
NGNG
>>92
これって本当にやっている人いたよな～
内心、「何これ？意味ねぇじゃん」っておもったけど
それをヘッダファイルに定義した人が、ちょっと怖そうな先輩だったので
裸の王様には何も言わないでおいておきました。
96デフォルトの名無しさん
垢版 |
NGNG
かっこいい変数名とかを考えるスレは無いの？
NGNG
char _g_busho12mysoftware_rgstrShiteidekiruMojiretsu[BUSHO12_SHITEIDEKIRU_MOJIRETSU_NO_MOJISUU];
NGNG
>>95
環境によって
#define VALUE_ONE (1 << 12)
などと、切り替える場合がある。
99デフォルトの名無しさん
垢版 |
NGNG
９９
100Yura
垢版 |
NGNG
>92＞#define VALUE_ONE 1
これにコメントが付いているなら意味があるように思います（昔似たような事をやってる人がいた）
NGNG
定数を直接コードに書かないのはわかるけど
問題は名前の付け方
NGNG
変数名・関数名じゃないけど、Blu-Rayって名称決めたヤツ痛すぎ。
NGNG
double[] iAvgTime1 = null;double[] iAvgSpeed1 = null;
double[] iAvgTime2 = null;double[] iAvgSpeed2 = null;
double[] iAvgTime3 = null;double[] iAvgSpeed3 = null;
double[] iAvgTime4 = null;double[] iAvgSpeed4 = null;
double[] iAvgTime5 = null;double[] iAvgSpeed5 = null;
double[] iAvgTime6 = null;double[] iAvgSpeed6 = null;
double[] iAvgTime7 = null;double[] iAvgSpeed7 = null;
double[] iAvgTime8 = null;double[] iAvgSpeed8 = null;
double[] iAvgTime9 = null;double[] iAvgSpeed9 = null;
double[] iAvgTime10 = null;double[] iAvgSpeed10 = null;

外注が作った、public void ナンタラカンタラ の中にこんなのが6ブロックも書いてあるPGを修正しています。
気軽にループ出来ないおかげで3000～4000行もあるpublic void ナンタラカンタラ が数カ所あります。
外注は「ウチもコノ仕事で赤字になった、次出すと言っていた仕事も出てこないしもうこれ以上
おまいの会社とは取引出来ない」と言って瑕疵対応期間中であるにも関わらずバグ修正してくれません。
なにか下さい。
NGNG
>>103
つ⑩
NGNG
>>102
次期XBOXがHD（高品位画像）に対応するとの発表に「HD DVD圧勝、Blu-Rayあぼーん決定」
という勘違いをする人もいるので、どっちもどっちかと。
106デフォルトの名無しさん
垢版 |
05/01/28 19:37:37
var ぬるぽ;
05/01/28 20:04:28
>>14,22
すいません・・。
いつもそんな感じの変数名付けてます・・。
英語わかんないし・・。
05/01/28 23:04:18
>>107
辞書を買う金もないとは言わんよな。
ネットがあれば英訳するのに辞書もいらないが
05/01/29 02:15:07
DBのフィールド名は日本語入れたりするな
とくに勘定系とか。英語だけだとわけわかんなくなるし
05/01/29 10:26:07
>>107
今すぐ　死　ん　で　く　だ　さ　い
05/01/29 20:50:13
int atoDeTsukauAtaiHozon;
05/01/30 02:10:31
いち、にぃ、はんがりあん♪
05/02/24 22:05:55
>>1
assert(BrainCheck(">1"));

アサートが通らん気がする。
114デフォルトの名無しさん
垢版 |
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はどこ行った？ってなるわけだ。
120デフォルトの名無しさん
垢版 |
2005/05/25(水) 07:09:20
転職した先輩から引き継いだソースコードを読んでたら、sinjiやらasuka
といったアニメキャラの名前っぽい変数名が沢山出てきてちょっとひいた。
121デフォルトの名無しさん
垢版 |
2005/05/25(水) 08:48:13
おいらは

hoge

とか出てきたらブチまわしたくなる衝動がある
2005/05/25(水) 10:00:13
HogeHoge。

123デフォルトの名無しさん
垢版 |
2005/05/25(水) 10:03:58
　　　　　　　　　 ﾉﾉﾉﾉ　　　 　＿　＿　　　　　.'　　，　．.　　 ∧＿∧
　　　　　 　　　(ﾟ∈ﾟ　)_　- ― ＝￣　￣`:,　.∴　'　　　　　 （　　　　）
　　　　　　　　　ヽ-''￣　　　　＿＿――＝',　・,‘　ｒ⌒>　 _／ ／
　　　　　　　　／　　,,-―　￣￣　　￣"'"　. 　　’ |　ｙ'⌒　　⌒i
　　　　　　　/　　　ノ＼＼　　　　　　　　　　　 .　　|　　／　　ノ ｜
　　 　 　 　/　 　 /　　 ＼＼ 　　　　　　　　　 　　, ー' 　／´ヾ_ﾉ
　　　　　 　ﾚ　　ノ　　　　　ヽ_つ　　 　　　　　　／　,　　ノ
　　　　　　/　 /　　　　　　　　　　　　　　　　.／　／ ／
　　　 　　/　 /|　　　　　　　　　　　　　　　／　／　,'
　 　　 　（　（　､　　　　　　　　　　　　　／　 ／| 　|
　　　 　　|　 |､　＼　　　　　　　　　 　!､＿／ / 　 〉
　　 　.　　|　/　＼　⌒l　　　　　　　　　 　　　|_／
　　　　　　|　|　　　）　/
　　　　　ノ　 ）　　 し'
　　　　(＿／　　　　　　　　　　　　　-=＝≡≡≡＝==-
2005/05/25(水) 10:11:03
hoge??
125デフォルトの名無しさん
垢版 |
2005/05/25(水) 10:15:46
　　　 _, ,_ 　パーン
　 （　゜д゜）
　　　⊂彡☆＝＝＝＝＝＝ /|
　　　　　 　＿＿　　　　　　　|/
　　　　　 ヽ| l　l│
　　　　　　┷┷┷
2005/05/26(木) 15:53:59
>>120
おまいの先輩はｴｳﾞｧｦﾀか？
127デフォルトの名無しさん
垢版 |
2005/08/10(水) 02:21:38
Dim 初号機
2005/08/10(水) 09:29:16
declare @nifty
129デフォルトの名無しさん
垢版 |
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;
とかなら実際に見た事がある。
131デフォルトの名無しさん
垢版 |
2005/10/09(日) 18:47:34
ﾜﾛｽ
132デフォルトの名無しさん
垢版 |
2005/10/09(日) 22:19:14
int i, ii, iii, iii, iiiiiiii, iiiiii
133デフォルトの名無しさん
垢版 |
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目ならべ(マルバツ)のソースより。実話。
135デフォルトの名無しさん
垢版 |
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
後者はどう言う命名の仕方なんだ？
140デフォルトの名無しさん
垢版 |
2005/12/02(金) 01:24:11
>>136
ローカル変数だと、長すぎる変数名は邪魔なだけだからある程度の略語はしょうがない。
しかし推測できないのや、長い関数だと駄目だが。
2005/12/02(金) 01:59:00
変数名ではないが、某企業のライブラリの実装ガイドの
サンプルコードに

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

orz
2005/12/02(金) 02:08:03
>>137
後のほうで使うデータ保存する変数を、使われる場所がすぐにわからない
ような前の方に宣言すること自体、痛いのでは。
143137
垢版 |
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 ...
がいいな、と。
146137
垢版 |
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
派閥味噌
153デフォルトの名無しさん
垢版 |
2005/12/03(土) 17:14:53
>>151
どっちがマシとか決めるのが無意味なレベルだと思うけどな
154デフォルトの名無しさん
垢版 |
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];
156デフォルトの名無しさん
垢版 |
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進数である事を祈る。
163158
垢版 |
2005/12/05(月) 13:49:56
>>161
あーいや、ある意味翁の取り上げた関数を超えている。
なんせ、2000行の中にgotoで仕込まれたループが6つもあるんだ。
それぞれのループの中は200行ほどの同じような演算だと言うのに。
膨大なローカル変数だけでなくグローバル変数も使っているから
どこで分離して関数化するかが悩みどころなんだ。
#しかも、コードを見る限り大幅にコードを削減した旨のコメントがあるんだよね。
164デフォルトの名無しさん
垢版 |
2005/12/06(火) 12:56:57
Cプログラミング診断室自体の質はどうなんだろう・・・
2005/12/06(火) 13:28:18
古いけど悪くはないと思うよ。
166デフォルトの名無しさん
垢版 |
2005/12/06(火) 14:29:45
マジか
　　　　ﾍ⌒ヽﾌ
　　　（　・ω・）　㌧
　　　/ ~つと）
167デフォルトの名無しさん
垢版 |
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
それは何か勘違いをしているのではないかな?
紀元節とは今の建国記念日のことなのだが。
まさか、天皇否定の三国人が紀元節なんて書くはずもないし。
170デフォルトの名無しさん
垢版 |
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こくらい);

■ このスレッドは過去ログ倉庫に格納されています
5ちゃんねるの広告が気に入らない場合は、こちらをクリックしてください。
ニューススポーツなんでも実況