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

! \# v9 N: f) t: S& Z 3.2、菜单功能: . ^5 w* X n) R3 {
' u1 m5 J5 i% P6 R 编码规约扫描:开始扫描代码2 C$ q7 ^! z N# ]5 w6 b
打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能4 `3 v, e6 ?( m; C( ~2 H `
切换语言至英文:中英文切换0 x4 E5 z F8 _9 ?& V% N6 D, A1 z
" {- B6 m2 x7 f- }/ {) v: Z9 c, S7 r" X
3.3、运行结果:
. m' M& ]( L. R: i 扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。
9 O4 B% }4 A% A& Z( ?' \$ Y 
& `# y7 v3 \( O9 y. j( C* x1 B; W 选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):
; W) ]* S4 {, B7 h  * C8 u/ V& U# O5 g
(1)指定区域搜索同一类问题:
6 H. r0 Y8 R, m3 s 当点击③处的按钮时,会弹出如下按钮: 9 f3 u$ S& p+ p
 0 ]7 s1 d' G& B4 h5 F
这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示: 3 r" ~' q. {1 Y, g" c( B
 / n, l+ \+ v8 l3 r i1 R% j
这里我们可以看到,显示了整个Project中的所有该类的问题。
1 j& p% N$ ?1 M: a$ p5 u! p (2)预览具体的不规范代码: * X; h+ b0 x- e
如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。
- f9 T$ S0 i1 O `8 M9 E& j 
6 W0 A1 P1 u% p5 m$ A. ]2 P9 X 3.4、工具栏功能介绍:
! F' n- `, }0 [- T! N" ^- K 
5 U% d( u* Y1 G
4 s2 A6 a" J1 F: P m! J0 t Rerun Inspection:重新运行一次扫描* U( Z# T. @6 r3 X
Close:关闭真个AJCG面板! ]" Z: e' ~: D8 u# {
Expand All:展开结果的树状结构,整个结果是树状结构的。
# p8 F$ n) C1 w+ Z Collapse All:收起结果的树状结构
0 C. e- @- E# d- h S. F Go Pre Problem:选择上一个问题
" }; T8 Q, S+ h3 z Go Next Problem:选择下一个问题
/ d) {# }; s6 L! z Help:帮助
1 H# S+ Y5 o% | k$ E; J Group by Serverity:(不知道如何描述)/ I# ?$ r* S7 _3 \
Group by derectory:按目录分组/按类名分组间切换: `8 \/ L" i5 w8 [
Filter resoled items:过滤掉已经解决的项
( G s ~( D4 d) [% K: G3 n Autoscroll to Source:自动滚动到源码
8 |/ J8 v+ K1 { s: c- W& l Export:导出,可以导出为XML和HTML两种格式- q G3 z" i( D+ V6 x$ k( W$ n
Edit Settings:编辑设置
$ h! k$ J8 Z; E
$ Y$ q2 L# o. b 二、CheckStyle: + ]% k0 Z+ Y" R7 \# ^
1、整体介绍: 7 a) N# Y% C8 n! L! T' I
CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
3 k5 u: C/ X2 W2 { Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。 " J7 e5 G' W( b4 `
2、安装步骤:
+ u1 T6 `2 T+ U! W: R 通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 ' r$ {6 {9 Y% q) l, x1 B4 p j
3、使用说明:
) M I6 M0 n: W- P8 ?5 a, C 
) C! r+ k% |( S& u% i 可以看到基本都是一些缩进啥的编码规范,可以不用太关注
: H" [; _* Q' ~9 H8 o) d4 e6 A3 Y 三、PMD
8 ~ D( M8 ~! j( C4 X+ k/ V2 e# w4 E 1、整体介绍:
5 r. l i, W0 v' ^2 x PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容: - G( h, r! ^7 J% C {. w
. b [& A' {7 `# |# D A. i* N
潜在的bug:空的try/catch/finally/switch语句
0 A/ e1 W9 A3 C& u& d 未使用的代码(Dead code):未使用的变量、参数、私有方法等, Q5 F1 C/ j+ D7 L8 e) z& b4 E2 v
可选的代码:String/StringBuffer的滥用
% d; k3 t$ N: e4 d" w) }$ W 复杂的表达式:不必须的if语句、可被while替代的for循环
4 Q" }4 a i! ]. _: E+ ], x, n4 d; f 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs% g& o+ M9 L: }5 s, l) _
循环体创建新对象:尽量不要在循环体内实例化新对象
' z( {0 @3 \% r2 m: @ |3 h 资源关闭:Connect,Result,Statement等使用之后确保关闭掉 ]# i5 t% y+ r6 Y. `/ _$ ?3 s! u
- f+ p' V& N# X2 J/ v, i' R ^1 F; Q 2、安装步骤:
1 i6 |4 \' Q9 y- n; A 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可
$ |/ W. p; x" j' k U) V7 w* Z1 l 3、使用说明: 6 T0 B: ?% `9 ~2 Y& L9 @% {. ?# u
. e6 o! R' p+ Q# }1 ~: h( _% A 参考文章:http://wjhsh.net/andy-songwei-p-11830812.html5 o, q( b G" T6 |: z/ q
7 p" b; S+ r4 r5 j5 c& ?
3.1、运行方式: % B' P% u7 P" h8 W& H
(1)从Tools菜单中启动: - H l" X) q# o5 Z" t( W) g1 s/ R
通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。
" D8 ?8 c0 ~0 ^+ R7 ?4 v! N
1 U/ V# K! m* J+ p5 M5 Q Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。
% }7 O& J1 x* r8 \. b4 [ Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。/ \4 A( E7 d4 ^4 T/ H
: j& X3 |4 D9 d5 A2 u- @, O# g (2)从右键菜单中启动:
# ^) g- o8 c# A- a" h; Z* [* w$ N 在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 2 f3 A2 n' ]* u( s" J+ \9 M
3.2、运行结果:
# Y4 F- m% f1 a! P0 h( u  ) j; v. d; A6 b' R) g) [, N
运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。
6 O$ s+ C6 F2 _0 O i3 Y ]8 H+ n% V: y 3.3、配置检测规则: 0 E$ C! g" b2 b, ]" m
通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面: 5 W8 D; }9 H/ e: {6 I& O

" U0 F1 ]1 T5 P) Q/ }# l 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。
3 y; T2 H; X' D1 E# G4 @8 f- x$ e 点击“Options”选项卡,在其中可以配置一些检测规则选项: 8 N! W, ]5 G+ Y# V4 t# K2 ~
 % H$ E/ ?# U' T& F+ G
其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。
2 A: r3 A5 k5 h3 i4 U) [% Q# ?' V 四、FindBugs: / R! ]4 |/ ?1 u" j" I* b9 Z* m8 s
1、整体介绍:
L7 y- G8 @2 B: \ FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考
g5 P; D H5 m% R 2、安装步骤: ; s: K2 H# c) W Z& l: a9 e' F
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可 , a Y6 B1 V/ D$ f
3、使用说明:
; X% O' w0 Y; F9 \9 u FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
, G: a4 H! T5 x8 ^# {3 o 
1 N$ n# S% v0 s; s 分析完之后就会出现结果面板
9 r! U5 T" h0 Z0 u2 E. ^9 h; d  7 M8 ]/ N+ V9 ?7 G9 d4 g" I
点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行 1 u( H0 K q# V8 H, l# k7 Z
- L$ M9 b0 g( q' c: ]' ?- c
4、附:常见的错误信息 8 c$ T! ?! _; Z, D: l. S, g$ x, z
4.1、Bad practice 代码坏习惯:
" C! M( ]# k% ?( {6 w* x4 y, g7 m: v- t# ]7 _* d( R
图片
8 O9 U8 D: U5 N: N 4.2、Dodgy code 糟糕的代码: 7 G7 a5 t+ ^5 g0 @7 Z
7 b$ J! s: \+ [ m+ L. h5 F2 {) n
4.3、Internationalization 代码国际化相关:
3 Y' V+ M3 o9 i 4.4、Performance 代码性能相关:
9 y) Z7 \# o3 [3 j# \
, j( A3 R' o! P' e 4.5、Experimental: 3 n& z8 q$ Y r$ _ G( i4 y
P& P( u% q9 I4 @; z 4.6、Malicious code vulnerability 恶意破坏代码相关:
( @9 \. K3 i p9 L) Q
8 q4 u& R. p4 {' p6 b1 o 4.7、Multithreaded correctness 多线程代码正确性相关: 0 t& U) l0 T* ^9 G$ T% W
4.8、Correctness 代码正确性相关:
- S5 U! |$ f2 L3 J" V4 ?, j5 _* o+ n* D1 _/ @
五、SonarLint: 1 R9 `3 o3 `+ \5 \% O: w2 A
1、整体介绍:
2 k6 ?9 ^, i& J: r2 k9 v! w+ f sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
[( b) \$ I" s 2、安装步骤: $ p# I3 [; O6 M$ n4 `) @
通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
* M- ~% V+ Y! C0 P 3、使用说明:
3 Y1 |6 v s! K+ h
7 O7 P) \" C% P' Y- n7 o 右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。
. d' F7 X k8 [+ t9 X6 W7 C" o
0 j/ x9 q. r# T+ E1 `* Z 4、配置 SonarLint 服务端:
; A; K: ]) I7 g 4.1、配置 Sonar 服务器:
9 D( i, e( h7 A" H6 F sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: 2 P# ~+ H$ B3 r) Z# @* j8 S- Y
/ l( q+ K% Y G4 Y2 q 点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 3 l' t/ R- R5 L4 Q. r* f" o ^7 J# u- S
$ x$ {: h# s2 \8 O 4.2、具体 Sonar工程配置: + R4 ^. Q: K* }
配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: & z: R) U- U2 z
: r' T; m5 p) L# t# K1 S$ _) q- Y
4.3、使用 SonarLint 检查: . d9 Q& r) e( n6 x- @" Q6 m
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查
$ |- {6 X$ {" H% V7 l! q2 ~
. P p: _1 I: G9 c 总结 1 Z# C. \! u5 [) _7 c; j
) w% Q3 s/ ~ W! d! ^0 r$ G+ V 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了
, \! ~& D- L2 @ 找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:0 |. n7 \, |) s
PMD 自定义能力强,用来自定义项目BUG规则非常好用
6 b+ e3 q1 C% [! \ _ Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。
7 w+ W6 ]/ X+ O& N5 s SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景& f+ ? f) k# T$ j$ w! \
0 r6 ]3 ~4 O" c7 N* P6 [$ j
文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多 ( I8 r. X1 n2 w5 S
`% Q9 n% M. w: d+ k& p
责任编辑: + `! d$ |' a/ j o
; y, }1 [" ^" p) q: P- ?3 E( X
- }6 d& p$ Z2 c4 q% J- ^
" X( t! h" ]* A& ~# q) }$ I1 M/ q2 t9 v' v I
|