减少代码中该死的 if else 嵌套

  • 时间:
  • 浏览:0

doShareText(item.content, listener);

String title;

super(TYPE_LINK);

}

public void share(ShareItem item, ShareListener listener) {

}

}

} else if (item.type == TYPE_IMAGE_TEXT) {

if (!TextUtils.isEmpty(item.link) && !TextUtils.isEmpty(item.title)) {

} else {

if (item == null) {

}

可能性其他同学会说,原来用户也需额外了解多好多个土方法。我我各自 我着实让用户了解多好多个土方法好过了解多好多个类,而已土方法名一看就能知道意图,成本还是挺小,是还须要接受的。

}

} else {

嘻嘻,怎么,内内外部接口有另有有一一两个if else都都没人,是都是很酷~ 可能性你什儿 分享功能是我各自 App后边的功能,都是第三方SDK,到这里可能性没问题了。但可能性是第三方分享SDK的功能句子,原来暴露给用户的类增加了或多或少(各ShareItem的子类,离米 把if else抛给用户了),用户的接入成本提高,违背了“迪米特原则”了。

if (listener == null) {

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

public ShareItem createImageTextShareItem(String ImagePath, String content) {

@Override

private void init() {

利用多态,帕累托图业务单独避免,在接口不再做任何业务判断。把ShareItem抽象出来,作为基础类,但会 针对帕累托图业务各自 实现其子类:

Class<? extends ShareItem> shareItemClass = map.get(type);

我我着实你什儿 清况 ,更多人想到的是使用工厂模式。嗯,工厂模式能避免你什儿 问题(我我着实也须要用户额外了解多好多个type类型),但工厂模式难免又引入分支,我们都我们都还须要用Map消除分支。

我们都我们都线程池员的脑力不应该花费在无止境的分支句子里的,应该专注于业务两种。或多或少我们都我们都很有必要避免写出多分支嵌套的句子。好的,我们都我们都来分析下后边的代码多分支的原困 :

if else作为帕累托图编程语言都是可或缺的条件句子,我们都我们都在编程前会 血块的用到。但if else一般不建议嵌套超过三层,可能性一段代码存在不多的if else嵌套,代码的可读性就会疾速下降,后期维护难度也大大提高。或多或少,我们都我们都线程池员都应该尽量避免不多的if else嵌套。下面可能性谈谈我在工作中怎么减少if else嵌套的。

if (listener != null) {

super(TYPE_TEXT);

map.put(TYPE_LINK, Link.class);

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

}

public class Text extends ShareItem {

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

本文来自云栖社区媒体战略合作伙伴“安卓巴士Android开发者门户”,了解相关信息还须要关注“安卓巴士Android开发者门户”。

if (listener != null) {

if (listener != null) {

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

}

listener.onCallback(ShareListener.STATE_FAIL, "分享信息不全版");

@Override

void onCallback(int state, String msg);

// 分享链接

避免你什儿 清况 也很简单,再次封装一层即可。把ShareItem的子类的访问权限降低,在暴露给用户的主类里定义好多个土方法,在内内外部帮助用户创建具体的分享类型,原来用户就不多再知道具体的类了:

public interface ShareListener {

}

想象下有另有有一一两个简单分享的业务需求:支持分享链接、图片、文本和图文,分享结果回调给用户(为了不跑题,这里简略了业务,实际复杂得多)。当接手到没人有另有有一一两个业务时,是都是我着实很简单,稍动下脑就还须要动手了:

};

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

private void shareImpl (ShareItem item, ShareListener listener) {

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

}

listener.onCallback(ShareListener.STATE_FAIL, "不支持的分享类型");

if (item != null) {

来,看代码更加直观:

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

但还须要看到,shareImpl里还是蕴含分享类型的判断,也即业务判断,我们都我们都都清楚产品经理的脑洞有多大了,分享的类型随前会 改变或加上。嗯说到这里相信我们都我们都都想到用多态了。多态不但能应付业务改变的清况 ,也还须要用来减少if else的嵌套。

}

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

}

在谈我的土方法也不,不妨先用个例子来说明if else嵌套不多的弊端。

if (!TextUtils.isEmpty(item.link) &amp;&amp; !TextUtils.isEmpty(item.title)) {

} else {

public void onCallback(int state, String msg) {

到此,简单的分享模型就做出来了。有没问题?老实说,可能性没哪些追求句子,还真没哪些问题,离米 思路是清晰的。但一周后呢?有另有有一一两个月后呢?可能性一年后呢?share土方法的分支有15条,这原困 你每次回看代码得让我各自 的大脑变成微型的避免器,考虑15种清况 。可能性经常出现 bug,你又得考虑15种清况 ,并15种清况 都是测试下。再可能性现在须要加多分享小视频功能,你又得加上多一两个分支,须要改代码,或多或少都是“开放-闭合”。再再可能性后边项目交接给他人跟进,他人又要把我各自 大脑变成避免器来想每个分支的作用,我敢肯定有百分之八十的人前会 吐槽代码。

} else if (item.type == TYPE_TEXT) {

listener.onCallback(ShareListener.STATE_FAIL, "不支持的分享类型");

}

if (listener == null) {

讲到这里我们都我们都有没收获呢?总结下减少if else的土方法:

好了,但会 在定义个分享接口,对帕累托图类型分别进行分享就ok了:

}

}

public void share(ShareItem item, ShareListener listener) {

} else {

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

}

}

String imagePath;

super(TYPE_IMAGE_TEXT);

// do share

}

if (true) {

public void onCallback(int state, String msg) {

return new ImageText(ImagePath, content);

listener.onCallback(ShareListener.STATE_FAIL, "分享信息不全版");

if (true) {

}

}

}

}

}

