- 追加された行はこのように表示されます。
- 削除された行は
このように表示されます。
{{amazon 4894712288}}
一言で言うと、これを読むことでプログラミングが楽しくなります。
プログラマー必読といってもよいのでは?
「動くコードはさわるな」とよく言われますけど、動くコードの振る舞いを変えることなく構造を「きれいに」する指針を与えてくれます。
今は結構IDEがサポートしてくれてるので、重宝してます。
個人的にもっとも影響を受けた(仕事上)本の一冊です。
まぁ実際問題として、「動いているコードをさわる」勇気はないんですけど・・・
以下、確認用メモ。
!!リファクタリング原則
::定義
外部から見たときの振る舞いを保ちつつ、理解や修正が簡単になるように、ソフトウェアの内部構造を変化させること。
一連のリファクタリングを行って、外部からみた振る舞いを変えずにソフトウェアを再構築する。
!!コードの不吉なにおい
*重複したコード
*長すぎるメソッド
*巨大なクラス
*多すぎる引数
*変更の発散(変更箇所が特定できない)
*変更の分散(変更を行うたびにあちこちのクラスが少しずつ書き換わる)
*属性、操作の横恋慕(あるメソッドが自分のクラスより、他のクラスに興味を持つ)
*データの群れ(複数個のデータがグループで出現)
*基本データ型への執着
*スイッチ文
*パラレル継承(新たなサブクラスを定義するたび、別の継承木にもサブクラスを追加しなければいけない)
*怠け者クラス(十分な仕事をしないクラス)
*疑わしき一般化(いつかこの機能が必要になるさ)
*一時的属性(インスタンス変数の値が、特定の状況でしかセットされない)
*メッセージの連鎖(連鎖の構造に依存)
*仲介人(メソッドの大半が委譲しているだけのクラス)
*不適切な関係
*クラスのインターフェース不一致
*未熟なクラスライブラリ
*データクラス
*接続拒否(継承した操作、属性が利用されず混乱が引き起こされている)
*コメント(非常にわかりにくいコードを補うためのコメント)
!!リファクタリングカタログ
!メソッドの構成
::メソッドの抽出
+まとめられるコードの断片がある
+断片をメソッドにして、目的をあらわすような名前をつける
::メソッドのインライン化
+メソッドの本体が、名前をつけるほどではなく明らか
+メソッド本体をコール元にインライン化
::一時変数のインライン化
+簡単な式によって一度だけ代入される一時変数があり、他のリファクタリングの障害となっている
+一時変数への参照を式で置き換え
double basePrice = anOrder.basePrice();
return (basePrice > 1000);
{{ref_image arrow_b.jpg}}
return (anOrder.basePrice() > 1000);
::問い合わせによる一時変数の置き換え
+一時変数を使って式の結果を保持している
+式をメソッドに抽出する。一時変数への参照を式へ置き換える。新たなメソッドが他のメソッドでも利用できるようになる。
double basePrice = _quantity * _itemPrice;
if (basePrice > 1000)
return basePrice * 0.95;
else
return basePrice * 0.98;
{{ref_image arrow_b.jpg}}
if (basePrice() > 1000)
:
double basePrice() {
return _quantity * _itemPrice;
}
::説明用変数の導入
+複雑な式がある
+式の結果または部分的な結果を目的を説明する名前をつけた一時変数に代入する
if ( (platform.tuUpperCase().indexOf("MAC") > -1 ) &&
(browser.toUpperCase().indexOf("IE") > -1) {
:
}
{{ref_image arrow_b.jpg}}
final boolean isMacOs = platform.tuUpperCase().indexOf("MAC") > -1;
final boolean isIEBrowser = browser.toUpperCase().indexOf("IE") > -1;
if (isMacOs && isIEBrowser) { ... }
::一時変数の分離
+複数回代入される一時変数があるが、ループ変数でも、一時変数を集めるものでもない
+代入ごとに一時変数を分ける
double tmp = 2 * (_height * _width);
System.out.println(tmp);
tmp = _height * _width;
System.out.println(tmp);
{{ref_image arrow_b.jpg}}
final double perimeter 2 * (_height * _width);
System.out.println(perimeter);
final double area = _height * _width;
System.out.println(area);
::パラメータへの代入の除去
+引数への代入が行われている
+一時変数を使う
::オブジェクトによるメソッドの置き換え
+長いメソッドで、メソッドの抽出を適用できないようなローカル変数の使い方をしている
+メソッド自身をオブジェクトとして、ローカル変数をそのオブジェクトのフィールドとする。
class Order...
double hoge(){ ... }
double price() {
double primaryBasePrice;
double secondaryBasePrice;
double tertiaryBasePrice;
/* 長い処理 */
:
primaryBasePrice * hoge();
}
{{ref_image arrow_b.jpg}}
class Order...
double price() {
return new PriceCaluculator(this,
primaryBasePrice,
secondaryBasePrice,
tertiaryBasePrice
).compute();
}
}
class PriceCaluculator...
private Order order;
private double primaryBasePrice;
private double secondaryBasePrice;
private double tertiaryBasePrice;
public PriceCaluculator(Order order,
double primaryBasePrice,
double secondaryBasePrice,
double tertiaryBasePrice ) {
this.order = order;
this.primaryBasePrice = primaryBasePrice;
this.secondaryBasePrice = secondaryBasePrice;
this.tertiaryBasePrice = tertiaryBasePrice;
}
public double compute() {
:
primaryBasePrice * order.hoge(); /* 元クラスへの特性へのコール */
:
}
}
::アルゴリズムの取替え
+メソッドの本体を新たなアルゴリズムで置き換える
!オブジェクト間での特性の移動
::メソッドの移動
+あるクラスでメソッドが定義されているが、そのクラスの特性よりも他のクラスの特性の方が、そのメソッドの使用、被使用頻度が高い
+同様の本体を持つメソッドを多用するクラスにも作成する。元のメソッドは委譲とするか、取り除く
::フィールドの移動
+あるクラスに定義されているフィールドが、他のクラスから使用されることの方が多い
+移動先クラスに新たなフィールドを作って、利用先側をすべて変更する
::クラスの抽出
+2つのクラスでなされるべき作業を1つのクラスで行っている
+クラスを新たに作成し、適当なフィールドとメソッドを移動する
::クラスのインライン化
+クラスのやっていることはたいしたことではない
+別のクラスに特性を移動し、削除する
::委譲の隠蔽
+クライアントがあるオブジェクトの委譲クラスをコールしている
+サーバーにメソッドをつくって委譲を隠す
{{ref_image 157.jpg}}
::仲介人の除去
+クラスがやっていることは単純な委譲だけ
+クライアントに委譲オブジェクトを直接コールさせる
委譲の隠蔽の逆
::外部メソッドの導入
+利用中のサーバクラスにメソッドを追加する必要があるが、そのクラスを変更できない
+クライアントクラスに、サーバクラスのインスタンスを第1引数にとるメソッドを作る
Data newStart = new Date(previousEnd.getYear(),
previousEnd.getMonth(),
previousEnd.getDate() + 1);
{{ref_image arrow_b.jpg}}
Date newStart = nextDay(previousEnd);
private static Date nextDay(Date arg) {
return new Date(arg.getYear(), arg.getMonth(), arg.getDate() + 1);
}
::局所的拡張の導入
+利用中のサーバクラスにメソッドをいくつか追加する必要があるが、クラスを変更できない
+追加されるメソッドを備えた新たなクラスを作る(サブクラス、ラッパー)
{{ref_image 164.jpg}}
!データの再編成
::自己カプセル化フィールド
+フィールドを直接アクセスしているが、結合関係が煩わしくなってきた
+getメソッド、setメソッドを作成し、それだけを使ってアクセスするように変更する
private int _low, _high;
boolean includes(int arg) {
return arg >= _low && arg <= _high;
}
{{ref_image arrow_b.jpg}}
private int _low, high;
boolean includes(int arg) {
return arg >= getLow() && arg <= getHigh();
}
int getLow() { return _low; }
int getHigh() { return _high; }
::オブジェクトによるデータ値の置き換え
+いくつかのデータや振る舞いが必要なデータ項目がある
+そのデータ項目をオブジェクトに変える
{{ref_image 175_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 175_2.jpg}}
::値から参照への変更
+同じインスタンスが多数存在するクラスがあり、1つのオブジェクトに置き換えたい
+そのオブジェクトを参照オブジェクトに変える
/* 顧客クラス */
class Customer {
public Customer(String name) { _name = name; }
pulbic String getName() { return _name; }
private final String _name;
}
/* 注文クラス */
class Order...
public Order(String customerName) {
_customer = new Customer(customerName);
}
public void setCustomer(String customerName) {
_customer = new Customer(custmerName);
}
public String getCustomerName() {
return _customer.getName();
}
private Customer _customer;
}
/* クライアントコード */
private static int numberOfOrdersFor(Collection orders, String customer) {
int result = 0;
Iterator iter = orders.iterator();
while (iter.hasNext()) {
Order each = (Order) iter.next();
if (each.getCustomerName().equals(customer)) result++;
}
return result;
}
{{ref_image arrow_b.jpg}}
class Customer...
private Customer(String name) { _name = name; }
static void loadCustomers() {
new Customer("Lemon Car Hire").store();
:
}
private void store() {
_instandces.put(this.getName(), this);
}
public static Customer getNamed(String name) {
return (Customer) _instances.get(name);
}
private static Dictionary _instances = new Hashtable();
}
顧客名毎にCustomerオブジェクトが唯一存在するようにする
::参照から値への変更
+小さくて、不変で、コントロールが煩わしい参照オブジェクトがある
+値オブジェクトに変える
::オブジェクトによる配列の置き換え
+配列の各要素が、それぞれ異なる意味を持っている
+その配列を要素ごとに対応したフィールドをもつオブジェクトに置き換える
String[] row = new String[3];
row[0] = "Liverpool";
row[1] = "15";
{{ref_image arrow_b.jpg}}
Performance row new Performance();
row.setName("Liverpool");
row.setWins("15");
::観察されるデータの複製
+あるGUIコントロールでのみ有効なドメインデータがあり、ドメインメソッドからもアクセスする必要がある
+データをドメインオブジェクトにコピーして、それらを同期させるためのオブザーバを設ける
::単方向関連の双方向への変更
+2つのクラスが互いにその特性を使う必要があるが、単方向のリンクしかない
+逆ポインタを加えて、両方の集合を更新するように更新操作を変更する
::双方向関連の単方向への変更
+双方向関連があるが、一方のクラスはもはや他方の特性を必要としていない
+不要になった関連の一方を削除する
::シンボリック定数によるマジックナンバーの置き換え
+特別な意味を持った数字リテラルがある
+定数を作り、それにふさわしい名前をつけて、そのリテラルを置き換える
::フィールドのカプセル化
+公開フィールドがある
+非公開にして、アクセサを用意する
::コレクションのカプセル化
+メソッドがコレクションを返す
+読み取り専用のビューを返して、追加と削除のメソッドを提供する
java.util.Collections.unmodifiableSet
{{ref_image 208_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 208_2.jpg}}
::データクラスによるレコードの置き換え
+古いプログラミング環境のレコード構造とインターフェースをとる必要がある
+そのレコード用に、振る舞いを持たないデータオブジェクトを作る
::クラスによるタイプコードの置き換え
+振る舞いに影響しない数字のタイプコードを持つクラスがある
+その数字を新しいクラスで置き換える
{{ref_image 218_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 218_2.jpg}}
::サブクラスによるタイプコードの置き換え
+クラスの振る舞いに影響を与える不変のタイプコードがある
+そのタイプコードをサブクラスに置き換える
{{ref_image 223_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 223_2.jpg}}
::State/Strategyによるタイプコードの置き換え
+クラスの振る舞いに影響するタイプコードがあるが、サブクラス化はできない
+状態オブジェクトでタイプコードを置き換える
{{ref_image 223_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 227.jpg}}
::フィールドによるサブクラスの置き換え
+定数データを返すメソッドだけが異なるサブクラスがある
+メソッドをサブクラスのフィールドに変更して、サブクラスを削除する
!条件記述の単純化
::条件記述の分解
+複雑な条件記述(if-the-else)がある
+条件記述部をthen部およびelse部からメソッドを抽出する
if (date.before(SUMMER_START) || date.after(SUMMER_END))
charge = quantity * _winterRate + _winterServiceCharge;
else charge = quantity * _summerRate;
{{ref_image arrow_b.jpg}}
if (notSummer(date))
charge = winterCharge(quantity);
else charge = summerCharge(quantity);
::条件記述の統合
+同じ結果を持つ一連の条件判断がある
+1つの条件記述にまとめてから抽出する
double disabilityAmount() {
if (_seniority < 2) return 0;
if (_monthisDisabled > 12) return 0;
if (_isPartTime) return 0;
{{ref_image arrow_b.jpg}}
double disabilityAmount() {
if (isNotEligibleForDisablity()) return 0;
}
::重複した条件記述の断片の統合
+条件式のすべての分岐に同じコードの断片がある
+それを式の外側に移動する
if (isSpecialDeal()) {
total = price * 0.95;
send();
} else {
total = price * 0.98;
send();
}
{{ref_image arrow_b.jpg}}
if (isSpecialDeal()) {
total = price * 0.95;
} else {
total = price * 0.98;
}
send();
::制御フラグの削除
+一連の論理型の式に対して制御フラグとして機能する1つの変数がある
+代わりにbreakかreturnを使う
::ガード節による入れ子条件記述の置き換え
+メソッドに正常ルートが不明確な条件付振る舞いがある
+特殊ケースすべてに対してガード節を使う
+特殊ケースすべてに対してガード節(メソッド内で、処理がメインロジックに到達するのを防ぐためのコード)を使う
double getPayAmount() {
double result;
if (_isDead) result = deadAmount();
else {
if (_isSeparated) result = separatedAmount();
esle {
if (_isRetired) result = retiredAmount();
else result = normalPayAmount();
}
}
return result;
}
{{ref_image arrow_b.jpg}}
double getPayAmount() {
if (_isDead) return deadAmount();
if (_isSeparated) return separatedAmount();
if (_isRetired) return retiredAmount();
return normalPayAmount();
}
出口が1つだけというルールは本当は有益ではありません。1つにすることでメソッドが明瞭になるならば出口は1つでいいが、そうでなければ使うべきではない
::ポリモーフィズムによる条件記述の置き換え
+オブジェクトのタイプによって異なる振る舞いを選択する条件記述がある
+条件記述の各アクション部をサブクラスのオーバーライドメソッドに移動する。元のメソッドはabstractにする
double getSpeed() {
switch (_type) {
case EUROPEAN:
return getBaseSpeed();
case AFRICAN:
return getBaseSpeed() - getLoadFactor() * _numberOfCoconuts;
case NORWEGIAN_BLUE:
return (_isNailed)?0:getBaseSpeed(_voltage);
}
throw new RuntimeException("ここには来ないはず");
}
{{ref_image arrow_b.jpg}}
{{ref_image 255.jpg}}
::ヌルオブジェクトの導入
+null値のチェックが繰り返し現れる
+null値をヌルオブジェクトで置き換える
if (customer == null) plan = billingPaln.basic();
else plan = customer.getPlan();
{{ref_image arrow_b.jpg}}
{{ref_image 260.jpg}}
::表明の導入
+コードのある部分が、そのプログラムの状態について何らかの仮定を持っている
+仮定を表明を使って明示する
double getExpenseLimit() {
/* 支出上限か優先プロジェクトか、どちらかを持つこと */
return (_expenseLimit != NULL_EXPENSE) ?
_expenseLimit:
_primaryProject.getMemberExpenseLimt();
}
{{ref_image arrow_b.jpg}}
double getExpenseLimit() {
Assert.isTrue(_expenseLimit != NULLEXPENSE || _primaryProject != null);
return (_expenseLimit != NULL_EXPENSE) ?
_expenseLimit:
_primaryProject.getMemberExpenseLimt();
}
}
!メソッド呼び出しの単純化
::メソッド名の変更
+メソッドの名前がその目的を正しく表現できていない
+メソッドの名前を変更する
::引数の追加
+あるメソッドが、呼び出し元からより多くの情報を必要としている
+情報を渡すために引数を追加する
::引数の削除
+ある引数が、もはやメソッド本体から使われていない
+引数を削除する
::問い合わせと更新の分離
+1つのメソッドが返す値と同時にオブジェクトの状態を変更している
+問い合わせようと行進用の2つのメソッドをそれぞれ作成する
{{ref_image 279_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 279_2.jpg}}
::メソッドのパラメータ化
+複数のメソッドが、異なる値に対してよく似た振る舞いをしている
+異なる値を1つの引数として受け取るメソッドを作成する
{{ref_image 283_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 283_2.jpg}}
::明示的なメソッド群による引数の置き換え
+引数の特定の値によって異なるコードが実行されるメソッドがある
+引数の値に値に対応する別々のメソッドを作成する
void setValue (String name, int value) {
if (name.equals("height"))
_heigth = vaue;
else if (name.equals("width"))
_width = value
else
Assert.shouldNeverReacheHere();
}
{{ref_image arrow_b.jpg}}
void setHeight (int arg) {
_height = arg;
}
void setWidth (int arg) {
_width = arg;
}
::オブジェクトそのものの受け渡し
+あるオブジェクトから複数の値を取得し、それらの値をメソッド呼び出しの引数として渡している
+オブジェクトそのものを渡す
int low = daysTempRange().getLow();
int high = daysTempRange().getHigh();
withinPlan = plan.withinRange(low, high);
{{ref_image arrow_b.jpg}}
withinPlan = plan.withinRange(daysTempRange());
::メソッドによる引数の置き換え
+あるオブジェクトがメソッドを呼び出し、その戻り値を別のメソッドの引数として渡している。受信側はそのメソッドを呼び出し可能である
+引数を削除し、受信側にそのメソッドを呼び出させる
int basePrice = _quantity * _itemPrice;
discountLevel = getDiscountLevel();
double finalPrice = discountedPrice(basePrice, discountLevel);
{{ref_image arrow_b.jpg}}
int basePrice =~ _quantity * _itemPrice;
double finalPrice = discountedPrice(basePrice);
::引数オブジェクトの導入
+本来まとめて扱うべき一連の引数がある
+それらをオブジェクトに置き換える
{{ref_image 295_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 295_2.jpg}}
::setメソッドの削除
+フィールドの値が生成時に設定され、決して変更されない
+そのフィールドに対するすべてのsetメソッドを削除する
::メソッドの隠蔽
+メソッドが自分の定義されているクラス以外からはまったく使用されていない
+メソッドを非公開にする
::Factory Methodによるコンストラクタの置き換え
+オブジェクトを生成する際に、単純な生成以上のことをしたい
+ファクトリメソッドを使って、コンストラクタを置き換える
Employee (int type) {
_type = type;
}
{{ref_image arrow_b.jpg}}
static Employee create(int type) {
return new Employee(type);
}
::ダウンキャストのカプセル化
+メソッドが返すオブジェクトが、呼び出し側によってダウンキャストされる必要がある
+ダウンキャストをメソッド内に移動する
Object lastReading() {
return readings.lastElements();
}
{{ref_image arrow_b.jpg}}
Reading lastReading() {
return (Reading) readlings.lastElement();
}
::例外によるエラーコードの置き換え
+メソッドがエラーを示す特別なコードを返す
+代わりに例外を発生させる
::条件判定による例外の置き換え
+チェックされる例外を発生させているが、本来は呼び出し側が先にチェックすべきだる
+最初に条件判定をするように呼び出し側を修正する
double getValueForPeriod(int periodNumber) {
try {
return _values[periodNumber];
} catch (ArrayIndexOutOfBoundsException e) {
return 0;
}
}
{{ref_image arrow_b.jpg}}
double getValueForPeriod(int periodNumber) {
if (periodNumber >= _values.length) return 0;
return _values[periodNumber];
}
!継承の取り扱い
::フィールドの引き上げ
+2つのサブクラスが同じフィールドを持っている
+そのフィールドをスーパークラスへ移動する
::メソッドの引き上げ
+同じ結果をもたらすメソッドがふくすうのサブクラスに存在する
+それらをスーパークラスに移動する
::コンストラクタ本体の引き上げ
+複数のサブクラスに内容がほとんど同一のコンストラクタがある
+スーパークラスのコンストラクタを作成して、サブクラスから呼び出す
::メソッドの引き下げ
+スーパークラスの振る舞いが、いくつかのサブクラスだけに関係している
+そのメソッドをサブクラスに移動する
::フィールドの引き下げ
+フィールドがいくつかのサブクラスだけで使われている
+そのフィールドを、サブクラスに移動する
::サブクラスの抽出
+あるクラス特定のインスタンスだけに必要な特性がある
+その一部の特性を持つサブクラスを作成する
{{ref_image 330_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 330_2.jpg}}
::スーパークラスの抽出
+似通った特性を持つ2つのクラスがある
+スーパークラスを作成して、共通の特性を移動する
::インターフェースの抽出
+複数のクライアントが、あるクラスのひとまとまりのインターフェースを使っている。または2つのクラス間でインターフェースの一部が共通である
+共通部分をインターフェースとして抽出する
::階層の平坦化
+スーパークラスとサブクラスにそれほど大きな違いがない
+あわせてまとめてしまう
::Templete Methodの形成
+異なるサブクラスの2つのメソッドが、類似の処理を同じ順序で実行しているが、各処理は異なっている
+元のメソッドが同一になるように、各処理を同じシグネチャを持つメソッドにする。そしてそれらを引き上げる。
{{ref_image 345_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 345_2.jpg}}
::委譲による継承の置き換え
+サブクラスがスーパークラスの一部のインターフェースだけを使っている。あるいはデータを継承したくない
+スーパークラス用のフィールドを作成して、メソッドをスーパークラスに委譲するように変更し、継承をやめる
{{ref_image 352_1.jpg}}
{{ref_image arrow_b.jpg}}
{{ref_image 352_2.jpg}}