腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!

[复制链接]
7 |% M/ q7 ] [7 ]- t/ q5 G

原标题:腾憬文化科技香港科技事业部 | 五款IDEA 插件,堪称代码质量检查利器!

' Z% s/ B" D$ B3 D % x) s1 d' Y& s

随着业务的发展,系统会越来越庞大,原本简单稳定的功能,可能在不断迭代后复杂度上升,潜在的风险也随之暴露,导致最终服务不稳定,造成业务价值的损失。而为了减少这种情况,其中一种比较好的方式就是提高代码质量,比如通过代码审查,从而降低错误风险,但是,代码审查难度大,代码缺陷、漏洞不易发现,且审查工作随着代码量增加而增加,审查效率低。

8 e0 E5 y6 a t+ S

工欲善其事,必先利其器,因此,这篇文章给大家介绍几种检查代码质量的利器,Alibaba Java Coding Guidelines、CheckStyle、PMD、FindBugs、SonarLint,让你在关注代码质量的同时,减少 code review 的工作量,提高 code review 的效率,并通过代码质量分析去反向提升我们的代码编写能力

' \; z2 }3 Q- ^' ~: `* N' {

一、Alibaba Java Coding Guidelines

: E" O7 v* S4 h' _

1、整体介绍:

' C9 b: N0 t3 E8 y

Alibaba Java Coding Guidelines 专注于Java代码规范,目的是让开发者更加方便、快速规范代码格式。该插件在扫描代码后,将不符合规约的代码按 Blocker、Critical、Major 三个等级显示出来,并且大部分可以自动修复,它还基于 Inspection 机制提供了实时检测功能,编写代码的同时也能快速发现问题所在。

6 \" @- \+ n; t& D+ O. K8 J8 z' ~

阿里巴巴规约扫描包括:

; h/ X9 N% J/ u$ R - c" ? [% `+ `7 @: |8 ]. S! M OOP规约 2 n# e& l; k( v$ l. |0 C 并发处理0 ?3 U$ X! M; e; [1 D; {8 { 控制语句 : h% e/ x% ^/ }# ?6 { 命名规约 7 `* h, W" n% G! ~ 常量定义3 o4 e+ Q. ^% H' i 注释规范5 ]! g$ U$ o' Q3 N# n 0 j: i/ o$ |( ~! j

2、安装步骤:

7 R9 W1 e3 I" Y/ v0 M$ F

File > Settings > Plugins > Marketplace 搜索 “Alibaba Java Coding Guidelines”,按照提示进行安装,然后重启即可。

" d: V1 l+ T. g; u# q2 E

3、使用说明:

9 j" s: ~' F( A8 c3 w

3.1、运行方式:

+ i! k; ]* B& E- {+ R4 O9 ^ e2 }

(1)可以Tools > 阿里编码规约 > 编码规约扫描

7 r) S7 Q# d. g. H' c7 x

# i: L- B2 y \. d% u2 W0 f

(2)在编辑界面或者项目区域点击右键,在右键菜单中选择“编码规约扫描”即可:

, ^$ w0 z$ n, I2 ]5 \ o+ I

3 o$ x5 a' m) D& Z0 R

3.2、菜单功能:

: f' M+ m$ c/ j$ H, h $ d( ^- @! V7 T) p 编码规约扫描:开始扫描代码 * y Q4 Y" }* O5 h2 j 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能 - Q$ `1 m; E8 F+ w+ ^ 切换语言至英文:中英文切换 : y$ K) b! G2 T0 J* L 0 {$ Z. Z" M# `2 Y

3.3、运行结果:

$ @0 M/ ~2 _! r" y7 R% y

扫描完成后显示结果如下,我们可以看到扫描结果主要分为 Blocker(阻挡者)、Critical(严重问题)、Major(主要的)三个大类,它们表示的是问题的严重程度,严重程度由高到低为:Blocker > Critical > Major,至于每一类中都会包含什么样的问题,图中的内容已经说明了一切。

( u, b' p# H# Q- l

4 O& p6 j; P0 L3 }. R7 V& o5 Y% d

选中其中的一个问题项目,会出现如下内容(如果当前鼠标点击的是最终项,右边区域显示的是其它的内容,后面会再讲到):

3 ]0 }0 J9 F/ X) u" ^8 w# e8 P

" o S0 r/ ]: h, r ]7 w

(1)指定区域搜索同一类问题:

6 \' C/ l* y* ]. F

当点击③处的按钮时,会弹出如下按钮:

