减少该死的 if else 嵌套

来源:https://www.jianshu.com/p/57c065b124c4


写在前面

不知大家有没遇到过像“横放着的金字塔”一样的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和分享回调类:

privatestaticfinalintTYPE_LINK =0;

privatestaticfinalintTYPE_IMAGE =1;

privatestaticfinalintTYPE_TEXT =2;

privatestaticfinalintTYPE_IMAGE_TEXT =3;

publicclassShareItem{

inttype;

String title;

String content;

String imagePath;

String link;

}

publicinterfaceShareListener{

intSTATE_SUCC =0;

intSTATE_FAIL =1;

voidonCallback(intstate, String msg);

}

好了,然后在定义个分享接口,对每种类型分别进行分享就ok了:

publicvoidshare(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,"分享信息不完整");

}

}

}elseif(item.type == TYPE_IMAGE) {

// 分享图片

if(!TextUtils.isEmpty(item.imagePath)) {

doShareImage(item.imagePath, listener);

}else{

if(listener !=null) {

listener.onCallback(ShareListener.STATE_FAIL,"分享信息不完整");

}

}

}elseif(item.type == TYPE_TEXT) {

// 分享文本

if(!TextUtils.isEmpty(item.content)) {

doShareText(item.content, listener);

}else{

if(listener !=null) {

listener.onCallback(ShareListener.STATE_FAIL,"分享信息不完整");

}

}

}elseif(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 方法一:接口分层

所谓接口分层指的是:把接口分为外部和内部接口,所有空值判断放在外部接口完成,只处理一次;而内部接口传入的变量由外部接口保证不为空,从而减少空值判断。

来,看代码更加直观:

publicvoidshare(ShareItem item, ShareListener listener){

if(item ==null) {

if(listener !=null) {

listener.onCallback(ShareListener.STATE_FAIL,"ShareItem 不能为 null");

}

return;

}

if(listener ==null) {

listener =newShareListener() {

@Override

publicvoidonCallback(intstate, String msg){

Log.i("DEBUG","ShareListener is null");

}

};

}

shareImpl(item, listener);

}

privatevoidshareImpl(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,"分享信息不完整");

}

}elseif(item.type == TYPE_IMAGE) {

// 分享图片

if(!TextUtils.isEmpty(item.imagePath)) {

doShareImage(item.imagePath, listener);

}else{

listener.onCallback(ShareListener.STATE_FAIL,"分享信息不完整");

}

}elseif(item.type == TYPE_TEXT) {

// 分享文本

if(!TextUtils.isEmpty(item.content)) {

doShareText(item.content, listener);

}else{

listener.onCallback(ShareListener.STATE_FAIL,"分享信息不完整");

}

}elseif(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抽象出来,作为基础类,然后针对每种业务各自实现其子类:

publicabstractclassShareItem{

inttype;

publicShareItem(inttype){

this.type = type;

}

publicabstractvoiddoShare(ShareListener listener);

}

publicclassLinkextendsShareItem{

String title;

String content;

String link;

publicLink(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";

}

@Override

publicvoiddoShare(ShareListener listener){

// do share

}

}

publicclassImageextendsShareItem{

String imagePath;

publicImage(String imagePath){

super(TYPE_IMAGE);

this.imagePath = !TextUtils.isEmpty(imagePath) ? imagePath :"default";

}

@Override

publicvoiddoShare(ShareListener listener){

// do share

}

}

publicclassTextextendsShareItem{

String content;

publicText(String content){

super(TYPE_TEXT);

this.content = !TextUtils.isEmpty(content) ? content :"default";

}

@Override

publicvoiddoShare(ShareListener listener){

// do share

}

}

publicclassImageTextextendsShareItem{

String content;

String imagePath;

publicImageText(String imagePath, String content){

super(TYPE_IMAGE_TEXT);

this.imagePath = !TextUtils.isEmpty(imagePath) ? imagePath :"default";

this.content = !TextUtils.isEmpty(content) ? content :"default";

}

@Override

publicvoiddoShare(ShareListener listener){

// do share

}

}

(注意:上面每个子类的构造方法还对每个字段做了空值处理,为空的话,赋值default,这样如果用户传了空值,在调试就会发现问题。)

实现了多态后,分享接口的就简洁多了:

publicvoidshare(ShareItem item, ShareListener listener){

if(item ==null) {

if(listener !=null) {

listener.onCallback(ShareListener.STATE_FAIL,"ShareItem 不能为 null");

}

return;

}

if(listener ==null) {

listener =newShareListener() {

@Override

publicvoidonCallback(intstate, String msg){

Log.i("DEBUG","ShareListener is null");

}

};

}

shareImpl(item, listener);

}

privatevoidshareImpl(ShareItem item, ShareListener listener){

item.doShare(listener);

}

嘻嘻,怎样,内部接口一个if else都没了,是不是很酷~ 如果这个分享功能是自己App里面的功能,不是第三方SDK,到这里已经没问题了。但如果是第三方分享SDK的功能的话,这样暴露给用户的类增加了很多(各ShareItem的子类,相当于把if else抛给用户了),用户的接入成本提高,违背了“迪米特原则”了。

处理这种情况也很简单,再次封装一层即可。把ShareItem的子类的访问权限降低,在暴露给用户的主类里定义几个方法,在内部帮助用户创建具体的分享类型,这样用户就无需知道具体的类了:

publicShareItemcreateLinkShareItem(String link, String title, String content){

returnnewLink(link, title, content);

}

publicShareItemcreateImageShareItem(String ImagePath){

returnnewImage(ImagePath);

}

publicShareItemcreateTextShareItem(String content){

returnnewText(content);

}

publicShareItemcreateImageTextShareItem(String ImagePath, String content){

returnnewImageText(ImagePath, content);

}

或者有人会说,这样用户也需额外了解多几个方法。我个人觉得让用户了解多几个方法好过了解多几个类,而已方法名一看就能知道意图,成本还是挺小,是可以接受的。

其实这种情况,更多人想到的是使用工厂模式。嗯,工厂模式能解决这个问题(其实也需要用户额外了解多几个type类型),但工厂模式难免又引入分支,我们可以用Map消除分支。

减少 if else 方法三:使用Map替代分支语句

把所有分享类型预先缓存在Map里,那么就可以直接get获取具体类型,消除分支:

privateMap>map=newHashMap<>();

privatevoidinit(){

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);

}

