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

■ このスレッドは過去ログ倉庫に格納されています
1デフォルトの名無しさん
垢版 |
NGNG
var unko;
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こくらい);

2005/12/07(水) 12:39:34
さすがにそれだとそのプロジェクト特定できるような

最後のThankYouはなんだ？本当にありがとうございました。みたいな感じ？
176174
垢版 |
2005/12/07(水) 20:59:07
ネタにマジレスすんなアホ
2005/12/07(水) 23:55:54
むしろ176の方がネタにマジレスに見える件。
178174
垢版 |
2005/12/08(木) 00:56:57
負け惜しみなんか言って恥ずかしくないのかな。
179デフォルトの名無しさん
垢版 |
2005/12/08(木) 03:47:09
おまいら、テストデータはどんなんにしてる？
180174
垢版 |
2005/12/08(木) 09:33:52
何でそんなこと教えなきゃならねんだよ、糞して寝ろ。
2005/12/08(木) 12:22:43
あー、もしかして>>174が大まじめに書いたソースが
このスレで晒されちゃったとか？それで粘着してんの？
182174
垢版 |
2005/12/08(木) 20:03:49
やかましいわ　氏ね
2005/12/08(木) 20:13:48
>>174
君を見てると昔の僕を見るようだ
184174
垢版 |
2005/12/08(木) 22:48:32
>>183
お前を見てると未来の旦那様を見ているようだ
185174
垢版 |
2005/12/08(木) 23:59:12
俺厨房なんですぐクビつって死んできます
本当にありがとうございました。
2005/12/09(金) 00:16:43
久しぶりにID無し板の楽しさを味わった気がするよ。
187174
垢版 |
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
どこでボタンを掛け違えちゃったんだろうな
200デフォルトの名無しさん
垢版 |
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"
　　　　　　　　};
203202
垢版 |
2005/12/14(水) 18:52:43
↑ごめんなさい・・・・でも本当なんです。
外注が作ったぷろぐらむなんです。Excelを使いたかったらしいんです。
他のclassでも同じ事を宣言してます。しっかりきっちりprivateで。

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

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


2005/12/14(水) 20:40:50
>>203
ソースもひどいが、受け入れテストもせずに納品出来てしまう管理体制も十分痛いな。
2005/12/14(水) 21:42:07
結論：そんな会社やめちまえ
206174
垢版 |
2005/12/14(水) 22:23:04
またネタにマジレスされた。
207174
垢版 |
2005/12/15(木) 04:38:50
ここはいい釣り堀ですね。
2005/12/15(木) 13:03:33
ネタじゃ無いですよ・・・
ネタだったらどれほど肩の荷が下りることか。
2005/12/15(木) 14:13:33
その仕事、私のところで引き受けてやろう。
#いや、その手の改修はお手の物で……_/￣|○
2005/12/16(金) 00:29:03
またネタにマジレス。いい加減見抜けよ。
211デフォルトの名無しさん
垢版 |
2005/12/16(金) 05:17:07
結構よくある話だよ。
ネタとか言ってるやつは無職鉄板。
2005/12/16(金) 05:46:51
大手メーカーのソフト開発の下請けで書いてる業者が
そういうコード書いてることが多い
2005/12/16(金) 10:50:19
>>212
下請けっつーか孫請け玄孫請けだろ。
2005/12/16(金) 10:54:50
>>209
いっとくけどすげーですよ？

class kyoutuuhensuu{
　　int pos;
　　

　　
}
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++ の標準入出力だというオチだったらどうする ?
2005/12/17(土) 13:46:17
C#です。カキコした部分は>>218のやり方で書き直しました。

>>219
その部分は楽勝なんですが、他にもグローバル変数的に使っている変数が
50以上あるし、コメントも最小限、
グローバル変数的に使ってる4次元位のジャグ配列も、
使う機能毎に要素の中身が違ったりすると、
もう何が何だか・・・ってあんばいです。
2005/12/17(土) 14:05:23
そんなん大した事ないし。
書いた方はたしかにレベルが低いが、
それをさっと直せないで何ヶ月も掛けてるのも最高にレベル低い。

第一 >>202の時点でスレ違いだろドアホウが。
224デフォルトの名無しさん
垢版 |
2005/12/25(日) 04:56:24
マジか
　　　　ﾍ⌒ヽﾌ
　　　（　・ω・）　㌧
　　　/ ~つと）
2005/12/25(日) 12:57:16
int death;
float hell;
double fuckyou;

これでGO
226デフォルトの名無しさん
垢版 |
2006/01/02(月) 15:07:57
const y=2006
227デフォルトの名無しさん
垢版 |
2006/01/03(火) 04:51:40
やたら長い変数は痛い?
英語で端的な説明になってるけど、略語になってない。
しかも、単語間はアンダースコアでつなげてある。
数個のグローバル変数がこういう名前なんだけど単語5つ以上だと結構長い。
2006/01/03(火) 04:56:00
グローバル変数は長くて醜くて入力しにくい名前のほうが良い。
229デフォルトの名無しさん
垢版 |
2006/01/03(火) 08:25:14
なんで？
2006/01/03(火) 08:35:46
>>229 軽い気持ちで使われちゃ困るからだろ。
231デフォルトの名無しさん
垢版 |
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色に染められてんでしょうか
235デフォルトの名無しさん
垢版 |
2006/01/13(金) 05:35:50
単に長いっていうヲチではなくて（ｒｙ
2006/02/03(金) 21:52:28
UNKNOWNのスペルミスでUNKOWN
237デフォルトの名無しさん
垢版 |
2006/02/07(火) 03:56:34
LINKがRINK
■ このスレッドは過去ログ倉庫に格納されています
5ちゃんねるの広告が気に入らない場合は、こちらをクリックしてください。
ニューススポーツなんでも実況