1 名前:デフォルトの名無しさん [04/02/19 05:59] var unko;
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はなんだ?本当にありがとうございました。みたいな感じ?
176 名前:174 mailto:sage [2005/12/07(水) 20:59:07 ] ネタにマジレスすんなアホ
177 名前:デフォルトの名無しさん mailto:sage [2005/12/07(水) 23:55:54 ] むしろ176の方がネタにマジレスに見える件。
178 名前:174 mailto:sage [2005/12/08(木) 00:56:57 ] 負け惜しみなんか言って恥ずかしくないのかな。
179 名前:デフォルトの名無しさん [2005/12/08(木) 03:47:09 ] おまいら、テストデータはどんなんにしてる?
180 名前:174 mailto:sage [2005/12/08(木) 09:33:52 ] 何でそんなこと教えなきゃならねんだよ、糞して寝ろ。
181 名前:デフォルトの名無しさん mailto:sage [2005/12/08(木) 12:22:43 ] あー、もしかして>>174 が大まじめに書いたソースが このスレで晒されちゃったとか?それで粘着してんの?
182 名前:174 mailto:sage [2005/12/08(木) 20:03:49 ] やかましいわ 氏ね
183 名前:デフォルトの名無しさん mailto:sage [2005/12/08(木) 20:13:48 ] >>174 君を見てると昔の僕を見るようだ
184 名前:174 mailto:sage [2005/12/08(木) 22:48:32 ] >>183 お前を見てると未来の旦那様を見ているようだ
185 名前:174 mailto:sage [2005/12/08(木) 23:59:12 ] 俺厨房なんですぐクビつって死んできます 本当にありがとうございました。
186 名前:デフォルトの名無しさん mailto:sage [2005/12/09(金) 00:16:43 ] 久しぶりにID無し板の楽しさを味わった気がするよ。
187 名前:174 [2005/12/09(金) 03:51:51 ] そんなおまいに空中元彌チョップ
188 名前:デフォルトの名無しさん mailto:sage [2005/12/09(金) 12:02:43 ] s/空中/宇宙/g
189 名前:デフォルトの名無しさん mailto:sage [2005/12/09(金) 12:29:00 ] グローバルにしたって1個しかマッチしないやんけ!
190 名前:デフォルトの名無しさん mailto:sage [2005/12/09(金) 14:06:15 ] 汚物:グローバルにしたって1個しかマッチしないやんけ! 普通:グローバルにしたって1個しかマッチせえへんやんけ!
191 名前:デフォルトの名無しさん mailto:sage [2005/12/09(金) 14:25:10 ] 拡張:グローバルにしたかて1個しかマッチせえへんやんけ!
192 名前:デフォルトの名無しさん mailto:sage [2005/12/09(金) 14:31:49 ] すげえ、俺の作る class より拡張性たけえや
193 名前:デフォルトの名無しさん mailto:sage [2005/12/10(土) 04:03:25 ] 何このスレのAA
194 名前:デフォルトの名無しさん mailto:sage [2005/12/10(土) 05:38:14 ] 汚物:何このスレのAA
195 名前:デフォルトの名無しさん mailto:sage [2005/12/10(土) 13:44:36 ] 吐瀉物:汚物:何このスレのAA
196 名前:デフォルトの名無しさん mailto:sage [2005/12/10(土) 15:11:17 ] 排泄物:吐瀉物:汚物:何このスレのAA
197 名前:デフォルトの名無しさん mailto:sage [2005/12/10(土) 22:59:02 ] 何このスレ・・・・・・・? /ヽ /ヽ / ヽ / ヽ ______ /U ヽ___/ ヽ | ____ / U :::::::::::U:\ | | // ___ \ ::::::::::::::| | | | | | U :::::::::::::| | | .|U | | ::::::U::::| | | | ├―-┤ U.....:::::::::::::::::::/ | |____ ヽ .....:::::::::::::::::::::::< └___/ ̄ ̄ :::::::::::::::::::::::::| |\ | :::::::::::::::::::::::| \ \ \___ ::::::
198 名前:デフォルトの名無しさん mailto:sage [2005/12/11(日) 07:13:25 ] ちょっと前まで良スレだったと思うんだ
199 名前:デフォルトの名無しさん mailto:sage [2005/12/11(日) 11:37:30 ] どこでボタンを掛け違えちゃったんだろうな
200 名前:デフォルトの名無しさん [2005/12/14(水) 09:13:39 ] CLS 3
201 名前:デフォルトの名無しさん mailto:sage [2005/12/14(水) 16:47:57 ] int tmp, tmp2, tmp3, ... tmp30; 痛い・・・
202 名前:デフォルトの名無しさん mailto:sage [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" };
203 名前:202 mailto:sage [2005/12/14(水) 18:52:43 ] ↑ごめんなさい・・・・でも本当なんです。 外注が作ったぷろぐらむなんです。Excelを使いたかったらしいんです。 他のclassでも同じ事を宣言してます。しっかりきっちりprivateで。 バグだらけ、と言うか完成度30%くらいで納品してきた物を、 リストラ対象だった前任者が受け入れテストをせずに客先に納品して辞めていきました。 完成度30%ですが、外注先からはこの件についてはこれ以上対応できない、と言うメールをもらいました。 以来何ヶ月もこんなコードをずっと直し続けてます・・・
204 名前:デフォルトの名無しさん mailto:sage [2005/12/14(水) 20:40:50 ] >>203 ソースもひどいが、受け入れテストもせずに納品出来てしまう管理体制も十分痛いな。
205 名前:デフォルトの名無しさん mailto:sage [2005/12/14(水) 21:42:07 ] 結論:そんな会社やめちまえ
206 名前:174 mailto:sage [2005/12/14(水) 22:23:04 ] またネタにマジレスされた。
207 名前:174 mailto:sage [2005/12/15(木) 04:38:50 ] ここはいい釣り堀ですね。
208 名前:202 ◆cofYHAha3E mailto:sage [2005/12/15(木) 13:03:33 ] ネタじゃ無いですよ・・・ ネタだったらどれほど肩の荷が下りることか。
209 名前:デフォルトの名無しさん mailto:sage [2005/12/15(木) 14:13:33 ] その仕事、私のところで引き受けてやろう。 #いや、その手の改修はお手の物で……_/ ̄|○
210 名前:208 ◆r5LpnmM55A mailto:sage [2005/12/16(金) 00:29:03 ] またネタにマジレス。いい加減見抜けよ。
211 名前:デフォルトの名無しさん [2005/12/16(金) 05:17:07 ] 結構よくある話だよ。 ネタとか言ってるやつは無職鉄板。
212 名前:デフォルトの名無しさん mailto:sage [2005/12/16(金) 05:46:51 ] 大手メーカーのソフト開発の下請けで書いてる業者が そういうコード書いてることが多い
213 名前:デフォルトの名無しさん mailto:sage [2005/12/16(金) 10:50:19 ] >>212 下請けっつーか孫請け玄孫請けだろ。
214 名前:202 ◆cofYHAha3E mailto:sage [2005/12/16(金) 10:54:50 ] >>209 いっとくけどすげーですよ? class kyoutuuhensuu{ int pos; }
215 名前:202 ◆cofYHAha3E mailto:sage [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 で取得してたり。。。 もう、なんか嫌がらせとしか思えないw C#初心者と言うよりもPG初心者だったんだろうなぁ・・・ 俺も昔はこんなコード書いてたかも、と思うとさらにガクブルです。
216 名前:デフォルトの名無しさん mailto:sage [2005/12/16(金) 15:54:31 ] もともとposはprivateだったのにpublicに書き換えた可能性あるな。 おそらくデスマーチが始まって最初に設計した香具師と違うアフォが 動員されたと。
217 名前:デフォルトの名無しさん mailto:sage [2005/12/16(金) 22:41:34 ] >>215 C#のプロパティの書き方ってそんな感じだっけ?
218 名前:デフォルトの名無しさん mailto:sage [2005/12/16(金) 22:48:12 ] get{};set{};
219 名前:デフォルトの名無しさん mailto:sage [2005/12/17(土) 00:23:05 ] つーか、そんなんリファクタリング楽勝じゃん。 そんなんで >いっとくけどすげーですよ? って、お前もスゲーよ。脳味噌の足りなさが。
220 名前:デフォルトの名無しさん mailto:sage [2005/12/17(土) 02:36:03 ] > グローバル変数的に使ってるクラスがあるんですが、 これがいちばんすげーよ
221 名前:デフォルトの名無しさん mailto:sage [2005/12/17(土) 05:16:09 ] >>220 実は C++ の標準入出力だというオチだったらどうする ?
222 名前:202 ◆cofYHAha3E mailto:sage [2005/12/17(土) 13:46:17 ] C#です。カキコした部分は>>218 のやり方で書き直しました。 >>219 その部分は楽勝なんですが、他にもグローバル変数的に使っている変数が 50以上あるし、コメントも最小限、 グローバル変数的に使ってる4次元位のジャグ配列も、 使う機能毎に要素の中身が違ったりすると、 もう何が何だか・・・ってあんばいです。
223 名前:デフォルトの名無しさん mailto:sage [2005/12/17(土) 14:05:23 ] そんなん大した事ないし。 書いた方はたしかにレベルが低いが、 それをさっと直せないで何ヶ月も掛けてるのも最高にレベル低い。 第一 >>202 の時点でスレ違いだろドアホウが。
224 名前:デフォルトの名無しさん [2005/12/25(日) 04:56:24 ] マジか ヘ⌒ヽフ ( ・ω・) d / ~つと)
225 名前:デフォルトの名無しさん mailto:sage [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つ以上だと結構長い。
228 名前:デフォルトの名無しさん mailto:sage [2006/01/03(火) 04:56:00 ] グローバル変数は長くて醜くて入力しにくい名前のほうが良い。
229 名前:デフォルトの名無しさん [2006/01/03(火) 08:25:14 ] なんで?
230 名前:デフォルトの名無しさん mailto:sage [2006/01/03(火) 08:35:46 ] >>229 軽い気持ちで使われちゃ困るからだろ。
231 名前:デフォルトの名無しさん [2006/01/03(火) 12:00:16 ] >>228 でも覚えやすい方がいいな 最低なのは短いのに忘れやすい名前 しかもよく使う言葉だとよく被るんだよなあ
232 名前:デフォルトの名無しさん mailto:sage [2006/01/03(火) 12:48:58 ] >>231 グローバル変数に flag なんて名前を使ってる馬鹿がいた。 おまけにその馬鹿、ローカル変数でも flag とか使ってるし。orz
233 名前:デフォルトの名無しさん mailto:sage [2006/01/03(火) 13:43:16 ] variable nanteguro-barunimoro-karunimotukatteruyatunanntemitakotoaruze
234 名前:デフォルトの名無しさん mailto:sage [2006/01/03(火) 23:28:50 ] nanteの跡でCtrl+Space打てば気楽に使えちゃうね、と思う俺は VS色に染められてんでしょうか
235 名前:デフォルトの名無しさん [2006/01/13(金) 05:35:50 ] 単に長いっていうヲチではなくて(ry
236 名前:デフォルトの名無しさん mailto:sage [2006/02/03(金) 21:52:28 ] UNKNOWNのスペルミスでUNKOWN
237 名前:デフォルトの名無しさん [2006/02/07(火) 03:56:34 ] LINKがRINK
238 名前:デフォルトの名無しさん mailto:sage [2006/02/07(火) 08:33:56 ] 2項演算子の間空けない奴多し。
239 名前:デフォルトの名無しさん mailto:sage [2006/02/08(水) 02:19:32 ] foo . barだとかfoz -> barだとか書く香具師は殺したくなりますが何か。 あー違うか。二項演算子の間だから foo < < bar ってことか? それじゃコンパイルできねぇべさ。
240 名前:デフォルトの名無しさん [2006/02/08(水) 03:48:38 ] 1項演算子はどーする?
241 名前:デフォルトの名無しさん mailto:sage [2006/02/08(水) 15:01:21 ] 普通は単項演算子と言うんだよ
242 名前:デフォルトの名無しさん [2006/02/09(木) 12:25:46 ] 対義語は複項演算子か
243 名前:デフォルトの名無しさん mailto:sage [2006/02/09(木) 12:42:01 ] >>242 なんで対義語の話がでてくるんだ? 1の反対は2で、2の反対は1とでもいうのか?
244 名前:俺様は242に非ず mailto:sage [2006/02/09(木) 12:59:47 ] 「単」の対義語を「複」とするのはおかしくないが 1の反対は2じゃ無いだろ。
245 名前:デフォルトの名無しさん mailto:sage [2006/02/09(木) 22:42:14 ] 2項演算子の間を「常時」空けるのはさすがにセンスを疑う
246 名前:デフォルトの名無しさん mailto:sage [2006/02/09(木) 23:00:33 ] // 手元のコードを常時空けてみた if (v . Chars[v . Length - 1] == "n") { this . First = false; v = v . Substring(0 , v . Length - 1); }
247 名前:デフォルトの名無しさん mailto:sage [2006/02/10(金) 00:40:03 ] vsが勝手に開けてくれますが。 そういう設定にしてるんだけど
248 名前:デフォルトの名無しさん mailto:sage [2006/02/12(日) 00:01:01 ] //全部詰めるか while((++n&4)==0) w[n]=v++; for(;n--;k>>=1) F[w[n]]=k&1; //全部開けるか while ( ( ++ n & 4 ) == 0 ) w [ n ] = v ++ ; for ( ; n -- ; k >>= 1 ) F [ w [ n ] ] = k & 1 ;
249 名前:デフォルトの名無しさん mailto:sage [2006/02/13(月) 15:56:34 ] VBのIDEはそろえたくなくても勝手にそろえてくれるので C#とかでも同じ書き方するところは倣ってます。 if (v.Chars[v.Length - 1] == "n"){ this.First = false; v = v.Substring(0, v.Length - 1); }
250 名前:デフォルトの名無しさん mailto:sage [2006/02/14(火) 14:57:51 ] int q,w,e,r,t,y,u,i,o,p;
251 名前:デフォルトの名無しさん mailto:sage [2006/02/14(火) 16:17:33 ] >>250 今、それに近い糞VBアプリを修正させられてる。 「この処理は時間がかかります [OK]」 とかの、アイコンも出さずキャンセルできない 鬼ダイアログとか、基本はきっちり押さえてあってむしろ心地よい。 こんな糞で1500kも取られたってんだから腹立つよな
252 名前:デフォルトの名無しさん [2006/02/14(火) 23:51:33 ] >>251 ウチのアホが作った帳票出力ツールはプリンタ印刷とpdf出力のダイアログが両方 「帳票を出力しますか?」だった。 いくつか連続して出力させることが出来るんだが、タイムラグやら、本人のコーディングミスやらで時々順番が逆になるのな。 ex) 帳票1のプリンタ出力確認ダイアログ →帳票1のpdf出力確認ダイアログ →帳票2のプリンタ出力確認ダイアログ →帳票2のpdf出力確認ダイアログ そいつ勇気があるもんで、運用チームにこのまま配布しようとしてた。 ちなみに「先輩」なんだが。 今日はそいつ、フォームの全てのコントロール(主にラベル、チェックボックスの名称etc)の値を格納する構造体を作成してサーバに渡そうとしている。
253 名前:252 mailto:sage [2006/02/14(火) 23:52:47 ] 間違えた。 ex) 帳票1のプリンタ出力確認ダイアログ →帳票1のpdf出力確認ダイアログ →帳票2のpdf出力確認ダイアログ →帳票2のプリンタ出力確認ダイアログ
254 名前:デフォルトの名無しさん mailto:sage [2006/02/15(水) 09:28:30 ] stdio
255 名前:デフォルトの名無しさん [2006/02/21(火) 05:24:01 ] インクルードファイルなし(((( ;゚Д゚)))ガクガクブルブル
256 名前:デフォルトの名無しさん mailto:sage [2006/02/21(火) 10:32:28 ] 勝手に他人の名前をつけた関数名…。そういうのは止めて欲しいぞ。
257 名前:デフォルトの名無しさん mailto:sage [2006/02/21(火) 14:13:56 ] Inisharaizu()
258 名前:デフォルトの名無しさん mailto:sage [2006/02/21(火) 21:39:16 ] 引数名が ToT。関数名が damepo。 関数を呼び出すと damepo(ToT)
259 名前:デフォルトの名無しさん mailto:sage [2006/02/21(火) 21:50:55 ] damepo(m_m) damepo(v_v) damepo(u_u) damepo(A_A) damepo(T_T) 良いねコレ