3 p4 E3 ]( g/ m: w8 X$ K6 `; x

. }# [) G3 |. Q2 I0 \8 U) p. \& f+ K5 F

这里选择扫描区域,来扫描鼠标选中的同类问题。如果按照默认选择,那么运行后的结果就如下图所示:

/ }2 I9 ^, v3 e

' Q q1 _2 U9 p& `

这里我们可以看到,显示了整个Project中的所有该类的问题。

! t$ i3 \: D. y0 e

(2)预览具体的不规范代码:

8 o8 z' c9 `, w# a8 J7 G; f

如果点击的是最终的问题点或者问题所在的类文件,那显示的就是如下界面,预览该处不规范的代码。

' i5 u) i1 g9 h3 ~# k

7 ?" _+ r$ h% r) ~% ~2 f

3.4、工具栏功能介绍:

& f& I: J7 q9 n& U a

. o& p! U- y7 l V/ `+ U, I, i1 U1 _: R# T7 h Q Rerun Inspection:重新运行一次扫描5 j! K) j( k2 @/ E, J4 e3 C( I Close:关闭真个AJCG面板 + `3 |) v& q0 o# y- \" T+ t4 E Expand All:展开结果的树状结构,整个结果是树状结构的。( J8 l' P5 T* q2 q) o Collapse All:收起结果的树状结构- j9 g9 X2 l* @: j& D" B" V4 X Go Pre Problem:选择上一个问题 " p- O, \% {- r* s( Q Go Next Problem:选择下一个问题 , s0 } ?' ~8 q% r3 K1 e Help:帮助0 ?& I3 _2 b( p Group by Serverity:(不知道如何描述)% m2 K. w' G* N2 } Group by derectory:按目录分组/按类名分组间切换- y' [0 H* p- @ Filter resoled items:过滤掉已经解决的项, Y* C$ S/ u, u$ z" h" R% I Autoscroll to Source:自动滚动到源码 " G2 d* F5 ^% X+ k' F Export:导出,可以导出为XML和HTML两种格式/ D q( L3 O, m; s) d Edit Settings:编辑设置" Z+ F% c; u4 D 4 u; Y# ^- j8 r1 @

二、CheckStyle:

( S9 F: d B2 t& ^" a

1、整体介绍:

& J6 Z2 S+ B1 }' ?( _; D

CheckStyle 侧重检查编码格式和代码风格规范,如命名规范、Javadoc注释规范、空格规范、size度量(如过长的方法)、重复代码、多余Imports等,从而有效约束开发人员更好地遵循代码编写规范。Checkstyle主要是文法层面的代码编写规范的分析,对bug几乎没什么发现能力。

& a8 f3 j3 m, o9 I

Checkstyle插件中默认内置有2个执行代码检查的配置文件(Sun Checks 和 Sun Checks),但是这两个文件检查的非常详细严格,即使优秀的开源项目也会检查出来有非常多的错误告信息,所以需要导入我们自定义的配置文件。

6 E) f2 l! t$ C

2、安装步骤:

" l k% x0 s9 V8 O. |+ L

