如何完成一次高质量的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

童子军规则