Code Review

以前我对code review是抗拒的,原因在于两方面:

  1. 总感觉有点脱光了让人看的感觉,没脸。
  2. 觉得是浪费时间,研发周期时间都不够还做什么code review。

不过后来发现之前的自己是多么愚蠢

Why

  • 倒逼团队成员写出更有质量的代码

    (基于编码规范等),让代码可以更好的组织起来,有更易读,有更高的维护性,同时可以达到知识共享,找到bug只是其中的副产品

  • 确认自己的设计和实现是一个清楚和简单、正确的

    并且通过review找到问题代码,减少错误和暗坑

  • 让更多的人了解你所写的模块,并能促进相互学习对方的长处和优点。

  • 提前暴露影响性能和安全的问题

When

  • 自动化测试之后,提测之前
  • 前后端联调之后(如果自动化测试还没实现)

What

评判标准

  • 编码规范(比如后端《阿里编程规范》,前端ESLint)
  • 需求覆盖度是否100%
  • 测试用例覆盖度至少满足80/20原则

How

  • 提升团队意识,让大家知道code review的好处和重要性,切忌别把时间安排与code review拿在一起说,它们是两回事,时间够不够条件够不够跟code review好不好无关。
  • code review落实到产线的专项目标,指定责任人。
  • 组件core view 团队,定义code view的关注点,持续改进。