[表示 : 全て 最新50 1-99 101- 201- 301- 401- 501- 601- 2chのread.cgiへ]
Update time : 05/09 15:07 / Filesize : 121 KB / Number-of Response : 678
[このスレッドの書き込みを削除する]
[+板 最近立ったスレ&熱いスレ一覧 : +板 最近立ったスレ/記者別一覧] [類似スレッド一覧]


↑キャッシュ検索、類似スレ動作を修正しました、ご迷惑をお掛けしました

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



1 名前:デフォルトの名無しさん [04/02/19 05:59]
var unko;

75 名前:デフォルトの名無しさん mailto:sage [04/11/03 11:42:29]
Exception e;
ApplicationException ae;
IOException ioe;
SQLException sqle;
ClassNotFoundException cnfe;
↑と言う感じで書いてたら↓に直せと言ってきた
Exception exception;
ApplicationException applicationException;
IOException ioException;
SQLException sqlException;
ClassNotFoundException classNotFoundException;

これって一般的には常識ですか?

76 名前:デフォルトの名無しさん mailto:sage [04/11/03 15:27:15]
>>74
全てPublicだからだろ。


77 名前:デフォルトの名無しさん mailto:sage [04/11/03 18:48:34]
>>75
Javaでは常識です。

78 名前:デフォルトの名無しさん mailto:sage [04/11/05 21:54:40]
ユーザ名:speed
管理パスワード:hitoe
一般パスワード:takako eriko hiroko
テストデータにて…
ところでみなさん
カラム名でなくテストデータにある程度自由な文字入れるのは自由ですよね?



79 名前:デフォルトの名無しさん [04/11/07 02:15:19]
>>78
公序良俗に反しなければw

80 名前:デフォルトの名無しさん mailto:sage [04/11/07 04:12:39]
突然客先からテストデータを要求されてもいいように、「あの客うぜぇ」なんてコメント入れるのはやめましょう。

81 名前:デフォルトの名無しさん [04/11/07 04:48:47]
>>75
めんどくせ
全部 e でいいよ

82 名前:デフォルトの名無しさん mailto:sage [04/11/07 04:49:42]
分かりきったローカルスコープに長い変数名使うのはイタイねぇ。

83 名前:デフォルトの名無しさん mailto:sage [04/11/07 13:08:04]
Javaって最低



84 名前:デフォルトの名無しさん mailto:sage [04/11/07 14:00:21]
比べてRuby最高!

85 名前:デフォルトの名無しさん [04/11/07 19:45:18]
パヒューーン!!!

86 名前:デフォルトの名無しさん mailto:sage [04/11/07 20:14:07]
>>75
直せっていう奴が馬鹿


87 名前:デフォルトの名無しさん [04/11/08 08:50:23]
long chinko
short chinko

88 名前:デフォルトの名無しさん mailto:sage [04/11/08 08:54:10]
long long ... // スゲー

double ... // キモイヨー
signed ... // ナンカツイテルヨー

89 名前:デフォルトの名無しさん mailto:sage [04/11/10 02:05:35]
汚物でましたので報告まで。つ●

90 名前:デフォルトの名無しさん [04/11/16 03:30:10]
うん

91 名前:デフォルトの名無しさん [04/11/22 15:26:13]


92 名前:デフォルトの名無しさん mailto:sage [04/11/23 03:00:47]
#define VALUE_ONE 1

93 名前:デフォルトの名無しさん [04/12/01 03:40:41]
#define VALUE_ICHI 1



94 名前:デフォルトの名無しさん [04/12/01 06:08:43]
ttp://www4.telge.kth.se/~d99_kme/

95 名前:デフォルトの名無しさん mailto:sage [04/12/21 00:46:31]
>>92
これって本当にやっている人いたよな〜
内心、「何これ?意味ねぇじゃん」っておもったけど
それをヘッダファイルに定義した人が、ちょっと怖そうな先輩だったので
裸の王様には何も言わないでおいておきました。

96 名前:デフォルトの名無しさん [05/01/02 00:00:26]
かっこいい変数名とかを考えるスレは無いの?

97 名前:デフォルトの名無しさん mailto:sage [05/01/02 09:23:07]
char _g_busho12mysoftware_rgstrShiteidekiruMojiretsu[BUSHO12_SHITEIDEKIRU_MOJIRETSU_NO_MOJISUU];

98 名前:デフォルトの名無しさん mailto:sage [05/01/03 10:33:33]
>>95
環境によって
#define VALUE_ONE (1 << 12)
などと、切り替える場合がある。

