系统重构对架构上的一些优化思路

前言：最近公司需要对系统进行重构，基于以前微服务系统架构上的不足，我在此总结了一些问题以及优化思路、方案等，适用于Springboot单体项目，希望能对眼前的你提供一些帮助。

问题描述：在系统中充斥着大量的try catch的代码，当接口出现问题时，由于你的粗心少写一个方法的try catch，给前端返回了一串Exception的异常错误信息。

@GetMapping("/get/user")
 public Re auth(){
     try {
         return servie.getUser();
     } catch (Exception e) {
         e.printStackTrace();
     }
     return Re.failed();
 }

解决思路 ：全局异常处理

解决方案：核心为 @RestControllerAdvice、@ExceptionHandler 注解。

@RestControllerAdvice
public class UnityExceptionAdvice {
    @ExceptionHandler(value =Exception.class)
    public final Re exceptionHandler(Exception e){
        log.error("系统异常错误:",e);
        return Re.failed();
    }
}

问题描述：在开发过程中，难免需要业务逻辑判断，直接返回Result结果集确实是一种方案，不过在service层之间调用时还需要处理Result是否成功，代码量剧增。在基于全局异常处理的架构下，抛出异常也是一种常用方案，不过却是在代码中写满了new RuntimeException() 且存在不能返回特定code码的Result结果集的问题。

示例1：
public Re getUser(String name){
    User user = userMapper.selectUser(name);
    if (user == null) {
        return Re.failed(ReCode.USER_NOT_FOUND);
    }
    return Re.success(user);
}

示例2：
public User getUser(String name){
    User user = userMapper.selectUser(name);
    if (user == null) {
        throw new RuntimeException("用户不存在");
    }
    return user;
}

解决思路 ：断言处理

解决方案：核心为Spring提供的工具类 org.springframework.util.Assert ，进行二次封装。此处需要避坑，默认的Assert工具类中的expression逻辑判断都是反着来的，且抛出的异常为new IllegalStateException()，以下代码的Asserts工具类中改为正向的处理，为自定义异常，具体可查看Assert源码。

public class Asserts extends Assert {

    
    public static void state(ReCode reCode) throws AnywayException{
        throw new AnywayException(reCode);
    }

    
    public static void state(boolean expression, ReCode reCode) throws AnywayException{
        if (expression) {
            throw new AnywayException(reCode);
        }
    }

    
    public static void state(boolean expression, ReCode reCode, String errorMsgTemplate, Object... params) throws AnywayException{
        if (expression) {
            throw new AnywayException(reCode, StrUtil.format(errorMsgTemplate, params));
        }
    }
}

自定义异常类：

@Data
@EqualsAndHashCode(callSuper = true)
public class AnywayException extends RuntimeException {

    
    private final ReCode reCode;

    
    public AnywayException(ReCode reCode) {
        super(reCode.getMsg());
        this.reCode = reCode;
    }

    
    public AnywayException(ReCode reCode, String msg) {
        super(msg);
        this.reCode = reCode;
    }
}

示例：

public User getUser(String name){
     User user = userMapper.selectUser(name);
     Asserts.state(user == null, ReCode.USER_NOT_FOUND);
     return user;
 }

问题描述：在数据响应的时候，我们都会进行一层包装，一成不变的返回Result结果集数据，此响应可能会在Service层就会包装，也可能在Controller层进行包装处理。对于一些新手玩家规范不到位，如果在Re结果集上不加泛型的话，会增加代码阅读的复杂度。

// 示例1:
@GetMapping("/get/user")
public Re auth(){        
    return servie.getUser();
}

// 示例2：
@GetMapping("/get/user")
public Re auth(){        
    return Re.success(servie.getUser());
}

解决思路 ：全局统一响应处理

解决方案：与全局异常处理一样，其核心为 @RestControllerAdvice 注解与 ResponseBodyAdvice 接口 。

@RestControllerAdvice
public class UnityResponseAdvice implements ResponseBodyAdvice