idea中的代码风险检查和提示,即黄底标注(Caught exception ‘e‘ is immediately rethrown是什么意思) 您所在的位置:网站首页 idea代码黄色背景是为啥 idea中的代码风险检查和提示,即黄底标注(Caught exception ‘e‘ is immediately rethrown是什么意思)

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

2024-07-08 17:33| 来源: 网络整理| 查看: 265

背景

挖掘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 将结果反过来呢?

重构的是会让你重新命名,然后会把调用的地方去掉!反过来,如下(方法名是自己决定的非自动) 在这里插入图片描述



【本文地址】

公司简介

联系我们

今日新闻

    推荐新闻

    专题文章
      CopyRight 2018-2019 实验室设备网 版权所有