SonarQube，SonarLint 检测代码修复问题汇总归纳，2021 京东最新 Java 面试真题解析

就我修改过程中碰到了，慢慢总结吧，持续更新。

1.try catch 的 catch 语句中,不要直接输出 e.printStackTrace(),检测会报(squid:S1148）也就是说不能直接打印在控制台，需要以日志的方式输出，这个我以前真的很少注意，但是现在知道了规范，以后写代码的时候可以注意一下。

应该改成

然后说一下 SonarLint,这个和 SonarQube 是一样的，只不过 SonarLint 是单机版的，便于我们自己在本地进行检测修改，安装 SonarLint 插件也很简单，我是使用 eclipse，在 help–Eclipse Marketplace 搜索 SonarLint，然后点击 install ，成功之后，重启 eclipse，然后右键需要检测的项目 SonarLint 就可以了，如图

好了回归正题

2.str.indexOf("/")或者 str.lastIndexOf("/")也是不规范的，会报(squid:S3027)

应该改成 str.indexOf(’/’)，str.lastIndexOf(’/’)

3.Java 7 引入了菱形运算符（<>）来减少泛型代码的冗长。 例如，您现在可以使用<>简化构造函数声明，而不必在其声明及其构造函数中声明 List 的类型，编译器将推断出类型。如下就会报 (squid:S2293)

应该修改成：

List<String> savedPhotosList = new ArrayList<>();

List<String> neededPhotosList = new ArrayList<>();

Map<String,String> typeMap = new HashMap<>();

Map<String,String> typeTableMap = new HashMap<>();

Map<String,String> typeTableMap2 = new HashMap<>();

List<String> typeList = new ArrayList<>();

List<String> typeList2 = new ArrayList<>();

当然 JDK 版本低于 1.7 就不能这样了，老老实实的把类型带上哈哈。

4.如果一个字符串会修改就不要定义 String，需要用 StringBuffer 或者 StringBuilder 不然会报(squid:S1643)，如下是不规范的，但是我们在写代码的时候往往就是这样写的，因为我们经常使用，所以就习惯了，但是这样确实是不好的，会造成资源浪费。

应该改成：

private String handlePhoto(String svalue) {

String []arr = svalue.split(",");

StringBuilder result = new StringBuilder();

for(String item:arr){

if(item.length()==0)continue;

String s = trimPath(item);

checkPhoto(s);

if(result.length()>0){

result.append(',');

}

result.append(s);

}

return result.toString();

}

我觉得这个现象在代码中真的很常见，所以以后写代码的时候还是要多多注意一下规范的。

![在这里插

【一线大厂Java面试题解析+核心总结学习笔记+最新架构讲解视频+实战项目源码讲义】
浏览器打开：qq.cn.hn/FTf 免费领取

入图片描述](https://static001.geekbang.org/infoq/75/756bfdada4f273bcd700ee9392f8de67.png)

这个是一个 bug (squid:S2184)：Math operands should be cast before assignment。我看了 sleep 方法参数是 long 类型的，数学操作数应该在赋值之前进行转换。

所以应该先赋值，然后再传递，这样就没有报错了。

7.如下图：

提示：