|
0 |/ V- [- r# y \) e$ O 原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器! ) _. z) Q( ^ k* l3 @8 t- u2 p2 R
3 V# w3 }2 @7 o* ~
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。 # ?1 r* K* ]/ H
工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力 5 ]& c9 c0 H9 i1 O' z: U" e3 e
一、Alibaba Java Coding Guidelines 7 _' V. y p) B: p3 r* u
1、整体介绍: ; Q% `& F- W2 \2 G$ s
Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。 / t( ?8 f7 a8 w9 j; s% m
阿里巴巴规约扫描包括:
/ s/ M/ j) t# ~/ K7 v0 Z; y9 [" o) B9 d0 H; s6 U3 @
OOP规约0 H. o% Q* w( L3 U
并发处理: m7 q! g4 J& F) b
控制语句1 ?9 p# |7 x9 R6 v5 F
命名规约, ~7 m: L9 Y; ? I- `7 M
常量定义
: N2 j" V3 s1 J" v4 i3 {" n3 \ 注释规范
6 j! J2 Q0 O! i+ |. Z% I
, ] V7 s5 z9 u+ K) U. A' L 2、安装步骤:
) R' p) g! B5 o" g1 @( |6 L* | File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。 + `- B7 f, z7 T* t! }5 X
3、使用说明:
, [ ^% I* c" _ 3.1、运行方式:
$ c: k! v- Q2 ]1 r7 ?8 s, B (1)可以Tools > 阿里编码规约 > 编码规约扫描
( D2 w2 x1 z5 r0 Y! G' n  + r3 l" W" g( \$ I7 C" d' o
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可:
, s/ u0 j6 u* `2 W2 _ 
# f/ }6 |( n$ R- k" h. q5 ? 3.2、菜单功能: : @- f0 z7 d( o# p
f' L8 f$ Z( F" r# ]+ p0 Y
编码规约扫描:开始扫描代码 h" Q- L7 j0 u( O! F5 a, a2 ~* x. l
打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能& d: r+ ^( t# V
切换语言至英文:中英文切换
2 k* x. l8 J+ F) J5 n
) I: j$ j8 k: c# _/ f 3.3、运行结果: . Z% S. d' @9 F& g) j8 m2 R4 k* S
扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。 - x5 T9 }' B" H& U& |) y
 / w7 _: N2 {! T9 y5 Q4 |
选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到): 1 o, ^8 I T4 m9 `' Q6 F- \1 q
 " L$ b$ } D i1 d% |( S7 M
(1)指定区域搜索同一类问题:
* F. k: N# W2 Z, { ~ 当点击③处的按钮时,会弹出如下按钮:
& g( Q2 U& V' X. S 
; ]; o6 o+ m! N' D 这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示:
0 x) `. }- Q0 E& r, Z6 l, F4 s  + q/ s3 d: b6 |* z R& I
这里我们可以看到,显示了整个Project中的所有该类的问题。 , l' N ]! y; ^' n
(2)预览具体的不规范代码:
% B7 ?8 A, U5 S0 R2 u, ~ 如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。 - m" Y5 H2 j, a( B7 W

2 {6 V& Q: p6 j L7 f! V$ V. p% w 3.4、工具栏功能介绍: ! h- T5 t# ]* G1 l

& A# Z* y3 r( Y9 l! D/ i( i* k5 l: l7 @
Rerun Inspection:重新运行一次扫描: s2 y/ {" l v8 e, c8 f$ ]
Close:关闭真个AJCG面板; ]$ c/ C X7 d1 d L m3 P- x
Expand All:展开结果的树状结构,整个结果是树状结构的。
X o$ v" e; W- H Collapse All:收起结果的树状结构. U% ~5 r3 o0 a8 w0 S" J
Go Pre Problem:选择上一个问题
) W0 d3 L; ]. w' G1 F Go Next Problem:选择下一个问题
" L* H' ]# K# R( ] Help:帮助 G1 F. B- k' O- c" Z
Group by Serverity:(不知道如何描述)
+ q) ]! G* Y6 C3 e, i Group by derectory:按目录分组/按类名分组间切换
/ s$ X0 n0 s3 w2 _# q Filter resoled items:过滤掉已经解决的项
0 Y6 ^% F4 a9 d$ r0 B$ {0 K7 r7 j Autoscroll to Source:自动滚动到源码
& Y+ n0 i5 Y# I& t Export:导出,可以导出为XML和HTML两种格式
3 v( [, y0 F$ z) g5 u- N$ f Edit Settings:编辑设置2 [7 j' m( X6 U) H: @* O
) K6 U) o4 t; D: E, D9 u r
二、CheckStyle: % g# L4 U$ w) u; G9 S# l
1、整体介绍:
& \+ j) H( m ~8 y( i; u CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
8 _' r4 e% y6 k& y! h/ Q Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。 " L4 T: f8 G( r3 G% K8 r
2、安装步骤: 8 y' r2 C# `1 u9 b, m
通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 2 ^: t C0 h9 ?; j8 R8 N7 s! s
3、使用说明: + x) u6 f9 |' i! G4 B1 d& b* T

