热文导读 | 点击标题阅读
老程序员被裁员,面试半年未果,同行:都四十了还是码农,怪谁?
来源: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() { @Override public 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"; } @Override public 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"; } @Override public 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"; } @Override public 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"; } @Override public 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() { @Override public 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> 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 extends ShareItem> shareItemClass = map.get(type); return shareItemClass.newInstance(); } catch (Exception e) { return new DefaultShareItem(); // 返回默认实现,不要返回null } }
这种方式跟上面分为几个方法的方式各有利弊,看大家取舍了~
写在最后
讲到这里大家有没收获呢?总结下减少if else
的方法:
把接口分为外部和内部接口,所有空值判断放在外部接口完成;而内部接口传入的变量由外部接口保证不为空,从而减少空值判断。
利用多态,把业务判断消除,各子类分别关注自己的实现,并实现子类的创建方法,避免用户了解过多的类。
把分支状态信息预先缓存在
Map
里,直接get
获取具体值,消除分支。
好了,到此就介绍完了,希望大家以后写代码能注意,有则避之,无则加勉。希望大家写的代码越来越简洁~
想进阿里吗?快加入我们的知识星球吧,如下:
如有收获,欢迎「分享 」
「点赞」「评论
」
妈妈常教导我,让我养成良好习惯。这样长大才能成为一个有用的人。良好的习惯是尊敬师长这样长大才能成为一个有用的人。良好的习惯是尊敬师长,爱护同学,对人有礼貌;是不粗心,做事情不拖拉;还是爱护公物,不浪费粮食。为什么呢?因为拥有良好习惯,做一个品德高尚的人,懂得尊重别人,才会得到别人的尊重。我要努力地做到这些。我有一些坏习惯,有时候学习很粗心,把一些会做的题做错。在生活上,也很粗心,有一次早上起床居然穿反了衣服。我吃饭很慢,有的时候还剩饭。我还起床磨蹭,本来应该迅速地穿好衣服,但是,我总是磨磨蹭蹭地,速度很慢。“我打算在这学期里,改掉这些坏习惯。早上起来,迅速地穿好衣服,不拖拉。学习不粗心,仔细完成每一道题。吃饭的时候,要很快的把饭吃完,不剩饭。我要从一点一滴做起,逐渐养成良好习惯。我相信自己一定能成为一名品学兼优的好学生!我打算在这学期里,改掉这些坏习惯。早上起来,迅速地穿好衣服,不拖拉。学习不粗心,仔细完成每一道题。吃饭的时候,要很快的把饭吃完,不剩饭。我要从一点一滴做起,逐渐养成良好习惯。我相信自己一定能成为一名品学兼优的好学生!” 在上幼儿园以前,我什么也不会干,就连穿衣服也是妈妈给我穿好,就要上幼儿园了,这样可不行,妈妈锻炼我要学会自己穿衣服。 有一天,妈妈把衣服摆在我面前,开始让我自己穿。一开始。我又哭又叫就是不穿,还把衣服扔的满地都是,然后坐在地上开始大哭,等了好长时间,妈妈还是不理我,我只好自己乖乖的把衣服穿好, 一出了房间门,妈妈就笑了起来,再看看我的衣服,毛衣和裤子都穿反了,我赶紧回房间又重新穿了一遍,这次穿好了,拿起外套,可是外套的扣子又扣不上了,扣子可调皮了,好像故意和我作对,我把扣子往扣眼——人类邪恶的根源;爱情——幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话:幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话“亲爱的!擦干你的眼泪,至高无上的爱情已经打开了我们的眼界,使我们成了它的崇拜者。是它,
妈妈常教导我,让我养成良好习惯。这样长大才能成为一个有用的人。良好的习惯是尊敬师长这样长大才能成为一个有用的人。良好的习惯是尊敬师长,爱护同学,对人有礼貌;是不粗心,做事情不拖拉;还是爱护公物,不浪费粮食。为什么呢?因为拥有良好习惯,做一个品德高尚的人,懂得尊重别人,才会得到别人的尊重。我要努力地做到这些。我有一些坏习惯,有时候学习很粗心,把一些会做的题做错。在生活上,也很粗心,有一次早上起床居然穿反了衣服。我吃饭很慢,有的时候还剩饭。我还起床磨蹭,本来应该迅速地穿好衣服,但是,我总是磨磨蹭蹭地,速度很慢。“我打算在这学期里,改掉这些坏习惯。早上起来,迅速地穿好衣服,不拖拉。学习不粗心,仔细完成每一道题。吃饭的时候,要很快的把饭吃完,不剩饭。我要从一点一滴做起,逐渐养成良好习惯。我相信自己一定能成为一名品学兼优的好学生!我打算在这学期里,改掉这些坏习惯。早上起来,迅速地穿好衣服,不拖拉。学习不粗心,仔细完成每一道题。吃饭的时候,要很快的把饭吃完,不剩饭。我要从一点一滴做起,逐渐养成良好习惯。我相信自己一定能成为一名品学兼优的好学生!” 在上幼儿园以前,我什么也不会干,就连穿衣服也是妈妈给我穿好,就要上幼儿园了,这样可不行,妈妈锻炼我要学会自己穿衣服。 有一天,妈妈把衣服摆在我面前,开始让我自己穿。一开始。我又哭又叫就是不穿,还把衣服扔的满地都是,然后坐在地上开始大哭,等了好长时间,妈妈还是不理我,我只好自己乖乖的把衣服穿好, 一出了房间门,妈妈就笑了起来,再看看我的衣服,毛衣和裤子都穿反了,我赶紧回房间又重新穿了一遍,这次穿好了,拿起外套,可是外套的扣子又扣不上了,扣子可调皮了,好像故意和我作对,我把扣子往扣眼——人类邪恶的根源;爱情——幸福和光明的源泉。我一直在这些思想的舞台上徘徊。突然我发现两个身影从我面前经过,坐在不远的草地上。这是一对从农田那边走过来的青年男女。农田那边有农民的茅舍。在一阵令人伤心的沉默之后,随着一声长叹,我听见从一个肺痨病人的嘴里说出了这样的话:“亲爱的!擦干你的眼泪,至高无上的爱情已经打开了我们的眼界,使我们成了它的崇拜者。是它,
如你有好的文章想和大家分享欢迎投稿,直接向我投递文章链接即可
最后,国庆福利来了,我们的知识星球已达到1000人了,之前说过到达1000人时将大大幅涨价到169元,为了反馈大家对我们的关注和厚爱,特此维持现价99元最后一天,今天后(今晚 00:00)后将涨到169元,欢迎大家加入我们的知识星球,更多星球信息参见:
微信扫描或者点击上方二维码领取Android\Python\AI\Java等高级进阶资源
更多学习资料点击下面的“阅读原文”获取
评论记录:
回复评论: