|
1 g. \; d' |' S9 ]* |: v 原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!
* r! r1 t+ Z+ ~. X' x
+ q1 o0 p3 p/ q$ O) b 随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。 3 [$ T4 S+ K% q0 b' ]+ Y
工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力 * g+ B+ }, t8 ?- z
一、Alibaba Java Coding Guidelines
e. D- B4 f9 p, x# u 1、整体介绍:
1 P+ Y" f. k% M; C0 A( } Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。
0 r2 Z2 o! o" t. U1 i) Z 阿里巴巴规约扫描包括: ; i3 z' |1 ?7 F9 D9 { U( o
( N/ J. \" o. M. z0 Y2 z* e OOP规约
& B3 t5 }9 t6 @& U 并发处理8 u8 G( K+ x6 ? i
控制语句2 Y, X- I- J# h
命名规约
7 e# b7 }' d' Y, Y6 c 常量定义
6 K' P2 t4 F( p. \ 注释规范9 ` z2 S- Y3 T7 ?( z2 u
# R- f5 |5 n+ ~3 _
2、安装步骤:
+ u( v5 A: n4 v7 d! n7 K File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。
' s' g: A( w/ B( n( Q) ` 3、使用说明: ' ?- J. m7 P4 q; ]' d
3.1、运行方式:
( w; J7 @4 F; S1 ^9 {! j (1)可以Tools > 阿里编码规约 > 编码规约扫描
8 t! U$ X2 y9 T7 E* u  ( ^! o, [: y* c2 M: W
(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可: 2 O& O" d6 s2 D. p3 _9 V

- X2 o* a5 Y$ g$ E0 O 3.2、菜单功能:
6 F; T# Y/ q& A$ P3 \+ O8 a" \8 D
编码规约扫描:开始扫描代码: C( K# l+ y8 A @: E0 t, m
打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能
0 C' h5 h9 X9 g7 F. q4 c7 L* a 切换语言至英文:中英文切换, ?' g, z p) B& Z' U
2 F& h7 X) a$ g/ `$ O 3.3、运行结果:
) u4 B. t% j, }8 P/ v 扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。 3 u# h2 m0 q" i

4 M# g3 J- R7 k( b9 w! w 选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到): ' r0 Z" j ^7 ^( H4 |/ u( C

8 c; \" t7 ?- \3 [+ }. z (1)指定区域搜索同一类问题: $ t+ i% j/ S; S3 E0 i! S: e0 I
当点击③处的按钮时,会弹出如下按钮:
+ j+ C1 c1 ^" j  0 r: ?5 x, {! @* U& I; i6 E2 \
这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示: ) T5 n) s7 ~1 c+ g2 H6 [

6 a" L0 j i6 _. @- c! ? 这里我们可以看到,显示了整个Project中的所有该类的问题。
0 m& H4 `9 H6 H (2)预览具体的不规范代码:
v: U4 B* V$ `0 u7 _; j- ? 如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。
* J* s* o B% n2 e: v" O U  - }" J9 k4 G, O
3.4、工具栏功能介绍: # t1 i* L7 Y, d
 ! z5 x% N2 y. K" Z* [% }1 J9 S5 `: H7 f+ v
4 Z4 L. Z6 O% n* P8 N
Rerun Inspection:重新运行一次扫描
7 a v/ a4 O- ?8 B# g; e( T3 l; q6 d Close:关闭真个AJCG面板
! ^; N D: C6 ?. S% U Expand All:展开结果的树状结构,整个结果是树状结构的。: E5 v _0 ~5 L& z6 {9 z
Collapse All:收起结果的树状结构* h) D' J6 l- ?5 Z
Go Pre Problem:选择上一个问题
6 @) s! I' Z4 t' s S; j Go Next Problem:选择下一个问题
. }9 R. F$ j: d" w/ G Help:帮助
" K( `( h5 p$ g. t K' @ Group by Serverity:(不知道如何描述)& m9 [/ f" {+ N( Z
Group by derectory:按目录分组/按类名分组间切换6 v& Y( i& }# _$ {3 }) w
Filter resoled items:过滤掉已经解决的项
3 |; ]" e t4 v: Q6 u2 M) u Autoscroll to Source:自动滚动到源码
7 r' E/ _. [% w. z0 G# }: b3 M. L( X* { Export:导出,可以导出为XML和HTML两种格式
' v" M6 Y# K( F5 A: R* J. e* e Edit Settings:编辑设置6 O/ t) l# v( Q, I) { z
# P5 |& S5 M% f+ ?
二、CheckStyle: * b |5 @2 L+ O( b" ^
1、整体介绍: 6 V- {. P9 q, ]9 R0 f
CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。 & x6 x2 a0 Z n
Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。
% u h: Q# t& y( T; i 2、安装步骤: - @) J: R0 Q7 W Y0 M
通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。 4 U, ] {; {) u" ]2 |- Y* t
3、使用说明:
% h& y5 Y4 |0 _- \/ z, Y 
/ v _0 E3 V4 V8 i3 `4 D 可以看到基本都是一些缩进啥的编码规范,可以不用太关注 ) m! g% J" T# ^' z8 \0 n
三、PMD
" ^) z/ B: }1 ^1 J# U 1、整体介绍:
( P- {2 r x0 d. v PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容: 5 O) L A y4 o2 {
& h) Z* d4 P3 U6 ~9 u 潜在的bug:空的try/catch/finally/switch语句# ?* I& n* I4 L& p6 f" Q) |
未使用的代码(Dead code):未使用的变量、参数、私有方法等
8 C- F' ~; v6 h 可选的代码:String/StringBuffer的滥用
+ ~1 @! t) w5 [) _3 {6 E 复杂的表达式:不必须的if语句、可被while替代的for循环3 Q4 H( U5 ~9 ?$ w7 O" o+ S
重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs# ?* R! K8 L. L6 I
循环体创建新对象:尽量不要在循环体内实例化新对象$ s5 m% e& b9 C2 S6 z
资源关闭:Connect,Result,Statement等使用之后确保关闭掉. A P' ?7 M h& A" ?) M6 M& G
* j3 v2 |" |; c. T
2、安装步骤:
! Y' ~: }: O, d+ I6 K. j' ^7 f 通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可
% I1 ^5 l% u- A1 I6 N8 g 3、使用说明:
) z& x* B% A% y# O7 T
/ I q( d4 B8 ?# n! O' a. ~0 p 参考文章:http://wjhsh.net/andy-songwei-p-11830812.html o5 j% J% K0 ^8 v4 x3 D% J
. S! E1 O: l; M4 r4 q1 |% R 3.1、运行方式: ; T* e2 r) s0 ^5 ]" W
(1)从Tools菜单中启动:
8 u2 |! ]) h7 q) K' f6 t 通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。 % U/ O9 n7 l& ~# j+ |+ q) f9 o
& h; J% I( s( ~ Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。
! \* r! R; _ E, d" o4 F( U Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。
2 S% O% G! H9 @4 Z7 |
$ f4 s2 Z" f) E$ i+ l3 X; V (2)从右键菜单中启动: 3 G- U' X& ]9 e5 M
在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。 0 ^9 l- u4 l9 I& L3 e$ H9 ?1 y
3.2、运行结果:
" m, j0 F- [5 Q; k- Y! ~ 
3 w3 }0 I1 k( P 运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。
* f$ a) T: ~( `$ c 3.3、配置检测规则:
: l9 f I: M& x/ k 通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面:
& A* N, U5 }8 U8 E 
* ~+ n9 U I4 K9 a* t, A1 } 在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。 ' {' X, M# V6 b4 i
点击“Options”选项卡,在其中可以配置一些检测规则选项:
' C2 W5 l" _6 B5 U& C' G  5 A* E! b: K: C f* X5 b' E# W9 k
其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。
, O5 u" W0 E" O* d6 S3 Y3 A 四、FindBugs: , ~9 \4 A) E0 o! l
1、整体介绍:
7 {6 S7 [4 j8 Z+ r t& ?' l- g- f2 q0 ~( A FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考
0 w! J' T( k% L 2、安装步骤: " s9 ~( y) X1 h2 R& I
通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可 4 h' [& c# z- h" z1 o# j2 L M
3、使用说明: 7 M! S% I, G5 Y+ P3 g% s
FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project
0 E* J5 z) C# n  9 v3 \; i$ |" g, L- Y
分析完之后就会出现结果面板
3 x/ q: F. c. G4 e' h9 m 
2 Z# C0 u/ r1 \8 V) j' l 点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行
6 _) [. Y: B, ?
0 P) d) d. l5 p" p" H 4、附:常见的错误信息 3 E# w8 o, @$ X0 N# b3 `
4.1、Bad practice 代码坏习惯:
0 O0 D7 f* o, d# a5 B/ x: B( `" i
/ H/ ~- A5 U3 a& k 图片
8 e6 k& z7 L7 K; G& l2 ?& t2 O" \ 4.2、Dodgy code 糟糕的代码:
7 P7 t/ c. E0 @' H3 n3 M, f2 Y" u( m# K& D$ E/ _ O
4.3、Internationalization 代码国际化相关: ( r- h! a( V1 L' q% s! ?% @
4.4、Performance 代码性能相关:
: e- |& U) ?9 J' N2 b1 ~# i5 |. @
$ ? N, ]/ s5 f& y9 y 4.5、Experimental: 6 _* S4 t2 N8 }, n
& t" p( L, f/ K9 N3 d' @5 f# E1 j 4.6、Malicious code vulnerability 恶意破坏代码相关:
$ I9 p- N% o# B$ k' e0 Z: J7 J% Z
4.7、Multithreaded correctness 多线程代码正确性相关: / i# s: c1 \' T: H. p c
4.8、Correctness 代码正确性相关:
2 d1 g9 R8 H) v1 Q( ]' o6 Z8 |4 w o$ }
五、SonarLint:
- E- s4 i2 F' b4 d& B 1、整体介绍: ; E; [. Z+ g! I/ H7 I; d' z
sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。 " e7 F4 G4 i8 G' y1 M
2、安装步骤:
% B t& s9 y1 q7 U 通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可 8 @5 g! y! ?2 ]! o/ a T& ?) _
3、使用说明:
1 m$ M8 \9 p* B* H: V0 c( ~& e( P9 ]4 L
右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。
& c; z( U( ?- Q2 Z
0 C' J8 _% e5 s- | 4、配置 SonarLint 服务端: % t1 i* \; m, N
4.1、配置 Sonar 服务器:
6 P" D+ k% e/ N0 O sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查: 6 B4 ^, f# H% ] _ ~
* [! }! n" ^+ U1 i* F) H0 P
点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码
J+ w' f7 K: q4 Y+ p/ N% p; p0 D; }/ g
4.2、具体 Sonar工程配置: 2 z# r; J r: |. l8 t
配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程:
* }. p5 E7 s$ k
. J' K, ?( D6 U8 y 4.3、使用 SonarLint 检查: s. n7 y) V+ r
配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查 5 Y- I G; l" ^- A3 `
: B1 o6 ~& p7 y9 ` b 总结
) N x% k, N1 J( h5 [
! M4 T } @" m# h+ \1 | 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了9 l. d8 f# j7 v3 K2 q5 f8 Z1 t: v: F
找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:
t5 n( U9 w# r* b. E PMD 自定义能力强,用来自定义项目BUG规则非常好用! \ O* t6 h. X& b
Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。
L" v5 `/ Z, c0 a Z. ] SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景
, s* a, g Y, H5 N9 B ]6 i
4 U4 g8 ?" j# ]$ k7 B4 W 文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多
& i% d- ^% I( i: e/ ~' p$ H, V; v
责任编辑:
' l( B. D+ t* J. o$ T; e6 A$ s1 p
7 Y( \; e4 ?' C9 Y7 Q+ p2 f2 F& |/ @: c3 j# @+ o; o
/ W2 W3 M, B7 y2 n6 [1 q" h& l4 z
|