|
/ m% T4 P5 C7 j9 E 原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
, @, A: N& k, h- h0 Z1 d& G5 k0 @3 u2 b- H0 n
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。 & {3 s" Z% p6 h+ S+ M) _; _" A$ s
工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力 7 W& n! S# p: l: @
一、Alibaba Java Coding Guidelines
, |) e2 K9 Y6 w& t+ f, M 1、整体介绍: - @0 P2 I# T# Y/ n5 u; F& O1 t
Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。
' h4 `- s' ` k7 y 阿里巴巴规约扫描包括:
* C) R, A9 Y- ?' } f. {" n8 d, j9 g: L s3 R
OOP规约9 L5 K* h' b' G& z6 _/ t b
并发处理8 c! @* o: C: U X( c- `0 R
控制语句
1 @4 V& U+ @/ E& K; v* m5 x; R7 t 命名规约5 |2 v& A. z6 y' A
常量定义
+ a) Z& { h1 @! v @& ~: C 注释规范# `9 Q1 o- [+ I* _
6 _! V. ]! ?2 O5 t: {% i3 Y2 B6 m 2、安装步骤: 2 N8 M7 l/ Y/ W
File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。 2 T9 S8 H& g9 ~/ z) J+ ]' \
3、使用说明:
! L6 c9 g: Z; g 3.1、运行方式: + z& C5 Z. A6 J% K* r. ?
(1)可以Tools > 阿里编码规约 > 编码规约扫描
" k- w8 d+ ?: p 
; x- s$ Y/ h( o9 @ (2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可: 7 c; ]* s/ A3 p$ o. Y* n$ J# b

3 H8 c' I" ]4 J: D# T: Y7 `3 f 3.2、菜单功能: $ J9 A5 l0 V3 ]! @
5 ~4 u0 ^/ z# h9 u
编码规约扫描:开始扫描代码
$ ]: `) O' K: h 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能; z8 m& ], M' j9 Y
切换语言至英文:中英文切换
' w, H* P: P# y7 X9 X
, Q: ^2 i9 G4 V! ~ q8 X 3.3、运行结果: " m- Z+ R; D o5 T$ p+ `# p
扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。 4 W# o, t/ X, L8 X! s; p4 k5 U
 : L& i0 X4 U& j
