一時変数のインライン化【リファクタリング】
状況
- 簡単な式によって一度だけ代入される一時変数があり、それが他のリファクタリングの障害になっている
var a = getAmount() return (a > 100)
対処
- 一時変数への参照を式に置き換え、一時変数を削除する
メリット
- 他のリファクタリングの障害を取り除くことができる
手順
代入の右辺によって副作用(オブジェクトの変更など)が起きないことを確認する
一時変数が定数でなければ、定数化して、代入が一度であることを確認する
一時変数への参照を代入の右辺(式)と置き換える
変更の度に、コンパイルしてテストする
変更を終えたら、一時変数の宣言と代入を取り除く
コンパイルしてテストする
return (getAmount() > 100)
楽天ブックス: リファクタリング新装版 - 既存のコードを安全に改善する - マーチン・ファウラー - 9784274050190 : 本