作者：慕容千語(yǔ)
juejin.im/post/5b893eaef265da434816006c
平時(shí)開(kāi)發(fā)中if-else用的多嗎？
其實(shí)這是個(gè)再正常不過(guò)的coding習(xí)慣，當(dāng)我們代碼量小的時(shí)候用來(lái)做條件判斷是再簡(jiǎn)單不過(guò)的了。
但對(duì)于優(yōu)秀程序員來(lái)說(shuō)，這并不是好代碼，
為啥？
拋開(kāi)劑量談毒性都是耍流氓
在使用條件判斷語(yǔ)句的地方，如果代碼量小，需要判斷的場(chǎng)景少的話(huà)，那么沒(méi)有比 if-else 更合適的語(yǔ)句，比如下面這樣
1
2
3
4
5
if(object.getIndex() > 0) {
//do something
} else {
//do other things
}
那在什么情況下 if-else 才會(huì)變差呢？
以上面的代碼為例子，當(dāng)需要判斷的情況逐漸增加的時(shí)候，上面的代碼可能會(huì)變的難以維護(hù)。
在進(jìn)階高級(jí)開(kāi)發(fā)的路上，應(yīng)該逐步培養(yǎng)起這種前瞻意識(shí)，即使在代碼還在起步階段，應(yīng)該要能夠看到將來(lái)代碼發(fā)展的趨勢(shì)，比如上面的代碼，當(dāng)情況越來(lái)越多的時(shí)候，if-else可能會(huì)發(fā)展出許多個(gè)分支：
為什么 if-else 不是好代碼
這是完全可能的，以我的經(jīng)驗(yàn)來(lái)說(shuō)就在不少項(xiàng)目上見(jiàn)過(guò)這樣的代碼。
而且代碼執(zhí)行塊中的邏輯可能在幾次迭代后變的非常復(fù)雜，就像下面這樣
為什么 if-else 不是好代碼
看到這段代碼第一感覺(jué)就是想殺個(gè)小伙伴祭天。
如何重構(gòu)掉這段代碼
對(duì)于這種代碼我們重構(gòu)的目標(biāo)可以有兩個(gè)深度，看自己強(qiáng)迫癥的嚴(yán)重程度決定
繼續(xù)用 if-else，只達(dá)到剝離執(zhí)行代碼塊
用工廠模式去耦合
對(duì)于這兩種其實(shí)不是非此即彼的關(guān)系，而是優(yōu)化深度不同。第一種相對(duì)比較簡(jiǎn)單，可以重構(gòu)成下面這樣子
為什么 if-else 不是好代碼
代碼清爽了很多，
現(xiàn)在這段代碼可以清楚的看出來(lái)都處理了哪些情況，條件判斷的代碼只關(guān)注了條件的不同，
而對(duì)于不同條件的具體處理邏輯我們剝離到了其他地方，這樣即使寫(xiě)到腦袋迷糊，也不至于說(shuō)漏了哪個(gè)條件沒(méi)判斷。
進(jìn)一步優(yōu)化
在上面的優(yōu)化之后，如何再用工廠模式來(lái)繼續(xù)重構(gòu)呢？
從上的代碼看的出來(lái)，不同的條件下，執(zhí)行的邏輯是不同的，那么可以把這種執(zhí)行邏輯抽象出來(lái)，用多態(tài)的概念來(lái)定義不同的執(zhí)行方式。
為什么 if-else 不是好代碼
完成了這一步之后，就可以把代碼塊中不同條件下的方法抽到各個(gè)不同的具體類(lèi)里面去了，
為什么 if-else 不是好代碼
還可以進(jìn)一步優(yōu)化嗎？可以的，甚至這里的條件判斷都可以不要，我們可以定義一個(gè)工廠來(lái)把 new ExecutorWithTag()這件事給包了，
為什么 if-else 不是好代碼
對(duì)工廠模式還有印象嗎，上面這段代碼在我之前的工廠模式一文里出現(xiàn)過(guò)，這里可以算是工廠模式的一個(gè)實(shí)際應(yīng)用。
在經(jīng)過(guò)這一輪重構(gòu)之后，我們之前在一個(gè)類(lèi)里面寫(xiě)的那堆代碼已經(jīng)抽離到多個(gè)不同的類(lèi)里了，現(xiàn)在在原來(lái)的類(lèi)里的代碼變成怎樣了呢，
為什么 if-else 不是好代碼
重構(gòu)之后各個(gè)Executor和主類(lèi)中的耦合已經(jīng)降到很低了，
而且代碼整潔度提高了很多，之前那個(gè)類(lèi)的一段50+行的代碼變成了2行，這就是重構(gòu)的意義。
