|
1 \* l1 g) W: @# b5 Y1 K
原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
1 [6 X5 V$ M7 u( K7 e
3 O4 I3 D* R& N 随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。
- H. f) i9 a1 n. T, f 工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力
& d! L0 O5 e) Q& r4 a( x 一、Alibaba Java Coding Guidelines 0 a" @6 o- e' Z1 d0 S9 H3 v
1、整体介绍: / e" b( I) O: C6 i2 W) P% L4 Y) `
Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。 ' x! M+ ?9 o7 N7 K- Z# U0 i) c
阿里巴巴规约扫描包括:
; S3 ?3 M7 l/ t6 L$ s6 m: T) S
& H% _" [2 ^$ j& A) l, B OOP规约% s# b2 r/ a4 l8 C; o) n5 c+ l
并发处理
4 Z$ M4 X+ B! m! N* S 控制语句7 O, p) y" }! ]7 j c9 w
命名规约 ~, b% f4 d7 m$ W- b$ R
常量定义
0 U9 R Z- m# [+ _9 E7 }5 A/ K u p 注释规范
9 @) | F K4 e1 R+ @+ m- b ~$ r2 \+ u" ] [; c
2、安装步骤: 2 D7 Q6 U( b) m# o% i% l1 f5 V
File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。
7 ~/ Z& V: Q) R) y! F6 l 3、使用说明: & b0 b p4 P9 F2 M0 I9 ]" R2 P% h, A
3.1、运行方式:
; j2 a8 u+ N+ M3 a9 [ (1)可以Tools > 阿里编码规约 > 编码规约扫描 - b) G. }' a: j: R: m; L
 - E8 K: |4 g; }/ X3 E: u. q& h$ u1 X
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可: * S& K- |8 ?, s& p2 o) y; O

) Q% V! E7 T6 I6 w- I; M 3.2、菜单功能:
! Q2 Z: j9 D- U9 u" w- F% @1 z/ C7 S& v
编码规约扫描:开始扫描代码7 s$ k& v3 X+ J( [! Y$ k2 p
打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能
! ~, j: f! L- a) ~0 s: R0 O 切换语言至英文:中英文切换
4 Q: h# `. F$ q
, J! N$ p, N! r$ S 3.3、运行结果: ; U( [( K: v% i; J( @
扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。 " N$ s& g- G$ E- C0 Y. {
 - l$ L4 X4 R: D+ k
选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到): 1 G; C. t+ _; P2 V3 Z

0 A. W9 P1 c# o4 ^; `8 i- w- i6 m (1)指定区域搜索同一类问题: 6 ]6 k3 Z. B1 Z3 c) ?" N1 g
当点击③处的按钮时,会弹出如下按钮:
% w: ~. j5 B4 I: S5 s6 J 
& s) B& T# Q8 P5 q- N 这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示:
3 |; V7 G" D1 w* Q% s( h  $ C$ h' T7 p* S) p7 \
这里我们可以看到,显示了整个Project中的所有该类的问题。
# Y h! f: B9 [: F4 t9 S% e) B (2)预览具体的不规范代码:
' B: G+ A; [( Z: W* R 如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。
/ v, }; Y; O; k7 J8 }4 y0 n7 M 
& c% n. J% r* \' H z) t6 v. W 3.4、工具栏功能介绍:
" R: c( T# k! G# @ 
6 f/ i6 @, f! c/ U9 E
( d# F) K: z/ A# [ Rerun Inspection:重新运行一次扫描
' i% U* R/ ?+ p% D A Close:关闭真个AJCG面板
& ]% l1 T. N; p: I$ r7 _ Expand All:展开结果的树状结构,整个结果是树状结构的。
$ j8 W9 W9 A) P1 Q Collapse All:收起结果的树状结构
- F7 _, t A3 _ Go Pre Problem:选择上一个问题0 @" U- r5 J& v. p: r* U
Go Next Problem:选择下一个问题
! f0 G7 s, ~" i$ ?9 f$ o Help:帮助
% H3 V+ _7 O/ u" X$ N7 H Group by Serverity:(不知道如何描述)
8 N: c E: ~- i/ `0 V5 q Group by derectory:按目录分组/按类名分组间切换0 Z, `8 d+ ?+ ~# R6 e! c
Filter resoled items:过滤掉已经解决的项
s$ w- `, S( K+ J Autoscroll to Source:自动滚动到源码
. ^- c3 p/ e# Z Export:导出,可以导出为XML和HTML两种格式
: F5 Q: H: V% a Edit Settings:编辑设置
' m; ]/ Y1 D& J' C l8 w/ A$ c4 G) F$ P1 o
二、CheckStyle:
, [5 S* j! I6 |( O! v1 _4 \, [ 1、整体介绍: 6 m, H! M; i) G& j' m
CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。
( Z3 n) f* W- {& _! T' K Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。
. D: T4 }% k/ S3 r, [' p 2、安装步骤: m$ k# M5 Y; g) u1 ~; {4 f/ a \
通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 ' m% m* O5 q0 a# Q3 t- n
3、使用说明:
, W# {5 ]2 ~, i4 x4 d 
- {3 V" P& ^# F6 d1 I0 a2 } 可以看到基本都是一些缩进啥的编码规范,可以不用太关注
8 U6 n, ?: P: p; `2 t: f7 } 三、PMD
9 ]& w/ U% Q0 i9 x 1、整体介绍: 5 V$ Y4 [9 w5 ^$ H/ ]( U
PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
8 K! s7 b. p/ w" Z7 K( x- h. n" o* T( z2 F3 j
潜在的bug:空的try/catch/finally/switch语句
& f# y) ]& B0 q( A! O3 O' |" W 未使用的代码(Dead code):未使用的变量、参数、私有方法等
# Q7 ~: g1 i2 a. K: ^9 T% S2 H! Y 可选的代码:String/StringBuffer的滥用
) \* B* u% P7 Q( _ 复杂的表达式:不必须的if语句、可被while替代的for循环. Z6 V' Z9 ^ e' c
重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs2 m; F6 q$ p. w/ `1 ~1 o
循环体创建新对象:尽量不要在循环体内实例化新对象
4 I7 D, [- ]0 x; }0 ]- Q; e 资源关闭:Connect,Result,Statement等使用之后确保关闭掉+ w' B0 V3 ?0 Z
+ b! O# ?/ { g
2、安装步骤:
0 T3 I, e; t# i3 M$ h 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可
' y# g' l4 G1 J0 I% z 3、使用说明:
7 ]+ \2 h7 @6 |+ ?) e+ |) n5 ]; T. f4 W+ D) f8 q4 W& B1 g
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html
8 ~1 t8 M; K- Z/ [3 ]: H8 z) H5 e; y0 ~: z
3.1、运行方式:
# [. e" P$ B2 X (1)从Tools菜单中启动: . {4 e/ H! g! S2 T
通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。
6 l% z v3 Y$ C- M+ S2 f: T$ p+ Q' g# d( s* b/ Y7 {# s( H
Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。
# m' I/ @+ n+ k2 |2 O: F Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。. u7 A: s7 P9 S* F
; \2 m5 a& V( K s$ g! s" `$ N0 v X
(2)从右键菜单中启动:
5 X5 v- S3 s& s0 _- {6 V, M, f( F( M 在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 / s% Q! g; ?3 I& m' E
3.2、运行结果:
0 ~2 ^6 f9 f0 }$ h; d' r/ L% V 
& m! N6 M! S: H 运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。
1 c" o( @* ` ~ 3.3、配置检测规则: 0 X/ J r: r5 d3 o
通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面: ) ^& x3 `) c8 N& A: o4 W: y
 3 p! k. `' ~0 w y, I# l
在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。
8 D# z, C6 j1 x# m' G 点击“Options”选项卡,在其中可以配置一些检测规则选项:
$ J) c( {% |: F+ h& j 
5 S: ~6 N0 l! {7 M$ [% N) y3 U 其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。 ) k" d7 a1 A# ^4 [# ]9 B e# ~
四、FindBugs:
; r3 B2 J# u3 B3 L 1、整体介绍:
! C' ^! h, U2 A6 w FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考 % x) t: T; A/ }
2、安装步骤:
5 d2 R; K2 h% D% ^3 c9 i( Y% @ 通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可
) U! i% c4 G# Y2 \ 3、使用说明:
C+ n, y9 {9 T# t# l1 ^% ` FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
9 b: s( P1 {+ w* z4 g- ?/ f  9 h% E( m1 `% F& V. D2 @- M( ?4 V
分析完之后就会出现结果面板 0 ?5 l& }: R& v3 R; y. k. A' Y5 F1 b2 q
 6 Y: M; `# u' T& n. a
点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行 # I+ s3 y9 [ D. P* y" o" u
% O3 X- r3 ^6 S& o; T% s3 y
4、附:常见的错误信息
1 f6 Q& j" N! m 4.1、Bad practice 代码坏习惯:
% B" M* E/ V; F5 N
- D# x( B+ e# D$ o: A' W 图片
& S: ?9 a( B3 I, y* j 4.2、Dodgy code 糟糕的代码: ( I$ |4 J5 f! [( g/ @9 ~
" v9 x9 p7 c( `9 J7 e9 D0 t 4.3、Internationalization 代码国际化相关: . e V$ n; O7 e" b6 e
4.4、Performance 代码性能相关: % ^+ ~) R9 @1 T2 q" v( u; x
7 y( x* h: B) @2 Z' W- G I3 D) o" {; } 4.5、Experimental:
+ J3 [/ k% I: T: v$ ]; T- Y$ \ _0 ^0 y$ x3 n
4.6、Malicious code vulnerability 恶意破坏代码相关: 6 {: p9 c& w" \4 C+ j6 x
+ X0 g0 M! c4 G- O) m! I3 A 4.7、Multithreaded correctness 多线程代码正确性相关: 4 q! l: ]: Z1 B/ E, }5 w; K
4.8、Correctness 代码正确性相关: * v1 v5 I/ _- A, j n. `
/ e1 ?% A. ~! s
五、SonarLint: 3 p8 t9 b8 S3 v( [
1、整体介绍:
+ G8 [) A' Z, a& `, @: m sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。 ) E* D) |# P; m
2、安装步骤:
" i% m) L$ c! T( o 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可
' B6 W" T; }7 C, `. f' g2 _ 3、使用说明:
1 L$ b: l! J3 d+ @# e
6 i$ b8 N5 T" M9 Z1 G 右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。
9 h( B; y4 q% t' A& L4 [5 Y: P4 o2 c- D
4、配置 SonarLint 服务端: * x% M7 W2 C' C1 Y* C
4.1、配置 Sonar 服务器:
6 ?+ B# [7 J; A2 M sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: 6 R) r2 q) o" C N/ o0 g0 Z
( M1 {9 `0 s' t% u 点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 4 S# Y" E. p7 U$ Y$ Y' d, m6 ]
2 Z H* H5 P% U0 H. a" Q
4.2、具体 Sonar工程配置:
+ e: j, m5 p3 H3 R7 G, y 配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程:
: x# L6 i o9 G: ]8 [, m2 |( t+ c% T% j+ w
4.3、使用 SonarLint 检查:
; `4 d+ A6 O: N/ i. c 配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查 4 k _( {6 B: o% I2 G7 g
d; s" H- k6 X( W 总结 , u0 c/ U: n1 D" G- a% Q
2 ]# H5 i+ S+ f9 Z& y4 N5 [ 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了- x" T6 y, l1 G7 g) }
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:1 s# r; `9 Y* {0 U8 o3 a U
PMD 自定义能力强,用来自定义项目BUG规则非常好用& N; K: n3 }0 r+ y& g
Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。- ] {6 i1 S/ t+ D0 S# O
SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景
) @9 n5 R( P! l- L$ `3 D& |
( B9 D/ m7 x: h% J: v( k 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多 ! X c% {! k+ s# b: v# g( [
9 Y; q6 q8 f+ ^5 k6 b- y1 y- x5 m5 J 责任编辑:
! |2 S }' y2 s/ J( Y# Y& R, T9 z/ T, z+ M" {. Q# N
0 ~' `, {. x, k
' m% W' Q# ]& J5 |7 v2 p' b& t
* r W5 y8 y* h: Q# N" a |