|
6 `: Z) K9 Y$ N! k$ b 原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
: Q' l* G3 F) H8 F5 u, K. C- C+ e: F4 y6 _, B
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。 , G% W' r, h' D( P+ G. I: F
工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力
; F' w# O0 l: f% s 一、Alibaba Java Coding Guidelines
, x4 S6 _' D+ E2 e 1、整体介绍:
7 {; b& f9 Y% M, W Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。 " E1 b. ]. l. d# _
阿里巴巴规约扫描包括:
5 }* W+ O6 B# v/ r- i9 J+ c! w' p+ c3 e. S$ D( j/ ~! M2 @6 W- v5 @
OOP规约4 L8 S; ], Z. q$ p! T0 S$ A. @
并发处理6 y$ I, j! b; c. Z% T3 z
控制语句
$ B3 ^% ^3 }) w4 {. S9 Z0 } 命名规约
0 x7 g0 a$ P0 S% G# a& Q4 t' { 常量定义4 M5 |, C) R) \# s& O5 O" }( A
注释规范
4 I- e3 }$ l* a" U% c" L
) Q! ^5 V P" O7 n0 O 2、安装步骤:
% J; m* E' u4 L" ]$ f File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。 $ {/ S, T' a F, h& _" Q# }
3、使用说明: ; s* M6 v3 s( ]# v# m0 I
3.1、运行方式: 5 a" m- k/ Q- c; C2 _) x2 s) P
(1)可以Tools > 阿里编码规约 > 编码规约扫描
2 T, w' G: t% I, S 
+ m6 P \ l4 r& _% i4 S (2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可: 6 N- g5 ~* w5 N

, U, E& V& d4 x' X% H8 A( W 3.2、菜单功能:
! N8 D# l8 {" L: @3 t" }, ?4 Y/ A+ g3 a( q* Q v! A$ m3 w, f. v: k
编码规约扫描:开始扫描代码* [ m5 _+ ~$ d6 g$ Q- B
打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能
3 G# P3 r6 {& j. C7 F9 C6 O# \ 切换语言至英文:中英文切换
1 U( c+ \1 |. Y' X, q7 S! j, d3 h; x
3.3、运行结果: 5 C* O+ \9 ^9 e* r0 O( Q
扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。
: |3 C% _! P+ D, U* k 
& V$ k* w& @: J8 E% z 选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):
5 T& W; B9 e9 U; R" N3 l  2 S# C7 V# D% {# m2 j/ B m
(1)指定区域搜索同一类问题: ! g/ ]; s4 S4 h0 Y3 f' S
当点击③处的按钮时,会弹出如下按钮:
: h: S8 i. J, k% t3 q, i 
6 ]+ n( `2 }' p( V 这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示:
% }6 D `$ ^# k  4 [* I5 R0 n1 S3 f( I# N/ s
这里我们可以看到,显示了整个Project中的所有该类的问题。 3 ]: U7 L7 P: n7 @
(2)预览具体的不规范代码:
( V3 Q2 U4 M4 B* Y, T0 m$ y4 @0 ? 如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。
+ h1 C; x; [6 Q+ f, c$ `1 c) E2 `1 a 
/ X( j' A) n3 s" Y) c8 ~" M7 l 3.4、工具栏功能介绍: F* |& n9 \9 Z- t. ]

& T4 R" @8 C( _1 ^1 A7 K6 S. H# w
5 T6 A/ Z. `& P6 E5 j" e Rerun Inspection:重新运行一次扫描( h& b+ @& p5 d9 X6 i4 C
Close:关闭真个AJCG面板
/ h r8 W: B! D8 ?4 V% a Expand All:展开结果的树状结构,整个结果是树状结构的。: F) t2 t6 k4 b( S: `' R' }" h' U ?
Collapse All:收起结果的树状结构! e: W- Y8 [* ~, P5 V4 k3 b
Go Pre Problem:选择上一个问题" U% N g/ g. k$ ^5 ^
Go Next Problem:选择下一个问题7 R1 O6 t( `" p. P8 O) {
Help:帮助
1 a0 a: _4 } O6 L* _* f1 f' r Group by Serverity:(不知道如何描述)
4 ^3 t$ o; E8 I# d1 |+ F, ~ Group by derectory:按目录分组/按类名分组间切换
/ p# g7 r9 I0 T5 D# k Filter resoled items:过滤掉已经解决的项7 c- r* R: {- ?+ X1 Z/ h
Autoscroll to Source:自动滚动到源码
5 g0 [$ h3 b1 S0 t Export:导出,可以导出为XML和HTML两种格式
: [% m$ [4 [$ n5 m; L) J/ O Edit Settings:编辑设置
* V, e+ B" q. y( P3 `) e: o U5 z/ Y" q1 x- X. ?2 G
二、CheckStyle: # i% {+ L* g$ B9 v+ @( K+ u
1、整体介绍:
" _; n3 @; S) d/ R( [ CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
' A/ n. `1 }# c: o% c) s& D Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。 : @7 J: t, _, w1 ^+ g% I! Z
2、安装步骤:
% U, Y: \, ?0 _ 通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 ! u* \! t0 K# d' B, A5 ^. \3 {3 r
3、使用说明:
$ r# Q# r5 e; U$ }( @ f [' k4 f  " V9 M( N" _; |, ^/ k& t- l
可以看到基本都是一些缩进啥的编码规范,可以不用太关注
2 _5 |* C; g) m2 c 三、PMD
% T0 w2 j3 y: u: ?9 q" b, H 1、整体介绍: X3 Q# K2 E: ~* L+ L& O; f. M! P
PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
! `0 V. s0 s- s' }6 G/ }) J& W! ?- S% Z+ t! n
潜在的bug:空的try/catch/finally/switch语句
: P" Z* l' }/ q& \2 b 未使用的代码(Dead code):未使用的变量、参数、私有方法等
+ e. g5 P! n5 x6 u- I* r 可选的代码:String/StringBuffer的滥用
/ v& ^% F6 M( i) A0 X$ g* S 复杂的表达式:不必须的if语句、可被while替代的for循环
) l) _* H) d1 b9 J3 ^ 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs
5 B3 p3 \5 s6 s' A8 o 循环体创建新对象:尽量不要在循环体内实例化新对象2 L) F) K" @3 d% ~! p
资源关闭:Connect,Result,Statement等使用之后确保关闭掉
6 ?3 ?& j: u: O/ J# n/ I; }
- F, r' n: |# k8 _+ X$ ` 2、安装步骤: ' Z6 t0 {- k( X( n
通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可
& ]1 ?) h1 @. u i 3、使用说明:
6 N3 l- \4 ]: u8 G z1 o/ P1 g9 e/ |$ h8 ~8 Y. U/ [8 a7 ?/ F
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html8 d2 D5 u4 i( r1 P+ }4 \
. [0 l) a3 Y. R) r- p4 @
3.1、运行方式: + n6 \3 K( o0 v! u
(1)从Tools菜单中启动: ; U6 ^% U7 m$ Z
通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。 0 k- g; L2 ^5 A/ G: ^5 O% c: ^
0 R( `) V K& Z3 T Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。
4 [* `9 S! _3 C+ P. D Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。2 p; d1 m& E! k9 O2 `3 `/ t
0 i; d! {. G+ h4 V& a4 e( a (2)从右键菜单中启动:
7 E6 B h/ f. A6 H/ C! y7 D/ {- H 在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 3 ]. L( U( l, x+ w3 O/ _6 j
3.2、运行结果: , e; ?' K; d0 z+ J5 g+ n

( |5 k, v" A$ t. S" P; t& ? 运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 2 h- p5 ?; R, I
3.3、配置检测规则:
' |9 Z3 b, q( c- f( a 通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面:
1 k9 y2 x- D6 f5 w1 W 
5 \, D' Y. o% m& \1 E 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。
. P0 m+ i# m/ q! y: `, { 点击“Options”选项卡,在其中可以配置一些检测规则选项:
& k5 E- V- L1 i5 L( @$ a  6 K6 M: B( M# a
其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。 % V7 e3 ^6 a) V) w& P( G
四、FindBugs:
; E5 c2 k/ {" Q" g- Y 1、整体介绍:
& q( U$ j+ T9 k4 G7 n" e; }3 H) ] FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考
1 [) D9 D/ d& F$ n 2、安装步骤: 6 `, h, v3 s8 K/ N1 b- I
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可 - Z( [. j3 ~) g( y7 \' f
3、使用说明:
# f$ M; j1 Z" B FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project , E! S: _5 ?$ \: R8 l
 1 R) s: Y' `* E: ?/ S+ ~( r
分析完之后就会出现结果面板
2 a8 _! ]# L, G) g2 { `  1 E6 j9 Q8 M3 {( ]( a
点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行
6 V# J$ B7 Z6 p( X$ C" X1 ^ k9 I' F/ t# T1 Q& z. ?6 b4 z8 g
4、附:常见的错误信息 * G/ p @) U2 A5 c
4.1、Bad practice 代码坏习惯:
8 A2 n9 e7 N* F5 b
6 v; S" n9 }9 ~- e( ^- l 图片 ( d1 B3 v- v5 q
4.2、Dodgy code 糟糕的代码:
# ~/ w3 k; U4 j* C
2 Y, H. i: D& }- e2 S 4.3、Internationalization 代码国际化相关:
, r# p; Y; j0 d7 S U 4.4、Performance 代码性能相关:
2 z- Q) w! E6 Y+ J! o8 l& [! ^* P8 ]2 M( [0 u e
4.5、Experimental:
: F/ A* N! \' k0 q! B! U$ E5 F: S& }7 c; Q$ c1 Q# K2 `
4.6、Malicious code vulnerability 恶意破坏代码相关: , u. X* m* N- }. a. e2 l
3 R9 O! H7 D, S& e0 q7 U 4.7、Multithreaded correctness 多线程代码正确性相关:
1 D4 d5 W0 E9 L4 d# w0 g: l 4.8、Correctness 代码正确性相关: ; \& J2 c9 w" Y2 v n {
0 ~7 Q/ _4 o' ]6 Q8 Z1 ^8 J
五、SonarLint: ; _" T E/ t$ @& @: A c
1、整体介绍:
( ^' x- i6 f& X$ J r. r- \ sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
7 H, \8 ]+ Y- b 2、安装步骤:
3 S Q2 [ V: s8 D4 m. X" X 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
- w6 ^+ e$ N5 S' Y6 j 3、使用说明:
# q& k2 l2 p) n! C$ L1 s
9 z9 W2 F& R; |& z g A, |/ S 右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。 * m+ g* M) S' S1 j! }% [% M8 `
6 D% [+ b; F! C# R) d 4、配置 SonarLint 服务端: 9 `$ H! J. c" H9 ^. A* C
4.1、配置 Sonar 服务器:
4 t% C7 Q" w! W# Q/ V+ X0 j sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查:
; _6 P' ?( N! @0 Z$ K$ u
2 b4 m: U# T N. b% \& s 点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 0 i& Y! d. v5 O O
: ]) r( g6 s. R! V8 u
4.2、具体 Sonar工程配置: " k% E1 c9 z$ x( e: T# `
配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: * s2 h$ \& D0 e& W3 x! j
9 k% M0 p$ F* E
4.3、使用 SonarLint 检查: , X$ K G" R# A6 K, \. h/ ~
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查 " c4 ` A; H. o$ [" x
5 W0 D/ x5 n2 w& ? 总结 / L2 p1 _: i! f k7 F' ~9 T
0 H! G- n$ l- T+ g. i( ]
检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了0 K$ x/ @7 U' e4 G* z% G. z
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:! q% h" B' c* R3 }
PMD 自定义能力强,用来自定义项目BUG规则非常好用
7 f1 Y5 S3 i1 ~: c: y Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。4 K9 T0 s- |" \
SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景7 ?9 }2 |1 N) |9 J2 ^
) i% e- |. X( Y, v% w1 W7 d3 |
文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多 $ r: J: |5 r! B& p" k$ B3 ^
m2 [4 U$ S* s. @1 v9 |, j7 {
责任编辑:
4 Y6 Y, Z- T' ] X' H, p4 k. y- s$ U2 x+ Z! x. n" s
. I! Y0 n e* V& o0 M3 C
0 [6 f/ L9 \9 p- L5 p& ]6 q. q2 C- q; O# X' }3 L `9 p
|