[TOC] # 第一步 每當我們要進行重構的時候,第一個步驟永遠相同,我們得為即將修改的代碼建立一組可靠的測試環境.這些測試是必要的,因為盡管遵循重構可以讓我們絕大多數避免引入bug.但我們畢竟是人.有可能犯錯,我們需要可靠的測試 重構之前,首先檢查自己有一套可靠的測試機制,這個測試機制有自我檢驗能力 # 分解并重組statement 代碼越小越容易管理,代碼的處理和移動也就越輕松  ~~~ private double getCharge(double thisAmount, Rental each) { switch (each.getMovie().getPriceCode()) { case Movice.REGULAR: thisAmount += 2; if (each.getDaysRented() > 2) thisAmount += (each.getDaysRented() - 2) * 1.5; break; case Movice.NEW_RELEASE: thisAmount += each.getDaysRented() * 3; break; case Movice.CHILDRENS: thisAmount += 1.5; if (each.getDaysRented() > 3) thisAmount += (each.getDaysRented() - 3) * 1.5; break; } return thisAmount; } ~~~ 重構技術就是以微小的步伐修改程序,如果你犯下錯誤,很容易便可發現他 任何一個傻瓜都能寫出計算機可以理解的代碼,唯有寫出人類理解的代碼,才是優秀的程序員 # 搬移金融計算代碼 觀察amountFor()時,我們發現這個函數來自Rental類的信息,卻沒有使用來自Customer類的信息 這立刻使我懷疑它是否被放錯了位置,絕大多數情況下,函數應該放在它所使用的數據的所屬對象內,所以amountFor()應該移到Rental類中 修改這些之后,下一件事就是去掉舊函數. 有時候會保留舊函數,讓他調用新函數,如果就函數是一個public函數,而我又不想改其他類的接口的時候 # 去除臨時變量 他們只是在自己所屬的函數中有效,所以他們會助長冗余而復雜的函數