|
8 R0 \ {0 [" D" v 原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器! # [4 K* M; j( R; s; |4 I
. T2 q# M y8 I- O8 P& u
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。 ( C z4 b0 w( W$ @& B! o
工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力
6 f/ G) u/ Q. L2 h# ~ 一、Alibaba Java Coding Guidelines
* @/ g& Y4 H- f# X) R! |, o 1、整体介绍: 6 W G* c: ]9 S0 O, E4 P5 _
Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。
0 o9 p4 L: W& I; U0 Z& Q9 |% V 阿里巴巴规约扫描包括: % V0 M7 ^2 Q, |8 v3 L. {( U& r
; u0 t1 V+ j" s OOP规约
$ N7 g6 G0 O, V1 T 并发处理) S9 r/ L$ B ?6 @" w8 m
控制语句
* n8 d: Y, W; k Q3 `' z/ t* ] 命名规约
$ x# F$ Y0 [: \4 M4 e" r 常量定义' L# N/ N+ M$ k8 q9 \$ f
注释规范5 o* {# M* i' u1 N
! k m( L2 \$ N$ d- b
2、安装步骤:
3 g& }; @# G L& v0 M" P File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。
% e/ Z- r) Q1 R7 R& I: R9 a! w+ A/ w 3、使用说明:
1 \8 Q" A! V( ~! @- K% G3 [5 @ 3.1、运行方式: * z8 B' p, m. @- ]
(1)可以Tools > 阿里编码规约 > 编码规约扫描
/ a% o% Q1 w& n! D1 w8 g  : c% [' j6 u: s$ V4 J% ^# y0 {
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可:
" l2 m; S7 o' m U: V; e5 l+ U 
# _9 |/ N8 R, h* n- n 3.2、菜单功能:
% |5 l5 K. K7 o- u w, c0 s# b b/ ^! y' c- ^6 u# I5 r$ P7 J
编码规约扫描:开始扫描代码 F: i+ I4 N; P! G. H# d2 P
打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能* o1 {+ A6 n9 h- I8 t2 j. g
切换语言至英文:中英文切换
5 V9 ?5 a0 t" e, C% G: Y7 K) c# d% M# F9 @9 z$ [
3.3、运行结果:
; L) r1 G$ `( y: k1 k1 b$ s0 |" S 扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。 8 }4 s0 C( U* e7 Z1 P7 l- h" R

- {; D: a6 Q8 b; x% a( i 选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):
( n, y& ^/ g9 k7 a& B$ [: v/ ? 
1 G! O3 v3 h! G$ J6 a N (1)指定区域搜索同一类问题:
* A* l4 g+ ]4 r! G9 I$ o 当点击③处的按钮时,会弹出如下按钮:
8 s( i H9 k; l8 T8 I! b1 n! H  1 u/ g7 Q3 \5 S$ Q1 B/ Q% }4 O
这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示: 8 t$ n2 w* A3 X' k( a
 . D4 W4 m f7 i4 l' b
这里我们可以看到,显示了整个Project中的所有该类的问题。 4 ?" x- e* |( a1 i# K7 S5 U8 D
(2)预览具体的不规范代码:
' _ U& p" i- s 如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。
, E! D8 H9 E0 a3 C1 Q5 p9 K& m. [ l 
+ n) K2 ?4 b7 N) a2 @$ b 3.4、工具栏功能介绍:
3 A7 o* Y g }0 I  6 m: t; u! {' k# O5 B7 l, Y U
9 ?' J* Q+ d; j6 L, ]0 I$ i4 ]
Rerun Inspection:重新运行一次扫描
* _! a9 X7 y, L$ t6 X. n- F Close:关闭真个AJCG面板
: P. [! k0 {. I E$ @( @0 _ Expand All:展开结果的树状结构,整个结果是树状结构的。- h* B/ Y2 u+ E) R. D6 p8 S4 r* l& K+ \
Collapse All:收起结果的树状结构9 V' a( `4 d7 h: a
Go Pre Problem:选择上一个问题) x! R6 q2 N3 I; l
Go Next Problem:选择下一个问题: m) m( H+ E7 S1 g' m, i
Help:帮助: |) q+ G+ `( c1 Y0 i$ M7 \, g
Group by Serverity:(不知道如何描述)0 ]% `( `5 e- s# u" n* o. R
Group by derectory:按目录分组/按类名分组间切换& L! t4 n/ x5 ?- D0 Q: d$ ]& X
Filter resoled items:过滤掉已经解决的项9 [! t1 \: v8 O. h
Autoscroll to Source:自动滚动到源码
4 o4 [, q& G) y Export:导出,可以导出为XML和HTML两种格式
$ E$ k* H/ u2 R# N1 f. G' V# P Edit Settings:编辑设置
% [$ o, p8 ^+ K% @
. V) m' I% N8 q) T9 ? 二、CheckStyle:
$ y5 e- h4 B* j; D1 x$ y 1、整体介绍:
7 N2 p N) `) d/ \ CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
+ ]7 f j; f5 o+ i2 | Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。 1 g" ? v( d) l
2、安装步骤: 3 }# [, l. u1 t; d, \4 e0 n. I
通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。
. K) g7 K6 L8 ^5 ~2 j. _" Q 3、使用说明:
8 d. ?6 ]. r2 o 
) q( |2 b' l/ O, Z* R$ D 可以看到基本都是一些缩进啥的编码规范,可以不用太关注 - ?8 L4 G% Y' n0 G" W
三、PMD $ A8 i' W$ f* R q1 M# m) Q. _
1、整体介绍:
4 r o% R# e% X PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
, \. i5 r3 @; D% h: u7 ]7 s
& p' P5 |& H$ d1 m ]( G- r 潜在的bug:空的try/catch/finally/switch语句
( ?7 T( F0 {- i( t# Y+ U2 N6 f 未使用的代码(Dead code):未使用的变量、参数、私有方法等3 w# [( |) f L
可选的代码:String/StringBuffer的滥用
2 a/ N$ R9 c0 M( M 复杂的表达式:不必须的if语句、可被while替代的for循环
8 l; X3 p0 ~6 u' B' R+ g& I, ` 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs# H7 v. K" `1 A: K: d
循环体创建新对象:尽量不要在循环体内实例化新对象/ c* K% j. b& F
资源关闭:Connect,Result,Statement等使用之后确保关闭掉
~* m* l+ g# @) B% [3 [
6 f5 n- @- e/ s 2、安装步骤: : |' e5 |+ |, r. g
通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可
G# l6 |1 t. a* {9 A% p 3、使用说明:
3 ?# p7 g4 F5 @1 o* y0 @, p4 ]9 f( j( d. |
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html
4 I$ p4 s' O' g* C# V
: B( K( e3 S+ a8 d 3.1、运行方式:
" Z1 V( \. {+ O$ Q (1)从Tools菜单中启动:
) @& p* E2 @5 J( ^6 A9 R+ B+ a 通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。 - T" n/ Y! |6 L8 O9 W* t
8 T n1 E5 T9 _- U+ f. R Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。; e" j; j- ^# x7 r2 N, o
Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。
u% B- C) p7 G) ]( W. P$ D0 `* W1 c9 i* _% w5 U3 Y7 N
(2)从右键菜单中启动:
- @. T& L% ]' J% t 在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。
+ t& g3 @1 O* k4 l1 f. A 3.2、运行结果: ! b, l7 Z# o: a
 7 L/ S; K, \1 V% z
运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 0 U* ?! x, S' h- Y0 j# L5 ?
3.3、配置检测规则:
, H, C2 k4 I: Y n, i 通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面:
3 }1 `, S: C, v8 i" S. Q& L0 K  - X1 ^, S$ b. ` e
在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。
1 F; W! o+ @0 h, s% w 点击“Options”选项卡,在其中可以配置一些检测规则选项:
- V) r0 T/ s) I! G! ` 
! q* V6 \+ n4 ^' {5 b6 j 其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。
# `/ B& F- j9 V- `0 j3 r4 R, x 四、FindBugs: ( {1 T$ O9 ? r' [
1、整体介绍: ) C1 }+ d4 i1 U" g& X) y5 r) @1 U
FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考 * V1 }) l" U2 `: B
2、安装步骤: ) D, ` g1 U f# l! {. m
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可
0 Q0 I, U- Y: F9 |; O7 y* k, B: P 3、使用说明: . ?0 M/ V6 p0 C- N, j
FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
& n: F5 V7 i8 \$ K1 D2 l( O6 \ 
6 p) A) A, a# ] 分析完之后就会出现结果面板
: Z' R* q- \2 Z- x% {) s 
( j) V' @$ T9 N1 R' k7 H0 Q; v 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行 4 l% P" l+ C( W) t2 Z
# g; C/ L. }& M, Q0 e, i% N
4、附:常见的错误信息
, ?6 e' K5 Y q8 P+ p. @ 4.1、Bad practice 代码坏习惯: ' K( V8 }6 }2 b
6 r" ]. C' X; {) [& {/ S$ C 图片
, D& V- p1 Z# Y# `) ` 4.2、Dodgy code 糟糕的代码: - m# m5 m. T0 r# u
) n7 O, a# c; D' K7 O7 d
4.3、Internationalization 代码国际化相关:
' ^- d; e& I* X2 V 4.4、Performance 代码性能相关: % w* k% W6 }2 q" Z) h$ Y1 C. l
" w3 }+ x" }5 C7 S
4.5、Experimental:
$ P" B! r; Z9 y$ p
1 {3 O# R% ?" V# K" g6 ?+ I 4.6、Malicious code vulnerability 恶意破坏代码相关:
6 ^2 Z( m7 P6 Q3 W2 Z1 H; ^! _, S) T
4.7、Multithreaded correctness 多线程代码正确性相关:
5 p8 ]/ _* p6 O+ j9 B. g0 c* Q0 Q 4.8、Correctness 代码正确性相关: : r. e2 h% U- I
' T0 ?" y3 }9 E; [& U/ B 五、SonarLint: 7 H5 V' R9 R: A7 S( {; [
1、整体介绍:
& A) }$ m! M8 C9 ] sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
4 S# ]* J( l# x0 J9 R" q; Y3 h+ y 2、安装步骤:
# B' w& |! f9 I: [5 \, r; O 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可 5 o$ _6 o; h3 f0 n9 B
3、使用说明:
2 }2 f. k) `: z9 C `2 q9 v1 v5 j, }
右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。
% z* ?7 _% w3 V* l/ e/ C5 j1 K! B) N J, ~
4、配置 SonarLint 服务端:
/ @$ R' d* l+ S [3 m7 ? 4.1、配置 Sonar 服务器:
( B0 Z- H% Q% M. P6 ~ sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: : D. _9 g9 ]2 b" q
2 J! R M0 ?( g0 S8 G6 g* n 点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 ! X9 B2 J6 m; J) o$ s3 z7 Y; L
5 ^9 y$ ` \, w4 \; I6 A! A1 @$ q, I# m 4.2、具体 Sonar工程配置: 8 [2 K& G' H. G& e0 h. V1 k+ o9 G
配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: / b/ }) n. M( s' _ z% c/ |: ^
, ]- r; ]5 E" C6 f1 p! S' e$ q R
4.3、使用 SonarLint 检查: , |+ o$ s" D7 O' i9 U. B8 I
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查 - p2 A4 d$ a1 u: ?
1 @% r, f/ B2 e 总结 9 w9 O9 H1 w4 v; a y
) Y, o% u' d6 F& B/ n 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了/ b; Q0 G1 S7 t0 W/ ^
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:$ b$ v: g$ u& w3 F4 z0 w
PMD 自定义能力强,用来自定义项目BUG规则非常好用. N6 O2 Z& I9 B' H& `: \+ [1 r* |
Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。# g8 r4 L- b1 {% V
SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景! i% e3 m+ x. [# t0 @* T, w
5 C% H7 ~: ]6 y4 N; n$ o* U6 ^ 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多
2 s3 e* X. {6 n7 l8 I- B9 x# V0 C" L) \- Q6 s% a' N
责任编辑: 8 v* [ ?* ?2 A5 s5 Y
5 F9 W: j2 n" Z$ f
# r3 W' s9 G+ v7 @/ M5 e4 ^
+ Y8 b) z# U. V; w6 `/ O2 M
- n' D1 R- Q* O2 O' j7 v |