map.put(TYPE_IMAGE_TEXT, ImageText.class);

item.doShare(listener);

}

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

}

@Override

try {

}

if (listener != null) {

private static final int TYPE_IMAGE = 1;

后边的代码我是用java写的,对于java线程池员来说,空值判断我们都我们都说使人很气恼,你会身心疲惫。后边的代码每次回调都是判断一次listener算是为空,又要判断用户传入的ShareItem算是为空,须要判断ShareItem后边的字段算是为空……

}

先定义分享的类型、分享Bean和分享回调类:

String imagePath;

public void doShare(ShareListener listener) {

}

// 分享图片

int STATE_SUCC = 0;

// 分享图文

public void share (ShareItem item, ShareListener listener) {

}

String link;

// do share

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

return new Image(ImagePath);

}

@Override

doShareImageAndText(item.imagePath, item.content, listener);

public ShareItem(int type) {

if (listener != null) {

private static final int TYPE_IMAGE_TEXT = 3;

}

对于你什儿 清况 ,我采用的土方法很简单:接口分层。

private static final int TYPE_LINK = 0;

listener = new ShareListener() {

} else {

return new Link(link, title, content);

}

listener.onCallback(ShareListener.STATE_FAIL, "分享信息不全版");

if (item.type == TYPE_LINK) {

}

}

}

private void shareImpl (ShareItem item, ShareListener listener) {

if (listener != null) {

} else {

String content;

} else if (item.type == TYPE_IMAGE) {

}

}

// 分享图文

} else {

public void doShare(ShareListener listener) {

原文发布时间为:2018-11-12

}

// 分享文本

};

return shareItemClass.newInstance();

