テストを書いてからリファクタリングなんてのは幻想

**デフォルトの名無しさん** · 2012/10/03(水) 00:24:26.88

テストを書いてからリファクタリングするというけれど、
コードの内容によっては、それが現実的に不可能な場合がある。

汚いコードであればあるほど、リファクタリングの前に
テストを書くのは難しくなる。

テストが書けるのは、単機能の関数になっているものだけ。
1000行以上からなる複数の処理を行う関数などテストを先に書くなんてまず不可能。

テストを書くためには、コードの再配置を先にやらなくてはいけない。
コードの順番を変えたりモジュールに分離するなどして、小さな処理にまとめて関数化する。
そこまでやってやっとテストが書ける。

現実的な修正の順番としては

コード再配置 → テストコード記述 → リファクタリング

にならざるをえない。

コード再配置はテストがない状態で行うから非常に神経を使う。
ミスを起こさないような再配置しかやってはいけない。

**デフォルトの名無しさん** · 2012/10/03(水) 00:29:16.55

See: Working Effectively with Legacy Code

**デフォルトの名無しさん** · 2012/10/03(水) 03:59:19.12

このスレッドは天才チンパンジー「アイちゃん」が
言語訓練のために立てたものです。

アイと研究員とのやり取りに利用するスレッドなので、
関係者以外は書きこまないで下さい。

　　　　　　　　　　　　　　　　　　京都大学霊長類研究所

**デフォルトの名無しさん** · 2012/10/03(水) 14:23:07.28

>>1
> 1000行以上からなる複数の処理を行う関数などテストを先に書くなんてまず不可能。

そうでもないよ。

================================== 終了 ===============================

**デフォルトの名無しさん** · 2012/10/14(日) 16:37:35.20

関数をリファクタリングするのは簡単だが、
明らかに関数になってない処理、
つまりいろんな手続きのあつまりを修正するのは難しい。

なぜなら、その処理の集まりを実行すると
何十もある状態（データベースなどの値）が一気に変化するから。

関数に値を数個入れて、一個の値を返すのではなく
何十個も値を入れて、何十個も値を返すという状態になってる。
そんなものにどうやってテストコードを書くのか。

こういうのはテストコードを書く前に、処理の集まりの中から
関数になる部分を抜き取るという作業になる。

抜き出した部分にテスト書くことはできるが、
処理の集まりの方に、テストを書くのはまず不可能。

**デフォルトの名無しさん** · 2012/10/14(日) 17:42:49.53

関数 {

　ここからAの処理
　　　：
　　　：

　ここからBの処理
　　　：
　　　：

　ここからCの処理
　　　：
　　　：

}
こういうのはリファクタリングしやすい。