リファクタリング：　Split Temporary Variable（一時変数の分割）

Split Temporary Variable（一時変数の分割）

HOME > プログラミング > リファクタリング > Split Temporary Variable

Split Temporary Variableとは、一時変数の使い回しをなくし、バグの入り込む余地を少なくするリファクタリングです。
リファクタリング後の一時変数には、途中で変更されてよけいなバグを発生させないようにreadonly属性を指定しておきます。

リファクタリング前：

double temp = circle.radius * circle.radius * PI;
MessageBox.Show("円の面積は" + temp.ToString() + "です。");

temp = circle.radius * 2 * PI;
MessageBox.Show("円周は" + temp.ToString() + "です。");

リファクタリング後：

readonly double area = circle.radius * circle.radius * PI;
MessageBox.Show("円の面積は" + area.ToString() + "です。");

readonly double perimiter = circle.radius * 2 * PI;
MessageBox.Show("円周は" + perimeter.ToString() + "です。");