) b. E( q. H) ^8 ]( O% U O/ G k: J 原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
r. n* ]( V! b. M" q. x; Z9 B$ P: W& ?
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。
% ]- @) x( }! b8 U \# o( Z( H 工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力 : \; j6 y5 r8 g8 b; ?* c
一、Alibaba Java Coding Guidelines 1 |) |9 B' M3 g3 M
1、整体介绍:
' o" {0 D# g P& j1 v Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。 " g0 K/ ^7 m) n( {
阿里巴巴规约扫描包括:
: A7 B( D- |+ B7 D- P
: b. W. e l- B! _& F OOP规约
2 f* N. p1 k, Z9 b4 W4 a! b 并发处理
/ o5 o7 {+ G0 b1 ^ 控制语句0 B# t( t9 ?: [) U, D* z; v. b
命名规约
4 i7 A6 Z( K. o4 | 常量定义
5 G9 {7 o Y2 _* K 注释规范& ^) ^9 ^7 S6 F! |! g0 v$ E
9 `) J. H' [, L0 K) Q
2、安装步骤: # i( Z8 H4 a4 N8 M& f5 a* K2 m2 s; [
File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。
0 l" B% \6 u% s% q2 ] 3、使用说明:
" g$ V2 ~) d3 U9 u4 R7 d4 X 3.1、运行方式: 7 S+ x, |2 R3 q2 H0 l
(1)可以Tools > 阿里编码规约 > 编码规约扫描 / z0 g0 o8 z1 B; _
 8 _- E/ N/ V/ _. `; r% M0 R7 Y( t
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可: r9 w; Z2 R! h
 & A4 X* V# p. H6 W, G) f
3.2、菜单功能:
. X! d& V& y! {( u% e+ I% i
3 A% g. G- Z' b/ M q 编码规约扫描:开始扫描代码
# l% U- s. J: S4 @. p. u 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能; ~8 l0 S7 W0 F0 o- S
切换语言至英文:中英文切换
' e; l$ @) ]) ~6 ^. H9 n3 t2 A/ L. f' D0 |& x$ K4 U8 V' J
3.3、运行结果: 9 r* D9 A+ p& B h( T' `& G, L
扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。
" v! I, r) S, J. |, R  2 d: o8 R, n; [& h/ Q1 o! h3 l
选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):
$ y; g& ^; G- A. z( l, F 
. Y4 B2 s$ O! v0 y" J H (1)指定区域搜索同一类问题:
/ p9 |! X1 Z- C' l3 @ 当点击③处的按钮时,会弹出如下按钮: ( y4 @# H/ C' F

( A2 Y5 K0 h1 I! t. v/ f 这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示: ( F* H4 I4 U* V# w4 e) V
 6 s: l/ X. \2 l+ z
这里我们可以看到,显示了整个Project中的所有该类的问题。
$ a% U q+ J( p" \2 T: P3 f1 r (2)预览具体的不规范代码: " u& p. m7 `; v
如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。 7 D3 |* P Z5 z( A* @( o f
 * T% E# [3 z5 a8 y+ w. R8 F! L8 _
3.4、工具栏功能介绍: - m0 ?: _* r7 {; D h4 M# p
 9 e$ E6 j2 C+ J8 `, y- x3 M. I) \" u, Z; \
9 v) y6 ^. G( H- Q Rerun Inspection:重新运行一次扫描
Z# ~* x5 D0 |3 P Close:关闭真个AJCG面板( S# T; s& |4 m0 {( ]
Expand All:展开结果的树状结构,整个结果是树状结构的。8 _6 j/ [7 _2 I& B' E
Collapse All:收起结果的树状结构" h6 D" U7 k" A
Go Pre Problem:选择上一个问题
1 Q4 m. S( n% B( ~0 Z Go Next Problem:选择下一个问题# I! i& \/ _3 ?# `# I* Y% C
Help:帮助 c$ ~! s& `. ~* `
Group by Serverity:(不知道如何描述)2 \2 y" T1 t$ O+ f: M1 C
Group by derectory:按目录分组/按类名分组间切换, u4 x7 q" ?; R* L# I2 p& o- z- y4 u
Filter resoled items:过滤掉已经解决的项
( y/ Z x) z7 t* ?* m9 S Autoscroll to Source:自动滚动到源码% [) d; j& `1 E; z! F* g
Export:导出,可以导出为XML和HTML两种格式1 p# u( W' o0 |9 ?
Edit Settings:编辑设置$ Q0 k# c2 W+ [( L8 r
( z; F/ l4 S& W/ h- T3 c' M
二、CheckStyle:
9 L% s& s8 {+ ] 1、整体介绍:
6 _& p5 Q/ M7 ?: @5 H9 } g c CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
1 N# J( H9 n9 i# ^& a \ Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。 6 J0 ^- E# S [7 @& a2 o$ H
2、安装步骤: ( Y3 S" p+ C; g& o. }: R! U" n
通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 % V, Y& R' N2 B3 z
3、使用说明:
! k. S2 d5 u \5 |( o* ~% S 
) S: o5 }+ C9 m* D 可以看到基本都是一些缩进啥的编码规范,可以不用太关注 7 ~8 j. s4 [9 {/ `
三、PMD 7 U3 N4 n! C# q1 I* _" j
1、整体介绍:
; ~. v$ }5 h {. z' _5 V PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
/ X2 }5 C9 n' |; u6 u
' A( r9 m9 B# }; S) u; Q% G) q 潜在的bug:空的try/catch/finally/switch语句! } L' |% C! m5 H: S
未使用的代码(Dead code):未使用的变量、参数、私有方法等
0 A! f5 e& c+ H1 Q5 R 可选的代码:String/StringBuffer的滥用( _7 y( [- Q& q
复杂的表达式:不必须的if语句、可被while替代的for循环: |1 l0 T9 k- R( i, g
重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs. _+ G. {) E3 {
循环体创建新对象:尽量不要在循环体内实例化新对象2 x! A1 D: A9 Y- W7 l7 X
资源关闭:Connect,Result,Statement等使用之后确保关闭掉) E+ o; b i+ p- i- R1 g
8 A1 l+ \: Q g" x- L8 P$ v9 b 2、安装步骤: 6 o t0 A& I. E! Q
通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可 1 R; M, [. x w1 D" ^
3、使用说明:
: @6 n7 M5 G+ Q, g0 O T; h8 ^1 q2 g" y
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html1 G w# F7 g. Y0 }" q- U
/ Q- Z# Z. k6 \# J4 l
3.1、运行方式: ' ^' ~, }: T; W1 [* J
(1)从Tools菜单中启动: . H: T& Y$ X) r+ A: T! j+ {, e/ h5 k
通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。
- i9 | K' k- c4 W* S# B( g7 b/ ^: R. u) n% @4 u8 Z
Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。, X0 R; d* Q5 M% e2 e% T8 l% u3 v& B0 T
Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。4 T8 f" U7 U+ H9 [5 P. u, z9 L
3 V" G; Q- ]+ S- V* A! W
(2)从右键菜单中启动: " _7 ~& Q) j3 X: H. T) T, W4 c
在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 7 `/ |3 I# Q4 x$ e0 e2 [
3.2、运行结果:
, ?6 M4 ~ P& n6 b, U9 m  1 v9 X* ? I" a" x
运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 3 [0 Z, P7 O$ ] a* ]8 Y9 `, x
3.3、配置检测规则:
; G5 x# D4 k; l) t 通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面: : Q5 y' i6 ]3 A6 V0 \
 5 p) }7 @& U2 |
在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。 1 E, C: a# [. j% c* }
点击“Options”选项卡,在其中可以配置一些检测规则选项:
$ }- c2 s3 {1 k 
& O2 y* s) C) D9 p 其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。
7 @' t8 i6 t% Q" C 四、FindBugs:
: E% s/ Q+ i* c8 u3 T1 ?9 f 1、整体介绍: K* F; ~/ R+ A( s; X) i0 t# K
FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考
l& {# A2 T$ |# I" r5 F& p 2、安装步骤:
+ Y- {1 B. l9 [0 U" Y- E! u! @ 通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可 # i/ r5 v, |! t! V: r
3、使用说明: . \& v" n' X/ n4 |; c, T
FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
( l. b8 P6 Y* P6 X8 S' v9 \! g  $ m2 X$ q) `4 p
分析完之后就会出现结果面板 8 C$ F+ v* ?) G* v* a

# U( U6 Q& b2 N" w6 V2 X/ s4 J 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行
. L7 C+ r! [2 T: ^1 z
( I z% R7 _6 M9 T4 T. B 4、附:常见的错误信息 & D* [" z- \1 X0 t& ]5 `
4.1、Bad practice 代码坏习惯: 9 ~- ]+ @6 c# w G
# ]- p* [# Z Z1 p! Y( k
图片
- l$ `; p, D) a5 }! _% r& w- i 4.2、Dodgy code 糟糕的代码: 0 Y# e6 l- Q. b7 Y+ a
! m8 x6 ]! [6 v- K1 o
4.3、Internationalization 代码国际化相关: 0 p9 d. K# w# r
4.4、Performance 代码性能相关: 2 D" \: u+ K. b& _* B' g( i
8 o# ]6 f* G" n% ~4 I0 y9 Q
4.5、Experimental: 6 P+ o8 E) k; M- s# M
! @) x& ]) u% m5 ]5 {
4.6、Malicious code vulnerability 恶意破坏代码相关: $ T' t. U8 S# R& X6 s
; I/ R% [9 [+ n 4.7、Multithreaded correctness 多线程代码正确性相关: ! I1 I& e6 D! b- q; |. ~1 o' @5 I) r0 E
4.8、Correctness 代码正确性相关: " V, ?) S1 e. ~
$ C( C9 D" M* v8 H" o( w 五、SonarLint: 7 e; b& I# k% {+ G9 X
1、整体介绍:
* s4 L/ f# S5 ^0 ] sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
* v$ O% s/ e S& q0 D1 a; F @* s 2、安装步骤:
7 U0 [8 t) ?9 g/ C% N# L7 I% I 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可 8 T3 x2 }! P5 j
3、使用说明:
/ _0 S2 {2 A6 N5 C2 V% z- c. u% k' c9 v9 V
右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。
. I% R) _& E- w, ]& p8 B7 J; T
6 L6 p/ J# ^% M! e. q) f# G 4、配置 SonarLint 服务端: / H% `9 _( t9 s- R
4.1、配置 Sonar 服务器: " j! U- T5 L8 u6 J6 F2 [2 B! P
sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: / @" H' o+ }! N! e
) _. w" V! @ l- g
点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 ! A }& s& D- Z t5 z
( p7 a* W! Y: C% T8 Z, w1 m 4.2、具体 Sonar工程配置: + j& x. V8 {$ R H
配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: % ]% O/ U+ x) J8 W
1 W; H2 X! p) a! k 4.3、使用 SonarLint 检查:
5 Q5 v+ M) M" h2 `- b0 { 配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查
2 s- |* c# O F7 C) |+ h, P8 e' O) x3 z4 B
总结
Z$ l4 V# ]7 p8 a4 Q3 J/ G1 ^4 @$ B/ g8 r
检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了. e& b, q5 l5 S: [/ W; C+ X
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:. l. p7 u0 v7 T1 q2 a
PMD 自定义能力强,用来自定义项目BUG规则非常好用
u) e3 ?2 U- I; ?: G Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。; f! ~& Y) k6 O+ `
SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景
# V6 w, K) N) G! ]2 l* Z
. a4 n% m9 e: ]5 q. b* Y' X 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多
/ y4 {/ O* v) L6 L# y+ D8 }, m; ~# T3 L" ?7 @
责任编辑:
/ Z, ?8 h7 v0 v% [( Z! u& O2 H+ v# R; r' B& @0 a
+ D- v* ^& t( U/ N4 `3 b# \+ H% m, x8 o: O; `& Q4 f4 e1 }0 _ s
' `: I( {0 e0 {3 |/ @ |