本文介绍了 IBM Rational Team Concert(RTC)的代码评审功能(Code Review)。这一功能可以使代码评审流程变得更加规范,完善代码提交流程;对于不同区域的成员可以更高效的">协同工作,在代码提交前发现到潜在的问题,尽快修复,提高代码质量,有效减少缺陷数。
代码评审的重要性
多数情况下,评审者能够发现开发人员自身不能发现的问题或潜在问题,毕竟思维方式和考虑的点等会有差异
优秀的评审者通过评审,不仅可以发现开发人员的不足,更可以通过开发人员的代码学到很多知识 对团队而言,通过评审以及互相评审,了解到其他团队成员负责的任务,必要时互相帮忙,互为后援,提高项目开发效率,降低项目风险
代码评审的规则
从逻辑上讲,本次修改是否符合需求,是否有类似的地方需要修改 可读性上,是否有足够的解释说明,以便以后维护 就代码规范而言,是否符合公司代码规范,以保持代码风格一致性 从代码性能上讲,是否有更有效的方案可以实现 单元测试(如果必要),测试用例是否能覆盖本次的修改
RTC 对代码评审的支持
作为目前主流的代码管理工具,RTC 对代码评审的功能已经有了很好的支持,比如利用邮件作为代码提交者与评审者通信的工具,代码评审过程中各个角色的划分,代码提交的权限设置等等,本节将具体介绍 RTC 在代码评审过程中所涉及的概念及详细配置。
对邮件的支持
在整个代码评审过程中,邮件是作为代码提交者及其他相关人员之间的重要通信工具,比如在代码评审前提交、对代码添加修改意见,团队相关人员都应收到相应的邮件提醒,用户可以通过如下配置是否启用邮件支持:
打开
Repository
Connections 界面并连接
图 1. Repository 连接界面
打开用户编辑窗口,如图 2 所示:
图 2. Open My User Editor
找到 Mail Configuration 页面,进行相应的配置并保存,如图 3 所示:
图 3. 邮件配置界面
代码评审中所涉及的角色的划分