第22章モンスターメソッドを変更する必要がありますが、テストを書くことができません

スプラウトメソッドやスプラウトクラスで、長い間リファクタリングを避けることができるが、リファクタリングせざるを得ないのは事実

モンスターメソッドは何百行・何千行となり、インデントもバラバラで、解読するのはほぼ不可能

モンスターメソッドの変種

殆どは、純粋な箇条書きメソッドや錯乱メソッドではなく、その中間に位置している

リファクタリングツールを使って、リファクタリングする場合
テストが無いときは、ツールだけを利用するその後、テストを整備して手作業でリファクタリングをする

手作業でリファクタリングをする場合

検出用変数の導入
- 変数を追加して、リファクタリング対象の状態を検出できる場合がある
- 必要なリファクタリングが終わったあとに変数を取り除くと、きれいな状態戻せる
理解している部分の抽出
- まずは小さくわかりやすい箇所から抽出していく
依存関係の落ち穂拾い
- まず、変えてはならないロジックを確認するためのテストを書く
  - その後、テスト対象になっていない部分を抽出する
  - そうして、重要な振る舞いが変わっていないことを確信する
メソッドオブジェクトの取り出し
- ？？？？

モンスターメソッドの解体の際に、構造上のトレードオフを判断するための指針

感想

検出用変数の導入は、binding.pry の操作に該当するのかな？

リファクタリングは、本当に隗より始めよの精神だな