一、结对，找到一个伙伴进行结对
小伙伴：曾景钊 ，博客链接：http://www.cnblogs.com/zengjingzhao/p/8594824.html
二、代码复审核查表
|
代码复审核查表 |
||
|
概要部分 |
代码符合需求和规格说明么？ |
比较符合 |
|
代码设计是否考虑周全？ |
无 |
|
|
代码可读性如何？ |
可读性一般 |
|
|
有冗余的或重复的代码吗? |
无 |
|
|
代码的每一行都执行并检查过了吗？ |
是 |
|
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式？ |
不遵循，只在主函数里进行一个测试 |
|
有没有硬编码或字符串/数字等存在？ |
有 |
|
|
代码有没有依赖于某一平台？ |
没有 |
|
|
有没有无用的代码可以清除？ |
有 |
|
|
代码规范部分 |
修改的部分符合代码标准和风格么？ |
基本符合 |
|
具体代码部分 |
数据结构中有没有用不到的元素？ |
无 |
|
对于调用的外部函数，是否检查了返回值 |
无，没有外部函数 |
|
|
效能 |
代码的效能如何？ |
一般 |
|
循环中是否有明显可优化的部分？ |
是 |
|
|
可读性 |
代码可读性如何？有没有足够的注释？ |
可读性一般，没有注释 |
|
可测试性 |
是否需要更新或创建新的单元测试 |
是 |
|
代码复审感想：小伙伴的代码中，编写规范并不是很好，而且实现方法处并没有注释；对于身份证逻辑性的判断只是进行简单的判断是不是合格的身份证，然而，对于地域、年份、男女生等的判断并没有进行到测试，所以，代码还可以进行更仔细的修改。 |
||