|
Avoid long parameter lists.
|
方法参数过多，最多允许5个
|
|
Missing a Javadoc comment.
|
需要注释
|
|
Expected @param tag for 'area'.
|
需要注释
|
|
Expected @return tag.
|
需要注释
|
|
Expected @throws tag for 'Exception'.
|
需要注释
|
|
Unused @param tag for 'licenseInfoMap'.
|
修改注释
|
|
Line is longer than 160 characters (found 162).
|
每一行代码不得超过160个字符，需要分行
|
|
Add a private constructor to hide the implicit public one.
|
常量类和工具类需要一个私有的构造器，同时将该类定义为final，不可继承
|
|
Variables that are final and static should be all capitals, 'logger' is not all capitals.
|
final修饰的变量名称需要全部大写
|
|
Only variables that are final should contain capitals (except for underscores in standard prefix/suffix), '_tar' is not final.
|
只有final修饰的变量名称中才可以有下划线，要么去掉下划线，要么加上final修饰符，Map，List和java对象加上final修饰符后，代表这个对象指向的内存地址不可变，但内容和值都是可以改变的
|
|
Name '_targetDataSources' must match pattern '(^[a-z][a-z0-9][a-zA-Z0-9]{0,50}$)'.
|
变量命名需要符合正则表达式
|
|
The field name indicates a constant but its modifiers do not
|
变量没有final修饰却使用了final变量的命名规则
|
|
Variables should start with a lowercase character, 'SUSPEND_MAP' starts with uppercase character.
|
普通变量（即没有final修饰）需要驼峰式命名，首字母为小写
|
|
More than 8 parameters (found 9).
|
构造方法只能有7个参数
|
|
Constructor has 9 parameters, which is greater than 7 authorized.
|
构造方法只能有7个参数
|
|
Define a constant instead of duplicating this literal "muluList" 6 times.
|
某个字符串出现了4次或以上，需要定义一个常量来代替
|
|
A method/constructor shouldnt explicitly throw java.lang.Exception
|
不能直接抛出Exception，需要精确到具体的异常，且最多抛出4个，超过请try catch
|
|
Define and throw a dedicated exception instead of using a generic one.
|
不能直接抛出Exception，需要精确到具体的异常，且最多抛出4个，超过请try catch
|
|
Remove this useless assignment to local variable "orgnIdL".
|
一个没有使用的变量，删掉即可
|
|
Avoid unused private fields such as 'fastGroup'.
|
一个没有使用的变量，删掉即可。如果不能删，则将private改为protected或default即可。
|
|
Exceptional return value of java.io.File.mkdirs() ignored in ...
|
mkdirs方法有返回值，如果是false则代表文件夹创建失败，需要处理。判断返回值是否正确，并进行处理，如记日志或抛异常等
|
|
Exceptional return value of java.io.File.delete() ignored in ...
|
delete方法有返回值，如果是false则代表文件删除失败，需要处理。判断返回值是否正确，并进行处理，如记日志或抛异常等
|
|
Use "java.nio.Files#delete" here for better messages on error conditions.
|
使用Files.delete(file.toPath())来代替file.delete()
|
|
Suspicious comparison of a Integer reference to constant in ...
|
Integer需要使用equals来比较，这是由于超出(-128~127)范围后，==无法比较，但equals性能较低
|
|
Refactor the code in order to not assign to this loop counter from within the loop body.
|
不要在循环体内操作计数器，重构代码
|
|
Rename "jdbcTemplate" which hides the field declared at line 116.
|
一个类中变量名称重复，重命名变量名称即可
|
|
Field ssoServer has the same name as a method
|
一个类中字段名称和方法名称重复，重命名即可
|
|
Merge this if statement with the enclosing one.
|
多个if可以合并，合并成一个
|
|
Replace the synchronized class "StringBuffer" by an unsynchronized one such as "StringBuilder".
|
StringBuffer需要换成StringBuilder
|
|
A "NullPointerException" could be thrown; "dataResults" is nullable here.
|
在使用对象之前需要对这个对象进行非空判断
|
|
Possible null pointer dereference of track in ...
|
在使用对象之前需要对这个对象进行非空判断
|
|
Update this method so that its implementation is not identical to "getDvStatus" on line 149.
|
两个方法的内容完全一致，不用写两遍，在其中一个方法中调用另一个方法即可。或者删掉一个，全部使用另一个方法
|
|
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
|
空的方法体内部需要写一个注释
|
|
Avoid throwing raw exception types.
|
自定义一个异常来继承RuntimeException，然后抛出这个自定义的异常，而不是抛出通用的RuntimeException
|
|
Define and throw a dedicated exception instead of using a generic one.
|
自定义一个异常来继承RuntimeException，然后抛出这个自定义的异常，而不是抛出通用的RuntimeException
|
|
New exception is thrown in catch block, original stack trace may be lost
|
在catch中抛出异常会使抛出的异常混淆，所以抛出的异常必须带着原异常信息。例如：throw new BusinessException(errorMsg, e)这里第二个参数是必须的
|
|
Avoid empty catch blocks
|
catch代码块不能为空
|
|
Either remove or fill this block of code.
|
删掉或填充这个代码块
|
|
Return an empty collection instead of null.
|
返回空集合Collections.emptyList()而不是null
|
|
Put single-quotes around '-' to use the faster "indexOf(char)" method.
|
将双引号改为单引号，理由是用char来找字符比用String来找效率更高
|
|
Method length is 221 lines (max allowed is 200).
|
一个方法最多200行，试着压缩，实在不行只能拆成多个方法
|
|
Avoid printStackTrace(); use a logger call instead.
|
catch中使用logger.error来代替e.printStackTrace()。Logger logger = LoggerFactory.getLogger(this.getClass())。记住，logger不可以在数据库对象类中使用
|
|
Remove this expression which always evaluates to "true"
|
这个判断语句总是为true，删掉即可
|
|
... concatenates strings using + in a loop
|
在循环中String不要用加号来拼接，而是使用StringBuilder
|
|
Avoid using implementation types like 'LinkedHashMap'; use the interface instead
|
在方法的参数中不要使用具体的实现类对象，而是使用它的接口对象。例如：不用LinkedHashMap而用Map
|
|
Iterate over the "entrySet" instead of the "keySet".
|
遍历Map不使用keySet而使用entrySet
|
|
... makes inefficient use of keySet iterator instead of entrySet iterator
|
遍历Map不使用keySet而使用entrySet
|
|
Move constants to a class or enum.
|
接口中不要定义常量，移到常量类中
|
|
Boxing/unboxing to parse a primitive ...
|
使用parse方法来代替valueOf方法
|
|
Null passed for non-null parameter of ...
|
将一个可能为空的参数传到了一个方法中，但在该方法中，该参数不可为空。修改这个方法，在使用变量前进行非空判断，则这个参数就成为了可为空参数
|
|
Remove this "Integer" constructor
|
不要使用new Integer来初始化变量的值，使用Integer.valueOf
|
|
new com.model.CatalogInfo() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead
|
不要使用new Integer来初始化变量的值，使用Integer.valueOf
|
|
HTTP parameter directly written to HTTP header output in ...
|
resp.setHeader("Content-Disposition", "attachment;filename = " + fileName)改为resp.setHeader("Content-Disposition", "attachment;filename = " + new String(fileName.getBytes("GBK"), "ISO8859-1"))
|
|
Relative path traversal in ...
|
new File(filePath + fileName)改为new File(filePath, fileName)。还有使用new FileInputStream(File srcFile)而不是使用new FileInputStream(filePath + fileName)
|
|
Boxing/unboxing to parse a primitive ...
|
.valueOf改为.parseXXX，这是由于valueOf返回的是包装类对象，而parseXXX返回的是值
|
|
Use try-with-resources or close this "FileInputStream" in a "finally" clause.
|
使用try(FileInputStream inFile = new FileInputStream(file)){System.out.println("...")}catch{}且不需要finally代码块来关闭，这是jdk1.7的新写法，可以将一些可自动关闭的资源写在try()中，就可以直接实现自动关闭
|
|
Change this "try" to a try-with-resources. (sonar.java.source not set. Assuming 7 or greater.)
|
使用try(FileInputStream inFile = new FileInputStream(file)){System.out.println("...")}catch{}且不需要finally代码块来关闭，这是jdk1.7的新写法，可以将一些可自动关闭的资源写在try()中，就可以直接实现自动关闭
|
|
Replace this use of System.out or System.err by a logger.
|
System.out换成logger.info。记住，logger不可以在数据库对象类中使用。
|
|
System.out.println is used
|
System.out换成logger.info。记住，logger不可以在数据库对象类中使用。
|
|
Variable 'auditTypeName' must be private and have accessor methods.
|
只有静态变量才可以用public修饰，所以public改为private或protected或default
|
|
The return value of "valueOf" must be used.
|
这个语句的返回值必须要被使用，删掉或者使用它
|
|
Return value of String.valueOf(Object) ignored in ...
|
这个语句的返回值必须要被使用，删掉或者使用它
|
|
Synchronize this method to match the synchronization on "setSuspendMap".
|
一组get和set方法，如果其中一个方法被synchronized修饰，则另一个也需要使用synchronized 修饰
|
|
An empty statement (semicolon) not part of a loop
|
双分号导致sonar认为两个分号之间有一个空语句，删掉一个分号即可
|
|
Avoid empty if statements
|
if代码块不能为空，删掉或填充代码
|
|
A class which only has private constructors should be final
|
有private构造函数的类需定义为final类
|
|
Class Convert2Track should be declared as final.
|
该类需定义为final类
|
|
Verify this is the key that was intended; it was already set before.
|
该key之前已被塞入过一次，一个key塞两次没有意义，去掉一次
|
|
Extract this nested try block into a separate method.
|
不要在try代码块中再次使用try，去掉内部的，或者把内部的try代码块拉到外面去
|
|
Absolute path traversal in ...
|
直接将文件的绝对路径传到后台，然后生成File对象即会报错。如D:\1.jpg,不可直接传绝对路径，如果知道这个文件会在d盘，可定义常量D:\，然后传值为1.jpg，后台进行拼接后生成File对象即可
|
|
Only one statement per line allowed.
|
每一行只允许一个语句，可能是多写了分号，去掉即可
|
|
Make this anonymous inner class a lambda (sonar.java.source not set. Assuming 8 or greater.)
|
jdk1.8之后推荐使用兰布达表达式。请参考：https://blog.csdn.net/qq_33865313/article/details/81203412
|
|
Non-virtual method call in ... passes null for non-null parameter of ...
|
将一个可能为空的参数传到了一个方法中，但在该方法中，该参数不可为空。修改这个方法，在使用变量前进行非空判断，则这个参数就成为了可为空参数
|
|
A "NullPointerException" could be thrown; "printStream" is nullable here.
|
可能会抛出空指针异常，先进行非空判断
|
|
... could be null and is guaranteed to be dereferenced in ...
|
可能会抛出空指针异常，先进行非空判断
|
|
Unused import - ...
|
引入了一个类，但是没有使用，删掉即可
|
|
2' is a magic number.
|
这是一个魔术数字，需要定义一个常量，然后引用常量，而不是直接使用这个数字
|
|
+' is preceded with whitespace.
|
代码之前不能有空格，删掉空格即可
|
|
;' is not followed by whitespace.
|
代码之后没有空格，需要添加一个空格
|
|
?' is not preceded with whitespace.
|
代码之前没有空格，需要添加一个空格
|
|
Redundant 'private' modifier.
|
这个修饰符是多余的，删掉该修饰符即可
|
|
Incorrect lazy initialization of static field
|
对静态变量的延迟初始化有问题。一般可以试着把static修饰符去掉，或者在静态代码块中进行static变量的初始化。
|
|
... Service not allowed Autowired field ...
|
Service中不允许自动注解Manager,一般Service中都是注解其他Service，删掉这个Manager，换成相同功能的Service
|
|
Unread field: ... should this field be static?
|
final修饰的字段需要加上static修饰符
|
|
member def modifier' has incorrect indentation level 8, expected level should be 4.
|
缩进不对，可以删掉前面的缩进，用Tab键缩进，或者MyEclipse用Ctrl+Shift+F格式化代码
|
|
Change this code to use a stronger protocol.
|
修改编码以使用更强壮的协议，照着给的编码改就行
|
|
for child at indentation level 21 not at correct indentation, 20
|
首行缩进不对，使用Tab键进行缩进
|
|
0 is a valid index, but is ignored by this check.
|
0是一个有效的值，但在这里不能用。一般都是要用特殊的值，比如indexOf应该使用>=0或>-1
|
|
Use already-defined constant '...' instead of duplicating its value here.
|
使用已经定义的常量，而不是在这里复制它的值
|
|
Using the '.*' form of import should be avoided - ...
|
不要使用.*的方式将所有类导入进来，而应该详细的导入每一个用到的类
|
