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

3 ~( m. s* V% P3 U0 U 3.2、菜单功能:
3 ~* _; q2 `9 y& g) j' p, ^1 R' Q; R4 v: S) u @& ^# `
编码规约扫描:开始扫描代码 l. v4 e. t. \
打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能
y( Y7 l$ a5 p 切换语言至英文:中英文切换
+ d6 B+ h' V" h2 Y" ?+ z: ^ V0 B; A- u; L- C
3.3、运行结果:
8 H7 m+ z$ Q: ~ 扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。 ' n! i. A1 E9 ^

) J" x4 ^7 {+ ], v% `3 |/ k 选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到): 7 M" y: J) i: r, \- |+ I4 [ z7 ~
 r3 ]1 [! \" l2 F/ [ X
(1)指定区域搜索同一类问题:
8 b1 K1 \$ y( K3 |+ B2 ^ 当点击③处的按钮时,会弹出如下按钮:
' S& |' X6 [% v( \: I9 F Q  8 t6 S4 p. I3 X. I
这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示:
9 ~5 X6 b5 s. z$ Q 
3 T. ]! f. @' p: q 这里我们可以看到,显示了整个Project中的所有该类的问题。 7 c- ^% ^' l& x+ r$ U
(2)预览具体的不规范代码: $ j0 L/ V. T1 L
如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。
S. ~# `, P. z- L0 O1 c  ( x( X& ~5 Z h# M' ^# @
3.4、工具栏功能介绍:
3 X+ D, F) T2 Q# w  ?4 N5 [1 J+ r$ E* J
1 B2 V, l z5 M+ j
Rerun Inspection:重新运行一次扫描
8 [8 n- Z; i6 C- v) d, F Close:关闭真个AJCG面板+ t- d7 p8 t+ k
Expand All:展开结果的树状结构,整个结果是树状结构的。& n$ x9 f- t8 `& k! Q
Collapse All:收起结果的树状结构
7 o7 q3 z9 Z4 t Go Pre Problem:选择上一个问题
8 j* y9 v3 P* ]! J1 r! n Go Next Problem:选择下一个问题4 ]8 T7 L& h( T- v' a6 t& V7 U
Help:帮助7 n$ `. E1 A3 Y; P+ W
Group by Serverity:(不知道如何描述)5 I! w' ]) o; w% f
Group by derectory:按目录分组/按类名分组间切换
' _5 s; A/ p% { Filter resoled items:过滤掉已经解决的项6 L+ D4 V+ Z+ v. W+ C* P* V4 w7 k7 `
Autoscroll to Source:自动滚动到源码
+ S3 [. s$ y1 j8 ?9 P Export:导出,可以导出为XML和HTML两种格式
0 s) \( D) @) j( n6 I! Q6 a Edit Settings:编辑设置
}" R7 Y2 A4 V: t, r* W9 G6 G4 S4 n9 [9 E# q
二、CheckStyle: 2 S' q: c% |* U" Q k
1、整体介绍:
7 T+ q& B1 f+ x" K d6 I" K CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。 . h5 e. M+ |' d: s9 X# A6 Q2 e* Z* ?0 ^; V
Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。 8 _7 `( P9 |8 m, K( M8 h8 p1 A7 k
2、安装步骤:
: @/ b, ^( t7 h- g* [( m; d 通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 + Z7 Z: e& y5 D# l& e' q
3、使用说明: . p& A5 M% C# D4 _# Q. O4 ?

, h) `- ?! ^! O( ]9 x9 p 可以看到基本都是一些缩进啥的编码规范,可以不用太关注
9 w3 X5 M- A0 \. b2 \( k 三、PMD
4 u9 s5 M' k% z 1、整体介绍:
+ ~( I ?* H2 v, D PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
/ d/ ?; B' d+ u0 B( Q" y
$ Y" E* B5 @4 L5 E) [ 潜在的bug:空的try/catch/finally/switch语句$ e8 V* ~7 z8 Y% T/ v" O: r* o
未使用的代码(Dead code):未使用的变量、参数、私有方法等1 C- M6 w0 U* P# Q
可选的代码:String/StringBuffer的滥用
& h, r @" @+ ?( e 复杂的表达式:不必须的if语句、可被while替代的for循环+ n# d: t% x2 C1 U( u* }
重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs" P' H2 |3 T/ v
循环体创建新对象:尽量不要在循环体内实例化新对象
+ c; U9 F( j" t 资源关闭:Connect,Result,Statement等使用之后确保关闭掉: a! W& [; H$ \+ J; O; o7 i. B
: j) d6 z5 P+ w6 n' K1 V 2、安装步骤: ' o8 R+ {' k7 M) F9 K* F, d( }
通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可 7 f* r1 d; q& \* ^7 b0 K
3、使用说明: ; E8 u' ]4 ~7 L" z0 @- J
) o5 s* W6 x# J& f [
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html( X+ Z# J7 J4 k: q" [9 H% B
% Q6 T& u& R2 d2 T% R* Q& u& e3 z
3.1、运行方式: 2 B) e0 A$ B' ] B* R
(1)从Tools菜单中启动:
. [: i K0 \' w3 k0 a: E, v; t/ o 通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。
/ d5 E" P( {7 |" B
0 i, f# a1 l+ L/ g Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。0 {9 y7 U: h+ R5 X& ^
Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。
, ^6 F5 N. S; T/ u" l4 F; Z- N6 x0 j6 k- G
(2)从右键菜单中启动:
0 }) s- C2 Y6 I# r 在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 8 n9 x# q. _& e# H# @
3.2、运行结果: % S& i: _0 p% C ]$ R& I4 L. ?% s
 8 X6 v J% n7 f
运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 # f; U# h- y3 w! w9 q% ~; c
3.3、配置检测规则: + c" s6 b, P1 d6 j; j
通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面: * ^. R3 }# l+ j* _& n

