在codereview应有等态度

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

一流代码的特性

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

关于bad code的简单判断

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

code review的注意事项

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

review质量

  • 自身代码能力强