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