java怎么简洁用if_记一次java简单的if语句使用多态重构

场景描述:

一个controller中,部门领导有布置任务,查看任务整体情况,查看部门成员,查看部门成员完成情况,导出任务详情,如下:

@RestController

@RequestMapping(value= "/task")public classTaskController{

@PostMapping(value= "/saveTask")publicJSONObject saveTask(Task task){

...

}

@PostMapping(value= "/taskDetail")publicJSONObject taskDetail(Task task){

...

}

@PostMapping(value= "/memberList")publicJSONObject memberList(Task task){

...

}

@PostMapping(value= "/memberDetail")publicJSONObject memberDetail(Task task){

...

}

@PostMapping(value= "/exportTask")publicJSONObject exportTask(Task task){

...

}

}

过了一段时间后,新需求来了,之前任务是需求任务,现在任务种类多了一个测试任务,但是接口要公用一套。所以Task加了一个type的字段用来区分不同的任务,1:原来类型的任务,2:测试任务,controller变成如下(Task的类型type应该写在常量里面):

@RestController

@RequestMapping(value = "/task")

public class TaskController{

@PostMapping(value = "/saveTask")

public JSONObject saveTask(Task task){

if( Constants.COMMON_TASK == task.type){

...

}else if( Constants.TEST_TASK == task.type){

...

}

}

@PostMapping(value = "/taskDetail")

public JSONObject taskDetail(Task task){

if( Constants.COMMON_TASK == task.type){

...

}else if( Constants.TEST_TASK == task.type){

...

}

}

@PostMapping(value = "/memberList")

public JSONObject memberList(Task task){

if( Constants.COMMON_TASK == task.type){

...

}else if( Constants.TEST_TASK == task.type){

...

}

}

@PostMapping(value = "/memberDetail")

public JSONObject memberDetail(Task task){

if( Constants.COMMON_TASK == task.type){

...

}else if( Constants.TEST_TASK == task.type){

...

}

}

@PostMapping(value = "/exportTask")

public JSONObject exportTask(Task task){

if( Constants.COMMON_TASK == task.type){

...

}else if( Constants.TEST_TASK == task.type){

...

}

}

}

再之后又有新任务类型加进来,继续往这个controller的几个方法中增加if-else。目前的开发流程是controller-->service-->dao,判断那种任务,然后走到service调用这个任务的业务方法,现在使用多态的方法把if-else语句重构掉

首先创建一个抽象类,把共同的方法抽离出来

@Service

public abstract class BaseTaskService{

public abstract JSONObject saveTask(Task task);

public abstract JSONObject taskDetail(Task task);

public abstract JSONObject memberList(Task task);

public abstract JSONObject memberDetail(Task task);

public abstract JSONObject exportTask(Task task);

}

然后不同的任务继承这个抽象类,具体的业务逻辑在自己的类上处理,需求任务:

@Service

public class CommonTask extends BaseTaskService{

@Ovreride

public JSONObject saveTask(Task task){

...

}

@Ovreride

public JSONObject taskDetail(Task task){

...

}

@Ovreride

public JSONObject memberList(Task task){

...

}

@Ovreride

public JSONObject memberDetail(Task task){

...

}

@Ovreride

public JSONObject exportTask(Task task){

...

}

}

测试任务:

@Service

public class TestTask extends BaseTaskService{

@Ovreride

public JSONObject saveTask(Task task){

...

}

@Ovreride

public JSONObject taskDetail(Task task){

...

}

@Ovreride

public JSONObject memberList(Task task){

...

}

@Ovreride

public JSONObject memberDetail(Task task){

...

}

@Ovreride

public JSONObject exportTask(Task task){

...

}

}

再之后创建工厂类返回任务类

@Service

public class TaskFactory{

@Autowired

private CommonTask commonTask;

@Autowired

private TestTask testTask;

...

public BaseTaskService getTask(Integer type){

if(Constants.COMMON_TASK.equals(type){

return commonTask;

}else if(Constants.TestTask.equals(type){

return testTask;

}else if(...){

...

}

...

}

}

最后修改controller就大功告成

@RestController

@RequestMapping(value = "/task")

public class TaskController{

@Autowired

private TaskFactory taskFactory;

@PostMapping(value = "/saveTask")

public JSONObject saveTask(Task task){

BaseTaskService currentTask = taskFactory.getTask(task);

return currentTask.saveTask(task);

}

@PostMapping(value = "/taskDetail")

public JSONObject taskDetail(Task task){

BaseTaskService currentTask = taskFactory.getTask(task);

return currentTask.taskDetail(task);

}

@PostMapping(value = "/memberList")

public JSONObject memberList(Task task){

BaseTaskService currentTask = taskFactory.getTask(task);

return currentTask.memberList(task);

}

@PostMapping(value = "/memberDetail")

public JSONObject memberDetail(Task task){

BaseTaskService currentTask = taskFactory.getTask(task);

return currentTask.memberDetail(task);

}

@PostMapping(value = "/exportTask")

public JSONObject exportTask(Task task){

BaseTaskService currentTask = taskFactory.getTask(task);

return exportTask(task);

}

}

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
/* * 原始需求背景: * 网宿CDN要按月收取客户的服务费用,根据流量的大小、 * 服务的类型等,收取不同的费用,收费规则如下: * web应用:1000元/M * 流媒体应用:1000元/M*0.7 * 下载应用:1000元/M*0.5 * 月末打印报表时,要罗列每个用户每个频道的费用、客户总费用, * 还要打印该客户的重要性指数,重要性指数=网页流/100+下载流量/600; * * 需求变场景: * 系统已经开发出来了,接下来,运维部门现在希望对系统做一点修改, * 首先,他们希望能够输出xml,这样可以被其它系统读取和处理,但是, * 这段代码根本不可能在输出xml的代码中复用report()的任何行为,唯一 * 可以做的就是重一个xmlReport(),大量重复report()中的行为,当然, * 现在这个修改还不费劲,拷贝一份report()直接修改就是了。 * 不久,成本中心又要求修改计费规则,于是我们必须同时修改xmlReport() * 和report(),并确保其一致性,当后续还要修改的时候,复制-黏贴的问题就 * 浮现出来了,这造成了潜在的威胁。 * 再后来,客服部门希望修改服务类型和用户重要性指数的计算规则, * 但还没决定怎么改,他们设想了几种方案,这些方案会影响用户的计费规则, * 程序必须再次同时修改xmlReport()和report(),随着各种规则变得越来越复杂, * 适当的修改点越 来越难找,不犯错误的机会越来越少。 * 现在,我们运用所学的OO原则和方法开始进行改吧。 */

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值