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


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

ぱっと見て「ヘタだなぁ」と思うコード その5



1 名前:デフォルトの名無しさん [2006/08/12(土) 01:56:11 ]
禁止ネタ(超既出)
・長い関数
・深いネスト
・グローバル変数
・goto
・memset
・malloc - free
・局所ブロック
・サンプルコードのtypo
・記述スタイル
・関数・変数名

過去スレ
その4: pc8.2ch.net/test/read.cgi/tech/1153312202/
その3: pc8.2ch.net/test/read.cgi/tech/1149986051/
その2: pc8.2ch.net/test/read.cgi/tech/1142741989/
初代 : pc8.2ch.net/test/read.cgi/tech/1141867015/


534 名前:デフォルトの名無しさん mailto:sage [2007/06/18(月) 21:19:32 ]
>>533
禿道
「かもしれない」可能性論で実装するのはアホ
大体、消費税が大きく変わったら、そこで予算取って仕事にするんだから、
今下らない理由で無駄な作業をやらせるのは愚行

>「累進税率になったらどうしよう?」
ならなかったら、この作業分の無駄をどうpayするんだ?
>「品目別税率になるかもしれないね」
ならなかったら、この作業分の無駄をどうpayするんだ?


535 名前:デフォルトの名無しさん mailto:sage [2007/06/18(月) 21:24:41 ]
>>534
真の問題はその作業分が無駄になるだけじゃなくソースコードが
肥大化することでメンテナンスコストが増大することだろ。

536 名前:デフォルトの名無しさん mailto:sage [2007/06/18(月) 22:02:40 ]
むしろ真の問題は帰りがけに背中を刺されること。

537 名前:デフォルトの名無しさん mailto:sage [2007/06/18(月) 22:05:17 ]
むしろそれは最善の解決策

538 名前:デフォルトの名無しさん mailto:sage [2007/06/18(月) 22:08:36 ]
とりあえず >>531 みたいな馬鹿がいるとこ就職しちまったヤツには深く同情する。

539 名前:デフォルトの名無しさん mailto:sage [2007/06/18(月) 22:19:23 ]
将来の事を考えてバリアント型にしておきましたッ!!

540 名前:デフォルトの名無しさん mailto:sage [2007/06/18(月) 22:27:50 ]
お前の将来は何時までたっても泥沼なのかyo

541 名前:デフォルトの名無しさん mailto:sage [2007/06/18(月) 22:51:03 ]
> 税額計算してるとこ全部書き直すの?

そもそも、税額計算してるところが複数ある設計に問題があるだろ...。

542 名前:デフォルトの名無しさん mailto:sage [2007/06/19(火) 00:20:34 ]
みんな、531をそんなに苛めるなよ。
531は会社で評価されていなくて、新人を苛めるくらいしか仕事が無いんだよ。
本人は、「俺って先を読んで設計できる上級者」とか思っているんだから。
余り苛めると引きこもりになっちゃうぞ。




543 名前:デフォルトの名無しさん mailto:sage [2007/06/19(火) 07:13:54 ]
でもリファクタリングがはやって、思考停止するやつが増えたような気もする。
今回の流れみたいな雰囲気で話するやつが多い。
今回はの流れはわかるけど。
極論を話すやつもいるから。単に仕事をするのがいやって理由だったりする。
そのくせ、締め切りまぎわにまでリファクタリングとかって引っ張る。
設計の思考停止を実装でカバーってな風。


544 名前:デフォルトの名無しさん mailto:sage [2007/06/19(火) 08:10:33 ]
それなんて俺?

545 名前:デフォルトの名無しさん mailto:sage [2007/06/19(火) 15:17:45 ]
> そのくせ、締め切りまぎわにまでリファクタリングとかって引っ張る。
おれおれ

546 名前:デフォルトの名無しさん mailto:sage [2007/06/19(火) 21:51:18 ]
>>531みたいに未来の仕様を先回りして実装するのは下手だが
実際に仕様が追加されたとき(>>531の例でいうと累進税率や品目別税率)に
何箇所も直したりプログラムの構造を変えなくちゃいけなくなったりするのも下手
すなわちYAGNI

547 名前:デフォルトの名無しさん mailto:sage [2007/06/20(水) 00:13:58 ]
CVSとgrepと正規表現置換ができるエディタがあれば無問題

548 名前:527 mailto:sage [2007/06/20(水) 01:10:39 ]
>>531
一個だけ補足。
新人じゃないよ。

549 名前:デフォルトの名無しさん mailto:sage [2007/06/20(水) 02:16:37 ]
>>548
オチだとしたらできすぎだよw
おつかれ。


550 名前:デフォルトの名無しさん mailto:sage [2007/06/20(水) 07:19:46 ]
設計の妥当性の検証って1週間〜1ヶ月くらいはかかると見ておいた方がいいね。
例えば、ふと思いついた設計が、実は糞仕様だったのでその実装をやり直すってよくあるだろ。
まあ、実装してみなければわからん糞さってのもあるけど。
そこを熟成させるために実装を遅らせる効果は大きい。
バージョン管理していても、その糞実装したコストは変わらないわけで。

551 名前:デフォルトの名無しさん mailto:sage [2007/06/21(木) 00:52:35 ]
規模も複雑さも書かずに1週間〜1ヶ月と言われてもなぁ。

552 名前:デフォルトの名無しさん [2007/09/03(月) 15:47:43 ]
よしあげてやる



553 名前:デフォルトの名無しさん [2007/09/04(火) 16:13:00 ]
>>132
>それ以前に何故にthisを明記するか。
this->とやると、入力補完出来るんだよ。


554 名前:デフォルトの名無しさん mailto:sage [2007/09/04(火) 19:03:30 ]
this入力しないとメンバも保管できないIDEってwwww

555 名前:デフォルトの名無しさん mailto:sage [2007/09/04(火) 19:11:58 ]
if (f.isVisible() == true)

みたいな不要なことをするやつ。

#define SIZE 10
for (int i = 0; i < SIZE; i++)

みたいにして意地でも配列を使う奴。

typedefしまくってるやつ。

556 名前:デフォルトの名無しさん mailto:sage [2007/09/04(火) 19:52:53 ]
>>554
井出って、誰だよwwww


557 名前:デフォルトの名無しさん mailto:sage [2007/09/04(火) 20:40:53 ]
>ぱっと見て「ヘタだなぁ」と思うコード

俺のコード。






[ 新着レスの取得/表示 (agate) ] / [ 携帯版 ]

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

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