如何完成一次高质量的Code Review?

 

Emac

2016/5

 

code review是一次白盒测试,也是一场思维体操。

There are only two hard things in computer science: cache invalidation and naming things.
-- Martin Fowler

基本原则

  • Don't make me think!
  • 优先使用英文,其次使用拼音
  • 不要出现拼写错误

OR

每件事情都应该尽可能的简单,如果不能更简单的话。

 

-- 爱因斯坦    

基本原则

  • DRY
  • KISS
  • YAGNI
  • SRP

Check list

  • 条件分支(业务逻辑)
  • 循环处理(应用性能)
  • 模块划分
  • 模式抽取

Bad smell

  • 命名不合理
  • 格式不规范
  • 存放位置不当
  • 魔法数字
  • “巨无霸”方法/类
  • 临时代码
  • 没有注释
  • 大段注释代码

Last but not least

童子军规则

Made with Slides.com