Java CompletableStage中优雅区分异常类型的实现方案咨询

嘿，我懂这种代码能用但看着别扭的感觉！你当前的实现逻辑是没问题的，但确实可以通过几种方式让它更干净、更符合Java的编码风格，下面给你几个实用的优化方向：

方案1：用handle方法替代exceptionally #

exceptionally更适合单一异常场景的兜底处理，而handle可以同时处理正常结果和异常分支，整体结构会更清晰直观：

public CompletableFuture<DoStuffResponse> doStuff(DoStuffRequest request) {
    return stuffClient
        .doClientStuff(request)
        .thenApply(response -> {
            // 保留你的业务逻辑，比如停止计时器
            timer.stop();
            return response;
        })
        .handle((response, e) -> {
            if (e != null) {
                Throwable cause = e.getCause();
                if (cause instanceof ValidationException) {
                    throw new CustomException("Validation failed doing stuff: " + e.getMessage(), e);
                } else {
                    throw new OtherCustomException("Failed doing stuff: " + e.getMessage(), e);
                }
            }
            return response;
        })
        .toCompletableFuture();
}

方案2：提取异常处理逻辑到单独方法 #

把重复的异常判断逻辑抽成独立的工具方法，让主方法的链式调用完全聚焦核心业务流程：

public CompletableFuture<DoStuffResponse> doStuff(DoStuffRequest request) {
    return stuffClient
        .doClientStuff(request)
        .thenApply(response -> {
            timer.stop();
            return response;
        })
        .exceptionally(this::mapToCustomException)
        .toCompletableFuture();
}

private DoStuffResponse mapToCustomException(Throwable e) {
    Throwable cause = e.getCause();
    if (cause instanceof ValidationException) {
        throw new CustomException("Validation failed doing stuff: " + e.getMessage(), e);
    } else {
        throw new OtherCustomException("Failed doing stuff: " + e.getMessage(), e);
    }
}

这样主方法的代码链清爽很多，异常处理的逻辑也可以单独维护，后续要加新的异常类型也更方便。

方案3：Java 17+ 用模式匹配简化判断 #

如果你的项目已经升级到Java 17或更高版本，可以利用instanceof的模式匹配直接绑定变量，少写额外的赋值代码，让判断更简洁：

.exceptionally(e -> {
    if (e.getCause() instanceof ValidationException validationEx) {
        // 还能直接用validationEx做更多操作，比如取它的特定字段
        throw new CustomException("Validation failed doing stuff: " + validationEx.getMessage(), e);
    } else {
        throw new OtherCustomException("Failed doing stuff: " + e.getMessage(), e);
    }
})

这个小语法糖能让代码更紧凑，可读性也更强。

另外提个小细节：你代码里的StufClient接口拼写少了个f，应该是StuffClient，虽然不影响功能，但统一命名规范能让团队协作更顺畅~

备注：内容来源于stack exchange，提问作者r2d2