Code Review 是一场苦涩但有意思的修行
猿人谷
共 4338字,需浏览 9分钟
·
2020-12-08 10:54
Photo @ Wyron A
文 | 孤尽
前言
业务跑得这么快,没时间 Code Review
代码是讲道理的
每一行代码的存在是有意义的
更加严格地说,每一个字符的存在都应该是有意义的。如果某行代码的存在完全是可有可无的,这个时候,我们考虑过 JVM 的感受吗?凭白无故地要编译这些字节码,然后栈进栈出的忙活一阵子,然后告诉它,你的劳动是没有任何价值的。比如,Boolean assetFlag = Boolean.true ; 这里都已经明确地给给出来显示的初始值,可是在调用端,居然还有这样的判断:if ( assetFlag != null && assetFlag == true) {...},什么情况下为 null 值啊?另外参数在框架里已经做了值的判断,那么下边又是 n 行,对所有参数重新判断一遍,是对我们的代码有多少不自信,还是对框架不自信?每一行的代码,相当于生命,它的存在一定是有意义的,一定是能够被执行到并且能够为实际的业务负责的。
我们比拼的不是代码行数
用户视角的成功与失败
有重复使用的量一定要找个地方集中隔离
单测没必要代码 Code Review
需要调试来查找错误时,往往是一种对异常处理机制的侮辱
多个 return 的语句,概率高的一定先进行判定
吝啬空行
命名太随意
Photo @ Jon Tyson
注释是电影的旁白
满天飞的函数式编程好吗?
孤尽说:“亲力亲为写代码,写稳定和优质的代码,才能激发强大生产力”。我们希望通过“文化月”的形式,传播技术文化,向代码致敬。
很多公司都因为各方面的原因忽视详设和CR,拿到需求就写,遇到bug就干,架构设计不存在的,CR压根没有的事儿。
CR是不仅能让大家一起进行思维碰撞,共同进步,也能进一步降低代码的风险。简单点说,CR就是对设计和实现的再次确认,在反复较量的过程中,相互学习和成长。
但有一个前提非常重要,就是参与CR的人都需要心胸宽广,不然CR容易变成互怼。
评论