最近在讀《重構》這本書,試著把書中對我啟發較大的一些點記錄下來,以便回顧學習。
一、replace temp with query(以查詢取代臨時變量)
在寫程序的過程中,難免會在函數中定義一些臨時變量,有些時候是無法避免的。但因為臨時變量可能使用起來簡單、快捷,因而有時會出現不恰當的使用。因為有些臨時變量在定義之后只進行了一次賦值,那么這種臨時變量是否有存在的價值就值得考慮。把臨時相關的賦值操作抽象為一個函數,以一個恰當的函數名命名這個函數,那么便可將相應的計算封裝起來,這樣計的算規則封裝在一個函數中,可以將變化隔離獨立出來,便于修改。
以書中的一段代碼為例:
double getPrice(){
int basePrice = _quantity * _itemPrice;
double discountFactor;
if(basePrice > 1000) {
discountFactor = 0.95;
} else {
discountFactor = 0.98;
}
return baseprice * discountFactor;
}
重構后的代碼為:
double getPrice(){
return baseprice() * discountFactor();
}
int basePrice() {
return _quantity * _itemPrice;
}
double discountFactor() {
if(basePrice() > 1000) {
return 0.95;
} else {
return 0.98;
}
}
個人認為這樣做的好處是basePrice和discountFactor的計算都隔離出來,如果發生變化容易修改。此外,discountFactor()函數的1000,0.95等數字的可以通過常量的方式定義,更加合理一點。
二、replace method with method object(以函數對象取代函數)
有時,可能會出現某個函數特別復雜,里面有大量的臨時變量以及復雜的運算,那么我們可以采用“以函數對象取代函數”的方法來對這個復雜的函數內容進行重構。
做法為將復雜的函數中使用到的臨時變量作為函數對象(MethodObject)的成員變量,原先函數中復雜的計算封裝到函數對象的某一個方法中(如compute())。
這樣即可以在原先復雜的函數中以new MethodObject(...).compute();取代原先的代碼。這樣的好處在于這段復雜的代碼方便復用、重構以及變更。
三、Hide Delegate(隱藏委托關系)
- 首先是隱藏“委托關系”
有時客戶端需要先通過服務對象A的方法得到另一個對象B,然后再調用B的函數b,這會產生一個“級聯調用”。
有時我們需要隱藏這種“關系”,我們可以在在服務對象A上放置一個簡單的“委托函數”a,在函數a的內部調用B的函數b。
這樣的話客戶端只需要調用A對象的a函數即可。
《重構》書中給出的例子如下:
class Person {
private Department department;
public Department getDepartment() {
return this.department;
}
public void setDepartment(Department department) {
this.department = department;
}
}
class Department {
private String code;
private Person manager;
public Department(Person manager) {
this.manager = manager;
}
public Person getPerson() {
return this.manager;
}
}
- 調用場景及問題如下如下:
要獲取某部門的經理:manager = john.getDepartment().getManager();
這樣的調用會暴露內部實現,并且調用端會和Department類產生耦合。
- 解決辦法:
在Person類中添加一個委托方法:
public Person getManager() {
return department.getManager();
}
manager = john.getManager();
這樣調用端就不需要知道Department類的信息,Department類發生變化時只需要通知Person類即可,易于維護。
- 額外說明:
當服務端的委托函數過多時,可能之前的重構方案出現問題,需要采用新的重構方案。
如讓客戶端依賴受委托的對象。
當然,進行重構時還應該具體情況具體分析。
持續更新
如有不正確的地方,歡迎指出!謝謝大家!