Robinson的博客

Facebook产品经理王准的一些建议：作为审查者，一定要读懂diff；所有被接受的diff必须是在读懂的前提下。做审查者的人要有“将来如果这些代码线上出问题，我要帮助支持”的心理准备。代码审查应该被每个工程师当做工作的重要一部分。应当在24小时内给回复，这应当变成共识。感觉有问题的代码，一定要在相应的行上做出评论 （inline comments)，以让作者明白问题所在。尽可能把对修

版权所有，欢迎转载，但请注明作者，译者和出处。如翻译欠妥，欢迎不吝指正。欢迎关注微博：风再起时我就走英文原文链接：点击打开链接 在我的职业生涯中，碰到过各种各样的代码所有制模式。我把它们归纳为三大类：1、         强代码所有制：把程序分解组织为模块（类，函数，文件），然后为每一个模块指定一个程序员。程序员只能修改归属于自己的模块代码。如果他们需要修改其他模块

为什么执行代码评审1.      代码评审的最大好处是纯社会性的，当你知道你的每一行代码都有另外一个人看，自然而然会更加卖力的表现，拿出最好的状态编码，因为每个人都有虚荣心嘛；2.      代码评审可以及时发现一些容易发现的BUG，而不必将发现BUG的时间点推迟到测试阶段；3.      代码评审可以保证至少有两个人都理解任何一份代码。当出现员工休假，离职等情况的时候，至少保证团队的