记一次多租户项目的后端代码重构优化 1、背景 2、重新梳理设

1、背景

分享一次代码优化的案例,一个多租户项目,基于数据库字段进行租户隔离,多租户之间用的同一个后端服务,使用MVC架构,在经过数次迭代之后,逐渐发现该项目已存在严重问题:租户之间业务代码耦合度太高。

  1. 虽然租户之间大多业务场景类似,但也有不同,用if-else判断租户进行不同的业务逻辑处理。
  2. 增加测试成本,某一租户的业务逻辑需要修改,同时使用到这一段代码的所有租户都需要进行测试。
  3. service臃肿,一个service 1000多行代码,包含了所有租户的业务逻辑,不利于后续维护和扩展。

2、重新梳理设计

  1. 在DDD领域驱动设计中,负责处理业务基本规则的是领域层,而不是应用层,我们可以借鉴这种思想,把各个租户的业务逻辑内聚在各自的租户下,做到各个租户之前代码的相互隔离,互不影响。
  2. 根据依赖倒置原则,抽象不应该依赖细节,细节应该依赖抽象,对于一些租户公共的业务逻辑行为,提取为抽象类,具体的细节差异由各个租户各自实现。
  3. 工厂模式,根据不同的租户拿到对应的租户对象,进行对应的业务处理逻辑。
    image.png

image.png

3、重构代码示例

controller

1
2
3
4
5
6
java复制代码@PostMapping("/infoByNamePage")
@ApiOperation(value = "分页查询", notes = "分页查询")
public R<Page<OrderInfoPageVO>> infoByNamePage(@Valid @RequestBody OrderInfoVoRequest orderInfoVoRequest) {
OrderAction orderAction = OrderFactory.getOrderAction(CurrentUserUtil.getTenantId());
return orderAction.infoByNamePage(orderInfoVoRequest);
}

其中OrderAction是顶层接口,拥有租户所有的业务行为

OrderFactory工厂实现

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
java复制代码public class OrderFactory implements ApplicationContextAware {
// 租户A
private static final String ORDER_TENANT_A = "A";
// 租户B
private static final String ORDER_TENANT_B = "B";

@Autowired
private OrderTenantConfig config;

private static Map<String, OrderAction> orderActionMap = new HashMap<>(2);

/**
* 根据租户id获取对应租户
**/
public static OrderAction getOrderAction(String tenantId) {
OrderAction orderAction = orderActionMap.get(tenantId);
if (ObjectUtil.isEmpty(orderAction)){
throw new OrderBusinessException(OrderCodeEnum.TENANT_ID_IS_NULL);
}
return orderAction;
}

@Override
public void setApplicationContext(ApplicationContext applicationContext) throws BeansException {
// AOrderAction、BOrderAction 两个租户各自的实现类
orderActionMap.put(config.getMap().get(ORDER_TENANT_A), applicationContext.getBean(AOrderAction.class));
orderActionMap.put(config.getMap().get(ORDER_TENANT_B), applicationContext.getBean(BOrderAction.class));
}
}

image.png
重构之后,1000多行的service业务逻辑主要分布在了AbstractOrderAction、AOrderAction与BOrderAction三个类中,AbstractOrderAction也500多行代码,各自租户之间完成了解耦,更利于后续的维护和扩展。

4、思考与总结

这种设计并不适用于业务场景复杂且过多的场景

其中OrderAction顶层接口中包含了所有业务行为,如果业务一旦复杂过多,会导致AbstractOrderAction抽象类过度臃肿,进而又形成了一个1000多行的’service’,只能再度进行拆分重构。总之根据项目业务的体量做最合适的设计。没有最好的,只有最合适的,最合适的也就是最好的

若有错误,还望批评指正

本文转载自: 掘金

开发者博客 – 和开发相关的 这里全都有

0%