起因
作为程序员，保证线上质量的重要一环是代码review，或者说代码review的质量。最近看到多起其他组因为代码review没有做好，导致很多能避免的线上事故没有避免。我思考了一下目前小组的review方式，决定做一些改变。
关注点
- 保证代码得到高质量的review
- reviewer能快速理解代码逻辑
- 不要因为代码review给项目组带来很多的任务压力
改变
目前组里的review方案为项目代码必须由一个老手来review，因为大家对相关领域比较熟悉，这个方案还可行。为了更好的提高质量，更改如下：
- 非P0级项目，需要一个老手做代码review
- P0级项目
- 至少需要两个同学review代码，一个老手，一个是熟悉相关模块的同学
- 项目测试中期让同学开始review
- 编写人需要讲述写代码的思路，改动地方，时长15分钟左右
- 提供文档可以查看项目和代码内容，包括上线配置、上线计划等
这么做有如下几个好处：
- 对于P0级项目，两个熟悉的同学review，总体能保证review质量
- 其他同学做的事情，至少有两个同学熟悉
- 可以整理出每个人熟悉的模块，有的范围广，有的范围窄，可以从这个尺度判断同学的广度与深度
- 代码review带来的工作量不会太大
延伸
- 自动化测试能在更大程度上保证代码质量
- 代码编写者的素质也很重要