SonarLint，idea安装以及扫描bug修复记录2

继续代码质量检查，持续更新遇到的问题

1、Remove useless curly braces around statement

代码：这里采用的java8的新特性

提示：

分析：

有两种方法可以编写包含单个语句的lambda，但其中一种方法肯定比另一种更紧凑、更可读。

请注意，当项目的声纳被激活时，此规则将自动禁用。JAVA来源低于8。

解决：

去掉不需要的括号，根据提示进行去除修改就可以了

list.forEach(definition -> this.routeDefinitionWriter.delete(Mono.just(definition.getId())));

2、Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'

代码：这里我定义的超时时间常量，但是是小写了

private static final Long timeout = 5000l;

提示：

分析：

共享的编码约定允许团队高效地协作。此规则检查所有常量名称是否与提供的正则表达式匹配。其实这里就是检查你起名的规范性，认为定义的常量是遵循'^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$' 规则，所以看你需要吧，为了代码整洁统一，后边的开发人员熟悉，可以统一一下

解决：

根据提示的规范进行修改就可以了

private static final Long TIMEOUT = 5000l;

3、Replace this use of System.out or System.err by a logger .

代码：这是因为我代码中在调试的时候在控制台输出看内容

System.out.println(content);

提示：

 分析

记录消息时，必须满足以下几个重要要求：

• 用户必须能够轻松检索日志
• 所有记录信息的格式必须统一，以便用户轻松阅读日志
• 必须实际记录记录的数据
• 敏感数据必须安全记录
• 如果一个程序直接写入标准输出，则绝对无法满足这些要求。这就是为什么强烈建议定义和使用专用记录器的原因。

解决

其实就是说用这种方式写不优雅，让改成打日志的形式，这就看你需不需了，如果需要就改成日志形式，不需要删除就行了