关于代码review的思考

起因

作为程序员，保证线上质量的重要一环是代码review，或者说代码review的质量。最近看到多起其他组因为代码review没有做好，导致很多能避免的线上事故没有避免。我思考了一下目前小组的review方式，决定做一些改变。

关注点

1. 保证代码得到高质量的review
2. reviewer能快速理解代码逻辑
3. 不要因为代码review给项目组带来很多的任务压力

改变

目前组里的review方案为项目代码必须由一个老手来review，因为大家对相关领域比较熟悉，这个方案还可行。为了更好的提高质量，更改如下：

• 非P0级项目，需要一个老手做代码review
• P0级项目
  • 至少需要两个同学review代码，一个老手，一个是熟悉相关模块的同学
  • 项目测试中期让同学开始review
  • 编写人需要讲述写代码的思路，改动地方，时长15分钟左右
  • 提供文档可以查看项目和代码内容，包括上线配置、上线计划等

这么做有如下几个好处：

1. 对于P0级项目，两个熟悉的同学review，总体能保证review质量
2. 其他同学做的事情，至少有两个同学熟悉
3. 可以整理出每个人熟悉的模块，有的范围广，有的范围窄，可以从这个尺度判断同学的广度与深度
4. 代码review带来的工作量不会太大

延伸

1. 自动化测试能在更大程度上保证代码质量
2. 代码编写者的素质也很重要

最后

大家如果喜欢我的文章，可以关注我的公众号（程序员麻辣烫）

mp.weixin.qq.com/s/M0ExjoIGH…