当前位置：首页 > news >正文

@SneakyThrows不合理使用，是真的坑

news 2025/7/28 13:25:06

public static void main(String[] args) {int a =1;int b = 2;String result = getResult(a, b);System.out.println(result);}@SneakyThrowspublic static String getResult(Integer a,Integer b){if (a.equals(b)){return "成功！";}else{throw new InterruptedException("失败！");}}

报错了

SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/D:/maven/jp_maven/ch/qos/logback/logback-classic/1.2.3/logback-classic-1.2.3.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/D:/maven/jp_maven/org/slf4j/slf4j-simple/1.7.32/slf4j-simple-1.7.32.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [ch.qos.logback.classic.util.ContextSelectorStaticBinder]
Exception in thread "main" java.lang.InterruptedException: 失败！at com.juepeiscm.express.controller.openapi.TrackRePushOpenController.getResult(TrackRePushOpenController.java:87)at com.juepeiscm.express.controller.openapi.TrackRePushOpenController.main(TrackRePushOpenController.java:78)

> 1. 安全性和异常处理

问题: @SneakyThrows
注解虽然可以简化异常处理，但它隐藏了异常处理的细节，可能会导致潜在的错误被忽略。此外，InterruptedException
是一个受检异常，通常用于线程中断，这里使用它可能不合适。

建议: 使用更合适的异常类型，并显式地处理异常。

修改后的代码:

  public static void main(String[] args) {int a = 1;int b = 2;try {String result = getResult(a, b);System.out.println(result);} catch (Exception e) {System.err.println("发生错误: " + e.getMessage());}}public static String getResult(Integer a, Integer b) {if (a.equals(b)) {return "成功！";} else {throw new IllegalArgumentException("失败！");}}

2. 逻辑Bug

问题: a.equals(b) 可能会导致空指针异常，如果 a 或 b 为 null。

建议: 在调用 equals 方法之前，先检查 a 和 b 是否为 null。

修改后的代码:

  public static void main(String[] args) {int a = 1;int b = 2;try {String result = getResult(a, b);System.out.println(result);} catch (Exception e) {System.err.println("发生错误: " + e.getMessage());}}public static String getResult(Integer a, Integer b) {if (a == null || b == null) {throw new IllegalArgumentException("参数不能为空");}if (a.equals(b)) {return "成功！";} else {throw new IllegalArgumentException("失败！");}}

3. 性能效率

问题: 使用 Integer 类型的 equals 方法进行比较，虽然在大多数情况下性能影响不大，但使用基本类型 int 的比较会更高效。
建议: 将 Integer 类型的参数改为 int 类型。

修改后的代码:

  public static void main(String[] args) {int a = 1;int b = 2;try {String result = getResult(a, b);System.out.println(result);} catch (Exception e) {System.err.println("发生错误: " + e.getMessage());}}public static String getResult(int a, int b) {if (a == b) {return "成功！";} else {throw new IllegalArgumentException("失败！");}}

4. 可维护性

问题: 代码中的异常信息和返回值都是硬编码的字符串，这可能会导致维护困难。

建议: 使用常量来定义这些字符串，以便于管理和修改。

修改后的代码:

  public static final String SUCCESS_MESSAGE = "成功！";public static final String FAILURE_MESSAGE = "失败！";public static void main(String[] args) {int a = 1;int b = 2;try {String result = getResult(a, b);System.out.println(result);} catch (Exception e) {System.err.println("发生错误: " + e.getMessage());}}public static String getResult(int a, int b) {if (a == b) {return SUCCESS_MESSAGE;} else {throw new IllegalArgumentException(FAILURE_MESSAGE);}}