融入團(tuán)隊(duì)代碼風(fēng)格,代碼越寫越爛!
最近是真的有點(diǎn)忙啊,需求和bug都排到下一個月了,難得今天休息有空上來扯扯,主題是有關(guān)代碼質(zhì)量問題的。
在我初學(xué)的時候,我是挺喜歡專研各種Java語法和性能相關(guān)的問題的。舉個很明顯的例子,我們都知道Java的單例模式有好幾種寫法,什么餓漢式、簡單懶漢式(在方法聲明時加鎖)、DCL雙重檢驗(yàn)加鎖(進(jìn)階懶漢式)、靜態(tài)內(nèi)部類(優(yōu)雅懶漢式)、枚舉。
這貌似算是Java的高頻面試題了,所以這幾種寫法的優(yōu)缺點(diǎn)幾乎都已經(jīng)被說爛了。
1、所謂「餓漢式」指的就是還沒被用到,就直接初始化了對象。(浪費(fèi)內(nèi)存) 2、所謂「懶漢式」指的就是等用到的時候,才進(jìn)行初始化。(不浪費(fèi)內(nèi)存)
在剛學(xué)明白的時候,我一想:“我這以后肯定是用懶漢式啊,對內(nèi)存多友好。我不用簡單懶漢式(在方法聲明時加鎖),這會影響到性能。我可以用雙重檢驗(yàn)加鎖(進(jìn)階懶漢式)或者靜態(tài)內(nèi)部類(優(yōu)雅懶漢式)”。
后來工作了以后,發(fā)現(xiàn)工作代碼里什么牛鬼邪神都有,逐漸發(fā)現(xiàn):代碼也不是不能跑,編譯器遠(yuǎn)比我們想象得更加強(qiáng)。
于是,我用個錘子的懶漢式,我反手就是new出一個單例,代碼多簡單優(yōu)雅。我堂堂一個Java開發(fā)工程師,我還缺那點(diǎn)小內(nèi)存?真可笑。
剛進(jìn)團(tuán)隊(duì)的時候還想重構(gòu)臟代碼,后來隨著需求的累積,又有編譯器給兜底,也沒人code review監(jiān)管,寫代碼就越來越隨意了,融入團(tuán)隊(duì)代碼風(fēng)格就完事了。
后來出了點(diǎn)小故障,影響到線上的正常業(yè)務(wù)了,最后發(fā)現(xiàn)是臟代碼導(dǎo)致的,被老板吐槽了。讓我去找找有哪些工具可以排查下有沒有類似的問題(代碼質(zhì)量檢查),我一想,項(xiàng)目這里面的代碼的坑也太多了,不好搞啊。
工作還是得繼續(xù)干的,既然是代碼質(zhì)量檢查,找些IDEA現(xiàn)成的插件就OK咯。
Alibaba Java Coding Guidelines
集成了阿里巴巴的代碼規(guī)范和最佳實(shí)踐,幫助開發(fā)者保持代碼質(zhì)量和一致性,austin項(xiàng)目也一直用這個插件做代碼的掃描。
CheckStyle
以下是 CheckStyle 插件的一些常見功能和用途:
-
代碼規(guī)范檢查:CheckStyle 插件會分析項(xiàng)目中的源代碼,并根據(jù)預(yù)定義的規(guī)則檢查代碼,包括縮進(jìn)、命名約定、代碼注釋、代碼結(jié)構(gòu)等。它可以幫助團(tuán)隊(duì)確保代碼風(fēng)格一致,符合公司或項(xiàng)目的規(guī)范。
-
自定義規(guī)則:您可以自定義 CheckStyle 規(guī)則,以便根據(jù)您的項(xiàng)目需求和編碼標(biāo)準(zhǔn)添加自定義規(guī)則。這使得 CheckStyle 插件非常靈活,可以適應(yīng)各種項(xiàng)目。
-
代碼質(zhì)量改進(jìn):通過檢查代碼中的潛在問題和不良實(shí)踐,CheckStyle 插件可以幫助開發(fā)者識別和修復(fù)代碼中的問題,從而提高代碼質(zhì)量。
比如,我設(shè)置了一個規(guī)則文件:
<?xml version="1.0"?>
<!DOCTYPE module PUBLIC
"-//Puppy Crawl//DTD Check Configuration 1.3//EN"
"http://www.puppycrawl.com/dtds/configuration_1_3.dtd">
<!--checkStyle -->
<module name="Checker">
<!-- 檢查文件是否以一個空行結(jié)束 -->
<module name="NewlineAtEndOfFile"/>
<!-- 文件長度不超過1500行 -->
<module name="FileLength">
<property name="max" value="1500"/>
</module>
<!-- 長度檢查 -->
<!-- 每行不超過140個字符 -->
<module name="LineLength">
<property name="fileExtensions" value="java"/>
<property name="max" value="140"/>
<property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/>
</module>
<!--禁止打印e.printStackTrace錯誤信息-->
<module name="RegexpSingleline">
<property name="format" value="printStackTrace"/>
<property name="message" value="Prohibit invoking printStackTrace in source code !"/>
</module>
<!-- 每個java文件一個語法樹 -->
<module name="TreeWalker">
<!-- import檢查-->
<!-- 避免使用* -->
<module name="AvoidStarImport">
<property name="excludes" value="java.io,java.net,java.lang.Math"/>
<!-- 實(shí)例;import java.util.*;.-->
<property name="allowClassImports" value="false"/>
<!-- 實(shí)例 ;import static org.junit.Assert.*;-->
<property name="allowStaticMemberImports" value="true"/>
</module>
<!-- 檢查是否導(dǎo)入了多余的包 -->
<module name="RedundantImport"/>
<!-- 沒用的import檢查,比如:1.沒有被用到2.重復(fù)的3.import java.lang的4.import 與該類在同一個package的 -->
<module name="UnusedImports"/>
<!-- 注釋檢查 -->
<!-- 檢查構(gòu)造函數(shù)的javadoc -->
<module name="JavadocType">
<!--允許位置的tag如@date,@description等-->
<property name="allowUnknownTags" value="true"/>
<message key="javadoc.missing" value="類注釋:缺少Javadoc注釋。"/>
</module>
<!--方法注釋-->
<module name="JavadocMethod">
<property name="tokens" value="METHOD_DEF"/>
<!--允許get set 方法沒有注釋-->
<!-- <property name="allowMissingPropertyJavadoc" value="true"/>-->
<message key="javadoc.missing" value="方法注釋:缺少Javadoc注釋。"/>
</module>
<!--校驗(yàn)方法的注釋-->
<module name="MissingJavadocMethod">
<!--允許get set 方法沒有注釋-->
<property name="allowMissingPropertyJavadoc" value="true"/>
<!--允許構(gòu)造方法沒有注釋-->
<!-- <property name="" value="true"/>-->
<property name="scope" value="private"/>
</module>
<!-- 命名檢查 -->
<!-- 局部的final變量,包括catch中的參數(shù)的檢查 -->
<module name="LocalFinalVariableName"/>
<!-- 局部的非final型的變量,包括catch中的參數(shù)的檢查 -->
<module name="LocalVariableName"/>
<!-- 包名的檢查(只允許小寫字母),默認(rèn)^[a-z]+(\.[a-zA-Z_][a-zA-Z_0-9_]*)*$ -->
<module name="PackageName">
<property name="format" value="^[a-z]+(\.[a-z][a-z0-9]*)*$"/>
<message key="name.invalidPattern" value="包名 ''{0}'' 要符合 ''{1}''格式."/>
</module>
<!-- 僅僅是static型的變量(不包括static final型)的檢查 -->
<module name="StaticVariableName"/>
<!-- Class或Interface名檢查,默認(rèn)^[A-Z][a-zA-Z0-9]*$-->
<module name="TypeName">
<property name="severity" value="warning"/>
<message key="name.invalidPattern" value="名稱 ''{0}'' 要符合 ''{1}''格式."/>
</module>
<!-- 非static型變量的檢查 -->
<module name="MemberName"/>
<!-- 方法名的檢查 -->
<module name="MethodName"/>
<!-- 方法的參數(shù)名 -->
<module name="ParameterName "/>
<!-- 常量名的檢查(只允許大寫),默認(rèn)^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$ -->
<module name="ConstantName"/>
<!-- 定義檢查 -->
<!-- 檢查數(shù)組類型定義的樣式 -->
<module name="ArrayTypeStyle"/>
<!-- 檢查long型定義是否有大寫的“L” -->
<module name="UpperEll"/>
<!-- 方法不超過100行 -->
<module name="MethodLength">
<property name="tokens" value="METHOD_DEF"/>
<property name="max" value="100"/>
</module>
<!-- 方法的參數(shù)個數(shù)不超過5個。 并且不對構(gòu)造方法進(jìn)行檢查-->
<module name="ParameterNumber">
<property name="max" value="5"/>
<property name="ignoreOverriddenMethods" value="true"/>
<property name="tokens" value="METHOD_DEF"/>
</module>
<!-- 空格檢查-->
<!-- 方法名后跟左圓括號"(" -->
<module name="MethodParamPad"/>
<!-- 在類型轉(zhuǎn)換時,不允許左圓括號右邊有空格,也不允許與右圓括號左邊有空格 -->
<module name="TypecastParenPad"/>
<!-- 檢查在某個特定關(guān)鍵字之后應(yīng)保留空格 -->
<module name="NoWhitespaceAfter"/>
<!-- 檢查在某個特定關(guān)鍵字之前應(yīng)保留空格 -->
<module name="NoWhitespaceBefore"/>
<!-- 操作符換行策略檢查 -->
<module name="OperatorWrap"/>
<!-- 圓括號空白 -->
<module name="ParenPad"/>
<!-- 檢查分隔符是否在空白之后 -->
<module name="WhitespaceAfter"/>
<!-- 檢查分隔符周圍是否有空白 -->
<module name="WhitespaceAround"/>
<!-- 修飾符檢查 -->
<!-- 檢查修飾符的順序是否遵照java語言規(guī)范,默認(rèn)public、protected、private、abstract、static、final、transient、volatile、synchronized、native、strictfp -->
<module name="ModifierOrder"/>
<!-- 檢查接口和annotation中是否有多余修飾符,如接口方法不必使用public -->
<module name="RedundantModifier"/>
<!-- 代碼塊檢查 -->
<!-- 檢查是否有嵌套代碼塊 -->
<module name="AvoidNestedBlocks"/>
<!-- 檢查是否有空代碼塊 -->
<module name="EmptyBlock"/>
<!-- 檢查左大括號位置 -->
<module name="LeftCurly"/>
<!-- 檢查代碼塊是否缺失{} -->
<module name="NeedBraces"/>
<!-- 檢查右大括號位置 -->
<module name="RightCurly"/>
<!-- 代碼檢查 -->
<!-- 檢查空的代碼段 -->
<module name="EmptyStatement"/>
<!-- 檢查在重寫了equals方法后是否重寫了hashCode方法 -->
<module name="EqualsHashCode"/>
<!-- 檢查子表達(dá)式中是否有賦值操作 -->
<module name="InnerAssignment"/>
<!-- 檢查是否有"魔術(shù)"數(shù)字 -->
<module name="MagicNumber">
<property name="ignoreNumbers" value="0, 1"/>
<property name="ignoreAnnotation" value="true"/>
</module>
<!-- 檢查switch語句是否有default -->
<module name="MissingSwitchDefault"/>
<!-- 檢查是否有過度復(fù)雜的布爾表達(dá)式 -->
<module name="SimplifyBooleanExpression"/>
<!-- 檢查是否有過于復(fù)雜的布爾返回代碼段 -->
<module name="SimplifyBooleanReturn"/>
<!-- 類設(shè)計檢查 -->
<!-- 檢查類是否為擴(kuò)展設(shè)計l -->
<!-- 檢查只有private構(gòu)造函數(shù)的類是否聲明為final -->
<module name="FinalClass"/>
<!-- 語法 -->
<!-- String的比較不能用!= 和 == -->
<module name="StringLiteralEquality"/>
<!-- 限制for循環(huán)最多嵌套2層 -->
<module name="NestedForDepth">
<property name="max" value="2"/>
</module>
<!-- if最多嵌套3層 -->
<module name="NestedIfDepth">
<property name="max" value="3"/>
</module>
<!-- 檢查未被注釋的main方法,排除以Application結(jié)尾命名的類 -->
<module name="UncommentedMain">
<property name="excludedClasses" value=".*[Application,Test]$"/>
</module>
<!-- 禁止使用System.out.println -->
<module name="Regexp">
<property name="format" value="System\.out\.println"/>
<property name="illegalPattern" value="true"/>
</module>
<!-- return個數(shù) 3個-->
<module name="ReturnCount">
<property name="max" value="3"/>
</module>
<!--try catch 異常處理數(shù)量 3-->
<module name="NestedTryDepth ">
<property name="max" value="3"/>
</module>
</module>
</module>
當(dāng)我寫出幾層for循環(huán)的時候和常量的時候,就會被掃描和監(jiān)控到。
FindBugs
以下是Bug IDEA插件的一些常見用途和功能:
-
靜態(tài)代碼分析:Bug IDEA插件可以執(zhí)行靜態(tài)代碼分析,以檢測潛在的編碼錯誤、代碼風(fēng)格問題和性能問題。它可以識別諸如未使用的變量、空指針異常、不一致的縮進(jìn)、未處理的異常等問題。 -
代碼質(zhì)量評估:該插件可以生成代碼質(zhì)量報告,幫助開發(fā)人員了解代碼的質(zhì)量水平。這些報告通常包括代碼復(fù)雜性、代碼覆蓋率、代碼重復(fù)率等指標(biāo)。 -
檢查代碼規(guī)范:Bug IDEA插件可以根據(jù)您選擇的編碼標(biāo)準(zhǔn)或規(guī)范檢查代碼,確保團(tuán)隊(duì)的代碼風(fēng)格一致性,并提供建議以改進(jìn)代碼。 -
自動修復(fù)問題:插件通常提供了自動修復(fù)選項(xiàng),可以幫助開發(fā)人員快速修復(fù)代碼中的問題,提高代碼的質(zhì)量和可維護(hù)性。 -
安全漏洞檢測:有些Bug IDEA插件還可以檢測潛在的安全漏洞,如SQL注入、跨站腳本攻擊等,以提高應(yīng)用程序的安全性。 -
代碼重構(gòu)建議:插件還可以提供有關(guān)代碼重構(gòu)的建議,幫助改進(jìn)代碼的結(jié)構(gòu)和性能。
在這個位置可以掃描
掃描的結(jié)果:
SonarLint
SonarLint提供了更廣泛的代碼質(zhì)量和安全性檢查,讓你在編寫代碼的過程中就能發(fā)現(xiàn)和解決問題。相比之下,F(xiàn)indBugs更專注于靜態(tài)代碼分析,幫助開發(fā)人員發(fā)現(xiàn)潛在的bug模式。
下載IDEA插件:
在這個位置可以使用掃描:
能掃描出很多的優(yōu)化建議:
就這些吧,等我把a(bǔ)ustin的代碼都改改,通過了各種插件的掃描,再來推薦下我的股東服務(wù)
