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

[复制链接]
. ^8 U% e2 K0 j+ X

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

7 F5 D/ s7 |( I& n! e% @4 J 3 }* ~& E: A8 r% Z9 Y

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

# F% Y. k* U7 _7 l2 e! S9 m! r- x% }

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

/ c2 \. ?$ y% f! z

一、Alibaba Java Coding Guidelines

* r7 Q& Q: D% d* L: ]

1、整体介绍:

h( h0 i" g" F4 Q0 A' _1 P. j

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

1 z* I5 S$ S9 i" x; x. |4 K

阿里巴巴规约扫描包括:

2 Y$ F$ z4 s+ r/ ^" ` + A& W: K, Y0 |9 N2 Q, `: `8 C' q L OOP规约0 Q- q7 C- Y, J 并发处理9 X7 B1 K( d, I3 E) J, F5 L 控制语句8 T: @6 c) ?$ f# X 命名规约 ) M4 G# i& u$ C/ O% q( w9 y- }6 C) W5 W 常量定义" I" J0 X. Z# x 注释规范 8 y: A6 S' j7 {) t* n8 Q) x6 A3 @+ c7 _. P# o" G

2、安装步骤:

8 x% o# D* J( F8 N

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

; W! C5 i+ a) R& g4 F

3、使用说明:

/ Y: {6 R: v- Y: `4 x

3.1、运行方式:

0 O* b3 _1 `1 K* ^1 q

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

V# I/ o' ?! X- F* n

" a" \+ _1 J8 n* b

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

" U: W0 Y; v, m$ {& k# S& v

) }$ k$ j" y$ m2 ~

3.2、菜单功能:

9 @& r" J* U0 w& U4 `# I 0 M/ l1 l! ^/ P. S { 编码规约扫描:开始扫描代码 ! }) k6 w; u3 V6 W" p z 打开/关闭实时检测功能:实时检测代码,一般机器性能比较好的话可以开启这项功能4 |' l& W3 g. w+ J: L 切换语言至英文:中英文切换: p! L% c S0 Y8 a1 [- U/ ~ 7 f6 c& `& f( o' y* i

3.3、运行结果:

) `9 \8 P, X, e- r9 W. w6 g. K+ N6 Q

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

# h: M% h' B+ G4 P

- q$ p3 \8 g' @. x! ^

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

2 o0 O4 H) u. V: e! R

$ i5 u# w% y2 K$ l! F% u

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

) P ~& H# H" O4 n# c

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

1 F% V: N$ g9 x

; y s5 i$ `' b* v

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

" z. b* G' Y- f, i- B) ]4 X4 A& ~! \

: Z9 D" h- B" J! r

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

8 z o9 W% x* o- I7 R+ C

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

G6 X4 C+ P. Y" l1 s7 D( M/ j

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

; `* |. V% ^) L

, S6 i$ U4 g( p7 P: V9 [

3.4、工具栏功能介绍:

4 e8 G* \6 Y5 S# x' a% U& s, l) S

& e/ R4 k, A1 A, x; _( D / P' X4 D. ^% s, Z Rerun Inspection:重新运行一次扫描+ q+ p1 M& y( }9 p Close:关闭真个AJCG面板! F) b) F; W) U Expand All:展开结果的树状结构,整个结果是树状结构的。4 }6 u6 w. p" n5 e5 q+ a K; ~, s Collapse All:收起结果的树状结构 + b& F" f, U! S0 s+ {/ v8 o Go Pre Problem:选择上一个问题 : z' n9 v# x( a, Q) F* r6 J Go Next Problem:选择下一个问题 + B: y! T9 {6 F" {+ P- W Help:帮助# Y6 l. T$ F, o# s" S" N Group by Serverity:(不知道如何描述); G5 Y0 F5 U- R Group by derectory:按目录分组/按类名分组间切换& b9 k0 s# d* Y# z Filter resoled items:过滤掉已经解决的项 + P% k% X) w1 N; M* K Autoscroll to Source:自动滚动到源码 ' a5 J# X9 D$ B, ?8 o Export:导出,可以导出为XML和HTML两种格式 : w, `1 M# a7 v; m; i( k Edit Settings:编辑设置& Y1 q/ _1 p% j4 M$ T( \ $ G& r, c5 S; X: y* H2 y- Y- u