选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):
6 H, j2 o5 F2 ^; O1 t6 c 
* p1 M9 F& }% d/ Q0 j4 G2 ^ (1)指定区域搜索同一类问题:
) d, c/ i. p4 x! g3 ` 当点击③处的按钮时,会弹出如下按钮: + h' Y3 J* k1 \7 {+ k8 r
 + j) I- I. v$ ~) A2 t
这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示: " R7 G% f! j! S. P7 o

* q+ p8 Z5 V" }1 w- i: H7 f. s 这里我们可以看到,显示了整个Project中的所有该类的问题。
7 k- D8 [ n( L( D6 I+ S/ ^2 S (2)预览具体的不规范代码:
# L, e0 N- J- c" q$ \( B5 F& F 如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。
: G# Y1 z/ P: S+ W3 f- L1 n  . L% ]) q, b) N4 w; R
3.4、工具栏功能介绍: 6 k# j5 H }5 Q* [; b2 K8 ^4 k
 8 p* h; J! z5 L
- p" ~) d* Y' u: [1 {1 B
Rerun Inspection:重新运行一次扫描
0 d& X2 I6 h5 y# U+ { Close:关闭真个AJCG面板/ w2 e; ^4 f+ E* C2 B' y
Expand All:展开结果的树状结构,整个结果是树状结构的。' t ]( q# P5 T# S8 p% ?! Q
Collapse All:收起结果的树状结构4 A5 N0 \ Z& x6 V- L
Go Pre Problem:选择上一个问题, @ l% M6 O2 X F8 v$ J) B* x5 f
Go Next Problem:选择下一个问题' v8 v/ e7 u, A7 y2 @3 L; M5 j- H
Help:帮助
9 e& z0 O: G$ m8 p2 I; d Group by Serverity:(不知道如何描述)
3 Y5 g( K& W% d- Q3 g Group by derectory:按目录分组/按类名分组间切换0 o3 Z# d' l2 k$ y- J( d- r" K; [
Filter resoled items:过滤掉已经解决的项* `! y+ W0 ?1 u* u% r- h7 L
Autoscroll to Source:自动滚动到源码
6 Q* e1 U; N" J Export:导出,可以导出为XML和HTML两种格式" P7 L7 F/ V" V) w% N; I
Edit Settings:编辑设置4 f' j6 i8 X' g# v
c; `$ G7 {: @" W9 y! A
二、CheckStyle: 9 F$ ]: K- a0 ^8 r
1、整体介绍: + p$ t. f$ A9 H: g1 w$ v
CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
6 t) t! P1 r0 v& o0 u( N g Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。
& p0 l- _: n8 ] 2、安装步骤: 3 Q1 e; C3 H; J# z5 p
通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。
: C- A1 K2 k1 W) D' B0 C 3、使用说明: , u9 ], |. w: ~2 l2 F: h

: v9 x) W9 t* w) `2 a) ^/ R 可以看到基本都是一些缩进啥的编码规范,可以不用太关注 / Q' C. C% J& b. C, n
三、PMD 1 G- j' [0 V6 o6 k
1、整体介绍:
; q4 f% x+ X- t: |2 n; C3 F PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容: . ^) {. r5 a% U7 s$ @
( S' z) x9 i7 k- B
潜在的bug:空的try/catch/finally/switch语句
G, X$ @ E+ E8 I9 A, D. a" T 未使用的代码(Dead code):未使用的变量、参数、私有方法等: y/ {; S2 _6 C! Z' ~
可选的代码:String/StringBuffer的滥用: l L8 K" i& N- M3 _
复杂的表达式:不必须的if语句、可被while替代的for循环
1 i1 x2 m; m6 d0 t3 i+ f M0 q 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs4 v" c. X. y3 p" g
循环体创建新对象:尽量不要在循环体内实例化新对象6 k- x" ^' c3 z) Q* v1 B! e
资源关闭:Connect,Result,Statement等使用之后确保关闭掉) @' Y& u1 J# u% X. @8 u
8 G3 ]1 m1 x8 T: W9 e, { 2、安装步骤:
# |. L0 O" t; t+ A% E; j 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可 " M8 @# y) K6 J; j& B
3、使用说明:
1 C) H- P+ C1 X) T( t" {
' d2 p: s8 w: Q: B 参考文章:http://wjhsh.net/andy-songwei-p-11830812.html' v2 G* v$ O& D, `6 @% K
. k; ^. z2 W% ]# ^ 3.1、运行方式: : Y' B( E6 `* Y" h; P" Z
(1)从Tools菜单中启动:
0 Y) M" r$ R+ z- h" @; c, N5 X 通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。
, N m* Q, j3 I' |# ~
, e/ n# J+ Y. l( ` Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。
; S b j! }+ H& {8 Q6 \ Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。/ f# Y, Y5 o/ q) X1 u% J, H3 @& A" u0 S
7 x7 B$ t/ a( M( P) c
(2)从右键菜单中启动: ! L6 A$ Z" h7 x' s9 b, {8 g
在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 / q) K& F/ z$ k, g
3.2、运行结果:
P+ p& v& v& `1 k7 z9 j. \  # n2 b4 c- W" J& E$ X" J
运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。
4 p( `6 x5 S G 3.3、配置检测规则:
* v& ` B& ]! K1 T, W2 f 通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面:
( q( Q, h# K8 ~! e" l s 
1 k4 q6 a0 ?7 u2 ]$ | 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。
; D1 g2 G. u' K+ a; d- L% [ 点击“Options”选项卡,在其中可以配置一些检测规则选项:
7 g0 @1 M: ^9 S/ j1 h  4 x; \; f6 E2 b- w9 t1 E2 |
其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。 2 _! K5 r) g& J5 y: f i
四、FindBugs: & Z9 T* C2 Z# ]( D' J5 H
1、整体介绍:
& \7 Q' W' ~5 m% |4 L% |+ H0 h% C FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考 3 t" V; T! t9 ]9 f5 J1 I8 C1 S
2、安装步骤:
4 O, k1 r/ P2 U, U$ M: s7 W 通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可
% ]* y7 B/ @3 S 3、使用说明: 0 {: l( Y6 H+ Y7 s8 ^
FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project : `) C. k- e) [4 N

9 ]1 T6 |, C) R* n* R 分析完之后就会出现结果面板 % ?- M3 n5 E J& {' E& n8 S
 8 N. I/ d9 B3 R! X4 w4 R# [
点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行 G; _% i' w! d
5 O! c8 E" u5 |, W& W$ U 4、附:常见的错误信息
/ j i, p* q* L R5 h! Z( J 4.1、Bad practice 代码坏习惯: : b0 J/ ~8 |1 J# V) i6 c" Z
$ R/ t8 j% B; O) v+ E3 {
图片 # a3 @7 b n5 v
4.2、Dodgy code 糟糕的代码: + {8 p( N$ Z9 o' A% E2 V/ A4 Y
9 w& f! [7 g4 t+ C: H8 A" b 4.3、Internationalization 代码国际化相关:
% d2 t' F5 H6 B1 o 4.4、Performance 代码性能相关: 1 c) E1 b. k& M; X( N* s% D
2 H) [( e0 m: t; B
4.5、Experimental:
: J: ~& c! H/ Z3 [% e# Q) }$ i- ^- F
4.6、Malicious code vulnerability 恶意破坏代码相关: ( v: y1 M- ~3 F2 T4 I; U
+ ~" ^7 y. `8 ^) u$ e
4.7、Multithreaded correctness 多线程代码正确性相关:
6 |% y0 Q+ ~/ ~5 x1 ] 4.8、Correctness 代码正确性相关: + g4 o. j4 n; q
% S2 g0 X: M6 [$ s6 t7 D
五、SonarLint:
) j8 j e6 l7 L! v. i- h 1、整体介绍:
' e$ F: v, s+ P! T7 x( X sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。 * M7 N! i( T# o. P" D6 Z
2、安装步骤: " N& `( g( Q3 h. v4 Q8 M: ]" N
通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
: r3 o3 O& x# X1 K: g 3、使用说明:
$ k: L8 e- |, ~- Q- n8 o& J5 P* a7 [6 w3 y
右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。 8 b+ a; C+ a- V6 `$ V
$ V2 ?6 E; D2 A1 r# n
4、配置 SonarLint 服务端: / {. D1 Z) \9 h" J8 `" j
4.1、配置 Sonar 服务器: 2 N5 g7 Y! r" b7 g6 V& ^: ~' j* M
sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查:
7 C( J; |# B6 z* i$ |! n* S; i9 Y4 l) I* w9 M$ w$ s6 e5 h+ i
点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 - Y1 ?0 H5 X! i# o
, t! b4 p2 H2 b" R
4.2、具体 Sonar工程配置:
( t2 A3 J6 j5 v1 b" Q 配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: 5 U; p# l! u; v9 `0 l
3 M; B% y1 O! x, R
4.3、使用 SonarLint 检查: : A& u. h `" ]" ^6 w8 t W
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查
* t( o- k ]" ^; a, Q. D9 `: U4 F
2 |- y/ s6 p) @! e* e' S 总结 & ^# k1 D: g8 j, ]* g' t, m' t0 O0 Y
) q- I7 _: O4 j9 r/ H1 n
检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了
# I9 G5 v# {3 z* `' O 找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:
# |) b B6 v5 a8 k( F0 _- T B PMD 自定义能力强,用来自定义项目BUG规则非常好用4 z. P! n. W7 ?
Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。
$ l1 I7 Q0 _' f/ i SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景
; S1 r( A. q* r. m0 J
$ }, x) }' `4 | 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多
/ Q) O' U \! h, }; V, [6 N# b2 v L- B3 ^
' H: T: f S& C6 ?, i$ k+ R 责任编辑:
( K/ {3 m2 }6 U9 t2 z8 o! Y
9 v& x, n2 b, V: E' [% X3 K
& c9 h% U" Y- E0 h+ O; c' u- K" A. X9 T6 | l6 g7 @
! i7 U( T: s" O: w3 W" Z& o* K
|