リファクタリングをただのコード修正と思ってる人へ

**デフォルトの名無しさん** · 2010/05/29(土) 17:25:56

テストも書かないでリファクタリングとかうけるｗ

まずな、リファクタリングでは機能追加・修正は行わない。
動作はまったく同じでコードをきれいに書き換えること。

書き換えるといっても、これなら同じ動きだろ？って推測でやってはいけない。

まずテストを書く。ユニットテストをできるように、
単一のクラスでインスタンスを作る。

汚いコードなのだからたいていは依存関係のせいで単一ではクラスが生成できない
それを生成するために、クラスの動作を書き換える。
といっても元のコードは修正しない。継承やプリプロセッサを使って
依存関係を断ち切るために既存のコードを上書きする。
どうしてもそれが不可能な場合には、決められた手順で最小のコード修正を行う

そうやって既存のクラスのユニットテストを行う。
それでやっとリファクタリングが行える。
既存のクラスのユニットテストを通るように新たなコードに修正、
もしくは新規作成して置き換える。

この手順と考え方を守ってないのはリファクタリングではない。

で？

**デフォルトの名無しさん** · 2011/06/01(水) 22:25:20.30

リファクタリングなんてやって意味があると思ってる奴は消防

Ａ～Ｚまであるクラスの共通処理をＰクラスとして纏めたとするわな
したら、ＰクラスをいじるたびにＡ～Ｚまでの影響範囲を調べなあかんのやで

仮にＡにしか関係しないＰに入ってしまった共通処理を弄るのだって
一度結合したらはがしてＡクラス特有の処理としてＰクラスからＡクラスを避ける処理なんて
結構、邪魔な処理になるで

それでもＰとしてまとめてしまったらＡ～Ｚまでサポートしていかなあかんのやで
リファクタリング、結構、だが、誰がこの未来を予測できる？

Ａ～Ｚまでの共通処理が走ってるＰクラスを蹴飛ばしてＰＡクラスを新しく作って
Ａ－ＰＡ、Ｂ～Ｚ－Ｐの関係を新しくぶった切るような切込みはできんやろ？
せいぜいＰの処理にcase Aを入れるような修正が精一杯やないか？

今後を考えてリファクタリングなんてする意味があるんか？

**デフォルトの名無しさん** · 2011/06/01(水) 22:28:19.38

共通処理をまとめるだけがリファクタリングではないし、
自動テストが伴わないものはリファクタリングとは呼ばない。

**デフォルトの名無しさん** · 2011/06/01(水) 22:28:23.13

>>1

**デフォルトの名無しさん** · 2011/06/01(水) 22:38:24.11

>>50
いや、上で書いたのはあくまで例だよ
上の例は処理をまとめることによって明らかに構造が汚くなってるよね？
この先もまとめたＰクラスにＡ～Ｚへのどれかの追加が入るたびに
case B、case C・・・と追加されていくだろうね
そのときＰクラスを消滅する選択肢は誰か選べるんだろか？

こーゆーのってさ、その時点ででは確定しなくね？
じゃ、リファクタリングってなんのためにするの？

って根本的なところに目を向けたときに次の開発をしやすくするためでしょ？

したら、その仕様書もなしでリファクタリングをするってことは
仕様書もないのにプログラムの構造を「俺が好き」なだけの構造にただ意味もなく
書き換えてるだけのマスターベーションなんだよ