阿里孤尽:Code Review 是一场苦涩但有意思的修行
JAVA葵花宝典
共 4157字,需浏览 9分钟
·
2020-07-28 16:01
Photo @ Wyron A
文 | 孤尽
前言
业务跑得这么快,没时间 Code Review
代码是讲道理的
每一行代码的存在是有意义的
更加严格地说,每一个字符的存在都应该是有意义的。如果某行代码的存在完全是可有可无的,这个时候,我们考虑过 JVM 的感受吗?凭白无故地要编译这些字节码,然后栈进栈出的忙活一阵子,然后告诉它,你的劳动是没有任何价值的。比如,Boolean assetFlag = Boolean.true ; 这里都已经明确地给给出来显示的初始值,可是在调用端,居然还有这样的判断:if ( assetFlag != null && assetFlag == true) {...},什么情况下为 null 值啊?另外参数在框架里已经做了值的判断,那么下边又是 n 行,对所有参数重新判断一遍,是对我们的代码有多少不自信,还是对框架不自信?每一行的代码,相当于生命,它的存在一定是有意义的,一定是能够被执行到并且能够为实际的业务负责的。
我们比拼的不是代码行数
用户视角的成功与失败
有重复使用的量一定要找个地方集中隔离
单测没必要代码 Code Review
需要调试来查找错误时,往往是一种对异常处理机制的侮辱
多个 return 的语句,概率高的一定先进行判定
吝啬空行
命名太随意
Photo @ Jon Tyson
注释是电影的旁白
满天飞的函数式编程好吗?
推荐阅读:
RocketMQ实战:生产环境中,autoCreateTopicEnable为什么不能设置为true Java程序跑的快,全要靠线程带 用了这么多年的注解,你知道注解是干嘛的嘛? vim常用命令(命令模式、末行模式) 朕已阅
评论