|
& K& s& g; F& H. A) [" U
原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
+ D+ B; i7 R; N( ~% Z
; t2 N' D; O/ H- t% y1 } 随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。 - n# g: I# R x, q2 Q, y3 o+ L, a4 G
工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力 * d8 W( H* d8 M5 P2 p
一、Alibaba Java Coding Guidelines ) h: E8 K: q; D. J
1、整体介绍: * Y2 Q3 [7 G3 X+ b9 |) i
Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。 " G4 z/ N l5 q
阿里巴巴规约扫描包括:
6 G: i R8 b5 k: w" `) l3 H' w. k9 c0 _
OOP规约
# ^5 Y, u1 F0 Z 并发处理
2 X8 J0 j0 g% {. i& p 控制语句8 q2 r+ h- A, a+ u
命名规约2 ~) |# M- @& `8 o* |) \ S9 H& T
常量定义, p7 B0 ]8 W$ I
注释规范
4 T. w( E* ~ o
. V. L$ N2 N( m1 ]7 A$ Y2 f 2、安装步骤: 6 J; G6 ^: C+ j+ r3 U7 ?
File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。 " F O2 j/ p( t
3、使用说明:
6 P. b+ P/ f7 g8 s3 Q* Z 3.1、运行方式: ! K- H2 m+ D }9 I- d) R
(1)可以Tools > 阿里编码规约 > 编码规约扫描
7 T' X& l" o' D1 v- N! o 
! {7 Q/ J3 x0 ]+ C (2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可: 8 p2 n' m5 a- v; c0 e( b" Q

" A- \( [3 ?4 d* O 3.2、菜单功能: , d" G3 G/ T; Q( S
* H" ~1 e: Z1 I
编码规约扫描:开始扫描代码' r+ m# B- d9 a6 d6 o6 `
打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能
& h, S8 r* t( `- ?7 J 切换语言至英文:中英文切换3 W3 t5 x5 D# ^: C F/ e* n
" B# @- D( I+ N5 S' C0 r 3.3、运行结果:
* y* a0 S, z* t# y) _: `2 N 扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。
: f+ s8 i* C8 g) d" g' l  ( m- G. c1 M$ Q, g. u6 s
选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到): + J3 c1 Z2 m3 m" @

* W# B% C V% d, z/ ^( I2 k (1)指定区域搜索同一类问题:
& F8 Y9 K& P7 V! a 当点击③处的按钮时,会弹出如下按钮:
7 P" g% E7 }: I' K1 _5 k, w  5 ~! D8 v+ t( \7 |6 | e; i
这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示: . ?0 h$ L9 X; c! t- w

O- I ^' ?; _1 ?# y 这里我们可以看到,显示了整个Project中的所有该类的问题。 ; V: B1 l" ?1 @6 P6 b7 u6 X1 H) b, K! @
(2)预览具体的不规范代码:
) B7 \! s7 Y7 J2 l2 y 如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。 ( u9 t& V" S: q' a

& }5 n9 ~, m& F& d2 k7 N 3.4、工具栏功能介绍: * w0 p9 f8 A; D& ?: [% }
 % D/ d% z# u0 O& y2 a$ b, @8 C
