if else 深度優化
阿新 • • 發佈:2019-06-18
一. if else表示式過於複雜
if ((condition1 && condition2 ) || ((condition2 || condition3) && condition4)) {
}
問題解決:分解條件表示式,分解成多個獨立的函式,為分解的新函進行良好的命名,從而更清楚表達自己意圖。
if (date.before(SUMMER_START) || date.after(SUMMER_END)) {
}
// 優化後
if (notSummer(date)) {
}
二. if else 巢狀過深
public void workOrRest { if (isWorkDay()) { System.out.println("sad,is work day!!!"); } else { if (isWorkTime()) { System.out.println("shit,is work time!!!"); } else { rest(); System.out.println("happy happy happy!!!"); } } }
解決辦法:通過衛語句,有時候條件式可能出現在巢狀n次才能真正執行,其他分支只是簡單報錯返回的情況,對於這種情況,應該單獨檢查報錯返回的分支,當條件為真時立即返回,這樣的單獨檢查就是衛語句(guard clauses).衛語句可以把我們的視線從異常處理中解放出來,集中精力到正常處理的程式碼中。
public void workOrRest { if (isWorkDay()) { System.out.println("sad,is work day!!!"); return; } if (isWorkTime()) { System.out.println("shit,is work time!!!"); return; } rest(); System.out.println("happy happy happy!!!"); }
三. if else過多
if (houseName.equals("Targaryen")) { ... System.out.println("Blood and fire"); } else if (houseName.equals("Baratheon")) { ... System.out.println("Ours is the Fury"); } else if (houseName.equals("Stark")) { ... System.out.println("Winter is coming"); } else if (houseName.equals("Lannister")) { ... System.out.println("Hear Me Roar"); } else if (houseName.equals("Arryn")) { ... System.out.println("as High as Honor"); } else if (houseName.equals("Tyrell")) { ... System.out.println("Growing Strong"); } else if (houseName.equals("Tully")) { ... System.out.println("Family, Duty, Honor"); } else if (houseName.equals("Martell")) { ... System.out.println("Unbent, Unbowed, Unbroken"); } else if (houseName.equals("Greyjoy")) { ... System.out.println("We Do Not Sow"); } else { ... System.out.println("world peace"); }
問題解決:利用策略or狀態模式,消除業務判斷,各之類分別關注自己的實現,大大降低了系統各部分之間的依賴。利用Map快取分支狀態資訊。基本可以達到對修改封閉,對擴充套件開放。
<property name="thronesMap">
<map>
<entry key="Targaryen"><bean class="game.of.thrones.TargaryenStrategy"/></entry>
<entry key="Baratheon"><bean class="game.of.thrones.BaratheonStrategy"/></entry>
<entry key="Stark"><bean class="game.of.thrones.StarkStrategy"/></entry>
<entry key="Lannister"><bean class="game.of.thrones.LannisterStrategy"/></entry>
<entry key="Arryn"><bean class="game.of.thrones.ArrynStrategy"/></entry>
<entry key="Tyrell"><bean class="game.of.thrones.TyrellStrategy"/></entry>
<entry key="Tully"><bean class="game.of.thrones.TullyStrategy"/></entry>
<entry key="Martell"><bean class="game.of.thrones.MartellStrategy"/></entry>
<entry key="Greyjoy"><bean class="game.of.thrones.GreyjoyStrategy"/></entry>
<entry key="Default"><bean class="game.of.thrones.DefaultStrategy"/></entry>
</map>
</property>
public interface GameOfThronesStrategyService {
/**
* 具體實現處理
*/
void mainTitle();
}
thronesMap.get(houseName).mainTitle();