|
, r }+ E! |7 y4 v1 E) K7 I 原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器! 1 N( R) i$ e; p3 W1 @
; e, Z' g* K1 O7 }
随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。
) ~8 m6 f- ]3 o- s1 m! T( E' @ 工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力 ' B' c, h8 c# S9 G( X/ C
一、Alibaba Java Coding Guidelines
% C- ?5 W2 M' m, p$ S4 P 1、整体介绍: $ q6 a% \* p6 @: ?' L9 W6 h7 s
Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。 1 G3 i$ D% }* o7 H2 f
阿里巴巴规约扫描包括:
" ]5 q% I, [" ]; q, i
: ^& H" q, s6 D4 d3 R/ n OOP规约4 r! J, D) ?9 n$ R" h
并发处理
4 A6 Z" C5 u. k+ h6 k( F5 n' ? 控制语句
1 ]: ?3 v4 c! l 命名规约9 o3 X, M3 n5 Y% a+ {# M( c+ z
常量定义
: \9 c t5 X' L) ?: U 注释规范
7 ?$ Z/ n R4 v$ b1 l5 R# ]/ g* m+ e$ B7 Z& }6 T: ~6 s/ o
2、安装步骤:
& F4 i; K- t) g& D/ R' t; f K File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。
% W7 S# |% M, B4 e2 x. F 3、使用说明:
# ]8 y& N2 V7 l7 J9 r3 l 3.1、运行方式: $ W" W% d; V$ l% k9 z2 k
(1)可以Tools > 阿里编码规约 > 编码规约扫描 , b& j) z5 N5 x+ v$ w
 ) h7 O" F; U" | \
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可:
* p6 v5 Y7 r8 C) k! Z- ^* Q5 E q" @ 
! w) h' g4 u& J! v- O 3.2、菜单功能:
; @) k- @4 M7 \- P1 h) ~8 ]' w( [6 t+ B7 i; ^- r' ], x( d$ ~
编码规约扫描:开始扫描代码
' Y- X7 R6 ~( R/ C9 {% ~ 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能. r- Q5 s- x6 I- m# Z
切换语言至英文:中英文切换0 p$ `) \: {, x4 f. _1 ?1 u9 a' X/ |( L
; @0 V' W& Z8 M+ N: }; z+ x4 x 3.3、运行结果: 6 W* l i, B: \, k5 @, m, e
扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。 6 \( D/ J8 m+ p4 Q

/ P! h3 @5 [+ r. d6 d' z 选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):
7 o/ p( v5 k/ s% l x- o$ Z7 o  " C' b$ |6 m. \# G6 K3 m
(1)指定区域搜索同一类问题: : R; \# G& a* |& O8 T; ]8 D7 X3 i
当点击③处的按钮时,会弹出如下按钮: 5 n' o- R O7 |. U: g; {0 |) |/ G
 5 k" {; Q4 Y2 }7 s* y
这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示:
M, w3 `$ S+ v6 l% G 
% D+ A' `, ]& i 这里我们可以看到,显示了整个Project中的所有该类的问题。 ) f7 W+ g$ X$ z: j9 k. [
(2)预览具体的不规范代码: / ?" d/ {6 }1 W$ p* d" c4 Z% O
如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。 4 o3 e& h* d$ K/ [3 y

4 T; O4 _- V# @6 {9 o! R 3.4、工具栏功能介绍: 6 u: Z/ q+ M2 x7 b0 f7 ~
 ; J" p3 R0 R# Q. j" m
, p$ l: U; q' {' }1 D% p Rerun Inspection:重新运行一次扫描8 S; v9 p; d5 ^% S& }5 D: v
Close:关闭真个AJCG面板6 k' V4 ^- {* Q' Q5 {5 H
Expand All:展开结果的树状结构,整个结果是树状结构的。
# w5 {" U% c! s4 V# u Collapse All:收起结果的树状结构
' k5 X6 H) t7 ^8 l8 k7 F) P8 P Go Pre Problem:选择上一个问题
5 t( | G5 E$ W5 I% ], ] Go Next Problem:选择下一个问题& p7 n* v0 i6 W1 U5 f" S( Q
Help:帮助- l4 g+ g( B: {3 q: M, ]) b
Group by Serverity:(不知道如何描述)
. t$ o K& c5 B/ k* I# m Group by derectory:按目录分组/按类名分组间切换
2 T0 o* X% h/ j; S L( k' N/ Q Filter resoled items:过滤掉已经解决的项, O% Y" h5 V% }* A0 N, |
Autoscroll to Source:自动滚动到源码. [& W' ?$ ~7 m' l: J6 Q5 Z7 O
Export:导出,可以导出为XML和HTML两种格式
2 n- R6 d" R s+ E1 c. { Edit Settings:编辑设置( C$ i! K7 M; n( X, q& d
8 a8 [! Y2 H2 X# ~ 二、CheckStyle:
# T ]- i" w% n2 @/ f8 W 1、整体介绍: # G* e% S" K# ]9 J: E O9 f
CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。 , r6 |2 \2 H) L. \* d
Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。
4 G* M- R: R3 y 2、安装步骤:
( d- t3 d( h9 V* c8 U 通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。
% E( r4 R) {/ N( \8 j+ I; h 3、使用说明: % e, u5 |8 `' V5 ]! @

! l( m, M; y; J0 t7 J 可以看到基本都是一些缩进啥的编码规范,可以不用太关注
7 c" `9 X q3 V9 j% a; e 三、PMD # K; E! t# o; q) n; f& p! C2 W$ B# X
1、整体介绍: 5 _- K" J* U j. r0 |/ `
PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:
' ?. L' c3 J% \2 C) U! B- C0 p2 E) W# _, E5 R# R" r
潜在的bug:空的try/catch/finally/switch语句7 Z% P" o7 C$ w& \1 W* _! W' R
未使用的代码(Dead code):未使用的变量、参数、私有方法等
6 P4 J7 a) Y# x+ e! W 可选的代码:String/StringBuffer的滥用$ E* T" G! R5 @# {% G: ^+ j1 l
复杂的表达式:不必须的if语句、可被while替代的for循环
, H2 H8 `7 ^9 g, g, f9 ]3 O 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs3 q) h7 U) u; ~) g& H
循环体创建新对象:尽量不要在循环体内实例化新对象9 V* U3 t/ B/ l
资源关闭:Connect,Result,Statement等使用之后确保关闭掉. b: W' j5 m S1 T2 K( X2 f
- M3 O p1 P) P% l( U8 i
2、安装步骤: # b3 s2 ]3 [) ?6 m* h6 P7 W# [0 F
通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可
, O; o# J! ?7 c8 V6 ~ 3、使用说明:
! d* T- T9 h8 U/ m" F% S! J# m7 q3 p' J& x- G
参考文章:http://wjhsh.net/andy-songwei-p-11830812.html0 U' h; h4 |+ d: l& ~7 R( }
- m$ J; y$ Y. M$ E) y 3.1、运行方式:
+ C# g8 Q+ {, D* {1 ? ~ (1)从Tools菜单中启动: * U$ l! k) H/ U7 W o M
通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。 4 l) [. W+ V; u0 m7 c
7 k% b! W$ S, K4 T
Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。 y" R3 O6 H3 M! N+ R2 D" Z) I+ x+ o
Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。4 @. L; M/ Z$ ]- E0 z3 p) [; M1 N
9 t4 J+ b- {- r$ g+ G (2)从右键菜单中启动: 8 ^5 _! Y4 c) e" b1 i0 A, |& E
在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。
) B M. o, H$ I7 W E9 H' W 3.2、运行结果:
- V0 v( u$ M1 a$ r7 z. I 
# t5 }7 \9 n9 R+ D 运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。 - A X# m# P3 L) A% |8 h6 ~) F4 ^( J
3.3、配置检测规则: ( f" W3 `' @9 C: @7 j: I
通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面:
2 ?% e9 `& J( L, V# g 
" B* ^1 R' w8 u' S* N( l0 H5 c9 L 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。 4 w5 p( C9 b6 ?% V* p" e& W
点击“Options”选项卡,在其中可以配置一些检测规则选项: 4 H) G) C2 s. H( U) o4 G& b* d4 P

6 E7 u! C: V' E8 n 其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。
6 k. c8 Y% E0 @$ }& L& N 四、FindBugs:
; N+ `' \; i8 t9 _0 J 1、整体介绍:
( a$ c# O2 s V FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考 , \5 O+ _2 j. C
2、安装步骤: 1 {, P6 s& {2 \+ H
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可 . o) y8 E! t9 }5 \) y. N
3、使用说明:
* Q7 G0 e3 r& c FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project * q) B3 r3 B' o

; a& R8 V3 P. b: }- r) ~7 D4 j ~" h8 Z 分析完之后就会出现结果面板 " q- G/ N' R2 R& n1 r2 w4 y

