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

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

最近是真的有点忙啊,需求和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
随机主题
吴绮莉曾称家门永远向女儿敞开, 晒切蒜视频遭嘲, 被问怎么沦落了墨西哥一场总统竞选活动发生舞台倒塌事故 数十人死伤央视《法治在线》主播经蓓工作着装生活化, 背后原因用心良苦陈仓之战背景板, 传说中的无敌神刀王双【三国志战略版】万吨大驱率编队赴南海训练!它的兵力构成和课目设置有何讲究?1928年,张作霖被炸身亡,却意外拉进了张学良与蒋介石的关系丰田海狮即将上市布局5G导致千亿资金打水漂? 外籍院士揭露真实的状态!助力乡村共同富裕! 第十六届浙江一市·宁海白枇杷文化活动举行河南发布铁拳行动典型案例, 一燃气公司因充装不合规气瓶被罚锂电池产品在国际物流中的注意事项首趟豫鲁“郑日韩”铁海快线班列启程, 助力更多河南制造出海湖人引援新动向: 天时地利人和齐聚, 2年6950万锁定17+11内线巨塔什么值得买618年中大促开启, 首4小时GMV同比增长214%火箭越老越粗暴,希金斯防守瞬间变成马蜂窝,魔幻操控教科书清台米体: 两位独立董事将辞职, 马洛塔&安东内洛有望留在国米董事会数据: 4个希腊人当中就有1个老人卫报: 拜仁已经就孔帕尼补偿金问题与伯恩利展开谈判姚振华及宝能等新增2则被执行人信息, 执行标的合计22.88亿五款好玩的动作角色扮演单机游戏哪些下载量高森林北深夜发文澄清恋情, 汪峰离婚背后真相究竟如何?
最新回复(0)