99 名前:デフォルトの名無しさん [05/01/06 19:46:30]
99

100 名前:Yura [05/01/06 20:07:19]
>92>#define VALUE_ONE 1
これにコメントが付いているなら意味があるように思います(昔似たような事をやってる人がいた)

101 名前:デフォルトの名無しさん mailto:sage [05/01/06 20:33:47]
定数を直接コードに書かないのはわかるけど
問題は名前の付け方

102 名前:デフォルトの名無しさん mailto:sage [05/01/08 02:14:48]
変数名・関数名じゃないけど、Blu-Rayって名称決めたヤツ痛すぎ。

103 名前:デフォルトの名無しさん mailto:sage [05/01/10 20:57:32]
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 ナンタラカンタラ が数カ所あります。
外注は「ウチもコノ仕事で赤字になった、次出すと言っていた仕事も出てこないしもうこれ以上
おまいの会社とは取引出来ない」と言って瑕疵対応期間中であるにも関わらずバグ修正してくれません。
なにか下さい。



104 名前:デフォルトの名無しさん mailto:sage [05/01/10 22:16:51]
>>103
つI

105 名前:デフォルトの名無しさん mailto:sage [05/01/11 14:43:50]
>>102
次期XBOXがHD(高品位画像)に対応するとの発表に「HD DVD圧勝、Blu-Rayあぼーん決定」
という勘違いをする人もいるので、どっちもどっちかと。

106 名前:デフォルトの名無しさん [05/01/28 19:37:37 ]
var ぬるぽ;

107 名前:デフォルトの名無しさん mailto:sage [05/01/28 20:04:28 ]
>>14,22
すいません・・。
いつもそんな感じの変数名付けてます・・。
英語わかんないし・・。

108 名前:デフォルトの名無しさん mailto:sage [05/01/28 23:04:18 ]
>>107
辞書を買う金もないとは言わんよな。
ネットがあれば英訳するのに辞書もいらないが

109 名前:デフォルトの名無しさん mailto:sage [05/01/29 02:15:07 ]
DBのフィールド名は日本語入れたりするな
とくに勘定系とか。英語だけだとわけわかんなくなるし

110 名前:デフォルトの名無しさん mailto:sage [05/01/29 10:26:07 ]
>>107
今すぐ 死 ん で く だ さ い

111 名前:デフォルトの名無しさん mailto:sage [05/01/29 20:50:13 ]
int atoDeTsukauAtaiHozon;

112 名前:デフォルトの名無しさん mailto:sage [05/01/30 02:10:31 ]
いち、にぃ、はんがりあん♪

113 名前:デフォルトの名無しさん mailto:sage [05/02/24 22:05:55 ]
>>1
assert(BrainCheck(">1"));

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



114 名前:デフォルトの名無しさん [05/03/17 04:11:39 ]
var chan;

115 名前:デフォルトの名無しさん mailto:sage [05/03/18 15:12:19 ]
double ore;

ko_hai=ore;

116 名前:デフォルトの名無しさん mailto:sage [05/03/19 00:44:54 ]
以前の職場で

int _KONO_HENSU_KESUNA_

ってのがあったな。

117 名前:デフォルトの名無しさん mailto:sage [2005/03/21(月) 18:39:56 ]
int _HAYAKU_IENI_KAERITAI_

118 名前:デフォルトの名無しさん mailto:sage [2005/03/23(水) 16:47:09 ]
hogeFunc();
hogeFunc2();
hogeFunc3();
hogeFunc4();

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

