review 前准备

• 代码开发人员做好自review和自测试，杜绝代码中的低级错误
• 参加review的人员需要对代码业务有一定的了解，代码开发人员提前做好方案详细设计串讲
• 完成部分代码即可发起review，小步快跑，减少每次review代码量

review什么

• 专项review：参考《C/C++编程规范》、《C/C++安全编程规范》，这部分在开发人员自review时就应该做好
• 功能review：代码实现是否满足需求
• 可读性：圈复杂度、变量名命名合理、目录划分、函数代码行数、无冗余代码等等
• 可维护性/可扩展：数据结构/接口定义合理、兼容其他平台、提供测试接口、提供相关关键日志
• 性能review：算法实现是否最优，是否有执行效率低的代码等

写在最后

​ 对于代码要做到谁开发，谁保护；谁污染，谁治理。对产品代码划分责任田，责任田主对代码的质量长期负责。之前和若干合作方合作过，合作方交付的代码质量堪忧，我认为其中一条就是合作方不对代码质量长期负责，合作方交付代码之后就没人管了。第一版代码的开发人员对代码的日后演进有至关重要的作用，如果第一版质量就很差，以后代码问题会像滚雪球一样问题越来越严重。一个Cisco来的专家讲过他们review代码方式：每个源码文件都有相关的review人员，这些人员都是当初开发这段代码的人员或者维护过这段代码的人员，无论他们目前在公司做什么工作，一旦有人改了这段代码，这些人都会收到相关的邮件review通知。他就曾经半夜收到某大领导的电话，这个人多年前曾维护过这段代码，给他提了好多有效的review意见。

​ 最后的最后，代码的质量决定的根本因素还是开发代码人员，一个程序员的基本素养是啥？自己埋下的坑自己填，自己的BUG自己能解。