摘要:由於最近在做重構的項目,所以對重構又重新進行了一遍學習和整理,對31天重構最早接觸是在2009年 10月份,由於當時沒有訂閱Sean Chambers的blog,所以是在國外的社區上閒逛的時候鏈接過去的。記得當時一口氣看完了整個系列並沒有多少感覺,因爲這些基本上項目都在使用,只是我們沒有專門把它標示和整理出來,所以也沒有引起多大的重視。現在突然接手這個重構項目,由於團隊成員技術和經驗參差不齊,所以有必要專門整理一個重構的綱要,當然這個系列也非常適合做新系統的代碼規範參考,只要有代碼的地方,這個重構規範就很有價值。週末也不想出去閒逛,因爲在剛到這個美麗的城市,沒有親戚或者朋友,所以才能靜下心來兩天時間寫完這個重構參考規範。同時也感受了Windows Live writer寫文章的快感。當然重構的整體架構得另當別論(整體架構在我的這篇文章有專門的講解(http://www.cnblogs.com/zenghongliang/archive/2010/06/23/1763438.html)。大的架構設計好了以後,這些重構細節點就成了東風之後的大火,對整個項目也是至關重要。31天重構這個系列和《代碼大全》、《重構:改善既有代碼的設計》比較起來最大的特點就是比較簡單、淺顯易懂。那麼我這些文章也都是學習Sean Chambers的31天重構的筆記整理,所以如果大家對這個筆記有任何異議也可以指出。
具體也可以通過http://www.lostechies.com/blogs/sean_chambers/archive/2009/07/31/31-days-of-refactoring.aspx查看原文。
概念: 本文中的”儘快返回”是指把原來複雜的條件判斷等語句用盡快返回的方式簡化代碼。
正文:如首先聲明的是前面講的”分解複雜判斷“,簡單的來說,當你的代碼中有很深的嵌套條件時,花括號就會在代碼中形成一個長長的箭頭。我們經常在不同的代碼中看到這種情況,並且這種情況也會擾亂代碼的可讀性。下代碼所示,HasAccess方法裏面包含一些嵌套條件,如果再加一些條件或者增加複雜度,那麼代碼就很可能出現幾個問題:1,可讀性差 2,很容易出現異常 3,性能較差
using System.Collections.Generic;
using System.Linq;
using LosTechies.DaysOfRefactoring.SampleCode.BreakMethod.After;
using Customer = LosTechies.DaysOfRefactoring.BreakResponsibilities.After.Customer;
namespace LosTechies.DaysOfRefactoring.SampleCode.ReturnASAP.Before
{
public class Order
{
public Customer Customer { get; private set; }
public decimal CalculateOrder(Customer customer, IEnumerable<Product> products, decimal discounts)
{
Customer = customer;
decimal orderTotal = 0m;
if (products.Count() > 0)
{
orderTotal = products.Sum(p => p.Price);
if (discounts > 0)
{
orderTotal -= discounts;
}
}
return orderTotal;
}
}
}
那麼重構上面的代碼也很簡單,如果有可能的話,儘量將條件判斷從方法中移除,我們讓代碼在做處理任務之前先檢查條件,如果條件不滿足就儘快返回,不繼續執行。
下面是重構後的代碼:
using System.Collections.Generic;
using System.Linq;
using LosTechies.DaysOfRefactoring.SampleCode.BreakMethod.After;
using Customer = LosTechies.DaysOfRefactoring.BreakResponsibilities.After.Customer;
namespace LosTechies.DaysOfRefactoring.SampleCode.ReturnASAP.After
{
public class Order
{
public Customer Customer { get; private set; }
public decimal CalculateOrder(Customer customer, IEnumerable<Product> products, decimal discounts)
{
if (products.Count() == 0)
return 0;
Customer = customer;
decimal orderTotal = products.Sum(p => p.Price);
if (discounts == 0)
return orderTotal;
orderTotal -= discounts;
return orderTotal;
}
}
}
總結: 總結:這個重構很重要,它和前面講的”分解複雜判斷“有些類似,我們在做複雜的處理過程時,要經常考慮這個重構,用好了它,會對我們的幫助很大。