### Decompose Conditional（分解條件式） 你有一個復雜的條件（if-then-else）語句。 從if、then、else 三個段落中分別提煉出獨立函數。 ~~~ if (date.before (SUMMER_START) || date.after(SUMMER_END)) charge = quantity * _winterRate + _winterServiceCharge; else charge = quantity * _summerRate; ~~~ => ~~~ if (notSummer(date)) charge = winterCharge(quantity); else charge = summerCharge (quantity); ~~~ **動機（Motivation）** 程序之中，「復雜的條件邏輯」是最常導致復雜度上升的地點之一。你必須編寫代碼來檢查不同的條件分支、根據不同的分支做不同的事，然后，你很快就會得到一個相當長的函數。大型函數自身就會使代碼的可讀性下降，而條件邏輯則會使代碼更難閱讀。在帶有復雜條件邏輯的函數中，代碼（包括檢查條件分支的代碼和真正實現功能的代碼）會告訴你發生的事，但常常讓你弄不清楚為什么會發生這樣的事, 這就說明代碼的可讀性的確大大降低了。 和任何大塊頭代碼一樣，你可以將它分解為多個獨立函數，根據每個小塊代碼的用 途，為分解而得的新函數命名，并將原函數中對應的代碼替換成「對新建函數的調用」，從而更清楚地表達自己的意圖。對于條件邏輯，[將每個分支條件分解，形成新函數」還可以給你帶來更多好處：可以突出條件邏輯，更清楚地表明每個分支的作用，并且突出每個分支的原因。 **作法（Mechanics）** - 將if 段落提煉出來，構成一個獨立函數。 - 將then 段落和else 段落都提煉出來，各自構成一個獨立函數。 如果發現嵌套的（nested）條件邏輯，我通常會先觀察是否可以使用Replace Nested Conditional with Guard Clauses 。如果不行，才開始分解其中的每個條件。 **范例（Example）** 假設我要計算購買某樣商品的總價（總價=數量*單價），而這個商品在冬季和夏季的單價是不同的： ~~~ if (date.before (SUMMER_START) || date.after(SUMMER_END)) charge = quantity * _winterRate + _winterServiceCharge; else charge = quantity * _summerRate; ~~~ 我把每個分支的判斷條件都提煉到一個獨立函數中，如下所示： ~~~ if (notSummer(date)) charge = winterCharge(quantity); else charge = summerCharge (quantity); private boolean notSummer(Date date) { return date.before (SUMMER_START) || date.after(SUMMER_END); } private double summerCharge(int quantity) { return quantity * _summerRate; } private double winterCharge(int quantity) { return quantity * _winterRate + _winterServiceCharge; } ~~~ 通過這段代碼你可以看出，整個重構帶來的清晰性。實際工作中，我會逐步進行每一次提煉，并在每次提煉之后編譯并測試。 像這樣的情況下，許多程序員都不會去提煉分支條件。因為這些分支條件往往非常短，看上去似乎沒有提煉的必要。但是，盡管這些條件往往很短，在代碼意圖和代 碼自身之間往往存在不小的差距。哪怕在上面這樣一個小小例子中，notSummer(date) 這個語句也能夠比原本的代碼更好地表達自己的用途。對于原來的代碼，我必須看著它，想一想，才能說出其作用。當然，在我們這個簡單的例子中，這并不困難。不過，即使如此，提煉出來的函數可讀性也更高一些——它看上去就像一段注釋那樣清楚而明白。