public class Image extends ShareItem {

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

public void doShare(ShareListener listener) {

public ShareItem createTextShareItem(String content) {

public void doShare(ShareListener listener) {

public ShareItem createLinkShareItem(String link, String title, String content) {

doShareLink(item.link, item.title, item.content, listener);

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

}

if (true) {

if (item.type == TYPE_LINK) {

}

@Override

String link;

} else {

}

本文作者:leowudev

doShareLink(item.link, item.title, item.content, listener);

}

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

map.put(TYPE_IMAGE, Image.class);

}

你什儿 土方法跟后边分为好多个土方法的土方法各有利弊,看我们都我们都确定了~

}

@Override

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

public Text(String content) {

写在最后

if (true) {

shareImpl(item, listener);

String content;

shareImpl(item, listener);

if (true) {

String imagePath;

} else if (item.type == TYPE_IMAGE_TEXT) {

return;

public class ImageText extends ShareItem {

}

// 分享图片

int STATE_FAIL = 1;

}

}

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

// do share

把所有分享类型预先缓存在Map里,没人就还须要直接get获取具体类型,消除分支:

}

listener.onCallback(ShareListener.STATE_FAIL, "分享信息不全版");

public ImageText(String imagePath, String content) {

}

}

正文

// 分享链接

public ShareItem createImageShareItem(String ImagePath) {

}

减少 if else 土方法一:接口分层

(注意:后边每个子类的构造土方法还对每个字段做了空值避免,为空句子,赋值default,原来可能性用户传了空值,在调试就会发现问题。)

public class Link extends ShareItem {

doShareImage(item.imagePath, listener);

if (!TextUtils.isEmpty(item.imagePath) &amp;&amp; !TextUtils.isEmpty(item.content)) {

doShareImageAndText(item.imagePath, item.content, listener);

doShareImage(item.imagePath, listener);

if (item == null) {

} else {

}

if (!TextUtils.isEmpty(item.imagePath) &amp;&amp; !TextUtils.isEmpty(item.content)) {

return;

} else if (item.type == TYPE_TEXT) {

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

String content;

} else {

listener.onCallback(ShareListener.STATE_FAIL, "分享信息不全版");

if (listener != null) {

listener = new ShareListener() {

}

} else {

listener.onCallback(ShareListener.STATE_FAIL, "分享信息不全版");

if (true) {

}

return new Text(content);

}

我并没夸大其词,我是真的遇到过了!嵌套6、7层,有另有有一一两个函数几百行,简!直!看!死!人!

this.type = type;

int type;

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

public abstract void doShare(ShareListener listener);

if (listener != null) {

不知我们都我们都有没遇到过像“横放着的金字塔”一样的if else嵌套:

return new DefaultShareItem(); // 返回默认实现,无须返回null

String title;

String content;

public ShareItem createShareItem(int type) {

private Map<Integer, Class<? extends ShareItem>> map = new HashMap<>();

public Link(String link, String title, String content) {

}

} else if (item.type == TYPE_IMAGE) {

}

 ●  利用多态,把业务判断消除,各子类分别关注我各自 的实现,并实现子类的创建土方法,避免用户了解不多的类。 ●  把分支清况 信息预先缓存在Map里,直接get获取具体值,消除分支。

好了,到此就介绍完了,希望我们都我们都也不写代码能注意,有则避之,无则加勉。希望我们都我们都写的代码没人简洁~



public class ShareItem {

// 分享文本

listener.onCallback(ShareListener.STATE_FAIL, "分享信息不全版");

super(TYPE_IMAGE);

} catch (Exception e) {

public Image(String imagePath) {

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

map.put(TYPE_TEXT, Text.class);

int type;

// do share

}

}

private static final int TYPE_TEXT = 2;

doShareText(item.content, listener);

写在前面

}

还须要看到,后边的代码分为内外部接口share和内内外部接口shareImplShareItemShareListener的判断都倒入share里完成,没人shareImpl就减少了if else的嵌套了,离米 把if else分摊了。原来一来,代码的可读性好或多或少,嵌套或多或少我超过3层了。

减少 if else 土方法二:多态

listener.onCallback(ShareListener.STATE_FAIL, "分享信息不全版");

 ●  业务判断 ●  清况 判断

几乎所有的业务都离不开这好多个判断,从而原困 if else嵌套不多。那是都是没土方法避免了?答案肯定都是的。

听说有图会更多人看

public abstract class ShareItem {