我影響最非常深刻的一次代碼重構(gòu)

1. 業(yè)務(wù)說明

系統(tǒng)原先已有登錄功能，我們打算增加一個登錄IP和允許登錄時間的安全限制業(yè)務(wù)。

IP 分為內(nèi)網(wǎng)ip、外網(wǎng)ip，如果設(shè)置了，則該用戶只允許在這些ip登錄

2. 原有代碼貼圖

登錄代碼原先已有，這是增加的功能，該同事增加的代碼如下：

圖1：login方法中，判斷是否可以登錄的私有方法調(diào)用

圖2、3、4該私有業(yè)務(wù)實現(xiàn)方法。

3 原有代碼問題分析

從該方法的調(diào)用方式，到該方法的實現(xiàn)，代碼都存在不少問題，我先逐一分析，然后再貼上我重構(gòu)的代碼以及重構(gòu)的思路。

1. 調(diào)用方式

   用String匹配的方式判斷，直接用boolean判斷即可。

2. 時間hh:MM的處理 應(yīng)該寫成工具類，避免重復(fù)。

3. 字符串分割的處理 多個地方存在將字符串（逗號分隔）分割為集合的代碼，應(yīng)該寫成工具類，可讀性好，并避免重復(fù)。

4. 三種比較業(yè)務(wù)纏繞到一起，業(yè)務(wù)可讀性差。這里涉及允許時間判斷、內(nèi)網(wǎng)ip、外網(wǎng)ip三個業(yè)務(wù)判斷，但是代碼通過循環(huán)纏繞到一起，可讀性差，不夠聚合，難以修改。

5. 多處存在SecureLogEvent的構(gòu)造（構(gòu)造安全日志記錄），并且沒有將關(guān)鍵業(yè)務(wù)數(shù)據(jù)傳入。

6. 注釋很少，關(guān)鍵代碼不容易閱讀。

7. 方法命名不夠準(zhǔn)確、明確。

8. Magic Code太多，應(yīng)該重構(gòu)為常量。

4 重構(gòu)過程說明

由于重構(gòu)過程的代碼是反復(fù)修改，所以已經(jīng)不好拿回，我先說明一下我的重構(gòu)過程，然后將重構(gòu)結(jié)果代碼貼上，這樣讀者應(yīng)該可以基本理解了。

1. 先閱讀該部分代碼，觀察那里存在重復(fù)。
2. 將將時間處理分割出來，作為獨立私有子方法。
3. 寫好后，寫個main函數(shù)測試一下，沒問題了就替換原方法中時間處理部分的代碼。
4. 閱讀字符串分割相關(guān)代碼，找出其共性，然后寫私有方法、測試、替換。
5. 觀察原先代碼，【允許登錄時間】判斷是在兩重循環(huán)里面進(jìn)行判斷，但是從業(yè)務(wù)角度，只要不在允許范圍，哪怕ip允許也是一樣不行的，所以應(yīng)該單獨判斷，而不是放在循環(huán)里面。
6. 內(nèi)網(wǎng)ip判斷、外網(wǎng)ip判斷，同理，也應(yīng)該可以獨立判斷，所以兩重循環(huán)就可以拆成兩個獨立的循環(huán)了。
7. 業(yè)務(wù)理清后，代碼層次就清晰了。
8. 然后將日志增加有價值的業(yè)務(wù)數(shù)據(jù)、代碼加上注釋、魔術(shù)字重構(gòu)為常量等。
9. 將調(diào)用處（前面圖1）改為boolean方式。
10. 將整個規(guī)則判斷代碼從CreditController中移出去，新建一個合適的工具類存放，一來減少該Controller代碼，二來以后類似的擴(kuò)展都已放到該工具類中，職責(zé)更加分明。

5 重構(gòu)后的代碼

圖1：調(diào)用處，改為工具類，并且返回boolean，命名方面可讀性明確，禁止ip和訪問時間，如果返回true，就跳回登錄頁面。

圖2、3：

• 最上面是常量。
• 然后下面是目前本類唯一一個公有方法。
• 先判斷允許方法時間，調(diào)用私有方法forbitVisitTimeRange。
• 然后下面isInside（是否可以內(nèi)網(wǎng)訪問）、isOutside（是否可以外網(wǎng)訪問）分別判斷（調(diào)用checkRange私有方法），去掉了兩重循環(huán)。
• recordLog寫成私有方法，并允許傳入拼接信息，把有業(yè)務(wù)意義的關(guān)鍵數(shù)據(jù)也寫到日志中。

圖4、5、6三個圖。

• 時間處理：通過ToDay工具類（我的框架自帶）處理，可讀性更好。
• 逗號分隔的字符串的分割方法，用框架的工具類，并通過兩層的私有方法，讓代碼更容易維護(hù)。
• 寫日志的私有方法，增加了拼接業(yè)務(wù)數(shù)據(jù)字符串的參數(shù)供傳入。

6. 總結(jié)

• 避免重復(fù)代碼 看到重復(fù)代碼，務(wù)必想辦法把它抽離出來重用。
• 善用工具類 無論自己框架的還是第三方開源框架的，不要自己發(fā)明輪子，如果沒有，甚至自己寫一個工具方法，這樣可以讓代碼更關(guān)注業(yè)務(wù)。
• 涉及多重循環(huán)時，好好考慮一下是不是一定要這樣做才可以。能否每個業(yè)務(wù)一個獨立的子方法？
• 注釋不能省，而且對于關(guān)鍵代碼，有注釋可讀性大大提升。
• 魔術(shù)字要重構(gòu)為常量。
• 寫日志時，切記不要寫那種沒有參考價值的日志。你要考慮一下，如果以后業(yè)務(wù)出錯或者出現(xiàn)意外需要回看日志時，這些信息夠不夠，能不能讓你足夠的重現(xiàn)當(dāng)時的場景