目的

「リファクタリング」を理解するためにサンプルコードを PHP で書き換えてみました。

今回は「一時変数の分離」について書きます。

「一時変数の分離」 について

複数回代入される一時変数があるが、それは、ループ変数でも一時変数を集める変数でもない

1つの変数が2つのことに使われていると、コードを読む人が非常に混乱します。

例

変更前

$_height = 10;
$_width = 8;

$temp = 2 * ($_height * $_width);
echo $temp . "\n";

$temp = $_height * $_width;
echo $temp . "\n";

変更後

$_height = 10;
$_width = 8;

$perimeter = 2 * ($_height * $_width);
echo $perimeter . "\n";

$area = $_height * $_width;
echo $area . "\n";