代码评审123

一.代码评审的几种角色：

开发人员：编写某一模块程序的人员。

测试人员：测试任务的测试人员。

评审人员：参与代码评审的人员。

评审人员由相关模块比较强的同事担任。

二.主要评审的内容

1.       编码规范问题：注释不规范，命名不规范，打印语句 System.out…… 等。

2.       代码结构问题：重复代码、巨大的方法和类、分层不当、紧耦合、错误验证、异常处理、实现过于复杂、代码可读性不佳、扩展性不好




三.两种方案



方案一

一．方式：

会议，由开发人员，评审人员（评审相关模块比较强的同事担任）共同参加。



二．流程：

1.       确定评审模块

2.       由开发人员自己介绍此模块的需求。

3.       然后一步一步的介绍自己的开发步骤。

4.       评审人员根据开发人员的开发步骤，评审代码。

5.       根据陪审标准，然后列出评审问题清单。

6.       开发人员根据这个清单去评审修改。

7.       评审人员检查清单的修改情况。





方案二（并不是单纯的代码评审了，还包括测试，但是目的是一样的，使我们的系统健壮，强大）：

一．方式：

会议由开发人员，评审人员和测试人员共同参加。

二．流程

1.确定评审模块

2.由开发人员自己介绍此模块的需求。

3.测试人员和评审人员根据已完成的情况，提出存在的bug以及一些不合理的地方。列出bug清单。测试人员监督bug的修改情况。重点是功能、逻辑的测试。

4.然后由开发人员根据自己写的代码一步一步的介绍自己的开发步骤。

5.评审人员根据开发人员的代码介绍，评审代码。

6.根据陪审标准，发现代码的问题，然后列出评审问题清单。

7.开发人员根据这个清单去评审修改。

8.评审人员检查清单的修改情况。