idea中的代码风险检查和提示，即黄底标注（Caught exception ‘e‘ is immediately rethrown是什么意思）

Posted 2022-10-31 石头wang

背景

挖掘IDEA中有用的提示，当然了，有些提示不是很智能，这需要你进一步查看是否真的有问题，而且给出的修复方案有时候也挺傻的。

解释

对于上图，依次的解释是：
1、给出的修复方案比较傻，把方法的返回值改成void…

2、给出的修复方案是去掉入参直接写常量替换该方法内部的变量，也比较傻。。。

3、给出的解决方案我觉得比较不错，即inline variable，变成一行return ZonedDateTime.parse(dateStr, DATE_TIME_FORMATTER);

4、这个给出的方案比较意外，本以为会生成e.printStackTrace() 之类的，其实不是，给的建议是删除catch(Exception e)
Caught exception 'e' is immediately rethrown

这个给出的方案是删除catch块

**分析：**检查器认为异常捕获之后直接往外抛等价于删掉这个catch块。“删掉catch块” 即异常往上抛给调用者处理，如果最终没有调用者处理，则异常会抛给最顶层即JVM，所以删掉catch块是没问题的，因为两者是等价的

补充

稍微改了下代码，如下

则提示两个catch的处理方式完全一样：catch分支跟’DateTimeParseException’分支完全一样

这时候其实稍微改下，让两者不等，比如简单改下异常提示的字符串，就不再提醒

检查器实际给出的建议是合并catch块，如下：

折叠之后

可以看到，折叠成使用Exception的了

再补充：提示：Boolean method ‘xxx‘ is always inverted

这个提示的意思是你这个方法总是被反着来用的，即加了 !，那为什么不改成 isNotSameDate 将结果反过来呢？

重构的是会让你重新命名，然后会把调用的地方去掉!反过来，如下（方法名是自己决定的非自动）