通过Code Review我们不但能发现产品的问题,同样也是一个学习的过程,所谓开卷有益,看Code也一样,不管好的,还是差的。 今天也你看的Code 也许和你没有关系,明天也许你就要用到了,team内部,team之间的轮岗应该是个常态,以后会更加流行。
Code Review 的两种形式, P-2-P 和 Team Meeting。
P2P适合与日常工作中的Bug Fixedreview;
Team Meeting适合与框架,技术,Logic的review, 至于参与者相关人必须参加,感兴趣的人鼓励参加,想睡觉的:)就在位置上吧。 工具可以WebEx+Wiki, WebEx可以做record, Wiki做重点记录。
Name | Who review | Review 方式 | Frequency | 检查方式 | 目的 |
相关人Review | 上下调用依赖关系人Review | 代码精读,从逻辑到设计 | Frequently | 由Review人介绍别人开发的Code |
|
小组Review | 组内对这一块了解的人 | 关注设计和结构 | Every month/week | Project meeting | 减少设计,技术问题 |
组外飞检 | 组外人员,任意抽查一段代码 | 只关心“代码规范” | Milestone | Review人会出具Repor,这个是否通过 作为milestone的一个标志 | 保障制定的“代码规范”执行力度 |
架构Review | 各小组负责人参加,架构团队讲解 | 关注业务逻辑 和 整体设计 |
|
降低架构设计错误,带来的风险 |