8 Q1 n0 m L0 `* \ 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行 / Z# G. R, ~: b- g6 B* R8 |% l
% e2 T* {$ k5 E" W2 h, \' N+ T
4、附:常见的错误信息 & v! A" F" S8 t& [7 h. Z
4.1、Bad practice 代码坏习惯:
+ s- C& ^% ^/ A" s& t& `, S/ e
0 r1 w8 O* X9 D' g$ q( | 图片
. m7 S( N) L# `1 W9 \ 4.2、Dodgy code 糟糕的代码: 4 D" ~/ Y: V* Z! ~9 q" _0 \
* I5 Y; A: [+ [: S6 }' l 4.3、Internationalization 代码国际化相关:
. f0 ~* N" j; q- f% t2 n 4.4、Performance 代码性能相关: 8 M5 ]4 U7 c4 }
7 s! V, u+ d7 v: K2 f/ y# e 4.5、Experimental:
- t: N0 i7 A3 H2 b; S B j0 v5 j( x1 b" ?* B1 W( G1 d e" G
4.6、Malicious code vulnerability 恶意破坏代码相关: 4 [8 ~5 x& y7 {0 {
t% U: e/ Z( T$ w 4.7、Multithreaded correctness 多线程代码正确性相关:
. ]+ u7 c) Y& k. A) p/ `; w& U 4.8、Correctness 代码正确性相关: ' h i5 r! \9 A* c
) ?1 M9 v8 a' e' N; ~7 D% }
五、SonarLint: 1 J6 [7 `$ f2 _# q, h
1、整体介绍: # H C" b* _6 \7 f% w
sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。
j+ ^8 I& }1 T+ G/ }8 B; z( X, ^ 2、安装步骤: 4 y1 W! x& s0 | v
通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可 , a3 Q6 T% V9 `0 }3 F( u
3、使用说明: 8 `9 j+ M% _- h
+ s# c4 @0 Q- N- ~# N, ?0 ` 右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。 $ D% q/ s; x# J4 A3 B/ ^
, }; F$ e9 M( g
4、配置 SonarLint 服务端: 1 S/ G' L& p2 D. z" w( [( O
4.1、配置 Sonar 服务器: 4 X$ T4 z* ]( n6 t" A7 Q! B
sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: . f; L9 q3 o O% ?7 {. A* v
! `2 `1 v* [" |9 @ J& I
点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码 6 x% L/ J. I" m: U
* h, E6 @! Q6 ~! _4 |1 T- g. f
4.2、具体 Sonar工程配置:
. |) c: B# S; ?7 V) { w7 [) E 配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程:
- i" z7 y8 c8 J0 U, O8 b9 I' L0 I2 J6 J0 _7 e
4.3、使用 SonarLint 检查: 2 P0 g# {: V% Q5 \% Q
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查
. O' ~2 }# ]3 x( g, a$ r- y& g( c7 R8 z v4 g; U+ ?( T+ P
总结 . h( S7 P( |% f: a) B7 M
' @+ ~% e/ {9 S3 ^" J/ x) c+ { 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了
" _6 F9 N+ P7 T+ u! ~ 找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:
+ l8 r' z% r- x! G' I PMD 自定义能力强,用来自定义项目BUG规则非常好用
, g, o }1 K- P8 z Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。
* n* w( K7 c6 k: C* H SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景
8 ^4 F) a0 m5 U- @
! x. |. g) j" ?+ x( m% H 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多
3 i* P$ n7 t5 g# h! d2 W n* K
7 r; X2 e* T2 z' {+ j& n 责任编辑: : z" q$ f1 j+ o) i, V
/ _; R _0 H5 A1 B1 Z" T
H3 R+ ^3 k1 l& S* p# Z
) C& F+ E- t4 f+ v v+ L0 \: M
0 K+ `* L+ E) [' ?
|