最近重构老项目连着熬了两个通宵。不是因为代码多难而是越改越憋屈。翻遍整个项目的git提交记录发现绝大部分线上bug压根不是什么高深的算法错误、不是并发锁失效、也不是框架底层坑。全都是一些特别低级特别离谱本该完全避免的人为问题。写代码这几年我一直有个误区。总觉得技术能力就是会多少源码、会调多少参数、能搞定多少高并发场景。直到最近维护祖传代码才彻底想明白一件事职场开发里稳定靠谱比技术牛逼重要太多了。一、很多bug都是“图省事”埋的雷不知道大家有没有这种习惯线上紧急改bug的时候为了快点上线直接在业务代码里硬写参数、临时if判断、临时兜底逻辑。心里想的都是先凑合用等版本迭代空闲了我再来重构优化。但真实的工作现状是临时代码就是永久代码。我这次重构的订单模块就踩了这个巨坑。两年前有个活动限时改价当时开发为了赶进度直接在结算逻辑里写死了一个价格判断没有配置、没有开关、没有注释。活动结束之后没人清理代码一直残留在核心流程里。这次新版结算逻辑上线新旧代码叠加直接导致部分用户订单金额计算错乱线上报了一堆异常测试环境还完全复现不出来。排查问题的时候所有人都在怀疑是新写的折扣算法出问题、怀疑缓存过期时间不对、怀疑数据库事务隔离级别异常。整整排查了八个小时最后才发现是两年前那段临时硬编码代码在搞事。真的当时看着那行代码心态直接崩了。明明一秒钟就能删掉的冗余代码拖了两年最后让整个团队通宵救火复盘整改。很多时候我们口中的技术难题其实都是过去无数次偷懒、省事、侥幸心理堆积出来的。代码里的每一次将就最后都会变成线上的事故狠狠打脸。二、注释真的不是写给自己看的以前我特别反感写注释一直觉得代码即注释逻辑清晰的代码不需要多余说明。而且网上很多教程也说烂代码才需要大量注释好代码自带可读性。工作久了才发现这话纯属误导新人。你写的代码逻辑再清晰也抵不过时间抵不过业务迭代抵不过接手的人不是你。我前段时间接手一个老的定时任务项目定时规则全是直接写在代码里的cron表达式一堆没有任何说明。哪个是跑日报的、哪个是同步第三方数据的、哪个是清理过期日志的完全分不清。有一次运维调整服务器资源问我能不能关掉几个低频定时任务。我对着几十行cron表达式看了一下午根本不敢乱删。你根本不知道哪一个看似没用的定时关联着核心业务数据同步删错就是线上事故。更离谱的是有些代码逻辑写的非常诡异既不符合常规业务逻辑也不符合通用编码规范。刚开始我一直以为是前任开发技术不行写错了逻辑。准备重构删掉的时候翻了旧的工单记录才知道是当年对接的第三方接口有bug必须特殊兼容不然数据对不上。这种历史兼容逻辑如果没有注释后续接手的人百分百会改错。注释从来不是多余的它是团队协作的接力棒。不要觉得自己看懂了就行半年后的你新来的同事接手项目的人全都看不懂你的一时灵感。省下十秒钟写注释的时间未来可能要花十个小时去排查问题。三、过度追求优雅也是一种毛病逛csdn看多了很容易陷入一个误区。总觉得代码必须极致优雅、设计模式拉满、抽象层级做到极致不然就是垃圾代码。我刚入行的时候特别痴迷写高级代码。一个简单的查询逻辑非要套策略模式、工厂模式一个普通的工具方法非要多层抽象封装。看起来逼格拉满代码整洁好看但是实际落地坑巨多。前段时间复盘自己以前写的代码真的想吐槽当初的自己。为了所谓的优雅过度抽象导致后续业务迭代新增一个小小的需求都要改一堆抽象层代码。原本三行代码能搞定的逻辑封装完之后排查问题要穿梭五六个类文件。很多新手程序员包括曾经的我本末倒置了。代码的第一优先级是可读、可维护、好排查不是优雅。业务简单的场景直白的if else远比花里胡哨的设计模式好用。普通的业务迭代简单直接的代码远比过度抽象的框架更稳。优雅是锦上添花稳定才是雪中送碳。不要为了炫技去写超出业务体量的代码纯属给自己和团队挖坑。四、自测敷衍是线上事故的源头现在很多开发的通病包括我自己偶尔也会犯。代码写完本地跑通一遍接口调通参数不报错就直接提测了。异常场景不测、边界条件不判断、空数据、null值、超长参数、并发场景全部默认不会出现。总觉得测试会兜底线上不会有这么奇葩的数据。但线上的数据永远比你想象的离谱。用户的输入、第三方的返回、数据库的历史脏数据总能精准命中你所有的未处理边界。上周线上的一个小bug就是因为没有做空字符串判断。正常用户传的都是正常参数但是有一个老旧商户系统会传空字符进来直接导致接口报错整条业务链路卡住。这种问题技术难度为零完全是自测不严谨导致的。可是造成的影响、排查的时间、修复的成本一点都不小。慢慢工作久了就明白真正靠谱的程序员不是不出bug是尽量不出低级bug。技术短板可以慢慢补编码习惯差真的会毁掉一个人的职场口碑。五、写在最后做开发这几年见过太多人天天啃源码、学新框架、卷新技术却连最基础的编码规范、自测习惯、代码清理、注释规范都做不好。我们总想着进阶、想着高薪、想着进阶架构师。却忽略了编程最核心的基本功从来不是炫技而是严谨。不偷懒、不侥幸、不敷衍、不炫技。临时代码及时清理复杂逻辑必写注释边界场景全部兜底代码简洁够用即可。看似都是不起眼的小事却是区分普通开发和资深开发的核心差距。技术可以慢慢积累习惯一旦养坏了真的很难改。希望所有同行都能少写一点兜底的烂代码少留一点技术债写的每一行代码都经得起线上的考验。