如何使用全局异常处理器捕获异常？Spring控制器异常处理疑问

你现在的写法确实没必要保留Controller里的try-catch，完全可以交给全局异常处理器来统一处理，原因如下：

1. 全局异常处理器的核心作用 #

@ControllerAdvice标记的GeneralExceptionHandler类就是用来统一处理整个应用中Controller层抛出的异常，它会自动拦截所有Controller方法抛出的ApiException，并执行对应的@ExceptionHandler方法生成响应。这种设计就是为了避免每个Controller接口都写重复的try-catch逻辑，减少代码冗余。

2. 当前代码的冗余问题 #

你现在在Controller里手动捕获ApiException后又调用了GeneralExceptionHandler.handleExceptions(e)，相当于把全局处理器的逻辑手动执行了一遍，而且还重复打印了日志（Controller里打了error级别的异常日志，全局处理器里又打了info级别的日志），完全是多余的操作。

3. 优化后的Controller代码 #

去掉try-catch块，直接调用服务层方法即可，异常会自动向上抛给全局异常处理器处理，代码更简洁清晰：

@GetMapping("/departments/{departmentId}")
public ResponseEntity<Object> getDepartmentById(@PathVariable("departmentId") Long departmentId) {
    logger.debug("Entering get item by id endpoint");
    DepartmentDTO departmentDTO = departmentService.getDepartmentById(departmentId);
    logger.info("Returned department with id: " + departmentId);
    return ResponseEntity.ok().body(departmentDTO);
}

补充：日志处理的建议 #

如果需要记录异常相关的日志，建议统一在GeneralExceptionHandler中处理，保持日志逻辑的一致性；如果这个接口有特殊的日志需求（比如需要记录更详细的上下文信息），可以在Controller里保留日志代码，但依然不需要捕获异常——异常会被全局处理器自动拦截，不会影响接口返回正确的响应。

内容的提问来源于stack exchange，提问作者Leobejj