减少代码中该死的 if else 嵌套
來源：http://www.apkbus.com/blog-970703-78964.html
寫在前面
不知大家有沒遇到過像“橫放著的金字塔”一樣的if else嵌套：
if?(true)?{if?(true)?{if?(true)?{if?(true)?{if?(true)?{if?(true)?{}}}}} }我并沒夸大其詞，我是真的遇到過了！嵌套6、7層，一個函數幾百行，簡！直！看！死！人！
if else作為每種編程語言都不可或缺的條件語句，我們在編程時會大量的用到。但if else一般不建議嵌套超過三層，如果一段代碼存在過多的if else嵌套，代碼的可讀性就會急速下降，后期維護難度也大大提高。所以，我們程序員都應該盡量避免過多的if else嵌套。下面將會談談我在工作中如何減少if else嵌套的。
正文
在談我的方法之前，不妨先用個例子來說明if else嵌套過多的弊端。
想象下一個簡單分享的業務需求：支持分享鏈接、圖片、文本和圖文，分享結果回調給用戶（為了不跑題，這里簡略了業務，實際復雜得多）。當接手到這么一個業務時，是不是覺得很簡單，稍動下腦就可以動手了：
先定義分享的類型、分享Bean和分享回調類：
private?static?final?int?TYPE_LINK?=?0; private?static?final?int?TYPE_IMAGE?=?1; private?static?final?int?TYPE_TEXT?=?2; private?static?final?int?TYPE_IMAGE_TEXT?=?3;public?class?ShareItem?{int?type;String?title;String?content;String?imagePath;String?link; }public?interface?ShareListener?{int?STATE_SUCC?=?0;int?STATE_FAIL?=?1;void?onCallback(int?state,?String?msg); }好了，然后在定義個分享接口，對每種類型分別進行分享就ok了：
public?void?share?(ShareItem?item,?ShareListener?listener)?{if?(item?!=?null)?{if?(item.type?==?TYPE_LINK)?{//?分享鏈接if?(!TextUtils.isEmpty(item.link)?&&?!TextUtils.isEmpty(item.title))?{doShareLink(item.link,?item.title,?item.content,?listener);}?else?{if?(listener?!=?null)?{listener.onCallback(ShareListener.STATE_FAIL,?"分享信息不完整");}}}?else?if?(item.type?==?TYPE_IMAGE)?{//?分享圖片if?(!TextUtils.isEmpty(item.imagePath))?{doShareImage(item.imagePath,?listener);}?else?{if?(listener?!=?null)?{listener.onCallback(ShareListener.STATE_FAIL,?"分享信息不完整");}}}?else?if?(item.type?==?TYPE_TEXT)?{//?分享文本if?(!TextUtils.isEmpty(item.content))?{doShareText(item.content,?listener);}?else?{if?(listener?!=?null)?{listener.onCallback(ShareListener.STATE_FAIL,?"分享信息不完整");}}}?else?if?(item.type?==?TYPE_IMAGE_TEXT)?{//?分享圖文if?(!TextUtils.isEmpty(item.imagePath)?&&?!TextUtils.isEmpty(item.content))?{doShareImageAndText(item.imagePath,?item.content,?listener);}?else?{if?(listener?!=?null)?{listener.onCallback(ShareListener.STATE_FAIL,?"分享信息不完整");}}}?else?{if?(listener?!=?null)?{listener.onCallback(ShareListener.STATE_FAIL,?"不支持的分享類型");}}}?else?{if?(listener?!=?null)?{listener.onCallback(ShareListener.STATE_FAIL,?"ShareItem?不能為?null");}} }到此，簡單的分享模型就做出來了。有沒問題？老實說，如果沒什么追求的話，還真沒什么問題，至少思路是清晰的。但一周后呢？一個月后呢？或者一年后呢？share方法的分支有15條，這意味著你每次回看代碼得讓自己的大腦變成微型的處理器，考慮15種情況。如果出現bug，你又得考慮15種情況，并15種情況都要測試下。再如果現在需要加多分享小視頻功能，你又得添加多3個分支，還要改代碼，一點都不“開放-閉合”。再再如果后面項目交接給他人跟進，他人又要把自己大腦變成處理器來想每個分支的作用，我敢肯定有百分之八十的人都會吐槽代碼。
我們程序員的腦力不應該花費在無止境的分支語句里的，應該專注于業務本身。所以我們很有必要避免寫出多分支嵌套的語句。好的，我們來分析下上面的代碼多分支的原因：
-
空值判斷
-
業務判斷
-
狀態判斷
幾乎所有的業務都離不開這幾個判斷，從而導致if else嵌套過多。那是不是沒辦法解決了？答案肯定不是的。
上面的代碼我是用java寫的，對于java程序員來說，空值判斷簡直使人很沮喪，讓人身心疲憊。上面的代碼每次回調都要判斷一次listener是否為空，又要判斷用戶傳入的ShareItem是否為空，還要判斷ShareItem里面的字段是否為空……
對于這種情況，我采用的方法很簡單：接口分層。
減少 if else 方法一：接口分層
所謂接口分層指的是：把接口分為外部和內部接口，所有空值判斷放在外部接口完成，只處理一次；而內部接口傳入的變量由外部接口保證不為空，從而減少空值判斷。
來，看代碼更加直觀：
public?void?share(ShareItem?item,?ShareListener?listener)?{if?(item?==?null)?{if?(listener?!=?null)?{listener.onCallback(ShareListener.STATE_FAIL,?"ShareItem?不能為?null");}return;}if?(listener?==?null)?{listener?=?new?ShareListener()?{@Overridepublic?void?onCallback(int?state,?String?msg)?{Log.i("DEBUG",?"ShareListener?is?null");}};}shareImpl(item,?listener); }private?void?shareImpl?(ShareItem?item,?ShareListener?listener)?{if?(item.type?==?TYPE_LINK)?{//?分享鏈接if?(!TextUtils.isEmpty(item.link)?&&?!TextUtils.isEmpty(item.title))?{doShareLink(item.link,?item.title,?item.content,?listener);}?else?{listener.onCallback(ShareListener.STATE_FAIL,?"分享信息不完整");}}?else?if?(item.type?==?TYPE_IMAGE)?{//?分享圖片if?(!TextUtils.isEmpty(item.imagePath))?{doShareImage(item.imagePath,?listener);}?else?{listener.onCallback(ShareListener.STATE_FAIL,?"分享信息不完整");}}?else?if?(item.type?==?TYPE_TEXT)?{//?分享文本if?(!TextUtils.isEmpty(item.content))?{doShareText(item.content,?listener);}?else?{listener.onCallback(ShareListener.STATE_FAIL,?"分享信息不完整");}}?else?if?(item.type?==?TYPE_IMAGE_TEXT)?{//?分享圖文if?(!TextUtils.isEmpty(item.imagePath)?&&?!TextUtils.isEmpty(item.content))?{doShareImageAndText(item.imagePath,?item.content,?listener);}?else?{listener.onCallback(ShareListener.STATE_FAIL,?"分享信息不完整");}}?else?{listener.onCallback(ShareListener.STATE_FAIL,?"不支持的分享類型");} }可以看到，上面的代碼分為外部接口share和內部接口shareImpl，ShareItem和ShareListener的判斷都放在share里完成，那么shareImpl就減少了if else的嵌套了，相當于把if else分攤了。這樣一來，代碼的可讀性好很多，嵌套也不超過3層了。
但可以看到，shareImpl里還是包含分享類型的判斷，也即業務判斷，我們都清楚產品經理的腦洞有多大了，分享的類型隨時會改變或添加。嗯說到這里相信大家都想到用多態了。多態不但能應付業務改變的情況，也可以用來減少if else的嵌套。
減少 if else 方法二：多態
利用多態，每種業務單獨處理，在接口不再做任何業務判斷。把ShareItem抽象出來，作為基礎類，然后針對每種業務各自實現其子類：
public?abstract?class?ShareItem?{int?type;public?ShareItem(int?type)?{this.type?=?type;}public?abstract?void?doShare(ShareListener?listener); }public?class?Link?extends?ShareItem?{String?title;String?content;String?link;public?Link(String?link,?String?title,?String?content)?{super(TYPE_LINK);this.link?=?!TextUtils.isEmpty(link)???link?:?"default";this.title?=?!TextUtils.isEmpty(title)???title?:?"default";this.content?=?!TextUtils.isEmpty(content)???content?:?"default";}@Overridepublic?void?doShare(ShareListener?listener)?{//?do?share} }public?class?Image?extends?ShareItem?{String?imagePath;public?Image(String?imagePath)?{super(TYPE_IMAGE);this.imagePath?=?!TextUtils.isEmpty(imagePath)???imagePath?:?"default";}@Overridepublic?void?doShare(ShareListener?listener)?{//?do?share} }public?class?Text?extends?ShareItem?{String?content;public?Text(String?content)?{super(TYPE_TEXT);this.content?=?!TextUtils.isEmpty(content)???content?:?"default";}@Overridepublic?void?doShare(ShareListener?listener)?{//?do?share} }public?class?ImageText?extends?ShareItem?{String?content;String?imagePath;public?ImageText(String?imagePath,?String?content)?{super(TYPE_IMAGE_TEXT);this.imagePath?=?!TextUtils.isEmpty(imagePath)???imagePath?:?"default";this.content?=?!TextUtils.isEmpty(content)???content?:?"default";}@Overridepublic?void?doShare(ShareListener?listener)?{//?do?share} }（注意：上面每個子類的構造方法還對每個字段做了空值處理，為空的話，賦值default，這樣如果用戶傳了空值，在調試就會發現問題。）
實現了多態后，分享接口的就簡潔多了：
public?void?share(ShareItem?item,?ShareListener?listener)?{if?(item?==?null)?{if?(listener?!=?null)?{listener.onCallback(ShareListener.STATE_FAIL,?"ShareItem?不能為?null");}return;}if?(listener?==?null)?{listener?=?new?ShareListener()?{@Overridepublic?void?onCallback(int?state,?String?msg)?{Log.i("DEBUG",?"ShareListener?is?null");}};}shareImpl(item,?listener); }private?void?shareImpl?(ShareItem?item,?ShareListener?listener)?{item.doShare(listener); }嘻嘻，怎樣，內部接口一個if else都沒了，是不是很酷~ 如果這個分享功能是自己App里面的功能，不是第三方SDK，到這里已經沒問題了。但如果是第三方分享SDK的功能的話，這樣暴露給用戶的類增加了很多（各ShareItem的子類，相當于把if else拋給用戶了），用戶的接入成本提高，違背了“迪米特原則”了。
處理這種情況也很簡單，再次封裝一層即可。把ShareItem的子類的訪問權限降低，在暴露給用戶的主類里定義幾個方法，在內部幫助用戶創建具體的分享類型，這樣用戶就無需知道具體的類了：
public?ShareItem?createLinkShareItem(String?link,?String?title,?String?content)?{return?new?Link(link,?title,?content); }public?ShareItem?createImageShareItem(String?ImagePath)?{return?new?Image(ImagePath); }public?ShareItem?createTextShareItem(String?content)?{return?new?Text(content); }public?ShareItem?createImageTextShareItem(String?ImagePath,?String?content)?{return?new?ImageText(ImagePath,?content); }或者有人會說，這樣用戶也需額外了解多幾個方法。我個人覺得讓用戶了解多幾個方法好過了解多幾個類，而已方法名一看就能知道意圖，成本還是挺小，是可以接受的。
其實這種情況，更多人想到的是使用工廠模式。嗯，工廠模式能解決這個問題（其實也需要用戶額外了解多幾個type類型），但工廠模式難免又引入分支，我們可以用Map消除分支。
減少 if else 方法三：使用Map替代分支語句
把所有分享類型預先緩存在Map里，那么就可以直接get獲取具體類型，消除分支：
private?Map<integer,?class>?map?=?new?HashMap<>();private?void?init()?{map.put(TYPE_LINK,?Link.class);map.put(TYPE_IMAGE,?Image.class);map.put(TYPE_TEXT,?Text.class);map.put(TYPE_IMAGE_TEXT,?ImageText.class); }public?ShareItem?createShareItem(int?type)?{try?{Class?shareItemClass?=?map.get(type);return?shareItemClass.newInstance();}?catch?(Exception?e)?{return?new?DefaultShareItem();?//?返回默認實現，不要返回null}? }這種方式跟上面分為幾個方法的方式各有利弊，看大家取舍了~
寫在最后
講到這里大家有沒收獲呢？總結下減少if else的方法：
-
把接口分為外部和內部接口，所有空值判斷放在外部接口完成；而內部接口傳入的變量由外部接口保證不為空，從而減少空值判斷。
-
利用多態，把業務判斷消除，各子類分別關注自己的實現，并實現子類的創建方法，避免用戶了解過多的類。
-
把分支狀態信息預先緩存在Map里，直接get獲取具體值，消除分支。
好了，到此就介紹完了，希望大家以后寫代碼能注意，有則避之，無則加勉。希望大家寫的代碼越來越簡潔~
總結
以上是生活随笔為你收集整理的减少代码中该死的 if else 嵌套的全部內容，希望文章能夠幫你解決所遇到的問題。
- 上一篇： 比特币拒绝第 340 次“被死亡”
- 下一篇： 小白科普：Java EE vs J2EE