二、CheckStyle:

7 `+ L% l- d2 w7 d

1、整体介绍:

/ X; d4 B0 Y% C6 C F

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

6 l1 J1 N1 N1 _7 F) f1 G$ Z2 o

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

/ Y! Q# v0 _9 [& q

2、安装步骤:

7 H0 i7 l' E$ c

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

/ L) E/ A0 p7 Q9 b5 o9 _* q- V

3、使用说明:

; _6 i. @0 Q1 K% o* r( x

+ p, c0 i& K7 r2 o9 t' z

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

! D3 ^# Y3 E2 d0 @

三、PMD

8 P( Z% ]( C; j: j6 O

1、整体介绍:

: J: v1 K) x7 x- z6 N( k5 u* x" @: x

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

- W; W5 o3 @% o5 v! Y: U ; G: G f3 t- d& D+ ? 潜在的bug:空的try/catch/finally/switch语句7 G' q$ W" ^% V, Y 未使用的代码(Dead code):未使用的变量、参数、私有方法等 * X3 c& O, k3 X$ C 可选的代码:String/StringBuffer的滥用 9 {: e: z# Q+ u2 E- I- W 复杂的表达式:不必须的if语句、可被while替代的for循环. @1 S: M0 j+ d4 p 重复的代码:拷贝/粘贴代码意味着拷贝/粘贴bugs ( s/ c6 W( h8 i3 O& f 循环体创建新对象:尽量不要在循环体内实例化新对象. C+ y+ G4 F# b" P 资源关闭:Connect,Result,Statement等使用之后确保关闭掉. {1 x% x6 g& y' z) @& Y+ R , Z1 t2 u. Y" z4 b0 ]

2、安装步骤:

8 Q; S. J" @9 O# K. d; Q/ D

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

( Y6 p$ E- U# G- W3 z

3、使用说明:

" Z5 d [ g; o! O4 `; s& J' t+ r( b; Z 参考文章:http://wjhsh.net/andy-songwei-p-11830812.html 4 v+ z: J: S1 Q8 k2 G( Z' X n" z

3.1、运行方式:

0 ^6 l/ s- H+ v' _

(1)从Tools菜单中启动:

- Q U4 m( ?2 }; y2 Z

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

0 l, }; n' o1 B! s) G & Q7 E: r& h. ^- C$ U Pre Defined:预定义的规则,也就是插件自带的检测规则。后面展开的列表中列出了所有的规则列表,想扫描哪一种类型的问题,点击即可。其中“All”表示使用所有的规则。 # G) T( i: G$ a0 m8 X/ J: d Custom Rules:自定义的检测规则,PMD允许用户根据需要自定义检查规则,默认这里是不可点击的,需要在设置中导入自定义规则文件后方可选择。& q7 u6 P2 P" k % ]9 z5 c! i" I. R

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

* |! } d7 _/ K6 X

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

0 |4 W/ f; ~4 n: W! I6 ]7 Y/ R

3.2、运行结果:

