**摘要：**由于最近在做重構的項目，所以對重構又重新進行了一遍學習和整理，對31天重構最早接觸是在2009年10月份，由于當時沒有訂閱[Sean Chambers](http://www.lostechies.com/blogs/sean_chambers/archive/2009/07/31/31-days-of-refactoring.aspx)的blog，所以是在國外的社區上閑逛的時候鏈接過去的。記得當時一口氣看完了整個系列并沒有多少感覺，因為這些基本上項目都在使用，只是我們沒有專門把它標示和整理出來，所以也沒有引起多大的重視。現在突然接手這個重構項目，由于團隊成員技術和經驗參差不齊，所以有必要專門整理一個重構的綱要，當然這個系列也非常適合做新系統的代碼規范參考，只要有代碼的地方，這個重構規范就很有價值。周末也不想出去閑逛，因為在剛到這個美麗的城市，沒有親戚或者朋友，所以才能靜下心來兩天時間寫完這個重構參考規范。同時也感受了Windows Live writer寫文章的快感。當然重構的整體架構得另當別論（整體架構在我的這篇文章有專門的講解（[http://www.cnblogs.com/zenghongliang/archive/2010/06/23/1763438.html](http://www.cnblogs.com/zenghongliang/archive/2010/06/archive/2010/06/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](http://www.lostechies.com/blogs/sean_chambers/archive/2009/07/31/31-days-of-refactoring.aspx)查看原文。 **概念：**本文中的”分解復雜判斷”是指把原來復雜的條件判斷等語句用盡快返回等方式簡化代碼。 **正文：**簡單的來說，當你的代碼中有很深的嵌套條件時，花括號就會在代碼中形成一個長長的箭頭。我們經常在不同的代碼中看到這種情況，并且這種情況也會擾亂代碼的可讀性。 如下代碼所示，HasAccess方法里面包含一些嵌套條件，如果再加一些條件或者增加復雜度，那么代碼就很可能出現幾個問題：1，可讀性差。 2，很容易出現異常。 3，性能較差。 ~~~ using System;using System.Collections.Generic;using System.Linq;using System.Text;namespace LosTechies.DaysOfRefactoring.SampleCode.ArrowheadAntipattern.Before{ public class Security { public ISecurityChecker SecurityChecker { get; set; } public Security(ISecurityChecker securityChecker) { SecurityChecker = securityChecker; } public bool HasAccess(User user, Permission permission, IEnumerable<Permission> exemptions) { bool hasPermission = false; if (user != null) { if (permission != null) { if (exemptions.Count() == 0) { if (SecurityChecker.CheckPermission(user, permission) || exemptions.Contains(permission)) { hasPermission = true; } } } } return hasPermission; } }} ~~~ 那么重構上面的代碼也很簡單，如果有可能的話，盡量將條件從方法中移除，我們讓代碼在做處理任務之前先檢查條件，如果條件不滿足就盡快返回，不繼續執行。下面是重構后的代碼： ~~~ using System.Collections.Generic;using System.Linq;namespace LosTechies.DaysOfRefactoring.SampleCode.ArrowheadAntipattern.After{ public class Security { public ISecurityChecker SecurityChecker { get; set; } public Security(ISecurityChecker securityChecker) { SecurityChecker = securityChecker; } public bool HasAccess(User user, Permission permission, IEnumerable<Permission> exemptions) { if (user == null || permission == null) return false; if (exemptions.Contains(permission)) return true; return SecurityChecker.CheckPermission(user, permission); } }} ~~~ [](http://11011.net/software/vspaste) **總結：**這個重構很重要，它和后面講的”盡快返回“有些類似，我們在做復雜的處理過程時，要經常考慮這個重構，用好了它，會對我們的幫助很大。