代码复审和两人合做

代码复审的步骤:程序员

  • 代码必须成功地编译,在全部要求的平台上,同时要编译DeBug| Retail版本。编译要用团队规定的最严格的编译警告等级(例如C/C++中的W4)。
  • 程序员必须测试过代码。什么叫测试过?最好的方法是在DeBugger中单步执行。
  • 程序员必须提供新的代码,以及文件差别分析工具。WindiffVSTS自带的工具均可以。VSTS中能够经过Shelveset来支持远程代码复审。
  • 复审者能够选择面对面的复审、独立复审或其余方式。
  • 在面对面的复审中,通常是开发者控制流程,讲述修改的来龙去脉。可是复审者有权在任什么时候候打断叙述,提出本身的意见。
  • 复审者必须把反馈意见逐一提出。注意,复审者有权提出不少看似吹毛求疵的问题,复审者没必要每一件事都要亲自调查,开发者有义务给出详尽的回答。
  • 开发者必须负责让全部的问题都获得满意的解释或解答,或者在TFS中建立新的工做项以确保这些问题未来会获得处理。
  • 对于复审的结果,双方必须达成一致的意见。                                 

复审的目的在于:

1)找出代码的错误。如:a. 编码错误,好比一些能碰巧骗过编译器的错误。算法

b. 不符合项目组的代码规范的地方。工具

2)发现逻辑错误,程序能够编译经过,可是代码的逻辑是错的。测试

3)发现算法错误,好比使用的算法不够优化。优化

4)发现潜在的错误和回归性错误——当前的修改致使之前修复的缺陷又从新出现。编码

5)发现可能改进的地方。spa

6)教育(互相教育)开发人员,传授经验,让更多的成员熟悉项目各部分的代码,同时熟悉和应用领域相关的实际知识。                                 代码规范


两人合做:两人在一块儿合做,天然都有本身的想法,在两人平等合做的状况下,不存在领导与被领导的关系。没有绝对正确或错误的方法,只有合适或不合适的方法。两我的的水平、目标不一致,不可能没有矛盾。任何人都不是完美的,都有能够改进的空间,在不断的改进中,团队之间相互磨合,最终共同完成任务。开发

相关文章
相关标签/搜索