- y! [" L/ T5 u9 f; E6 ~- D" q' y

& X5 c6 ^. A5 S. ?8 m3 x: [

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

8 X+ C4 ^3 b0 t+ X, ~* Z

3.3、配置检测规则:

8 G/ [3 u" r9 H$ V

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

# k# I; o( p, D, G/ H) q9 f2 A

9 ^ {2 {+ L: L) [' q

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

& I6 H; ?5 |/ g" N' e6 n

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

# m E6 e; U7 V* V& P% Z" S

4 W2 _. f# b& _# B$ z/ E- F

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

: }' S+ U. k+ A! x4 m$ V

四、FindBugs:

4 D9 F- o$ Y/ M' o% f9 g

1、整体介绍:

! g- G& j" X8 F' H+ U* h

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

- v4 F) K" ?3 r' W

2、安装步骤:

3 m; l. E& G5 x0 }7 j

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

f- Y% B* f8 b M( P

3、使用说明:

7 r+ l. J1 I$ f2 d0 [; `

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

+ Z% n* r& i+ F1 k. K6 v' k

# Z% M# O( L1 _

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

; X1 H# A+ ?2 p) F$ [9 m

6 n& B6 }/ ~: h) ~ \

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

" ~5 @; f6 e$ q8 ^" ] . q9 J$ g4 l% Z# A2 ]2 k) X5 M

4、附:常见的错误信息

6 F, {% b7 X+ j0 `9 Y3 j; \0 @

4.1、Bad practice 代码坏习惯:

9 P) C5 l& S$ S ) ~( _2 e# Y1 i' q! P6 D& S* L

图片

- w- ]4 _) Z# A1 ^

4.2、Dodgy code 糟糕的代码:

4 P2 `7 h7 W* I$ Q% Y$ u9 D2 g $ ^1 Q! W. o& l2 B* }& m% e/ C7 V

4.3、Internationalization 代码国际化相关:

* n0 o* {( ]( K" R! F1 v. a8 g" b

4.4、Performance 代码性能相关:

2 h$ P, P) G6 N+ h- J, ^9 s' d / x+ N# Z, N J# t$ ~: K8 @

4.5、Experimental:

7 z9 B- g9 Z# p7 e . x: b9 [" v1 B& ]( g7 k$ q1 L

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

" g! s3 Y3 H/ ^4 ^) w% B$ A * ^" i7 R4 W3 X( j _7 W; |( c( Q4 u

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

; i" K2 g0 x( x/ l" ?

4.8、Correctness 代码正确性相关:

0 c: r3 v. x2 Q3 Y; ` 1 B* c3 l3 o D p. }, F& W

五、SonarLint:

9 |4 R' [& q3 k( o# U7 h/ C# ~* G1 N

1、整体介绍:

" N. v' S- X5 K9 s$ [# k* _- S

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

9 ? x7 p% l0 t7 _

2、安装步骤:

0 q! `+ D N) G; v

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

; X$ O3 w* w+ _0 H) g

3、使用说明:

4 w8 k5 f, r6 q1 c/ g; b& W6 d6 F/ n % f# N, Q% R6 s" z% h1 z' c

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

# K: z, n8 P# @6 q- Q# h1 N8 B! L _) r( H- e" A

4、配置 SonarLint 服务端:

& m9 a5 _7 u, r+ R

4.1、配置 Sonar 服务器:

: J! E9 Y1 Z# H0 \

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

1 Y; x U) ] y/ C: c 7 Y( }$ @8 b% v+ v7 a

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

- d( `% W7 E$ N6 U 3 E& e, a5 o" a+ a6 A; I

4.2、具体 Sonar工程配置:

+ I4 K0 P! W0 P( e" `% ]0 H; }

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

5 r% T. o: s; P! c. y2 q' |& K3 a ' w* k/ N" @6 d. O, m. y7 [- `( ^" x

4.3、使用 SonarLint 检查:

; q1 A" m% d. ^0 E

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

: [/ l& ^8 x4 s6 C! h" S. m" K' v' G9 b! l( l) m3 _" r$ @8 u

总结

2 T& k( g- x8 F; N - x$ c; j9 @: H5 s( Z9 M( v6 e 检查代码规范的话,直接使用 Alibaba Java Coding Guidelines 就可以了 1 A$ d% V, F2 c' z* n* ]* m 找 BUG 的话,使用 PMD、Findbugs、SonarLint 相互补充:+ W) G4 k6 a9 R2 z, L* ]$ t PMD 自定义能力强,用来自定义项目BUG规则非常好用' b! \: `0 n6 U2 j4 }$ B Findbugs 找 BUG 能力很强,我们拿找到的BUG给新员工培训也很好。 5 l8 V5 V5 b( z SonarLint 规则丰富,比 Findbugs 能覆盖到更全的场景2 Y1 e0 ~( b1 H4 G; A8 }. W 0 B) s5 g- z! Z3 H

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

- C [$ U6 n+ ~+ [9 k0 _( W9 @7 `6 W% K! I0 E! O% R$ X- I3 w7 c

责任编辑:

( G9 F; A V2 w" X& L* M3 |3 J# ? U; V8 s - z1 o" ], z" W' {% Z ( v1 b' n% ], R: w& E: O. V3 }6 V 7 a1 N& J, G6 P: H4 ~* O7 h7 v
回复

举报 使用道具

相关帖子

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