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

[复制链接]
- z5 ?3 L( g/ V

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

" ?9 T0 h: r9 j9 D+ l6 w6 E: d3 n ! r, P. d2 Q* U1 T& ?/ Q8 V' T

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

5 ]7 \9 C# {9 W0 E+ a

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

5 k5 k: T5 ^9 e2 I. g

一、Alibaba Java Coding Guidelines

1 u U5 i6 u' |! i

1、整体介绍:

0 ?( ]9 h0 T0 Q8 q

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

1 u& g& Q! |9 D, K: [: x

阿里巴巴规约扫描包括:

/ O; U% S9 K; J # R* ^$ a& u- P3 d OOP规约# b9 [. }" V" ~1 U5 d 并发处理 # y- n% x+ P1 {8 {( j 控制语句 ; N5 H% M0 J1 g; B 命名规约 ; I* d, e6 r1 c" ~& U 常量定义 / r- T2 X( P* v3 r! j+ o 注释规范& C% o' I2 g% S7 Q8 r ; z7 b1 ?7 b" Q3 P

2、安装步骤:

: b; V7 ^: p) t

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

+ E; y0 {- r8 R& r& P6 u

3、使用说明:

# ^2 u# [4 O( l. z* t2 @$ v+ E

3.1、运行方式:

, |( P6 t1 [. W' |( v- y5 I* V

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

) C o- ~; \& I v0 g5 H8 ]7 ]& _

2 @6 \* s2 b2 A' `% h

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

" H L2 T+ z; x9 S1 ^

2 I7 ~, }* C5 R* n% p V; P

3.2、菜单功能:

! p2 O% r0 x: o2 \% w5 L* [ # w: s1 Z; a( @/ O6 N. I4 q 编码规约扫描:开始扫描代码 / \# ]3 F; q- u8 p: ^% v 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能3 k/ D6 A! [ w2 Q 切换语言至英文:中英文切换 h& I% {; z, ]- f1 u- \1 F% i& u 5 ~; p# w0 D# A% G! p4 G `

3.3、运行结果:

! ~ [' v6 D( J7 h

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

1 J U% j( ]3 B7 b* l

: q5 a/ @. F. o" L) a! D

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

I2 o, |* f( ~* L: ^

2 u, N" Z0 v% i

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

) f- f, R: I3 k5 { F( X

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

- C4 [& ?. A; p

5 f2 M9 Z3 _; S& U7 O1 _4 Z

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

) ]# D- U8 L2 E" m( n

- z, N/ \. V( x

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

4 M4 y( K( J% r7 O

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

# n$ ?! L0 ]# _5 U; n: R& b

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

, [; _5 M) i0 G9 r6 \; U4 R

* H2 c7 [, O5 q" g* a: E

3.4、工具栏功能介绍:

" P+ q/ @ @ u1 \" \ n C

) W" ~: n1 v% S& k6 M6 i* d4 I, e9 E) x. R Rerun Inspection:重新运行一次扫描" Y$ f$ G+ s2 O4 R& t Close:关闭真个AJCG面板9 S+ A. [' i5 O$ K* Y3 w/ b1 X Expand All:展开结果的树状结构,整个结果是树状结构的。8 C" h: ?# Q% @5 L4 j+ R9 T Collapse All:收起结果的树状结构 , F- g t2 |0 `7 t- b2 Q Go Pre Problem:选择上一个问题 ' J0 K" J; M# K8 |, N Go Next Problem:选择下一个问题# u; X8 ~# ~: f8 ]4 y0 O- C Help:帮助% R& p/ {; Y! T3 j% c- O Group by Serverity:(不知道如何描述) 0 y6 d+ t- A, { Group by derectory:按目录分组/按类名分组间切换 7 t u( q7 n; j' _4 O9 ~ Filter resoled items:过滤掉已经解决的项 $ j4 l7 s) f2 ~: o Autoscroll to Source:自动滚动到源码 " G+ P7 B4 k( S, W Export:导出,可以导出为XML和HTML两种格式! g4 u- z1 K- W Edit Settings:编辑设置 . T, o$ v& z, B9 L! R " }2 G0 f: `" ^& n1 @8 t, l

二、CheckStyle:

1 o o; \, k$ [& p

1、整体介绍:

$ B3 ?! R6 |* Y. e4 k

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

7 W6 \2 m/ {- h: S+ ?4 T) b* L

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

5 J- c- I- M4 q

2、安装步骤:

V' L( G& G& p2 u. |) z: _

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

# `: | D) Z; P8 g

3、使用说明:

7 W8 _- X' v1 \9 C$ p- [: P

7 P- w( Q! S3 }/ e' U

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

# r; S) u. G3 Y) ?% N, J* c

三、PMD

9 Z% O. `2 u; q' h8 d

1、整体介绍:

$ i* \* o7 E" w- _( o" i7 T

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

9 t0 U8 N5 a1 h. j 8 Z9 Z5 M0 h6 E 潜在的bug:空的try/catch/finally/switch语句) d. H. |* l& S 未使用的代码(Dead code):未使用的变量、参数、私有方法等 ( k5 |) [4 a5 b' Q+ J& W# w 可选的代码:String/StringBuffer的滥用 ! H2 j+ B+ v9 ~& r* w2 ~4 L5 Z 复杂的表达式:不必须的if语句、可被while替代的for循环 6 k; z8 L# M; `" W* g5 z 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs% z) Y0 D% ]. a5 O 循环体创建新对象:尽量不要在循环体内实例化新对象, g( ?2 d: D5 r' _/ U1 w5 D; J 资源关闭:Connect,Result,Statement等使用之后确保关闭掉 , S% w0 X3 G* J7 S2 ~ 5 }4 f& K) |+ ^% P

2、安装步骤:

' X) f1 b: X7 _; Z q+ z2 i. n

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

6 e# {0 r$ I, v( K0 S

3、使用说明:

4 K& M: X( Z4 \1 d8 z1 r/ H# @ 0 T. @2 W/ h% g: ~5 Z* s7 R( h' ` 参考文章:http://wjhsh.net/andy-songwei-p-11830812.html4 a3 p$ g% Y x8 y+ [! h4 B ! X9 t7 _0 A* R$ K, N3 p) C) |

3.1、运行方式:

0 I3 I% p: s( ~* M8 F

(1)从Tools菜单中启动:

: F, P6 I1 R) F: p0 y7 D

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

" L0 K( v. w; G' E7 U; [3 [# N& V5 N6 L. I& I; R9 Y' ~1 ~/ d Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。 # D9 h0 p3 W6 m Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。 : j' F$ d, Q. }$ h0 l' z4 J1 O; t! j8 K! w' L. E. O5 d. g" Y& o

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

7 y3 v9 R8 Q. n! \

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

/ c, Q+ h! {+ M7 \

3.2、运行结果:

y4 |* J; f h' U) y$ P

6 z p x$ }. A5 D/ p4 ^) o( ]

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

* v! \: {% q8 a

3.3、配置检测规则:

% d6 I: s! C/ P- y& m3 p1 S

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

: T) R ^, p: u: k

$ w' g, R& a% u7 x9 b6 J* @2 J

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

) i8 _2 L O0 F, |7 T

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

[) v0 D# i- J8 P. P* ~

( v4 ^' m& u R) R/ N& E

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

. c) _/ H3 J4 _7 n/ I0 q1 U! A

四、FindBugs:

M( D8 I7 u: d n1 o

1、整体介绍:

7 r2 D# P* W8 ^4 A/ w

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

$ h7 i z' [ L9 Q

2、安装步骤:

; @% C& o: @9 \( t8 t, ]0 S3 c; u: Z

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

, v- |0 `$ P$ t, g( O' q8 g( o# n+ Q; q. m

3、使用说明:

" ]8 e7 t* _& P5 j; Q6 m1 b$ ^

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

8 [+ K0 @1 k' u4 }6 r) h7 \

" @! m& L6 O5 h1 P" s

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

* X9 @2 N8 r0 X$ p- C

1 z$ }6 @: y. L! ]3 @

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

# `: p% |/ c* g' Y5 l+ J+ h9 G5 e+ X4 r$ S) [5 p

4、附:常见的错误信息

4 a- [# ~9 K$ i+ h: k5 Q! [

4.1、Bad practice 代码坏习惯:

, L( W; A( J0 j' a D- F: J9 U - m2 d* r! O5 w/ g

图片

7 C, j- U; z- }9 O, a# c" @

4.2、Dodgy code 糟糕的代码:

. S4 b% a$ n* {; S! J) o' @$ T" s

4.3、Internationalization 代码国际化相关:

) c0 k; G" L* G$ Z* G# i

4.4、Performance 代码性能相关:

0 F& \1 U9 Q) f* `! c, e [- W4 f. t9 e5 V' V/ o& h

4.5、Experimental:

* j& ^0 ]. B8 ^3 Z! [8 j' T* g, {9 P4 {$ b6 p/ M/ O

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

- C h% w, [6 Q: z- j) e- p2 U( k0 c6 Y3 ~3 j; e1 f, A3 K: o

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

5 Y. J9 I& P& e

4.8、Correctness 代码正确性相关:

+ L0 d- S, |6 g. I+ E% S& @. w# } 2 G5 j& F6 O- O% X* R; a

五、SonarLint:

8 V' P1 s: j/ w2 Z

1、整体介绍:

2 K7 K* L9 {( H) ]* ^

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

, I2 U; \0 i6 S( Z1 G8 K6 B8 g

2、安装步骤:

' _3 r7 C5 b: L" o1 u/ Z+ u

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

8 x& }* c. o6 H

3、使用说明:

3 X* v7 @- p; }9 \ # m" ^% n! {$ O( U; @) L

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

, O; c4 n; O' b+ B ; H6 J: b1 \' U. Z4 q. _( Q9 Z3 h! e

4、配置 SonarLint 服务端:

. e' s% ^# m6 `4 l5 h

4.1、配置 Sonar 服务器:

- \4 x, P. u0 x/ m$ i

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

, w/ N. m2 Y a* _ 6 }% l- c; \& S" q/ c" n7 ^

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

8 \- y6 [3 M5 t0 E, w; t # w/ a) ~' N/ v* m4 `3 o

4.2、具体 Sonar工程配置:

9 Q: V& l7 L$ ]' F

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

" k: k- L9 v3 I. a0 w( J b1 [# T: c w; O5 v* y

4.3、使用 SonarLint 检查:

/ ~& d* n6 n, C; \4 C! D9 V

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

$ b3 D+ T& X& T9 N# j' @; x6 R9 }( S0 m) T

总结

/ r: [) H8 y! x- C# J , u6 f( Q& x7 E3 Z8 V 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了* C: V7 y$ J! h( A) y 找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:- ^. V6 u3 [2 f( S' }, Y5 U" N PMD 自定义能力强,用来自定义项目BUG规则非常好用 ) Q I1 M$ z+ E# [/ F, I Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。 q2 d' u' T% u% d. g' ]- m SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景 x7 f; b9 F' N: k5 D! L " G4 T I! L$ ]' u8 V2 ?6 K3 K

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

& ?6 i# b8 n" r& q$ }. }5 F) n6 h! Z% J7 \5 U

责任编辑:

; L' v0 d" o% k) H: W/ F1 Z( \# \' u5 g, \& w$ q/ ? # R1 U0 E8 q; P) X+ U* U , h: k! i- V& b - O5 }+ R1 J3 ]

相关帖子

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