西南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，sonarqube 扫描报告，赋值，sonarqube 扫描报告，统一返回。

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

错误示例：

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

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

原因：参数应当是final的

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

sonarqube安装插件

通过web页面可以安装各种插件

目前有如下插件：

1、代码分析插件

2、其他语言：css   erlang     groovy    lua    puppet

3、外部分析工具集成

aem rules 、android lint、checkss 、findbugs 、jdepend 、pmd 、sonargraph 7、sonargraph integration 、xanitizer