请注意,我要开始吐槽了

307 阅读3分钟

说在前面的话

在现在的这家公司快呆了2年了,一直都在维护老项目,这个项目08年就已经开始动工了,经手的人怕也有十几个了吧!从来没有过代码审查,基本上就是完成任务就OK了,所以存在了很多问题,大多数代码都是很不规范的,遇到的问题特来此跟大家分享下。看看大家有没有遇到跟我类似的情况

命名风格迥异,也太随意了吧,懂的都懂

image.png

一个类一万七千行代码

每次打开这个类都会卡上好几秒钟,好歹我的笔记本可是7000块啊,瞧不起我呀

image.png

image.png

随处可见的e.printStackTrace(),高达600多处

每次线上出了问题,都是莫名其妙,异常日志都被吃了

image.png

如果报错了,问题大概率是NullPointerException

经常性的Integer直接转int

String变量或集合没有非空判断,拿来就用,这么着急吗

一个方法超过2000行,你敢信

一个方法动辄上百行,随处可见,数千行的方法也有,但是现在没有找到,之前反正被吓到了

image.png

image.png

深度循环,随处可见

image.png

冷不丁的冒出来一大群魔数,谁知道这啥意思嘛

image.png

拒绝重构,从我做起

哪里有类似的代码,我直接copy过来,快速解决战斗真好,反正大家都这么干,嘿嘿

image.png

一个controller解决一切的问题

我们的项目没有service层和dao层,来了一个请求,一个controller解决问题,就问你服不服

代码零注释,这是非你不可了是吧

经常要你去问写这代码的人,这啥意思啊?虽然给你解释了,但是还是不懂。害。坚持下来真难!

我如果说这是从1开始到200结束的你敢信吗?

项目里面有很多地方有类似的逻辑,就是直接COPY,你拿我怎样

image.png

一个switch有400行

image.png

总结

代码风格迥异是一种常见的情况,它可能会导致代码可读性降低、维护困难,甚至引发一些难以调试的问题。要解决这个问题,可以采取以下措施:

  1. 制定并遵守统一的代码规范:团队或项目可以制定一套统一的代码规范,包括命名规则、缩进风格、注释方式等,要求所有成员遵守。这样能够保证代码风格的一致性,提高代码的可读性和可维护性。
  2. 使用集成开发环境(IDE)或代码编辑器的自动格式化功能:许多IDE和代码编辑器都提供了自动格式化代码的功能,可以根据预设的规范自动调整代码风格。这样能够减少手动调整格式的工作量,同时保证代码风格的一致性。
  3. 审查代码:在代码提交之前,进行代码审查可以发现风格不一致的问题。审查人员可以根据统一的规范提出修改建议,促使代码风格一致。
  4. 培训和指导:对于新成员或经验不足的成员,可以进行代码规范的培训和指导,让他们了解并遵守团队或项目的代码规范。
  5. 使用版本控制系统:版本控制系统可以帮助记录代码的修改历史,方便追踪问题并进行回溯。同时,通过比较不同版本的代码,可以更容易地发现风格不一致的问题。

通过以上措施,可以有效减少代码风格迥异的问题,提高代码质量和可维护性。可以参考# 《阿里巴巴Java开发手册(纪念版)》.pdf 阅读笔记