publicShareItemcreateShareItem(inttype){

try{

Class shareItemClass =map.get(type);

returnshareItemClass.newInstance();

}catch(Exception e) {

returnnewDefaultShareItem();// 返回默认实现,不要返回null

}

}

这种方式跟上面分为几个方法的方式各有利弊,看大家取舍了~

写在最后

讲到这里大家有没收获呢?总结下减少if else的方法

把接口分为外部和内部接口,所有空值判断放在外部接口完成;而内部接口传入的变量由外部接口保证不为空,从而减少空值判断。

利用多态,把业务判断消除,各子类分别关注自己的实现,并实现子类的创建方法,避免用户了解过多的类。

把分支状态信息预先缓存在Map里,直接get获取具体值,消除分支。

好了,到此就介绍完了,希望大家以后写代码能注意,有则避之,无则加勉。希望大家写的代码越来越简洁~

扩展阅读

面试的时候问:你的期望薪资多少?怎么谈?

MySQL到底有多少种日志类型需要我们记住的!

僧多肉少的阿里实习,我面了5次才拿到offer!

这20个正则表达式,让你少写1,000行代码

全面对比 Redis 和 Memcached 的 6 点区别

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 213,752评论 6 493
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,100评论 3 387
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 159,244评论 0 349
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,099评论 1 286
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,210评论 6 385
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,307评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,346评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,133评论 0 269
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,546评论 1 306
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,849评论 2 328
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,019评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,702评论 4 337
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,331评论 3 319
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,030评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,260评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,871评论 2 365
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,898评论 2 351

推荐阅读更多精彩内容