|
2 V5 \$ E0 b$ F) y% k* ^. e
原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
& g7 m) p C* y! A; c5 G1 @4 e Q6 R+ S4 u
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。 ; e# b9 c% v0 ~6 [# ]
工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力
3 X p6 M' G4 E8 Y 一、Alibaba Java Coding Guidelines 2 ~3 F' ^4 N3 i/ C9 r, G
1、整体介绍:
" m% m! h$ o- C4 T) Z. w Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。 " g) q( x: _8 m4 `6 z& {& w( T# x
阿里巴巴规约扫描包括: 4 D2 W& g/ k/ ?7 f; Z& b5 `
3 y/ q- ?1 O: F) r$ z: K
OOP规约
7 o6 L- E) N, z4 {/ F( m 并发处理
# l0 T& w' W) F+ w2 |& { g; V 控制语句0 w7 B: P8 u" D3 K4 C6 G$ A3 M+ j# t
命名规约" z! W: k a5 e+ q* ]3 @7 \8 n$ P
常量定义0 e1 S, A2 Q: T) E# m Q
注释规范
/ Z+ @4 B0 A) {4 C" O# H. A7 h! w3 Z: \# N( s0 {
2、安装步骤:
, x- F$ t! l) G5 A9 B File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。 * K% w2 D& O( `/ C7 y) i
3、使用说明:
/ y, C1 k2 q% I1 ]2 \2 x" P 3.1、运行方式: : ?' p# F7 t- ^* t9 [9 h# v1 O" X
(1)可以Tools > 阿里编码规约 > 编码规约扫描 " H* Z8 C- {1 E2 n, o$ t3 V
 8 z9 l _- B4 j6 ]& z J
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可: ; B, g% ?/ B/ J* M& O& q* \
 * [! h$ n) x) H7 k; P! N
3.2、菜单功能: ' u/ s! l6 e' m# q p5 u* {9 h7 ]$ X
# p$ Y9 @; K7 y 编码规约扫描:开始扫描代码
! j7 X( Y( O8 @( F2 x 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能
3 _- }. V" a: ]( k3 n 切换语言至英文:中英文切换
8 b; @6 T/ V; H/ G; E1 _1 T" z) R* T- V$ F' |% o
3.3、运行结果: & S3 b) R6 Q) x. D/ N
扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。
7 N% H& y" v( v% w2 x8 ]  5 _4 g0 A2 {, w. \# f- ]
选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):
1 T+ y2 T7 C, M+ R  $ J- ]# k! q/ a$ e
(1)指定区域搜索同一类问题: " _! h' {- K8 u% i' M; i
当点击③处的按钮时,会弹出如下按钮: " Z8 g0 e( j, g8 L

( `: I- L' e7 W5 v$ n5 k( M 这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示:
9 P' l/ ^ v% I3 I* R  5 O0 d! _& k! q8 [
这里我们可以看到,显示了整个Project中的所有该类的问题。 % J/ D, p; [* l. v; P P
(2)预览具体的不规范代码: / V$ F9 E5 p o5 ^0 C5 J& m0 W
如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。 4 b: L5 r9 W0 E% [( I
 2 @3 X/ z: F, L3 A* f
3.4、工具栏功能介绍:
: y. O, o+ f+ L9 o! ] 
0 e& M( E0 r' M0 z) _- p' k/ G0 o; d: ?. J) o
Rerun Inspection:重新运行一次扫描; D; G% ^; e% |" _3 w
Close:关闭真个AJCG面板
3 e0 v$ V* \9 h Expand All:展开结果的树状结构,整个结果是树状结构的。% |) M* o I: }) o
Collapse All:收起结果的树状结构* L5 n V3 k, {. a+ H$ J
Go Pre Problem:选择上一个问题
4 h% Q B! c2 Z4 z, V+ O( p Go Next Problem:选择下一个问题0 [# \" P) a' k: G
Help:帮助5 l0 h: X# y" R3 m, |
Group by Serverity:(不知道如何描述)
. I, i/ J( P9 L4 ? Group by derectory:按目录分组/按类名分组间切换8 _1 ^6 e: b4 n& q$ i6 s/ s! D
Filter resoled items:过滤掉已经解决的项* ]& J2 d1 U' U3 A" V
Autoscroll to Source:自动滚动到源码
% M; T. E% [; T/ z2 G" m) M3 ? Export:导出,可以导出为XML和HTML两种格式: A2 T' ]9 ?6 I Q2 F3 s
Edit Settings:编辑设置- W/ M2 p0 e. o. Q/ J% o( Q/ L
/ X* M" |4 Z, _$ |, M, _
二、CheckStyle: + W1 t: p6 p2 r' r
1、整体介绍: # M5 C% U7 K6 g; ^0 J, F
CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。 1 B) r% G) D6 e# Y: Y
Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。
% J/ f- C% M1 e7 E1 C+ \" Y 2、安装步骤:
3 I0 q; C; ^9 p% _) O, ~ 通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 3 m- Z( G5 K* R) b/ E; l7 d
3、使用说明: 6 `6 X! H: m' W5 O6 N# K4 l
 \/ w( I' p3 F, ^' g: y; A
可以看到基本都是一些缩进啥的编码规范,可以不用太关注 : J* w+ P9 M$ H @$ r i7 U1 `5 A
三、PMD 1 J- @+ F9 S' t$ _8 y0 S
1、整体介绍: g7 M! L% U8 s) T0 o" G
PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容: ) k4 e5 ~9 q8 F+ `
0 P! ^7 b2 P: c0 U: _ 潜在的bug:空的try/catch/finally/switch语句) x2 [" \0 t# Q3 {) J$ R
未使用的代码(Dead code):未使用的变量、参数、私有方法等8 |8 e6 p* x0 N- L' C
可选的代码:String/StringBuffer的滥用6 ^- |7 y, t3 ^( s1 y
复杂的表达式:不必须的if语句、可被while替代的for循环) \: K5 ^) }( @* D
重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs
/ r' v! }- v2 B2 H) U# h 循环体创建新对象:尽量不要在循环体内实例化新对象& m6 U9 v1 C; e' x8 V
资源关闭:Connect,Result,Statement等使用之后确保关闭掉% F. D" H$ z' r% \& z3 V
" M4 |0 w" C1 T, b% q
2、安装步骤:
# r0 x& v: V0 B# i- u4 H 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可
4 |3 R; M* R3 ?2 `% S4 s* }1 U' N/ A 3、使用说明:
' d" p6 q6 [( ^' A6 l" u# q0 ^# N/ |3 s p/ A. K$ T
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html8 C0 k. j2 P( W0 u2 M* \
# g+ O- W' J! }( H 3.1、运行方式: + X/ r( I9 {7 r* i, t; b( X
(1)从Tools菜单中启动:
7 l. [3 J1 X+ M% p7 ]$ D6 a 通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。
3 ~+ H8 i) }$ [! a8 J1 ~' _2 @' q) s+ {' G8 D4 \3 r2 P8 w
Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。3 Z) s( p$ {& _* ~8 `. H
Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。% C! l: g$ q; s& m- x
% {# A+ a8 Y) [) ~1 V: ? (2)从右键菜单中启动: ' P0 E" ~; T# x0 G# [% `
在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 & g/ ?. s, x: M6 T) W9 z
3.2、运行结果: 9 Q. z1 Q3 d' u* K0 G
 2 D, I$ k: R7 |& n' h8 q( k- G7 J
运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 5 c- i7 j3 W1 ^+ l4 q1 Z
3.3、配置检测规则:
( p2 } c! y. y 通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面:
! {: b: c2 G) Z$ W& \: E, h  - N5 q% ?! N9 o) l
在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。 3 i3 E/ ?' r! z1 e
点击“Options”选项卡,在其中可以配置一些检测规则选项: & [/ {9 L; C* f" e" J
 % ]" L& b% K2 y v0 n) Z" N5 z* W
其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。 0 m1 J+ d8 t) g( M3 c
四、FindBugs:
$ A- e8 ^/ f' K 1、整体介绍:
/ z/ H- }4 y) p FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考 ) P( ], I. W) a; d! l( x/ I9 F
2、安装步骤:
6 t/ \* s1 x6 A4 g: L 通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可 1 R @' X) H) M6 Y" T. B
3、使用说明:
1 F2 h1 p: l, h6 u FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project 7 t {1 p" q, ^
 + x; N. i$ x$ L3 u3 h5 a4 p
分析完之后就会出现结果面板 1 X A1 f# H) X- y* e7 u1 l
 6 h1 {& K! F+ R5 t* A3 X
点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行
2 [6 i3 |, `9 x7 y s0 F8 P: U$ z. q/ }
4、附:常见的错误信息 ) c, Q/ M J. U# S* y5 u5 j8 Z* |
4.1、Bad practice 代码坏习惯:
4 {/ w& E* @) f9 T( ]
6 X* U/ Z6 Y! e" ` 图片
) b+ D* g. E3 u. d+ U 4.2、Dodgy code 糟糕的代码:
9 q, H' {( n# b0 H9 \: H6 [5 Y% C) @
- ^; l& \& j. |; y( K, D/ D; v+ U 4.3、Internationalization 代码国际化相关: 2 k9 A$ D0 G! s: V( u6 y! w/ F
4.4、Performance 代码性能相关: , T4 \. D. I$ t6 v0 B9 z7 T' v
$ @ F. K' Q8 b0 p% Y* P 4.5、Experimental:
" G2 Q6 Z$ p) P1 n2 W5 [6 `% w* J% Q
! U, B6 F2 b4 `" v, d0 D 4.6、Malicious code vulnerability 恶意破坏代码相关:
[/ p9 y, D A+ q
+ c; G* t/ T7 W( `8 F) X: S 4.7、Multithreaded correctness 多线程代码正确性相关:
8 f8 `4 E% W( T+ n: ^ 4.8、Correctness 代码正确性相关:
2 ^8 S) i# A1 u" z" \. k8 ]! z) Q+ e Y! D0 Z. Q
五、SonarLint: 4 u" Y# m8 h0 Q8 k% X6 z8 k& v
1、整体介绍: % S+ b3 B- t3 L% j
sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
+ r* ?# h! O6 z6 ?1 j 2、安装步骤: ) H# l9 E8 D5 x1 L) r+ ~
通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
: D3 i2 Z+ D2 e: K 3、使用说明:
/ I/ b7 c0 j6 {7 g
- @$ H+ B" Z3 O/ ^# G2 c 右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。
! @3 |# @3 M+ r6 `1 W' {" a$ g& O3 d3 z# h S- Q
4、配置 SonarLint 服务端:
- J/ T* z) u. ]; [6 d 4.1、配置 Sonar 服务器: ! B0 F* v+ l( I/ {
sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: " a w8 b; z3 ]# [7 c
5 c( B h8 t7 |* Q' g2 ` 点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 # Q; Y I+ H8 x$ I# I3 p
+ M$ m( U$ |8 a
4.2、具体 Sonar工程配置:
) ?$ h, S9 L/ N2 n P& ?( D2 _ 配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: ( p/ R* i% V5 ]* j9 ^+ L, r: g2 T
: D% f0 n2 y9 O) l2 ~/ m" V* N: } e
4.3、使用 SonarLint 检查: : `. A& { C* Y
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查 : O v8 r# B7 ^6 ?9 ?
. }- V( X! L& v4 i7 V* E
总结 % ~9 @8 L4 c, r/ ]6 [% G& T
# L, O2 h0 F- ~6 i8 v
检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了+ N4 Y* E6 s% l$ W* L, T! Z% B3 ~
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:
' T7 n7 [/ N4 A+ G f# O, w1 x PMD 自定义能力强,用来自定义项目BUG规则非常好用( i4 j8 E7 Q: {1 N8 }4 x9 t6 v* S
Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。
) L: X" [7 E: A9 q) k SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景 Q/ d! Y6 K( i0 k; R, R
) ?) d* b5 G- j' j. r: a4 { 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多
$ A# b$ o# [+ r
! Y" C# S3 I# D7 n T9 o% v' S 责任编辑:
. I: F( z, N! s
9 ^; L0 s* M4 ]+ x# j h2 h% c( k% _* B$ z1 L
5 Y+ L& _5 \2 V( ?! P
* m5 n, _! _ j; K
|