捕获SQLException后抛出并捕获新Exception的合理场景咨询

结论先行：这段代码没有任何合理的应用场景，属于冗余且错误的异常处理写法。 #

问题拆解 #

• 无意义的嵌套try-catch：抛出新Exception后立即捕获，除了多打印一次栈轨迹（e1.printStackTrace()），没有任何实际价值——既没有改变异常的传播路径，也没有补充任何有效处理逻辑。
• 异常信息丢失：新抛出的Exception没有将原SQLException作为异常原因传入（正确写法应为throw new Exception("xxx", e)），导致无法追溯问题根源，排查故障时会非常麻烦。
• 冗余日志打印：已经对原SQLException做了栈打印和日志记录，再次打印新异常的栈信息属于重复操作，只会增加日志噪音。

为什么会出现这种代码？ #

• 大概率是开发者对异常处理逻辑理解混乱：比如想将SQLException转换为通用异常抛出，但误操作成了“抛完立刻抓”，完全没实现异常向上传递的目的。
• 可能是IDE自动生成代码后未清理：比如IDE自动补全了catch块，但开发者没有删除或补充有效处理逻辑，留下了无意义的占位代码。

正确的替代写法 #

场景1：需要将异常向上抛出，让上层调用方处理

} catch (SQLException e) {
    log.error("SQLException occured in getSeverityList()", e);
    // 用RuntimeException包装原异常，保留异常链，同时避免受检异常的强制声明
    throw new RuntimeException("Error In getSeverityList(): " + e.getMessage(), e);
} finally {

场景2：在当前方法内完成兜底处理，不向上抛出

} catch (SQLException e) {
    log.error("SQLException occured in getSeverityList()", e);
    // 这里执行兜底逻辑，比如返回默认的 severity 列表、清理临时资源等
    return Collections.emptyList();
} finally {

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