|
5 H S- u; C4 s0 ^
原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
$ v$ v1 |5 G2 I5 b1 u* b6 ~& a; X" c/ g) p
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。
U) y9 ]: r/ K5 a+ s; z 工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力
3 g' |& R; V9 u# S9 z# {; v( | 一、Alibaba Java Coding Guidelines % S' B% y0 }4 v% O2 c0 \
1、整体介绍: + f5 A5 u* K) V$ e3 _$ L
Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。 ' @4 n: b1 l& S
阿里巴巴规约扫描包括:
4 X1 J6 W4 \& @: D# |; ~0 Q2 c
3 S+ _0 E2 M/ ?+ {4 b OOP规约( C! J; {5 y; G" S
并发处理
) _7 i- Z- T# |0 X/ Q1 T1 `; o 控制语句
- G$ P1 p4 V, Q6 W5 W6 A1 V/ Y 命名规约$ Z& q7 D+ D8 G# @! F$ z1 M, {0 l3 _
常量定义
! o7 T8 z; a5 g9 l 注释规范5 e# C$ K% ?9 n# v7 H4 w' `6 O
8 E8 W G2 l3 C, S' N/ B
2、安装步骤:
; t, G8 E4 m" s5 } File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。
8 E4 A; h! Z! x& v( {( [ 3、使用说明:
8 j: l! v% a+ w) o' N 3.1、运行方式:
4 u* z! f7 `* J' ]% F4 l" f (1)可以Tools > 阿里编码规约 > 编码规约扫描 % n) F" y$ \3 b( ^$ M
 8 x. F% A( v& X# H3 _
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可: 5 o1 b: p2 e0 w3 B' D3 R

; V; }" R8 K7 t 3.2、菜单功能: # @6 F% w' j" C7 \2 O
$ g+ n% [ d6 S 编码规约扫描:开始扫描代码
/ x- E5 B# K$ a5 Q' q7 r, w 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能; d; h$ e- L1 F6 g1 @
切换语言至英文:中英文切换4 Z/ d0 z, Q9 l# R" J
$ x; y8 o' m, |9 ~. r" Y' v
3.3、运行结果: ) w& f8 E& l4 }5 n2 y9 U
扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。
+ Q8 Y/ e0 ]7 Q8 A! n% q 
* d- U+ j% e# F" r4 m3 B V# C$ A 选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到): & u6 l2 X6 W% f8 s' P y: S3 A& Y

5 i9 c d: k* k5 v (1)指定区域搜索同一类问题: / N6 g+ @3 o# f6 Q8 k6 V) {6 P
当点击③处的按钮时,会弹出如下按钮:
) d; D' W# o9 D: V# @ 
6 t& F" g' l v# D( O* V8 U 这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示:
# C# S: I0 M# d. G$ Y  9 A- M9 l9 q' y. i5 ^* B
这里我们可以看到,显示了整个Project中的所有该类的问题。
D/ q7 _8 U( ` V- g (2)预览具体的不规范代码: & y( e5 T( g2 Q# r& Q0 R
如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。 0 W# R+ s8 {; }6 w

6 p$ c' F u6 E 3.4、工具栏功能介绍:
6 Y8 \5 V% D+ u y 
7 ^3 d# x% {$ i/ d% C# U! v2 a) C6 T" A) N9 B/ }- B
Rerun Inspection:重新运行一次扫描1 h. I5 ?; {$ m/ E
Close:关闭真个AJCG面板
) q9 d3 [& A# d9 n' O0 a Expand All:展开结果的树状结构,整个结果是树状结构的。1 u3 W& J% M8 `$ h }1 J
Collapse All:收起结果的树状结构
3 S6 O, d( a) c5 m8 o Go Pre Problem:选择上一个问题
# p' p0 Z" h& a# ?/ Q Go Next Problem:选择下一个问题, b: m& U- N5 b H& @+ y. ?
Help:帮助. D+ t0 m) P; y$ Q7 p( K
Group by Serverity:(不知道如何描述)
3 P1 x: p9 q5 G: F Group by derectory:按目录分组/按类名分组间切换
* W7 w3 V* I, T Filter resoled items:过滤掉已经解决的项
# b6 |; E' B0 A6 B Autoscroll to Source:自动滚动到源码2 a0 [: c% Y$ V, ]
Export:导出,可以导出为XML和HTML两种格式
; ?; z5 Q6 R8 T/ N* b2 W2 | Edit Settings:编辑设置
, H2 M3 u7 t$ b( [6 R
, {, ^; O0 w# m j/ w, E4 _; Z, `- P4 `4 s 二、CheckStyle:
: \9 ?2 D, K# d) M 1、整体介绍:
* b3 h" q V* H& ]' h CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
$ V% _! l+ F8 B Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。 : i4 h5 E) c) L _* v( c% A4 A
2、安装步骤:
4 m1 p3 }2 G) m$ z 通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 $ N5 c" S' v# g g
3、使用说明:
/ c1 t$ {: G6 e8 K  1 x! z% O3 B6 Q) L0 W+ _3 s# Y
可以看到基本都是一些缩进啥的编码规范,可以不用太关注
) F9 _0 }0 l8 i. L. v" _ 三、PMD 9 Q4 c! x% {5 z; P5 l: q) i
1、整体介绍:
3 q, j3 |9 I$ A( Z7 M4 T2 B& G PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容: 4 r( M7 H m; s0 _* s
( b) A. L+ A$ l7 h% l' A4 Y
潜在的bug:空的try/catch/finally/switch语句
$ m$ k1 T4 e+ B 未使用的代码(Dead code):未使用的变量、参数、私有方法等
' L5 X- z" Z+ G; d, l( M 可选的代码:String/StringBuffer的滥用
: E& D; a! ?. k; v 复杂的表达式:不必须的if语句、可被while替代的for循环
: O( Y+ f& K/ P2 `$ P' }; h5 T+ B 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs
9 U5 X. h& x5 C# ]. _, Y' F 循环体创建新对象:尽量不要在循环体内实例化新对象
: r# ]* c8 }6 P5 l4 k. N/ s 资源关闭:Connect,Result,Statement等使用之后确保关闭掉
% Z4 o) S! [8 ?. }4 F* a9 o; v* e" C0 ?/ x. k
2、安装步骤:
8 }3 V) W& @" J5 u8 ?- Y, P/ h: ? 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可 2 v5 O0 J- c4 M5 t- F/ ^
3、使用说明: p8 {, x- I, ~; Q. Y. @- U+ [% Z6 b
# B/ p; p+ B; @" ?) o$ F$ ]
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html
" t* H4 R5 k3 d( V3 d
: b1 Q1 Z9 s$ C 3.1、运行方式: " ]$ ?+ C& \/ H
(1)从Tools菜单中启动: 3 N( @% Y* [% e- {. y9 ?2 ?2 _
通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。 $ I) l% x! Q2 k& [
& X* y) o; k. d1 C: @
Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。
- O' I2 U2 g: `5 s/ b2 ~ P5 y Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。8 g! W) p/ O g& L( p! Q. p* e' ~& ?
: [8 h# q+ M" o1 ^0 ~- c5 ^+ L (2)从右键菜单中启动: 2 c4 c7 |5 e1 t" p" X, ~
在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 8 M! P m3 a7 E+ k$ b' `
3.2、运行结果:
% r W; J5 n4 u0 h1 } E) K8 E 
1 I5 w4 i# }8 E: J: D5 M/ m8 } 运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 + q+ ]! O! }+ @& q
3.3、配置检测规则:
3 Q# O2 J& `5 G; r5 Y 通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面: & B$ m" T3 i! @$ V
 3 c' p- Z' ~& B" p% j
在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。
' D8 K: u6 I7 \. ]; m0 }- \4 U 点击“Options”选项卡,在其中可以配置一些检测规则选项:
" M, b2 t0 F- y8 `! M7 f" \  7 y% q5 [3 p8 P
其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。 0 Q- b# W3 R& ]& C3 M
四、FindBugs:
! Y: K1 ^$ B) L& O6 g( O' J$ b 1、整体介绍: " M/ c0 [9 g7 A* I, L1 E6 Q
FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考
# k! y1 W- o$ ^0 N9 m+ z: E6 ^+ `" f 2、安装步骤: ; m& F1 h/ h' m
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可 k2 M/ G+ R* I: t5 w! V
3、使用说明:
+ V! B/ L/ X, g Y! e1 P! K FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
; P7 y. N& Y; |' I4 a 
) q- }- B0 d- @* m7 ?, K 分析完之后就会出现结果面板
- l/ N$ L' ^2 x: _ 
6 P+ j5 Y0 m' ~4 z9 R, `# L0 S# R 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行 0 N$ u- {5 C& U$ F
8 W3 v+ d& a$ k3 B* _) u 4、附:常见的错误信息
6 A4 P& b1 g4 ?" j 4.1、Bad practice 代码坏习惯:
7 U$ r, j% h% B4 M5 |, f
1 O6 Z( N3 }3 ~1 q S# _ 图片 / J' k6 s6 u$ Z* }# g5 M: g4 ]* O# a
4.2、Dodgy code 糟糕的代码:
/ ~* i& R: t/ l" ~& H. Q( k
' u- i$ |/ }# E/ P$ E 4.3、Internationalization 代码国际化相关:
/ X/ h" ]8 v" `( j 4.4、Performance 代码性能相关: , z" o3 u5 G/ g- T% S$ h* c; L& m3 l9 I
6 I, W" H+ ^* P( A: P- e- q1 r
4.5、Experimental: % }3 P& O9 g$ _ S/ T
, g# H6 R: J" P% x 4.6、Malicious code vulnerability 恶意破坏代码相关:
: }! b/ I% K/ c+ g# V# i. B1 \+ n: |
4.7、Multithreaded correctness 多线程代码正确性相关:
, F. r9 q' Q% f8 G 4.8、Correctness 代码正确性相关:
/ V3 z7 y6 _( I! Z- y& N3 J; M/ _0 o6 B7 s! R
五、SonarLint:
3 b* {+ N& l- ? E% q6 ? 1、整体介绍:
4 z* h G1 C# [" t sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
2 c8 j5 b' \# A/ S$ m 2、安装步骤:
: u; a9 h& s4 t- q) E+ o 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
3 d1 @8 ]$ s8 f( {* i/ g 3、使用说明: i3 I/ z3 b+ [; Z3 }9 x2 c
/ I9 s( H0 M f7 i4 E7 @ 右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。 2 b9 Z* R# [! e# W k, _6 F) g6 H
! O; A; d; U( p& W, K6 H- J( N 4、配置 SonarLint 服务端:
; m6 v( @! G. b# ^ 4.1、配置 Sonar 服务器: 1 o7 T& N$ u5 y. G- m3 Y
sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查:
7 }' U3 }$ _" C0 p2 G" R
0 A- g$ ^- q" f1 V8 n 点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码
" f' m3 O' z. N8 {6 c# J& F2 o
: E+ T! X# E$ v* a# S0 O 4.2、具体 Sonar工程配置:
F8 D7 X `' v" J& Y 配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: 3 T" m* p" J: U y# b8 c
1 B( ]) H2 h6 R
4.3、使用 SonarLint 检查: $ y4 `& C2 g6 }4 h# j. N, g/ o/ _
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查
7 @! H5 ^. k; b+ y7 d4 E: x% x1 H8 `2 m1 i6 j" T! p5 c9 Z
总结
& G- t! x8 c% [; C$ U3 ?
; M4 c/ O; I8 W* } 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了7 r, j( a C3 ?( s
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:
e% _# w0 f" z/ m' ^/ X( @ PMD 自定义能力强,用来自定义项目BUG规则非常好用
% _" I/ q4 u! i Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。. m3 T; F0 ?) O
SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景9 g8 `3 b/ Q6 W% s
2 S, Y" U; G7 A/ D- @ 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多 6 L0 @2 F5 V( y% X
0 I0 j0 u: k9 Q8 g$ J5 z, F) N 责任编辑:
2 m' l$ Y! U: _) u6 m6 a/ u% x# q. P3 j, P8 a
( v; w! e J9 o( I7 @6 V( l! ]) G0 J& n" B
6 w3 x& ?5 g8 k
|