|
, q U' [; ]) }0 E# r 原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
8 H% p7 w3 C' P Q' Z2 x+ E- Z0 C8 c. l8 D8 P
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。 . L( Y, I+ ]7 y: `& u# d
工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力 8 ]! u8 I6 d+ {) o. C' {% ~. V& Y
一、Alibaba Java Coding Guidelines , F' }7 W' D, \* v' ~. Y. W6 B* C
1、整体介绍:
5 T7 C8 v" h: d. K! L Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。
; k+ r. f/ J1 T/ S/ A, W1 t 阿里巴巴规约扫描包括:
5 Z, \% ?$ M7 U; N# k7 n% b; R& a5 q
OOP规约3 B% w; l2 _* N, c
并发处理8 d) J$ d( S" p
控制语句
- ]4 @4 N/ L9 N) F* U1 _ 命名规约
1 X, L) @( a4 _0 \1 [# I E 常量定义4 Q( l* t5 b# y0 i) }
注释规范+ Q4 i- i- G) i4 T+ m* c! w' P) |
& y7 |3 {, J P( ?
2、安装步骤: ) m) }3 [9 w$ R$ o1 h! ]1 P4 b& S
File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。 6 R! n% s' A# Z# q3 o9 V& h
3、使用说明:
6 d( z3 p& }1 g% b/ A5 O2 ] 3.1、运行方式: * C& Y% w. p i8 ~
(1)可以Tools > 阿里编码规约 > 编码规约扫描
; V, g$ o. t2 V, z. G# s  . q- @3 x( j4 H0 |( T4 w1 `
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可:
6 A% W; H+ @& r# v9 } 
$ K, h! I& X* N7 }$ G' y 3.2、菜单功能: , `. C, p3 [8 x0 ?! C& ?- ?8 v1 R/ g7 k7 u
$ |, S( |$ W3 e7 J7 z
编码规约扫描:开始扫描代码
$ ~; h" i6 J- E, R; M1 U1 V 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能
3 |. O1 t1 K7 `3 d& e$ U* T 切换语言至英文:中英文切换& `/ O% f) X: {4 k! ?8 i7 P0 F
3 |7 ^( x/ T# D. i: ^( r1 @; s 3.3、运行结果:
9 ~. K' O, s' E$ F 扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。 ' G, b8 y# e& s. t
 8 J; x2 o% y3 R* C, q
选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):
8 J+ x6 K2 ?" [  2 h9 W2 Y7 K" |( W" i5 D: l3 l
(1)指定区域搜索同一类问题: : P, g+ r7 |7 ^* A2 ~, T4 _
当点击③处的按钮时,会弹出如下按钮:
1 T0 w8 K9 o3 I N  0 ~1 k, l# w. d( v8 F
这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示: 0 s& \/ _5 C/ B8 s. B

! f% q9 Y- N7 s, n7 f 这里我们可以看到,显示了整个Project中的所有该类的问题。 1 y. c) f9 p# z/ ~! S
(2)预览具体的不规范代码:
/ S/ Y) \( N$ ~/ H 如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。 ! Q! i9 b9 h* y! ~

$ [" y( S& }3 U. ~' M# j 3.4、工具栏功能介绍:
, n4 V$ ]0 y# E& ` 
! j4 G# F+ T- N; e( K; V" J: L; w {
Rerun Inspection:重新运行一次扫描
9 j: _6 |( j. u2 \4 n4 Q! q' t Close:关闭真个AJCG面板
+ v. x8 Y9 L. D v/ C Expand All:展开结果的树状结构,整个结果是树状结构的。( P% a# q9 }& e. J" K! ?( R) q
Collapse All:收起结果的树状结构
$ n( t2 p" U9 U0 s2 \+ E Go Pre Problem:选择上一个问题) Y! k) z" d, P6 w, m
Go Next Problem:选择下一个问题
3 Y3 r( L. `. K Help:帮助7 `% y$ P/ f3 R; v: A3 G) d9 Y
Group by Serverity:(不知道如何描述)
. q* N7 F& ], y1 t6 N- M) l( I Group by derectory:按目录分组/按类名分组间切换( @' h1 _6 Y( X( g8 [
Filter resoled items:过滤掉已经解决的项5 |: _& o+ Q* z, ~
Autoscroll to Source:自动滚动到源码. Z9 d8 `4 q! H3 v
Export:导出,可以导出为XML和HTML两种格式
9 D+ X+ J0 E& a4 z! ^ Edit Settings:编辑设置
: I1 Q1 n9 _2 y- M# K, j
' |3 j; J2 N# i2 Q9 w 二、CheckStyle: " l" y' B+ V M/ P
1、整体介绍:
6 C, t& v; ]3 A B# B CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
6 n) |2 C7 u9 S# w' R) P Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。
' T: ~1 B3 `) P& o 2、安装步骤:
/ l& Z5 L. y1 c( N 通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。
2 ~* J. r& ]0 c* G/ L# O; R3 G 3、使用说明:
* v& \7 O8 G) j' D& s  # U" W$ }6 `) H* k% X
可以看到基本都是一些缩进啥的编码规范,可以不用太关注
3 k+ u# ^, R6 c) i 三、PMD 1 d% U1 {! h5 K8 V
1、整体介绍: 8 H a4 {5 }% Z7 _' b9 \. v0 m
PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
! P, D b, ?' Z. t0 w, t* r' J, G$ w" V5 d4 y1 e/ E
潜在的bug:空的try/catch/finally/switch语句 S" h. D J$ v: Z
未使用的代码(Dead code):未使用的变量、参数、私有方法等2 z) ?% J2 H. D- p: {; T
可选的代码:String/StringBuffer的滥用% @$ I: K f* Y9 R' z# ~2 U
复杂的表达式:不必须的if语句、可被while替代的for循环" a( C0 ~" k6 J F& e7 _0 m
重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs
# I$ V0 o/ E5 W( \, Z- c# ^ 循环体创建新对象:尽量不要在循环体内实例化新对象, i9 Y, X$ m1 z# S% K( c
资源关闭:Connect,Result,Statement等使用之后确保关闭掉; l9 y+ e s5 O1 N8 p2 X
/ ]! A. C, j% H; F1 p
2、安装步骤:
6 Q/ c( W2 w3 T" A$ O 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可 0 K3 u! @0 F0 e/ g
3、使用说明:
. D+ Z& C; L' V0 d7 Q+ z- i }+ M3 Z; {" V, d. N& r, \
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html {/ d* T5 q8 o- B0 r8 f. D7 E
* V# }# \) O x9 D! |8 @4 C8 m( S
3.1、运行方式: ! O' S% l1 F# B* }# z0 ^
(1)从Tools菜单中启动:
5 ]( y; i7 Y% z1 \% t( U 通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。
8 w4 @# X, f) G) \% C; o, }) Q
$ }. _- ]/ ^# P, G" J( O" _" m Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。
+ B1 _6 ^1 S1 h% w- s+ F( E Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。
3 e/ }* |+ s5 ^, H3 @
- {$ l* m9 J5 J; d- n (2)从右键菜单中启动:
+ `! c8 z, A6 K9 E$ D 在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 3 N1 K' a1 O) \: m
3.2、运行结果:
8 R# q8 G) _' ?* |: c 
1 `0 n: O0 H3 |' q, ?0 \6 ~# W 运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 0 [& _% L$ f' T. I: |0 z8 a. P
3.3、配置检测规则: 4 z* p6 A! K. j. H( o( _% _/ C
通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面: ) i5 b! | j) B% K; m5 U

' t4 \; v' x+ N( i/ D$ {$ v8 | 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。 ' x8 K- F9 w5 z6 T5 ^# V
点击“Options”选项卡,在其中可以配置一些检测规则选项:
% t% s0 ~& n: o. O  ! L* [" M Y2 @; I" X& z/ x
其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。
# U2 @8 ^0 R" j3 j6 o, E1 \ 四、FindBugs:
5 K9 U( {" O" \$ q5 M- H, j* o 1、整体介绍:
8 s) A. P7 u' m8 R8 _ FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考 ! X7 d8 ^) t) [0 c8 c! p. i; h
2、安装步骤: $ D! U2 B% |8 H7 C$ {& C p4 }% J, V! O
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可
5 y g K3 G1 C4 L# \$ p 3、使用说明: - A. u! L$ x3 Q4 i. }# N4 r9 b
FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
+ y. b7 g7 \1 D& ` 
7 e# d( r/ B: y, ^ 分析完之后就会出现结果面板
6 T8 S% b6 M& a/ H8 {* R 
. S% ?0 D; @/ f8 f5 l/ X8 w& N) `" G 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行
5 S% W1 ~" T( H6 v
0 z/ \! z G1 e) @" X 4、附:常见的错误信息
1 S) B! R( l0 o6 y( U5 W 4.1、Bad practice 代码坏习惯: 6 @& e0 L2 W/ Z+ Y* T/ J7 c. }
( c' L6 O, ]0 w( ?6 R
图片
7 f8 J- c9 Y# `0 j7 I% R 4.2、Dodgy code 糟糕的代码:
$ d! V" d" w7 V- j. j1 [1 @
5 r) @( r" a7 D9 Z 4.3、Internationalization 代码国际化相关: - a9 t# l" x- F: a0 N1 }5 E
4.4、Performance 代码性能相关: $ x% D6 e0 R5 a. }- N/ C
& m: K9 k: s, G! ? ?9 a
4.5、Experimental:
O s1 b4 B% w# Q6 l- a2 Q$ \$ d7 Q" N
4.6、Malicious code vulnerability 恶意破坏代码相关:
4 F5 T# J0 o6 g: v# S8 b6 W; v) {# e) Q1 s* B4 V
4.7、Multithreaded correctness 多线程代码正确性相关: ) _, J: O* n8 K0 {+ O" k) ?
4.8、Correctness 代码正确性相关:
' Q8 w8 W% U$ M" \
+ ]1 f* _, w( i4 ~ 五、SonarLint:
7 i& h4 ?6 K) e0 A 1、整体介绍:
5 _1 o! J7 z$ z5 Y$ J sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。 5 R8 _7 L8 w4 c7 Q9 @. R: h$ n
2、安装步骤:
, A# r& z& f* V! V* I* p+ R! x3 ] 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
; | m6 S) o) \( J% N" d 3、使用说明: / Q$ k+ J- ~3 \
( f/ _! R' t: h0 h 右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。 - S# ^1 _% A9 H5 b& B# j" U
9 H: ~+ A) v8 X3 p! I 4、配置 SonarLint 服务端: 3 V% V# h! e7 a1 d& H2 m
4.1、配置 Sonar 服务器:
$ f1 b, k2 x' U+ R& N sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查:
% }6 B! K7 K: H( B
' ~. m. J# w0 c' i) I+ ]) Y. ?) @' ] 点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码
# R' U% b' h/ k1 Q" Y/ A
9 s8 P0 d0 p, R. N/ N# `) c: b: | 4.2、具体 Sonar工程配置: + s6 O. u; e4 H: k+ O4 _
配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程:
% Q; l, H2 c$ w7 M& l0 P! o3 P& a' a! P
4.3、使用 SonarLint 检查: V" W* y3 c$ J& v+ p: u: e
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查 & m( K( g; X7 u' H9 y# V
0 M* J. o9 R& R5 S 总结 : @; I# U: p1 b3 X9 Z+ q6 Z2 _, X
5 v6 p6 k5 S6 u0 S" ]- `
检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了( D( h$ L& f: b$ d
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:
* W5 c5 x6 D) \! S1 Q) w+ N PMD 自定义能力强,用来自定义项目BUG规则非常好用
% @3 f. B1 T5 W- o Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。! _' d+ b v8 }/ P$ m
SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景
# J2 a: L3 K7 r& W/ [7 g2 r0 u/ P$ s1 n" Q
文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多
# z* }4 J# O! ?$ V7 _. {+ t8 [- h# X1 I+ P" j! X% J) f
责任编辑: 9 s: o8 z) G5 K6 K& ^9 m1 w
; k9 C7 g+ i9 R: }; x4 I
! n/ o( s% Z- V. ?& _: M" e$ `0 ]5 a+ P
' l+ `2 y. ~! L! D |