, r0 E: f- i9 c" V R, K% i: _ 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。 & d% F* ]# d) e, M) W% N; |1 Q
点击“Options”选项卡,在其中可以配置一些检测规则选项: ; W* D+ l" L L

$ o: R. f! {! k) A# ~) o6 d 其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。
$ } z& Q# [1 D/ |: j/ [ 四、FindBugs: " a- u0 B% u* A3 M$ K0 j
1、整体介绍: + Z, }# {! x7 E
FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考
" j3 l) |1 c$ T7 L- Q9 U 2、安装步骤: + E; [6 H9 P" l" ?
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可 2 Z4 p5 P3 |; t) a+ p
3、使用说明: / o J) s+ z6 G8 o7 U1 k5 E
FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project % _7 c6 N2 }9 z7 ?
 4 i, g$ u$ X; D ^/ |) {: n/ r2 r
分析完之后就会出现结果面板
' d" _. m7 k% M" H" b" E& ` 
+ [/ J6 k5 j; X3 @5 S/ a3 i5 I2 S; H 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行
6 o& O/ T. j, z! V# n
3 H8 H4 m1 F, d 4、附:常见的错误信息 1 R( U3 V1 n/ O% D- b/ e
4.1、Bad practice 代码坏习惯:
/ b2 d; W4 F( t
8 [# Y+ e* E' j: ]2 M 图片 9 W2 u6 q' z8 m$ K
4.2、Dodgy code 糟糕的代码:
; K5 p2 T$ x) b$ i" e0 W! I; {+ A# [8 U' k) [
4.3、Internationalization 代码国际化相关: ( \1 U0 v; U7 d2 K I' b- V
4.4、Performance 代码性能相关: 2 \, V; a$ r( ^: T1 ]
1 g0 S) G5 F' v. a6 z4 B
4.5、Experimental:
5 \) K0 O' o# | y6 H4 ^; j# `: I' D& H8 m
4.6、Malicious code vulnerability 恶意破坏代码相关:
/ a9 x0 ^! P" d( s4 \& R" u
- a7 b1 s$ U( u u, q/ Z 4.7、Multithreaded correctness 多线程代码正确性相关: ( Y8 I' h2 R) ^& E6 j
4.8、Correctness 代码正确性相关:
6 X7 ~6 G) j" G2 y8 T& |3 Z2 o
五、SonarLint: 9 \! b/ h. K2 P% M7 j" c0 u7 p
1、整体介绍: 4 x9 ~1 Z7 f# x7 L# i
sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。 2 G, E1 {% x, Z; g
2、安装步骤:
8 W: Q6 _+ I8 l% X! h/ V1 n 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
! i7 p2 ?( f& v: R 3、使用说明:
' g3 B# \7 m# l& s1 h f: l; Y! G
右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。
+ U! P' U$ r3 D( w
2 U! T% @3 j% | 4、配置 SonarLint 服务端:
" w, Q8 r. D. w" @& \% J5 D 4.1、配置 Sonar 服务器: 1 o7 O+ [7 l' }, i5 p
sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: ( `' W7 s) X; E# @9 U
\1 T1 `" m% G! |9 k; @) W# o
点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 " ^4 j$ M* d: o2 S* d" w* N
]$ T! @) a6 i3 A; c# T 4.2、具体 Sonar工程配置:
; z; N7 n. \- I( U% e' o1 T1 J 配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程:
' b* @" s! S& Y" K$ b
2 P# R5 @1 g; q5 C 4.3、使用 SonarLint 检查: $ A/ x1 q0 m; G9 ?; B [0 ]
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查 & U+ U* z5 D e3 {# N1 S
) {+ M) d3 l( C 总结
8 I+ S" G% V! k3 \$ z
. n3 f7 @: l8 _# z' K! a4 m, s 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了. o7 x P5 U5 w" W# o5 g9 F' T: {: o
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:
, i. Y* [% z" H" h7 p2 p7 a1 }' B PMD 自定义能力强,用来自定义项目BUG规则非常好用; A- [' c" n6 \: d( O& P; e
Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。/ Z" C M5 w) P8 L
SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景/ e a( J% N3 R: ~0 S) \" ?) K3 j( ~
2 X1 o3 H7 ~9 P4 t& w- A 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多
0 N! D( @% v% ?7 l4 c7 p3 Z' i1 S. ^! _) P2 P1 J
责任编辑: ) A2 p# N" r5 W& n
* b) Y# P- F- H% H: r
- U" u8 a4 J+ F8 `9 z' C
" c1 N$ T* c# d% M, a( F
+ g- C2 B: C1 a2 B! O
|