摘要：
控制器变成如下（任务类型应该写在常量中）：当前的开发过程是控制器-->；然后转到service调用此任务的业务方法，并从@ServicepublicabstractclassBaseTaskService｛publicabstract JSONObjectsaveTask（Tasktask）；
场景描述：
一个controller中，部门领导有布置任务，查看任务整体情况，查看部门成员，查看部门成员完成情况，导出任务详情，如下：
@RestController @RequestMapping(value = "/task") public class TaskController{ @PostMapping(value = "/saveTask") public JSONObject saveTask(Task task){ ... } @PostMapping(value = "/taskDetail") public JSONObject taskDetail(Task task){ ... } @PostMapping(value = "/memberList") public JSONObject memberList(Task task){ ... } @PostMapping(value = "/memberDetail") public JSONObject memberDetail(Task task){ ... } @PostMapping(value = "/exportTask") public JSONObject 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);
}
}