sonarqube 扫描报告-苏州华克斯公司

sonarqube问题修改总结

一、代码复杂度问题

1、类的复杂度过高。

说明：split this class into smaller&more spe---lized ones to reduce its dependencies on other classes from 24 to the maximum authorized 20/less.

原因：类之间的耦合度过高，sonarqube 扫描报告，引用了太多其他的类，

错误示例：

修改建议：根据面向对象的单一职责进行设计，进行降耦。

2、多重循环嵌套

说明：代码块中包含多个if、for、while、switch，sonarqube 扫描报告，影响阅读，难以维护

错误示例

sonarqube问题修改总结

三、可读性问题

1、用isempty()去做容器的空或者非空判断。

说明：use isempty() to check whether the collection is empty/not.

原因：增加可读性。

2、删除没有用的import、field、class、method。

原因：增加可维护性。

修改建议：删除无用代码块、申明、方法和引用。

3、if语句被拆分不利于可读

说明：merging collapsible if statements increases the codes readability.

3、方法内多个return出口

错误示例：

解决建议：定义一个return，赋值，统一返回。

4、修改了入参说明：修改了入参降低了代码的可读性，丢失了原始参数的值。

错误示例：

5、用equalsignorecase()方法替代字符串的---大小写比较。

说明：replace these touppercase()/tolowercase()&equals() calls with a single equalsignorecase() call.

原因：参数应当是final的

解决建议，利用一个临时变量充当需要变动的参数。