Android 代码优化：“这个需求很简单，移动开发工程师的岗位职责

before:

在我们APP启动过程,我们可能常常有这样的需求:在APP第一次进入的时候根据网络请求的结果弹一个对话框式的广告, ok~ 很简单,那么代码大致就是这样(这边都是伪代码,为了方便理解):

@Override

protected void onCreate(@Nullable Bundle savedInstanceState) {

checkFirstDialogIfNeed

《Android学习笔记总结+最新移动架构视频+大厂安卓面试真题+项目实战源码讲义》

【docs.qq.com/doc/DSkNLaERkbnFoS0ZF】 完整内容开源分享

();

}

/**

• step 1

*/

private void checkFirstDialogIfNeed(){

Utils.fakeRequest(“http://www.api1.com”, new HttpCallBack() {

@Override

public void onOk() {

showADialog();

}

@Override

public void onFailure() {

//do nothing

}

});

}

ok,做完上线

step1

过了一阵子,产品同学又跑过来了说:

“老板的需求,我们要在首页第一次进来的时候加一个注册协议的弹框”

程序员小A: “可是首页已经之前有一个弹框了,直接一起展示吗?”

产品同学: “一起弹出来体验不好,那你放在A弹框之后吧!”

ok,那么程序员小A寻思了一下,那应该就是这么改了:

@Override

protected void onCreate(@Nullable Bundle savedInstanceState) {

checkFirstDialogIfNeed();

}

private void checkFirstDialogIfNeed(){

Utils.fakeRequest(“http://www.api1.com”, new HttpCallBack() {

@Override

public void onOk() {

showADialog();

}

@Override

public void onFailure() {

//请求失败直接跳过，请求注册协议

checkRegisterAgreement();

}

});

}

private void showADialog() {

new AlertDialog.Builder(this)

.setTitle(“这是一条有态度的广告”)

.setPositiveButton(“我看完了”, null)

.setOnDismissListener(new DialogInterface.OnDismissListener() {

@Override

public void onDismiss(DialogInterface dialogInterface) {

//弹框结束后请求注册协议

checkRegisterAgreement();

}

}).create().show();

}

private void checkRegisterAgreement() {

Utils.fakeRequest(“http://www.api2.com”, new HttpCallBack() {

@Override

public void onOk() {

showBDialog();

}

@Override

public void onFailure() {

//do nothing

}

});

}

这么下来,我们先请求接口APi1,如果成功了弹广告弹框A,弹框结束了调用注册协议的方法,如果APi1 请求失败,直接跳过广告弹框A,继而请求注册协议,当注册协议请求成功了弹出我们的注册协议弹框,这么下来,我们确实可以保证注册协议弹框在广告弹框之后~

step2

过了几天,产品同学又来了:“这次我们首页初次进来再加一个H5页面跳转,这次就加在 广告和注册协议之间吧…”

于是乎,小A继续改了改代码,整个代码就成了这样:

@Override

protected void onCreate(@Nullable Bundle savedInstanceState) {

checkFirstDialogIfNeed();

}

private void checkFirstDialogIfNeed() {

Utils.fakeRequest(“http://www.api1.com”, new HttpCallBack() {

@Override

public void onOk() {

showADialog();

}

@Override

public void onFailure() {

// //请求失败直接跳过，请求注册协议

// checkRegisterAgreement();

checkNeedShowH5();

}

});

}

private void showADialog() {

new AlertDialog.Builder(this)

.setTitle(“这是一条有态度的广告”)

.setPositiveButton(“我看完了”, null)

.setOnDismissListener(new DialogInterface.OnDismissListener() {

@Override

public void onDismiss(DialogInterface dialogInterface) {

// //弹框结束后请求注册协议

// checkRegisterAgreement();

//现在产品要插入一个H5页面优先请求

checkNeedShowH5();

}

}).create().show();

}

private void checkRegisterAgreement() {

Utils.fakeRequest(“http://www.api2.com”, new HttpCallBack() {

@Override

public void onOk() {

showBDialog();

}

@Override

public void onFailure() {

//do nothing

}

});

}

private void showBDialog() {

new AlertDialog.Builder(this)

.setTitle(“这是注册协议”)

.setPositiveButton(“我看完了”, null)

.setOnDismissListener(new DialogInterface.OnDismissListener() {

@Override

public void onDismiss(DialogInterface dialogInterface) {

//do nothing

}

}).create().show();

}

private void checkNeedShowH5() {

Utils.fakeRequest(“http://www.api3.com”, new HttpCallBack() {

@Override

public void onOk() {

toH5Page();

}

@Override

public void onFailure() {

checkRegisterAgreement();

}

});

}

private void toH5Page() {

startActivityForResult(new Intent(this, TestH5Activity.class), REQUEST_CODE_H5);

}

@Override

