**摘要：**由于最近在做重構的項目，所以對重構又重新進行了一遍學習和整理，對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)查看原文。 **概念：**本文中的”避免雙重否定”是指把代碼中的雙重否定語句修改成簡單的肯定語句，這樣即讓代碼可讀，同時也給維護帶來了方便。 **正文：**避免雙重否定重構本身非常容易實現，但我們卻在太多的代碼中見過因為雙重否定降低了代碼的可讀性以致于非常讓人容易誤解真正意圖。存在雙重否定的代碼具有非常大的危害性，因為這種類型的代碼容易引起錯誤的假設，錯誤的假設又會導致書寫出錯誤的維護代碼，最終會導致bug產生。具體可以看下面的代碼： ~~~ using System.Collections.Generic;using LosTechies.DaysOfRefactoring.SampleCode.BreakMethod.After;namespace LosTechies.DaysOfRefactoring.SampleCode.DoubleNegative.Before{ public class Order { public void Checkout(IEnumerable<Product> products, Customer customer) { if (!customer.IsNotFlagged) { // the customer account is flagged // log some errors and return return; } // normal order processing } } public class Customer { public decimal Balance { get; private set; } public bool IsNotFlagged { get { return Balance < 30m; } } }} ~~~ [](http://11011.net/software/vspaste) ~~~ 如上代碼中的雙重否定可讀性非常低，因為我們很難搞明白雙重否定的正確值。要重構它也非常容易，如下是重構后的代碼： ~~~ ~~~ using System.Collections.Generic;using LosTechies.DaysOfRefactoring.SampleCode.BreakMethod.After;namespace LosTechies.DaysOfRefactoring.SampleCode.DoubleNegative.After{ public class Order { public void Checkout(IEnumerable<Product> products, Customer customer) { if (customer.IsFlagged) { // the customer account is flagged // log some errors and return return; } // normal order processing } } public class Customer { public decimal Balance { get; private set; } public bool IsFlagged { get { return Balance >= 30m; } } }} ~~~ [](http://11011.net/software/vspaste) **** **總結：**”雙重否定“很容易讓人產生錯誤的判斷，也很難讓人理解你的代碼，所以這個重構在我們的代碼中是很重要的，尤其是在判斷條件很多且業務復雜的時候。