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

架构的小事2024-05-09 11:14:22  53

最近是真的有点忙啊,需求和bug都排到下一个月了,难得今天休息有空上来扯扯,主题是有关代码质量问题的。

在我初学的时候,我是挺喜欢专研各种Java语法和性能相关的问题的。举个很明显的例子,我们都知道Java的单例模式有好几种写法,什么饿汉式、简单懒汉式(在方法声明时加锁)、DCL双重检验加锁(进阶懒汉式)、静态内部类(优雅懒汉式)、枚举。

这貌似算是Java的高频面试题了,所以这几种写法的优缺点几乎都已经被说烂了。

1、所谓「饿汉式」指的就是还没被用到,就直接初始化了对象。(浪费内存) 2、所谓「懒汉式」指的就是等用到的时候,才进行初始化。(不浪费内存)

在刚学明白的时候,我一想:“我这以后肯定是用懒汉式啊,对内存多友好。我不用简单懒汉式(在方法声明时加锁),这会影响到性能。我可以用双重检验加锁(进阶懒汉式)或者静态内部类(优雅懒汉式)”。

后来工作了以后,发现工作代码里什么牛鬼邪神都有,逐渐发现:代码也不是不能跑,编译器远比我们想象得更加强

于是,我用个锤子的懒汉式,我反手就是new出一个单例,代码多简单优雅。我堂堂一个Java开发工程师,我还缺那点小内存?真可笑。

刚进团队的时候还想重构脏代码,后来随着需求的累积,又有编译器给兜底,也没人code review监管,写代码就越来越随意了,融入团队代码风格就完事了。

后来出了点小故障,影响到线上的正常业务了,最后发现是脏代码导致的,被老板吐槽了。让我去找找有哪些工具可以排查下有没有类似的问题(代码质量检查),我一想,项目这里面的代码的坑也太多了,不好搞啊。

工作还是得继续干的,既然是代码质量检查,找些IDEA现成的插件就OK咯。

Alibaba Java Coding Guidelines

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

CheckStyle

以下是 CheckStyle 插件的一些常见功能和用途:

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

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

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

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

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

FindBugs

以下是Bug IDEA插件的一些常见用途和功能:

静态代码分析:Bug IDEA插件可以执行静态代码分析,以检测潜在的编码错误、代码风格问题和性能问题。它可以识别诸如未使用的变量、空指针异常、不一致的缩进、未处理的异常等问题。

代码质量评估:该插件可以生成代码质量报告,帮助开发人员了解代码的质量水平。这些报告通常包括代码复杂性、代码覆盖率、代码重复率等指标。

检查代码规范:Bug IDEA插件可以根据您选择的编码标准或规范检查代码,确保团队的代码风格一致性,并提供建议以改进代码。

自动修复问题:插件通常提供了自动修复选项,可以帮助开发人员快速修复代码中的问题,提高代码的质量和可维护性。

安全漏洞检测:有些Bug IDEA插件还可以检测潜在的安全漏洞,如SQL注入、跨站脚本攻击等,以提高应用程序的安全性。

代码重构建议:插件还可以提供有关代码重构的建议,帮助改进代码的结构和性能。

在这个位置可以扫描

扫描的结果:

SonarLint

SonarLint提供了更广泛的代码质量和安全性检查,让你在编写代码的过程中就能发现和解决问题。相比之下,FindBugs更专注于静态代码分析,帮助开发人员发现潜在的bug模式。

下载IDEA插件:

在这个位置可以使用扫描:

能扫描出很多的优化建议:

就这些吧,等我把austin的代码都改改,通过了各种插件的扫描,再来推荐下我的股东服务

转载此文是出于传递更多信息目的。若来源标注错误或侵犯了您的合法权益,请与本站联系,我们将及时更正、删除、谢谢。
https://www.414w.com/read/461182.html
0
最新回复(0)