2 _3 `5 B. }3 s$ A 可以看到基本都是一些缩进啥的编码规范,可以不用太关注 8 i u8 |% X W8 ~, z
三、PMD
3 W; G @) d6 Z) I0 w 1、整体介绍: % r1 |% ]' ^ P1 |* J
PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
. p; ~/ G3 h$ ~* ]8 b
( w, I3 \& ]8 s! K1 k! o! ^5 v 潜在的bug:空的try/catch/finally/switch语句9 J o1 F- n) O$ Q2 r8 a
未使用的代码(Dead code):未使用的变量、参数、私有方法等
* @0 o9 w4 J- F2 Q 可选的代码:String/StringBuffer的滥用
$ X1 F9 u+ c+ C. J' g# F" R# w \ 复杂的表达式:不必须的if语句、可被while替代的for循环
& b% L+ d+ g! k( u8 C2 q/ M 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs; c9 y; t- f) I' B
循环体创建新对象:尽量不要在循环体内实例化新对象# g2 X) @( `0 S/ ?/ w: k" |' x
资源关闭:Connect,Result,Statement等使用之后确保关闭掉$ u2 l: |4 ]# C. g8 {
* f' d2 k: e1 v& a 2、安装步骤:
, b+ b# L" ? F5 e& z# F 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可 , F" @% e. u. p3 T
3、使用说明: & \2 \: I" [. Y( t* C" {
2 d- S# B# L B0 D2 e 参考文章:http://wjhsh.net/andy-songwei-p-11830812.html
2 k* X& y3 x8 v8 d1 E) \7 l& m1 ~1 ^0 E- S, }6 C
3.1、运行方式: - G6 N$ o+ V! A }% [# }
(1)从Tools菜单中启动: 5 K1 q* g! i% q" C6 _: `
通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。 ( E0 |8 I. ~2 A* S
: V+ B' C0 V7 k. a# v
Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。
- a: |" u& x6 y Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。
1 [' G C' Z/ d% h* w3 o/ O3 Y3 q9 }6 Z+ S3 [- J/ M) r( k
(2)从右键菜单中启动:
+ U$ V2 r0 j" ]7 O 在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 5 j6 H4 k# Y2 u/ T
3.2、运行结果: * i% u; v3 L I: O

- J$ j/ V% S$ x& \, C* L* G 运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。
/ X8 t) j- _7 S- j 3.3、配置检测规则: ( w j" ^9 J+ ^9 c7 z. _
通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面: 8 q; u+ ]% [& y: y" O; U4 O

8 ~! A+ f& V$ s8 K7 k# ^ 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。
1 Z, r. \ P( U0 n9 T% Y 点击“Options”选项卡,在其中可以配置一些检测规则选项: 5 m8 |" B3 F L. P
 * S: F3 z7 D: p5 h g
其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。 % H* o4 l Y: [% h( c
四、FindBugs:
3 t# [7 H9 d9 H* |/ b ? 1、整体介绍: - g% l; Y% r3 \( c2 Z) e7 c- _6 S
FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考 " {, C3 x, ^+ R9 g; d# j9 ]
2、安装步骤: ; l- e# k5 U+ W/ N7 |; h: }- Q8 f) I
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可
+ V. O5 K) |. [" K; Y& l B1 v 3、使用说明: 7 `" e, D+ C+ x$ P
FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
; \: E7 J+ j1 U3 E9 h 
- ?) ]4 p+ k0 D7 p" ] 分析完之后就会出现结果面板 0 n0 u- C* w$ l* X+ y

! A: p, V4 X6 ]+ k- v; A7 u 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行
4 f) g) U1 d6 O
* W# k' q* _& Z9 M 4、附:常见的错误信息
9 A, C7 h8 M7 `1 X: i9 H B$ }0 k 4.1、Bad practice 代码坏习惯: , F1 l% Y0 b+ S6 ]$ ?5 g' A
6 I& d1 B6 n$ r' Y 图片 " J9 k+ ]: t0 g# t! H) v
4.2、Dodgy code 糟糕的代码: 3 H$ W( I% k- h& l3 o$ v! P- U, ]+ a
* ]! p% c% f+ ~; G# f4 g) c 4.3、Internationalization 代码国际化相关: 5 N2 G! V( r' g' k4 e
4.4、Performance 代码性能相关:
* ]7 F5 t, P8 `
5 z% [9 P' [6 {( q 4.5、Experimental:
2 ~! u7 M# R8 G7 C) J3 Z, E
! o1 g9 s7 B" ]2 J8 W& p& i 4.6、Malicious code vulnerability 恶意破坏代码相关:
) }5 ]8 T5 ^' ~# u3 ~! P, f
6 B, U+ p! C- h- i0 r% v; U 4.7、Multithreaded correctness 多线程代码正确性相关: 2 m0 P* j, M5 N
4.8、Correctness 代码正确性相关:
8 g/ N; b7 }8 u1 y7 S) L' |
! h R3 v) i7 _' R7 P( M& C 五、SonarLint:
, y( ^( {& k& g9 w* Q$ x 1、整体介绍: 3 Z: ]. S# g7 M& k8 }- G
sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
6 \( |5 y; Z8 r b3 |+ d 2、安装步骤:
4 s A! s, P) F7 E% @ ` 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
$ o1 i0 J) M% Q" C 3、使用说明:
6 h( ^ P/ W! K! d; K' b# K1 d2 U H; |' F9 Q9 [: m
右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。
# C8 q2 S: d* d* F: K
& z. Y- J+ D; } J 4、配置 SonarLint 服务端: 4 P9 G4 X( h( A$ n& L0 n+ E! B
4.1、配置 Sonar 服务器:
/ ]8 l# D3 I- ]6 e$ O2 Z sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: 2 a. E8 @0 m! f1 |3 C
. k5 q. h# Q9 U5 [& j( w9 q/ m
点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 - Q* E8 |4 C, ?6 @
2 ~% |$ P. I' s
4.2、具体 Sonar工程配置: 5 g, C e/ X- t9 [2 ~+ y
配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程: $ W' g, Y, A) V% R8 R3 z
. S4 i. O; j0 E, a W6 V% H
4.3、使用 SonarLint 检查: C+ P4 f0 {) p$ w
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查
1 w* Z- a2 S' \$ h2 ^" X7 _
" s( W0 D+ F+ c/ d 总结
# ~) s* |3 o% A. V4 m! L0 w6 J
* Y% k8 W; A8 b 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了$ C0 ~0 P0 }, L# x. S2 H6 l. R( }# K# c
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:& U$ e2 N4 b6 V5 z: m; V H
PMD 自定义能力强,用来自定义项目BUG规则非常好用
6 d3 }) ]0 n. _+ @0 W. Y Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。
; `, A. i( m% } SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景
+ g( p' W2 z. T. k6 l1 M$ f3 [# A' F- S# C! H7 O- Z
文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多 : F C9 n8 j) W9 w
% M l2 Z# L" N) p r
责任编辑:
1 ^+ L' h- J5 u. E3 r1 k% `6 O5 A# `3 h t- z# E, H
: W6 \# J4 h! A& e( a
- a; L8 ?' e5 R" h. [ Y
" e# Y5 w9 r0 m4 R |