119 名前:デフォルトの名無しさん mailto:sage [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

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

122 名前:デフォルトの名無しさん mailto:sage [2005/05/25(水) 10:00:13 ]
HogeHoge。



123 名前:デフォルトの名無しさん [2005/05/25(水) 10:03:58 ]
          ノノノノ     _ _     .'  , ..   ∧_∧
         (゚∈゚ )_ - ― = ̄  ̄`:, .∴ '      (    )
         ヽ-'' ̄    __――=', ・,‘ r⌒>  _/ /
        /  ,,-―  ̄ ̄   ̄"'" .   ’ | y'⌒  ⌒i
       /   ノ\\            .  |  /  ノ |
        /    /   \\             , ー'  /´ヾ_ノ
       レ  ノ     ヽ_つ         / ,  ノ
      /  /                ./ / /
      /  /|               / / ,'
      ( ( 、             /  /|  |
      |  |、 \           !、_/ /   〉
    .  | / \ ⌒l             |_/
      | |   ) /
     ノ  )   し'
    (_/             -==≡≡≡===-




124 名前:デフォルトの名無しさん mailto:sage [2005/05/25(水) 10:11:03 ]
hoge??

125 名前:デフォルトの名無しさん [2005/05/25(水) 10:15:46 ]
    _, ,_  パーン
  ( ゜д゜)
   ⊂彡☆====== /|
       __       |/
      ヽ| l l│
      ┷┷┷


126 名前:デフォルトの名無しさん mailto:sage [2005/05/26(木) 15:53:59 ]
>>120
おまいの先輩はエヴァヲタか?

127 名前:デフォルトの名無しさん [2005/08/10(水) 02:21:38 ]
Dim 初号機

128 名前:デフォルトの名無しさん mailto:sage [2005/08/10(水) 09:29:16 ]
declare @nifty

129 名前:デフォルトの名無しさん [2005/08/10(水) 12:03:18 ]
hage

130 名前:デフォルトの名無しさん mailto:sage [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

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



134 名前:デフォルトの名無しさん mailto:sage [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 ]
ワロタw

136 名前:デフォルトの名無しさん mailto:sage [2005/12/01(木) 10:33:37 ]
*pr = lna() + px; みたいなのがあったな。
略号だらけで意味がわからん。


137 名前:デフォルトの名無しさん mailto:sage [2005/12/01(木) 11:59:13 ]
ローマ字でも漢字でも変な省略でもまあ許すけど
変数にtempとかブール関数にcheckとか、「意味のある名前付けろ」の
意味を分かってない名前は勘弁してほしいんだけどほんとに多い。
たとえば後で使う販売店名称を保存するのに
The_temporary_variable_used_later
なんて英語の変数だったら HBT_MSO のほうが「まだまし」だと思う。

138 名前:デフォルトの名無しさん mailto:sage [2005/12/01(木) 17:04:36 ]
TOMOYO Linux

139 名前:デフォルトの名無しさん mailto:sage [2005/12/01(木) 22:49:23 ]
>>137
後者はどう言う命名の仕方なんだ?

140 名前:デフォルトの名無しさん [2005/12/02(金) 01:24:11 ]
>>136
ローカル変数だと、長すぎる変数名は邪魔なだけだからある程度の略語はしょうがない。
しかし推測できないのや、長い関数だと駄目だが。

141 名前:デフォルトの名無しさん mailto:sage [2005/12/02(金) 01:59:00 ]
変数名ではないが、某企業のライブラリの実装ガイドの
サンプルコードに

System.out.println("キタ----(゚∀゚)----!!!!");

orz

142 名前:デフォルトの名無しさん mailto:sage [2005/12/02(金) 02:08:03 ]
>>137
後のほうで使うデータ保存する変数を、使われる場所がすぐにわからない
ような前の方に宣言すること自体、痛いのでは。

143 名前:137 mailto:sage [2005/12/02(金) 15:20:31 ]
>>142
言語仕様上、前の方でしか宣言できない場合、どうするのかと。

# C99以前のCとか


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



144 名前:デフォルトの名無しさん mailto:sage [2005/12/02(金) 15:31:36 ]
HanBaiTen_MeiSyOu

145 名前:デフォルトの名無しさん mailto:sage [2005/12/02(金) 15:58:58 ]
>>139
>>143
いやHBT_MSOは>>22から拝借したの。販売店名称をローマ字にして適当に縮めたんじゃない?
ちょっとたとえがまずかったですか。つまり
「変更前の販売店名称を、後で使うために仮保存する」んだったら
「変更前の販売店名称」だと少しでも分かる名前にしてくれ、と言いたかった。
「後で(直後でもね)使うために(仮)保存する」のはあらゆる変数がそうなんだから。

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


146 名前:137 mailto:sage [2005/12/02(金) 22:24:41 ]
>>145
言いたい事はよくわかった。
が、例えで出した名称は>>22を拝借せず、
大抵の人がすぐ判る名前にすべきだったね。



# 販売店名称なら、安直に「ShopName」とかでいいんジャマイカ?

147 名前:デフォルトの名無しさん mailto:sage [2005/12/02(金) 23:06:34 ]
「HBT_MSO というのも相当ひどい変数名だが、それでも The_temporary_variable_used_later よりはマシだ」

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

148 名前:デフォルトの名無しさん mailto:sage [2005/12/02(金) 23:15:17 ]
int cunt;

149 名前:デフォルトの名無しさん mailto:sage [2005/12/02(金) 23:50:53 ]
double speeed; // (km/h)


150 名前:デフォルトの名無しさん mailto:sage [2005/12/02(金) 23:55:10 ]
int WindowChenge(int flg)

151 名前:デフォルトの名無しさん mailto:sage [2005/12/03(土) 16:17:18 ]
>>147
どちらも相当ひどいが、糞コードを押しつけられて何をやっているか解析しろと
言われた場合、全コードに渡って前者のような意味不明の略号変数が散乱して
るものよりも、後者の方がまだマシだと思うな

152 名前:デフォルトの名無しさん mailto:sage [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

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

155 名前:デフォルトの名無しさん mailto:sage [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行程度に納めるようなコントロールって
誰でもやってんじゃないのか???


157 名前:デフォルトの名無しさん mailto:sage [2005/12/03(土) 23:55:45 ]
そうそう。
変数宣言と実際に使うところの距離があるってことは
その関数が複雑になりすぎてるってこと。


158 名前:デフォルトの名無しさん mailto:sage [2005/12/04(日) 10:26:10 ]
今編集しているプログラム、main()だけで2000行。
取り敢えず関数分割前にローカル変数の整理を始めた。
全てmain()の先頭で宣言していたのだが、ここのブロック内に移動できるものはいどうしたわけだ。
…何故か変数が半分に減った挙句、処理速度が数%向上した……

159 名前:デフォルトの名無しさん mailto:sage [2005/12/04(日) 13:53:44 ]
生保系システムのレビューを行ったら、変数名だけで100文字ってのがあった。

160 名前:デフォルトの名無しさん mailto:sage [2005/12/04(日) 15:31:48 ]
>>159
レビューだけで良かったねw

161 名前:デフォルトの名無しさん mailto:sage [2005/12/05(月) 12:56:21 ]
>>158
世の中、上には上が居る。(w
# こう言う上は居て欲しくないがな。

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

162 名前:デフォルトの名無しさん mailto:sage [2005/12/05(月) 12:59:25 ]
>>154
そのインデックスが、『せめて』10進数である事を祈る。

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



164 名前:デフォルトの名無しさん [2005/12/06(火) 12:56:57 ]
Cプログラミング診断室自体の質はどうなんだろう・・・

165 名前:デフォルトの名無しさん mailto:sage [2005/12/06(火) 13:28:18 ]
古いけど悪くはないと思うよ。

166 名前:デフォルトの名無しさん [2005/12/06(火) 14:29:45 ]
マジか
    ヘ⌒ヽフ
   ( ・ω・) d
   / ~つと)

167 名前:デフォルトの名無しさん [2005/12/06(火) 20:27:35 ]
>>166
かわいいね、それ

168 名前:デフォルトの名無しさん mailto:sage [2005/12/06(火) 21:02:30 ]
if ($kigen == '1') {
print '昭和';
} else if ($kigen == '2') {
print '平成';
}

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

$kigen; //紀元節



169 名前:デフォルトの名無しさん mailto:sage [2005/12/06(火) 22:21:25 ]
それは何か勘違いをしているのではないかな?
紀元節とは今の建国記念日のことなのだが。
まさか、天皇否定の三国人が紀元節なんて書くはずもないし。

170 名前:デフォルトの名無しさん [2005/12/07(水) 04:22:13 ]
某王手のソースがこんなんでげんなりした

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


          1000行くらい


          }
        }
      }
    }
  }
} 

171 名前:デフォルトの名無しさん mailto:sage [2005/12/07(水) 06:42:56 ]
>>170
その書き方だと、全ての if が同じ条件式のように見えてキモさ5割増。

172 名前:デフォルトの名無しさん mailto:sage [2005/12/07(水) 08:25:47 ]
>>171
むしろ100倍

173 名前:デフォルトの名無しさん mailto:sage [2005/12/07(水) 09:02:14 ]
public boolean isSyoriShimashita() { return syoriShimashita; }
ってのなら新人時代にやったことがある。

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



174 名前:デフォルトの名無しさん mailto:sage [2005/12/07(水) 11:23:34 ]
今日こんなの目撃orz

GetAccessNameAndIDBeforePostForPurchaseProductsThankYou(引数6こくらい);



175 名前:デフォルトの名無しさん mailto:sage [2005/12/07(水) 12:39:34 ]
さすがにそれだとそのプロジェクト特定できるような

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






[ 続きを読む ] / [ 携帯版 ]

前100 次100 最新50 [ このスレをブックマーク! 携帯に送る ] 2chのread.cgiへ
[+板 最近立ったスレ&熱いスレ一覧 : +板 最近立ったスレ/記者別一覧]( ´∀`)<121KB

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef