融入团队代码风格,代码越写越烂!

共 20584字,需浏览 42分钟

 ·

2024-05-09 09:19

往期热门文章:


    

1、4 个 GitHub 上令人惊艳的项目!
2、90 后程序员辞职搞灰产,不到一年获利超 700 万,结局很刑!
3、Redis“叛逃”开源,得罪了几乎所有人
4、潜伏开源项目3年,混成核心开发者,竟然只为加一个后门
5、就因为开发环境用了 Docker,就被开除了

最近是真的有点忙啊,需求和bug都排到下一个月了,难得今天休息有空上来扯扯,主题是有关代码质量问题的。
在我初学的时候,我是挺喜欢专研各种Java语法和性能相关的问题的。举个很明显的例子,我们都知道Java的单例模式有好几种写法,什么饿汉式、简单懒汉式(在方法声明时加锁)、DCL双重检验加锁(进阶懒汉式)、静态内部类(优雅懒汉式)、枚举。
这貌似算是Java的高频面试题了,所以这几种写法的优缺点几乎都已经被说烂了。
1、所谓「饿汉式」指的就是还没被用到,就直接初始化了对象。(浪费内存) 2、所谓「懒汉式」指的就是等用到的时候,才进行初始化。(不浪费内存)
在刚学明白的时候,我一想:“我这以后肯定是用懒汉式啊,对内存多友好。我不用简单懒汉式(在方法声明时加锁),这会影响到性能。我可以用双重检验加锁(进阶懒汉式)或者静态内部类(优雅懒汉式)”。
后来工作了以后,发现工作代码里什么牛鬼邪神都有,逐渐发现:代码也不是不能跑,编译器远比我们想象得更加强
于是,我用个锤子的懒汉式,我反手就是new出一个单例,代码多简单优雅。我堂堂一个Java开发工程师,我还缺那点小内存?真可笑。
刚进团队的时候还想重构脏代码,后来随着需求的累积,又有编译器给兜底,也没人code review监管,写代码就越来越随意了,融入团队代码风格就完事了。
后来出了点小故障,影响到线上的正常业务了,最后发现是脏代码导致的,被老板吐槽了。让我去找找有哪些工具可以排查下有没有类似的问题(代码质量检查),我一想,项目这里面的代码的坑也太多了,不好搞啊。
工作还是得继续干的,既然是代码质量检查,找些IDEA现成的插件就OK咯。

Alibaba Java Coding Guidelines

集成阿里巴巴的代码规范和最佳实践,帮助开发者保持代码质量和一致性,austin项目也一直用这个插件做代码的扫描。

CheckStyle

以下是 CheckStyle 插件的一些常见功能和用途:
  • 代码规范检查:CheckStyle 插件会分析项目中的源代码,并根据预定义的规则检查代码,包括缩进、命名约定、代码注释、代码结构等。它可以帮助团队确保代码风格一致,符合公司或项目的规范。

  • 自定义规则:您可以自定义 CheckStyle 规则,以便根据您的项目需求和编码标准添加自定义规则。这使得 CheckStyle 插件非常灵活,可以适应各种项目。

  • 代码质量改进:通过检查代码中的潜在问题和不良实践,CheckStyle 插件可以帮助开发者识别和修复代码中的问题,从而提高代码质量。

比如,我设置了一个规则文件:
   

<?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">

    <!-- 检查文件是否以一个空行结束 -->
    <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"/>
            <!-- 实例;import java.util.*;.-->
            <property name="allowClassImports" value="false"/>
            <!-- 实例 ;import static org.junit.Assert.*;-->
            <property name="allowStaticMemberImports" value="true"/>
        </module>
        <!-- 检查是否导入了多余的包 -->
        <module name="RedundantImport"/>
        <!-- 没用的import检查,比如:1.没有被用到2.重复的3.import java.lang的4.import 与该类在同一个package的 -->
        <module name="UnusedImports"/>

        <!-- 注释检查 -->
        <!-- 检查构造函数的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>
        <!--校验方法的注释-->
        <module name="MissingJavadocMethod">
            <!--允许get set 方法没有注释-->
            <property name="allowMissingPropertyJavadoc" value="true"/>
            <!--允许构造方法没有注释-->
            <!--            <property name="" value="true"/>-->
            <property name="scope" value="private"/>
        </module>

        <!-- 命名检查 -->
        <!-- 局部的final变量,包括catch中的参数的检查 -->
        <module name="LocalFinalVariableName"/>
        <!-- 局部的非final型的变量,包括catch中的参数的检查 -->
        <module name="LocalVariableName"/>
        <!-- 包名的检查(只允许小写字母),默认^[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名检查,默认^[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"/>
        <!-- 方法的参数名 -->
        <module name="ParameterName "/>
        <!-- 常量名的检查(只允许大写),默认^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$ -->
        <module name="ConstantName"/>

        <!-- 定义检查 -->
        <!-- 检查数组类型定义的样式 -->
        <module name="ArrayTypeStyle"/>
        <!-- 检查long型定义是否有大写的“L” -->
        <module name="UpperEll"/>


        <!-- 方法不超过100行 -->
        <module name="MethodLength">
            <property name="tokens" value="METHOD_DEF"/>
            <property name="max" value="100"/>
        </module>
        <!-- 方法的参数个数不超过5个。 并且不对构造方法进行检查-->
        <module name="ParameterNumber">
            <property name="max" value="5"/>
            <property name="ignoreOverriddenMethods" value="true"/>
            <property name="tokens" value="METHOD_DEF"/>
        </module>

        <!-- 空格检查-->
        <!-- 方法名后跟左圆括号"(" -->
        <module name="MethodParamPad"/>
        <!-- 在类型转换时,不允许左圆括号右边有空格,也不允许与右圆括号左边有空格 -->
        <module name="TypecastParenPad"/>
        <!-- 检查在某个特定关键字之后应保留空格 -->
        <module name="NoWhitespaceAfter"/>
        <!-- 检查在某个特定关键字之前应保留空格 -->
        <module name="NoWhitespaceBefore"/>
        <!-- 操作符换行策略检查 -->
        <module name="OperatorWrap"/>
        <!-- 圆括号空白 -->
        <module name="ParenPad"/>
        <!-- 检查分隔符是否在空白之后 -->
        <module name="WhitespaceAfter"/>
        <!-- 检查分隔符周围是否有空白 -->
        <module name="WhitespaceAround"/>

        <!-- 修饰符检查 -->
        <!-- 检查修饰符的顺序是否遵照java语言规范,默认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"/>
        <!-- 检查子表达式中是否有赋值操作 -->
        <module name="InnerAssignment"/>
        <!-- 检查是否有"魔术"数字 -->
        <module name="MagicNumber">
            <property name="ignoreNumbers" value="0, 1"/>
            <property name="ignoreAnnotation" value="true"/>
        </module>
        <!-- 检查switch语句是否有default -->
        <module name="MissingSwitchDefault"/>
        <!-- 检查是否有过度复杂的布尔表达式 -->
        <module name="SimplifyBooleanExpression"/>
        <!-- 检查是否有过于复杂的布尔返回代码段 -->
        <module name="SimplifyBooleanReturn"/>

        <!-- 类设计检查 -->
        <!-- 检查类是否为扩展设计l -->
        <!-- 检查只有private构造函数的类是否声明为final -->
        <module name="FinalClass"/>
        <!-- 语法 -->
        <!-- String的比较不能用!= 和 == -->
        <module name="StringLiteralEquality"/>
        <!-- 限制for循环最多嵌套2层 -->
        <module name="NestedForDepth">
            <property name="max" value="2"/>
        </module>
        <!-- if最多嵌套3层 -->
        <module name="NestedIfDepth">
            <property name="max" value="3"/>
        </module>
        <!-- 检查未被注释的main方法,排除以Application结尾命名的类 -->
        <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个数 3个-->
        <module name="ReturnCount">
            <property name="max" value="3"/>
        </module>
        <!--try catch 异常处理数量 3-->
        <module name="NestedTryDepth ">
            <property name="max" value="3"/>
        </module>

    </module>
</module>

当我写出几层for循环的时候和常量的时候,就会被扫描和监控到。

FindBugs

以下是Bug IDEA插件的一些常见用途和功能:
  • 静态代码分析:Bug IDEA插件可以执行静态代码分析,以检测潜在的编码错误、代码风格问题和性能问题。它可以识别诸如未使用的变量、空指针异常、不一致的缩进、未处理的异常等问题。
  • 代码质量评估:该插件可以生成代码质量报告,帮助开发人员了解代码的质量水平。这些报告通常包括代码复杂性、代码覆盖率、代码重复率等指标。
  • 检查代码规范:Bug IDEA插件可以根据您选择的编码标准或规范检查代码,确保团队的代码风格一致性,并提供建议以改进代码。
  • 自动修复问题:插件通常提供了自动修复选项,可以帮助开发人员快速修复代码中的问题,提高代码的质量和可维护性。
  • 安全漏洞检测:有些Bug IDEA插件还可以检测潜在的安全漏洞,如SQL注入、跨站脚本攻击等,以提高应用程序的安全性。
  • 代码重构建议:插件还可以提供有关代码重构的建议,帮助改进代码的结构和性能。
在这个位置可以扫描
扫描的结果:

SonarLint

SonarLint提供了更广泛的代码质量和安全性检查,让你在编写代码的过程中就能发现和解决问题。相比之下,FindBugs更专注于静态代码分析,帮助开发人员发现潜在的bug模式。
下载IDEA插件:
在这个位置可以使用扫描:

能扫描出很多的优化建议:
就这些吧,等我把austin的代码都改改,通过了各种插件的扫描,再来推荐下我的股东服务
   
往期热门文章:

1有人问我数据库ER图为什么这么好看?
2、我用这10招,能减少了80%的BUG
3、突发!特斯拉大量毁约应届生offer…
4、警报炸锅了,FastJson 又立功了。。
5、一个小公司的技术开发心酸事(已倒闭)
6、JetBrains 如何看待自己的软件在中国被频繁破解?
7、程序员因Bug被要求归还4万多年终奖,网友:不还!
8、一套万能通用的异步处理方案
9、微服务全做错了!谷歌提出新方法,成本直接降9倍!
10、IntelliJ IDEA 2024 首个大版本发布,好用到爆!


浏览 281
点赞
评论
收藏
分享

手机扫一扫分享

分享
举报
评论
图片
表情
推荐
点赞
评论
收藏
分享

手机扫一扫分享

分享
举报