title: Code Review
author: Gamehu
tags:
  - 实践
categories:
  - 工作
date: 2018-08-01 09:47:00
---

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

1. 总感觉有点脱光了让人看的感觉，没脸。
2. 觉得是浪费时间，研发周期时间都不够还做什么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的关注点，持续改进。