一時変数のインライン化【リファクタリング】

状況

• 簡単な式によって一度だけ代入される一時変数があり、それが他のリファクタリングの障害になっている

var a = getAmount()
return (a > 100)

対処

• 一時変数への参照を式に置き換え、一時変数を削除する

メリット

• 他のリファクタリングの障害を取り除くことができる

手順

• 代入の右辺によって副作用（オブジェクトの変更など）が起きないことを確認する

• 一時変数が定数でなければ、定数化して、代入が一度であることを確認する

• 一時変数への参照を代入の右辺（式）と置き換える

• 変更の度に、コンパイルしてテストする

• 変更を終えたら、一時変数の宣言と代入を取り除く

• コンパイルしてテストする

return (getAmount() > 100)

楽天ブックス: リファクタリング新装版 - 既存のコードを安全に改善する - マーチン・ファウラー - 9784274050190 : 本