任务说明：
一、结对，找到一个伙伴进行结对；林坤煌http://www.cnblogs.com/LKH254974541/p/8604791.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审，形成“代码复审检查表”。
1．概要部分
（1）代码能符合需求和规格说明么？
答：可以符合需求与规格
（2）代码设计是否有周全的考虑？
答：不是很周全，修改了几次
（3）代码可读性如何？
答：可以简单读懂
（4）代码容易维护么？
答：代码容易维护
（5）代码的每一行都执行并检查过了吗？
答：检查过了
2．设计规范部分
（1）设计是否遵从已知的设计模式或项目中常用的模式？
答：是
（2）有没有硬编码或字符串/数字等存在？
答：有一部分
（3）代码有没有依赖于某一平台，是否会影响将来的移植（如Win32到Win64）
答：没有依赖，不会影响
（4）开发者新写的代码能否用已有的Library/SDK/Framework中的功能实现？在本项目中是否存在类似的功能可以调用而不用全部重新实现？
答：不能，不存在
（5）有没有无用的代码可以清除？（很多人想保留尽可能多的代码，因为以后可能会用上，这样导致程序文件中有很多注释掉的代码，这些代码都可以删除，因为源代码控制已经保存了原来的老代码。）
答：有一点无用的代码
3．代码规范部分
（1）修改的部分符合代码标准和风格么（详细条文略）?
答：符合代码标准
4．具体代码部分
（1）有没有对错误进行处理？对于调用的外部函数，是否检查了返回值或处理了异常？
答：没有对错误进行处理，没有检查返回值和处理异常
（2）参数传递有无错误，字符串的长度是字节的长度还是字符（可能是单/双字节）的长度，是以0开始计数还是以1开始计数？
答：有错误，字符串的长度是字节的长度，以1开始计数
（3）边界条件是如何处理的？Switch语句的Default是如何处理的？循环有没有可能出现死循环？
答：通过百度查询和书上的知识，或者问同学解决的，循环不会出现死循环
（4）有没有使用断言（Assert）来保证我们认为不变的条件真的满足？
答：没有使用
（5）对资源的利用，是在哪里申请，在哪里释放的？有没有可能导致资源泄露（内存、文件、各种GUI资源、数据库访问的连接，等等）？有没有可能优化？
答：资源从内存中申请资源，不会导致资源泄露，有可能。
（6）数据结构中是否有无用的元素？
答：检查过后没有
5．效能
（1）代码的效能（Performance）如何？最坏的情况是怎样的？
答：代码正确，程序运行正常。
（2）代码中，特别是循环中是否有明显可优化的部分（C++中反复创建类，C#中string的操作是否能用StringBuilder来优化）？
答：有可优化部分，但是代码正确就没有修改
（3）对于系统和网络调用是否会超时？如何处理?
答：不会超时，通过在网上查找处理
6．可读性
代码可读性如何？有没有足够的注释？
答：代码不是很复杂，有一点注释
7．可测试性
代码是否需要更新或创建新的单元测试？
还可以有针对特定领域开发（如数据库、网页、多线程等）的核查表。
答：需要
8、代码复审感想
看别人的代码，最大的感受就是很难看懂，因为注释量很少而且整个代码当中只有一个主函数没有其他的函数，对于阅读代码并且理解代码就会十分的缓慢了，
原文地址：https://www.cnblogs.com/362xu/p/8622604.html