通过 File > Settings > Plugins > Marketplace 搜索 “CheckStyle”,按照提示进行安装,然后重启即可。

) m4 X, [ K/ U, Y& Y# e

3、使用说明:

( G S; ^0 M: K" v/ V# `0 r

( ~0 ~7 D. o9 k, R

可以看到基本都是一些缩进啥的编码规范,可以不用太关注

( ^7 }5 {/ M* r4 A- u3 _1 ~1 R4 Y

三、PMD

q6 j" u- }" N1 F: |- j

1、整体介绍:

& _2 z3 W; Y% I6 G9 C$ Q

PMD侧重面向安全编码规则,且具备一定的数据流分析和路径分析能力,能力比CheckStyle稍微强点,并且 PMD 支持自定义规则,PMD可以直接使用的规则包括以下内容:

( R4 v$ |' C6 }1 [3 o 7 A' K2 j3 x. o, b; u) ^ x 潜在的bug:空的try/catch/finally/switch语句. e3 O/ ^5 H3 R6 ~ 未使用的代码(Dead code):未使用的变量、参数、私有方法等( m; @/ Z9 S/ E0 L 可选的代码:String/StringBuffer的滥用# Y4 b" _# J) n5 W* } 复杂的表达式:不必须的if语句、可被while替代的for循环0 @; @6 W$ Z! V- c 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs ! v6 h( }+ p1 ~; z# o ?+ z) o2 v3 x 循环体创建新对象:尽量不要在循环体内实例化新对象 % b; J0 Q y; W, d: j 资源关闭:Connect,Result,Statement等使用之后确保关闭掉 ! ~' ^0 Y, H% x7 v6 d$ c Q3 C 9 f. w) y1 h' r/ B# a

2、安装步骤:

( a* G6 I! k' u0 d3 Z$ h

通过 File > Settings > Plugins > Marketplace 搜索 “PMDPlugin”,按照提示进行安装,然后重启即可

( M; t" w' _5 | C, m# _' ?

3、使用说明:

. @9 ]5 x% m4 ]' R. Y: k 3 z* v* C% }* ~' n& p# ^. S 参考文章:http://wjhsh.net/andy-songwei-p-11830812.html 4 P) v1 j! |( E, t4 T# G& t 7 d( r. g* B5 w m) S* P

3.1、运行方式:

: {0 u9 a" W! h: _8 v+ Z" z' F, A

(1)从Tools菜单中启动:

+ l$ V# m1 F d) [

通过 Tools > Run PMD 可以看到如下的界面,如果通过该方式启动,扫描的范围就是整个项目中的文件了。

$ r+ k, O# c/ }* U0 A; z 4 O) c. h* h! D- o0 o' H7 j Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。" U) }9 @/ H$ Y9 l5 o m Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。 + x f6 J* z* _6 j& s8 D+ d% l5 U% A, X) r

(2)从右键菜单中启动:

( v* f, S) k9 \( O. A1 j9 S7 U

在文件或者编辑器中点击右键,也可以看到“Run PMD”选项,如果通过该方式启动, 检测范围取决于鼠标或光标当前所选中的区域。

+ w- X3 t; r0 K3 z- {+ ^ t" n* N

3.2、运行结果:

- N, o6 \ S6 ?0 h* c/ b* ?

! C" ?" |3 y) V) ~1 a9 R1 s

运行后会出现如上所示的面板,左边工具栏,鼠标停留在上面会提示其功能;右边显示了检测结果,当点击具体某一问题项时,会跳转到对应的源码中。

. l2 O X3 F( L1 Q$ b7 _8 A' o

3.3、配置检测规则:

/ T9 s. S! j6 R7 ^' ?6 H

通过 File > Settings > Other Settings > PMD 可以打开检测规则的设置界面:

. n+ b# G4 Z1 @& g5 @# A

) C* @4 a% v2 O! `) D8 E

在 “RuleSets(规则设置)” 界面可以管理自定义的检测规则。因为在实际工作中,可能需要根据实际情况自定义检测规则,就可以通过这里导入,如果要使用它,需要在启动PMD进行检测时选择该自定义规则。

6 S6 h2 w0 x4 G9 G6 T! e

点击“Options”选项卡,在其中可以配置一些检测规则选项:

. ~8 C) w. H! h6 ~" p, p

9 X' }9 i6 ?: U! f

其中重点需要留意的是“Skip TestSource”这一项,因为在项目中有不少Android Studio自动生成的测试代码,如下所示,选择上述选项后可以将其过滤掉。

& i5 M7 p& H. g0 H2 }. u

四、FindBugs:

6 S) ^0 Q( r9 x- R

1、整体介绍:

0 [& G5 n" j0 `3 t: M

FindBugs 侧重于发现代码中存在的bug,如运行时错误检测(空指针检查、未合理关闭资源、字符串相同判断错(==,而不是equals)等),它可以简单高效全面地帮助我们发现程序代码中存在的bug以及潜在隐患,针对各种问题,它提供了简单的修改意见供我们参考

& m# h1 E/ e' J6 E/ `! M7 J a" Y

2、安装步骤:

9 o p2 y6 P0 ^; X4 W" H0 t# Y" n0 L$ f

通过 File > Settings > Plugins > Marketplace 搜索 “FindBugs”,按照提示进行安装,然后重启即可

# b- R+ ~ r6 y# l4 y! M3 F

3、使用说明:

% d9 D* L0 q5 l& H2 K) K

FindBugs 可以分析单个文件、包下面的所有文件、整个module下的文件、整个project下的文件,右键想要分析的文件名/包名/module名/project

7 D8 C! ?0 o: ^8 j9 Y

" {6 P% _2 ~' O) G

分析完之后就会出现结果面板

: ?4 C1 x" K; ?

; _5 c* T6 @& j" g8 E, O

点击对应的item在右边会定位到具体的代码,这是根据提示进行处理修改就行

5 s) F7 G, }, N: ~% W4 L+ N + y' ]$ I; p% {5 j) q- l; c; A

4、附:常见的错误信息

( @1 }: Z/ o- O5 p1 v( I

4.1、Bad practice 代码坏习惯:

& H- ?% p$ }( [' Z1 }, o; _! t $ X! Q6 |7 L5 \9 j { X! n# r. {

图片

) X& T: h: L% `

4.2、Dodgy code 糟糕的代码:

5 i9 t& M% ` q$ f- H2 Z5 A- x - u4 Q& O, i. F3 B

4.3、Internationalization 代码国际化相关:

: M& W; a6 k5 E2 l

4.4、Performance 代码性能相关:

K. u2 e. f- D& v j& R9 \: N) A * o$ }* R: V$ Y5 T6 A3 B

4.5、Experimental:

$ v- u) f' h# Q F! ?: {: Y/ P; u( x# \7 r/ k

4.6、Malicious code vulnerability 恶意破坏代码相关:

% `% [* {6 Q: t; ~+ c/ H/ t7 B 7 e1 c' V% y5 I p3 Z- N

4.7、Multithreaded correctness 多线程代码正确性相关:

1 K! d( R, S3 l3 \

4.8、Correctness 代码正确性相关:

, A/ `/ J. {0 u. x * P8 _" C1 h2 }5 E8 \6 B

五、SonarLint:

L7 X5 a8 t# L; y# H+ m

1、整体介绍:

9 u+ t. [2 {6 |9 V

sonar 比 Findbugs 高了一个层级,它不仅关注常规静态BUG,还关注到了如代码质量、包与包、类与类之间的依赖情况,代码耦合情况,类、方法、文件的复杂度,代码中是否包含大量复制粘贴的代码,关注的是项目代码整体的健康情况。sonar 有两种使用方式:插件和客户端,sonar 的插件名称为 sonarLint。

( }4 i6 F% H" g

2、安装步骤:

2 h: t N; u; C6 j

通过 File > Settings > Plugins > Marketplace 搜索 “SonarLint”,按照提示进行安装,然后重启即可

# z, |, v! K, U4 o

3、使用说明:

8 N* e* G% O# |+ h1 }9 I/ l+ h! w9 R8 _3 v3 E: u

右键项目或者文件进行如上图所示操作,执行之后可以看到如下信息,如果代码中有不合理的地方会在report中显示,同时点击错误的地方在右边会给出建议的修改供参考。

/ t; a$ Q3 x0 c- U' @+ s- [9 e9 p + |! h. H* z3 ~* D: E9 Y. a

4、配置 SonarLint 服务端:

" ?2 F) i) l* t5 J$ c& t% G

4.1、配置 Sonar 服务器:

2 m! x5 }" @9 k( ?0 F! R

sonarLint 插件的使用场景是自用自审,但 sonar 也提供了平台版本,使用场景则是他审,sonar 平台的搭建就不在这篇文章介绍了,感兴趣的读者可以自己上网查看,我们这里主要介绍如何在 sonarLint 插件中配置关联 sonar 平台服务器的工程,进行本地检查:

' q' U: N8 ^# w- L0 s ; V5 [# p% a( X4 z" h

点击新增按钮,输入Configuration Name,配置sonarlint 服务器的地址,然后下拉框选择 Login/Password,输入 sonarlint服务器的账号密码

, P' O4 K( n7 {( {: w9 q - q% x6 C7 G2 O$ e( g

4.2、具体 Sonar工程配置:

+ q y- _ a y& \; r+ y8 K

配置完服务器之后,需要针对具体工程进行配置,点击 connection下拉框,选择上面配置好的服务器连接,然后点击 Search in list,找到对应的工程:

) F- w; w7 }; N( A1 c/ ` : m$ q- r! q2 k: P [: z

4.3、使用 SonarLint 检查:

6 L; F+ T2 P" ~

配置完上面两步之后,接下来就可以选择要进行检查的类或者目录进行 sonarlint 检查了(跟第3点的使用方式一致),同时,在 commit 代码的时候,勾选 “Perform Sonarlint analysis”,会针对你要提交的代码进行sonarlint检查

# E1 t) D. X! n' R+ f' M1 H4 P) K* f8 ?. N6 A

总结

. k7 x. n& \9 k9 X" o ' S9 k7 t9 t, i# R 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了; j8 c, j. }. ^; s7 g3 x 找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充: ( E- p) t- w" K& J6 z& V PMD 自定义能力强,用来自定义项目BUG规则非常好用 " a- X7 A+ ]$ H Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。 P7 ]- T( k2 F( Z. f9 J SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景" C- J* |+ I- u , R& z: _% h* C% ?" w' p

文章由重庆腾憬文化科技有限公司新闻事业部整理,仅用于学习交流,版权归原作者所有,如有问题,请及时与我们联系,我们将第一时间做出处理。返回搜狐,查看更多

3 K u( p5 l) ~6 v* }+ V3 C 6 _# K- O; J' n1 c) t, H8 {& F$ x

责任编辑:

* Z3 {) `2 k7 e$ z" Z; j ( j$ j; p G$ F6 t 0 p5 a+ P' q; u- l! e; Q1 e6 H/ V9 r0 }2 i7 G 2 j( u: l p0 m- v( s7 r* }$ ]5 m% J9 g
回复

举报 使用道具

相关帖子

全部回帖
暂无回帖,快来参与回复吧
懒得打字?点击右侧快捷回复 【吾爱海洋论坛发文有奖】
您需要登录后才可以回帖 登录 | 立即注册
家国依稀残梦里
活跃在7 天前
快速回复 返回顶部 返回列表