如何做一个好的codereview

在codereview应有等态度

• 对所有review代码逻辑都能看懂
• 好代码的标准，不仅仅是可以运行通过
  • 正确性，可读性，可维护性，可重用性，可运维性
• codereview和写代码一样重要

一流代码的特性

• 高效
• 鲁棒
• 简洁
• 共享
• 可测试性
• 可移植性
• 可监控性
• 可运维性
• 可扩展性

关于bad code的简单判断

• 5分钟内不能看懂的代码
• 需要思考才能看懂的代码
• 需要来回翻屏才能看懂的代码
• 没有空行/注释的代码

code review的注意事项

• 必要时，面对面沟通
• 关键模块，建立owner机制
• 对review中发现的问题，一追到底
• 一行代码都不能放过

review质量

• 自身代码能力强