不断更的 Code Review 记录

悠悠 2023-09-29 15:44 9阅读 0赞

此文章用来持续记录笔者参与的 Code Review 会议纪要，欢迎阅读斧正。

### 服务拆分 ###

主数据系统作为提供方要尽可能满足业务系统的数据要求，主要有两个目的：

1.  方便调用方使用，比如算金额：两个字段分别为数量和单价，主数据系统可以将两个字段返给调用方，但是更推荐提供方计算好提供给调用方使用
2.  减少调用方对提供方的网络调用次数
3.  如果提供的内容会被多个调用方按照同类规则组合使用，则由提供方统一封装，提高复用性。

### 是否存在 ###

如果有按照条件查询会有很多条结果而只需要判断有没有相同的，此时使用 SQL 限制为只取一条，对 JVM 友好

### 方法命名 ###

总的宗旨是见名知意，但是要注意不要加 Domain 对象名称（比如 StudentService 中有个方法为 `modifyStudentById`，此时就没必要有 `student`，改为 `modifyById` 即可）

### 方法抽取 ###

如果方法是针对某个 Domain，可以放在 Domain 的方法中，提高复用，方便测试。

### 分库分表 ###

2 次方（待完善）

### 服务调用 ###

比如有一个服务需要调用三个基础系统的接口，这个时候可以使用 `CompleteFuture` 来「异步」调用。

> 注意线程池 executor 使用

public UserInfo getUserDetail(Long userId) throws Exception {
        
        final UserInfo userInfo = new UserInfo();
        
        CompletableFuture serviceOne = CompletableFuture.supplyAsync(() -> {
        
            serviceOneMethod(id, userInfo);
            return Boolean.TRUE;
        }, executor);
        
        CompletableFuture serviceTwoMethod = CompletableFuture.supplyAsync(() -> {
        
            serviceTwoMethod(id, userInfo);
            return Boolean.TRUE;
        }, executor);
        
        CompletableFuture.allOf(serviceOneMethod, serviceTwoMethod).join();
    
        serviceOneMethod.get();
        serviceTwoMethod.get();
    
        return userInfo;
    }

### SQL 相关 ###

group\_concat 函数有长度限制，可以配置