首页 » Cisco » 正文

About Code Review

通过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
  1. 增加对相互模块的理解
  2. 人员备份
小组Review 组内对这一块了解的人 关注设计和结构 Every month/week Project meeting 减少设计,技术问题
组外飞检 组外人员,任意抽查一段代码 只关心“代码规范” Milestone Review人会出具Repor,这个是否通过 作为milestone的一个标志 保障制定的“代码规范”执行力度
架构Review 各小组负责人参加,架构团队讲解 关注业务逻辑 和 整体设计
  1. 代码开发前
  2. 每次重大架构变动
  3. Meeting
  4. Mail list discussion
降低架构设计错误,带来的风险