SonarQube，SonarLint检测代码修复问题汇总归纳

当然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();

}

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

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

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

7.如下图：

提示：

意思是不能按照类名进行比较，不要求类名是唯一的，只要它们在包中是唯一的。 因此，尝试根据类名确定对象的类型是一种充满危险的练习。 其中一个危险是恶意用户将发送与受信任类同名的对象，从而获得可信访问。

进行修改，如下：

private static boolean isTypeMatch(Class<?> one, Class<?> two) {

if (one.equals(two)) {

return true;

}

if (one.isPrimitive()) {

if (one.isAssignableFrom(int.class) && two.isAssignableFrom(java.lang.Integer.class)) {

return true;

}

if (one.isAssignableFrom(long.class) && two.isAssignableFrom(java.lang.Long.class)) {

return true;

}

if (one.isAssignableFrom(float.class) && two.isAssignableFrom(java.lang.Float.class)) {

return true;

}

if (one.isAssignableFrom(double.class) && two.isAssignableFrom(java.lang.Double.class)) {

return true;

}

if (one.isAssignableFrom(char.class) && two.isAssignableFrom(java.lang.Character.class)) {

return true;

}

if (one.isAssignableFrom(byte.class) && two.isAssignableFrom(java.lang.Byte.class)) {

return true;

}

if (one.isAssignableFrom(short.class) && two.isAssignableFrom(java.lang.Short.class)) {

return true;

}

if (one.isAssignableFrom(boolean.class) && two.isAssignableFrom(java.lang.Boolean.class)){

return true;

}

}

return false;

}

---

欢迎大家关注个人公众号
分享各种学习资料，包含java，linux，大数据等。资料包含视频文档以及源码，仅供学习交流和分享，不涉及任何商业用途。同时分享本人及投递的优质技术博文。