Code Review
以前我对code review是抗拒的,原因在于两方面:
- 总感觉有点脱光了让人看的感觉,没脸。
- 觉得是浪费时间,研发周期时间都不够还做什么code review。
不过后来发现之前的自己是多么愚蠢
Why
倒逼团队成员写出更有质量的代码
(基于编码规范等),让代码可以更好的组织起来,有更易读,有更高的维护性,同时可以达到知识共享,找到bug只是其中的副产品
确认自己的设计和实现是一个清楚和简单、正确的
并且通过review找到问题代码,减少错误和暗坑
让更多的人了解你所写的模块,并能促进相互学习对方的长处和优点。
提前暴露影响性能和安全的问题
When
- 自动化测试之后,提测之前
- 前后端联调之后(如果自动化测试还没实现)
What
评判标准
- 编码规范(比如后端《阿里编程规范》,前端ESLint)
- 需求覆盖度是否100%
- 测试用例覆盖度至少满足80/20原则
How
- 提升团队意识,让大家知道code review的好处和重要性,切忌别把时间安排与code review拿在一起说,它们是两回事,时间够不够条件够不够跟code review好不好无关。
- code review落实到产线的专项目标,指定责任人。
- 组件core view 团队,定义code view的关注点,持续改进。