4 U/ x U6 N: ^; q5 A Rerun Inspection:重新运行一次扫描
9 r& a' E# @$ Z4 W6 _# L% B# a Close:关闭真个AJCG面板$ B6 A d- t2 C! a/ s2 X& x, I
Expand All:展开结果的树状结构,整个结果是树状结构的。. p, N; P; r. ^$ Z
Collapse All:收起结果的树状结构 L; f) u) E. I8 @
Go Pre Problem:选择上一个问题
$ N- N0 g( m4 ?' v4 O4 Q Go Next Problem:选择下一个问题0 `0 a# W, p3 m6 ]* f1 f
Help:帮助5 _1 X8 ]% ?! W' `6 e3 t
Group by Serverity:(不知道如何描述)
) m- G$ F6 ]4 i' j/ I/ {! ^ Group by derectory:按目录分组/按类名分组间切换
9 @, u/ J, Z- s Filter resoled items:过滤掉已经解决的项
7 s6 X7 c' X9 D Autoscroll to Source:自动滚动到源码1 X& M, S* x2 I
Export:导出,可以导出为XML和HTML两种格式
: w' S( B! Y, c! K, b4 z Edit Settings:编辑设置
- y5 t d: v2 b- m r
6 V6 W; R2 u- |$ K5 a 二、CheckStyle:
: j, H( \0 D1 Q: k4 S 1、整体介绍:
u z" g _, r) ?# v/ i) t CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
4 N/ X1 B+ p' G c" X Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。 8 n) e) J& V% Z+ O) w
2、安装步骤: * A m/ J) c) G5 H; B# l8 u5 S$ W
通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 * B% e2 G- l* P, G* w& V3 n* ^. o
3、使用说明:
0 C5 s1 @( r! P! g! Y7 ]  ; @* N6 }; f& M* W R1 W, C
可以看到基本都是一些缩进啥的编码规范,可以不用太关注
. W& Q8 i z" A9 Q 三、PMD 2 ]8 {1 m; `+ n1 o0 l, _- I& H
1、整体介绍:
O6 Z m$ o. p" \' N* `% f PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
" P! k" z3 B/ J, @ l, _1 x
3 W7 s3 c2 T1 q* V, S 潜在的bug:空的try/catch/finally/switch语句* B" p5 H( V3 B( R- T6 i$ M
未使用的代码(Dead code):未使用的变量、参数、私有方法等1 e4 q% Z) l( @5 \
可选的代码:String/StringBuffer的滥用, S# V: T/ g: x N! z7 y
复杂的表达式:不必须的if语句、可被while替代的for循环
% `% h) Z) u J) S 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs4 h5 f3 V& b, V5 Z! F
循环体创建新对象:尽量不要在循环体内实例化新对象; L0 x3 S/ i0 P& Q
资源关闭:Connect,Result,Statement等使用之后确保关闭掉
1 J3 {) v& P; O& ~, M( ~. W4 F& D* X9 m5 s' e- \7 c
2、安装步骤:
* a6 u, E: }( _) G 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可 2 f. O) _+ R5 \3 y8 s$ C6 F
3、使用说明: ; W+ d R% D- c8 v! k0 G
* F' Q0 p* Q; r3 ~
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html
5 T' Q- v. ?8 J y4 x7 Z1 H5 {& z! n s5 [! D' h5 c
3.1、运行方式:
0 z+ N3 f2 U, K/ @3 T e4 ^ (1)从Tools菜单中启动:
# h3 f% }4 n7 G X5 q9 P9 v; _ 通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。 + C3 J1 {( W# O3 A7 y9 q; j
( }/ ~8 {7 _0 g
Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。: r6 s2 j% E# t& S7 `
Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。
) w: g+ d! V0 d5 F3 X, K4 Z- r9 @3 j c4 S
(2)从右键菜单中启动: " P( j" I g' f6 ?: @4 [
在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 ( c4 V' D, _% y! z
3.2、运行结果: & {7 E# I Q" E5 h u) d( }$ K
 : i# ?8 c0 y/ U6 s. C9 }
运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 : S. v/ \. k B4 K
3.3、配置检测规则: 6 [5 A8 Y! a5 q1 `, _1 e+ a$ \
通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面:
6 u$ b5 d; z+ b; Q' Z 
! x- s! r( ^4 @3 @/ h3 ?* B" A3 ~ 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。 3 G# W; f$ {: h8 I
点击“Options”选项卡,在其中可以配置一些检测规则选项: ! h$ |$ `! I" |4 u

) q: |3 R# q' n. \& m7 v 其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。 2 W5 s+ s' T0 _: e9 N
四、FindBugs:
9 K: x. o' _+ D4 z 1、整体介绍:
, N, z" H; X2 A' Z FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考 * c' @+ J2 ]3 A, T
2、安装步骤: 6 g6 Z% {; {2 v8 G% m4 m: h6 \9 c
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可
" p5 }6 ~* S. [# q$ { 3、使用说明:
9 l* D \/ B. G0 A) i FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
/ J. `$ F. `1 }; f3 e3 [( Q  7 X. W% Q/ D d3 q6 W/ G: Y% d
分析完之后就会出现结果面板 / \0 S( l3 z& q0 l: M

1 g. e. P; R! S 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行
1 u$ S& L1 I6 a4 U& v
. s( R+ P& p# I9 w, N, n/ f 4、附:常见的错误信息
; W& O1 W0 M( ], s- k% v# H9 q$ N 4.1、Bad practice 代码坏习惯:
" c! D! b/ X7 i$ I6 P, {# x% v7 b& I4 c& D0 J) Y
图片
2 O/ S1 e2 a9 S7 q8 O 4.2、Dodgy code 糟糕的代码:
# _! H: D h( e8 q, l+ o- v8 Y- _7 }
4.3、Internationalization 代码国际化相关:
7 T3 c. r3 k* w- l# I 4.4、Performance 代码性能相关:
" }1 D. m$ V3 r6 x. L6 Q3 j% y$ I( o) Z0 Z0 W* t
4.5、Experimental: 6 E A5 Q; r1 A* ?
1 J6 B! W1 p( w2 O. ^ 4.6、Malicious code vulnerability 恶意破坏代码相关:
8 j5 \7 C* I) y& ]& L3 x; C6 i2 x T- i* U# y% O) @5 u
4.7、Multithreaded correctness 多线程代码正确性相关: " W: t. [" T. i3 E H
4.8、Correctness 代码正确性相关: * q% B1 F d" Q) e
7 y& C6 q, G* k- M 五、SonarLint: - h _3 m% X& P! _2 {1 ^
1、整体介绍: * ]" \! `4 f; d9 E2 F6 A4 Y
sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
3 n; U9 @( M7 R 2、安装步骤: : C; l5 M# @2 J2 S y
通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可 {; a3 a# b$ z( q* o/ A
3、使用说明: * C; I" Y& ~6 U3 ]
4 k: z$ F& D: m2 v! _& Z 右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。 % C1 K' H3 o0 s, h% {1 x
8 p0 _1 ?/ }* h$ N0 R+ L 4、配置 SonarLint 服务端:
# @9 {0 R. {( q ~ 4.1、配置 Sonar 服务器: ) x; Q3 K2 }1 n& v
sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: * ]# b P- n" {, G: |
' K) f! s0 ~$ q# N$ \
点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 ! `6 b/ [# e! |2 I( ]! h
) ~$ u: ]/ y' G) w7 q$ v0 d+ [
4.2、具体 Sonar工程配置: & Z8 @1 I9 ~6 @: i
配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: 4 U7 n2 [ U" ~" P3 @% M
4 Y' v! P4 m$ Y$ j, A8 l
4.3、使用 SonarLint 检查: 8 ~4 x. R+ i6 w3 q2 ?% ~0 K
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查 1 K! ]0 a9 Z7 s( e) {3 X
: y- U* {: K/ ^* A: s' o: B; P/ x
总结 ( i' p" B3 a& t& y: D5 J( b; o' E! |4 o7 Q
- Y' l0 |! I$ X* P5 w& w. M5 Q 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了
3 e) J; ?4 E A 找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:- V g8 A. h, S- k- u
PMD 自定义能力强,用来自定义项目BUG规则非常好用4 P0 ?2 x/ M3 T) K; z: K) b
Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。3 r/ p" T7 F% n: ^6 ~: p4 g$ {
SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景; z) q4 J) N2 s, x& i
) j Y, D+ s: y& z/ e6 s 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多 4 ^( ^+ K8 ?4 V$ \( h
( V" Z2 p6 Z. F# g3 j! ^% |# F
责任编辑: ! @1 H' M. b+ z6 C, j, @- s7 ?: t2 A
6 [8 T* z9 N6 g, J$ Q& u! ^$ p- D
$ E, f/ w# O2 h5 f, A
! [( a+ X0 k$ a: G" @# r% g0 }
8 ^( n- L4 ` r# G |