protected void onActivityResult(int requestCode, int resultCode, @Nullable Intent data) {

super.onActivityResult(requestCode, resultCode, data);

switch (requestCode) {

case REQUEST_CODE_H5:

checkRegisterAgreement();

break;

default:

break;

}

}

首先将原有step1 结束之后不再调用注册协议,而是调用请求H5的方法.

由于去H5是一个Activity的跳转,所以我们在onActivityResult里我们继续之前的注册协议的调用.

看看大致demo效果:

再后来…几经迭代,首页首次的弹框和页面跳转已经有7、8个了,当每次产品有类似需求的时候,我们又要重复上面的过程了,每次大概梳理个大半天吧.

大家是否有发现什么问题吗?

---

1.首页次序之间强耦合,每次一旦要在二者之前插入一个其他弹框或者页面,我们每次都要修改在它之前和之后的调用链,至少要修改三处,很容易漏掉,但是实际上它们彼此之间除了次序之外,没有其他关联.

2.每次新需求来的时候都要完整梳理原有逻辑,比较浪费时间,影响效率.

怎么办？

---

思考:

1.能否在一个链中统一管理要处理的事情,每个事情之间互相不关联,只要简单配置就可以轻松替换它们之间的顺序.

2.后来维护者,很清晰的就能知道调用的次序,无需每次重新梳理整个业务代码.

设计:

1.我们是否可以把每件要做的事情抽象成一个节点,每个节点只关心自己的任务是否完成,它并不知道它是第几个,也不知道它前面或者后面的是谁.

2.每个节点统一由一个流来管理,它作为全局统筹者,可以控制从任意节点开始、控制整个流的开启与结束等,每个节点的顺序由流来管理.

实现:

带着以上设计思路,我对代码做了一下重构,代码变成了这样:

public class AfterActivity extends AppCompatActivity {

private static final int REQUEST_CODE_H5 = 1;

/**

• 初次广告弹框

*/

private static final int NODE_FIRST_AD = 10;

/**

• 初次进入h5页

*/

private static final int NODE_CHECK_H5 = 20;

/**

• 初次进入的注册协议

*/

private static final int NODE_REGISTER_AGREEMENT = 30;

private WorkFlow workFlow;

@Override

protected void onCreate(@Nullable Bundle savedInstanceState) {

super.onCreate(savedInstanceState);

startWorkFlow();

}

private void startWorkFlow() {

workFlow = new WorkFlow.Builder()

.withNode(getFirstAdNode())

.withNode(getShowRegisterAgreementNode())

.withNode(getShowH5Node())

.create();

workFlow.start();

}

private WorkNode getFirstAdNode() {

return WorkNode.build(NODE_FIRST_AD, new Worker() {

@Override

public void doWork(final Node current) {

Utils.fakeRequest(“http://www.api1.com”, new HttpCallBack() {

@Override

public void onOk() {

new AlertDialog.Builder(AfterActivity.this)

.setTitle(“这是一条有态度的广告”)

.setPositiveButton(“我看完了”, null)

.setOnDismissListener(new DialogInterface.OnDismissListener() {

@Override

public void onDismiss(DialogInterface dialogInterface) {

//仅仅只需关心自己是否完成，下一个节点会自动执行

current.onCompleted();

}

}).create().show();

}

@Override

public void onFailure() {

//仅仅只需关心自己是否完成，下一个节点会自动执行

current.onCompleted();

}

});

}

});

}

private WorkNode getShowRegisterAgreementNode() {

return WorkNode.build(NODE_REGISTER_AGREEMENT, new Worker() {

@Override

public void doWork(final Node current) {

Utils.fakeRequest(“http://www.api2.com”, new HttpCallBack() {

@Override

public void onOk() {

new AlertDialog.Builder(AfterActivity.this)

.setTitle(“这是注册协议”)

.setPositiveButton(“我看完了”, null)

.setOnDismissListener(new DialogInterface.OnDismissListener() {

@Override

public void onDismiss(DialogInterface dialogInterface) {

current.onCompleted();

}

}).create().show();

}

@Override

public void onFailure() {

current.onCompleted();

}

});

}

});

}

private WorkNode getShowH5Node() {

return (WorkNode.build(NODE_CHECK_H5, new Worker() {

@Override

public void doWork(final Node current) {

Utils.fakeRequest(“http://www.api3.com”, new HttpCallBack() {

@Override

public void onOk() {

startActivityForResult(new Intent(AfterActivity.this, TestH5Activity.class), REQUEST_CODE_H5);

}

@Override

public void onFailure() {

current.onCompleted();

}

});

}

}));

}

@Override

protected void onActivityResult(int requestCode, int resultCode, @Nullable Intent data) {

super.onActivityResult(requestCode, resultCode, data);

switch (requestCode) {

case REQUEST_CODE_H5:

workFlow.continueWork();

break;