用code workshop取代code review

Contents

Box Tech Blog » Effective learning through code workshops介绍了Box如何用code workshop而不是code review的形式来改善代码质量。

code review有2个显著的缺点：

1. reviewer现场看代码会带来浪费（有人快，有人慢，需要同步）。
2. 被review的人容易有反驳心理，影响学习效果。

作者提出的code workshop的方法有以下几个要点：

1. 制定一个审查人，他必须选择不是他自己写的代码来审查。
2. 审查人审查所有的代码，而不是diff（这很重要！！）。这样大家就能学习审查的代码中的全部逻辑。
3. 审查人要提前准备，阅读代码，记笔记。
4. workshop中所有人都关掉笔记本（除了审查人和记录员）。
5. 所有参会人员一起看代码然后提修改意见。
6. 审查人不能提到人名，目的是审查代码，不是人。所有成员都对代码质量负责。
7. 审查人鼓励大家讨论，寻找问题模式，提炼编码规范。

我自己也参加过类似的code workshop，但是我们是所有人都要提前看代码，我觉得这样效果更好。