"改修の時に気をつけること・・・どんなに汚いロジックだろうが、正常に動いている所は絶対に触らない、これ鉄則です。"　←これを守り続けてリファクタリングしないというのもなんだか。特に現状がひどすぎる場合など。 - GOTTi from Bookmarklet

1000行近くのSQL文と言うのを見てみたい。 - Fake

「正常に動いているところは触らない」→テストが無い、テストを構築するコストが無い、という場合においては、残念ながらこれが鉄則だと思いますねぇ。 - shin1ogawa

本当に「正常に動いている」なら、「触らない」が確かに鉄則なんでしょうけど、汚すぎる箇所は得てして実装にすでに（未発見の）バグがある可能性が高いんじゃないかなとも私は思ってます。 - GOTTi

理想はそうなんですけどね。コストさえ許すなら、できるだけ多くの負債を解消して行きたいのが技術者魂だし、技術的負債を放置しても結局どこかのタイミングでとんでもなく膨らんで返ってきてしまうのが目に見えてますし。 そういう「何とかすべきだけど、金と時間がない、それを確保できる交渉役もいない」ようなプロジェクトで、一番気をつける事は参加する本人の身の振り方なのかもしれません。苦労に見合う金をもらえないなら見捨てる、というのも十分有効な選択肢ですし…ね。メンバやプロダクトに愛着があるなら、あえて巻き込まれて苦労するのも良いですし…ね。 - shin1ogawa

なるほど、そういったプロジェクトや立場の場合だと、「自分の身を守るため」には鉄則なのかもしれませんね。 - GOTTi

「未発見のバグを修正して、新しいバグを1つも生まないでリファクタリングするコスト」が無視できるほど少ないのであれば、どんどん改修欲しいですね。 - Fake

；； - GOTTi

1000行近くのSQL（というかストアド）見たことありますよ。改行多いせいもあるけど、3、40項目くらいのselectを３つ４つunionでつないで、さらにそれがif分岐ごとにあって、当然対応するカーソル定義もあって。エラー処理だのログ吐きだのがトッピングされて1000行SPのできあがり。 - Regicat