任务说明:
一结对, 找到一个伙伴进行结对; 林坤煌 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 代码复审感想
看别人的代码, 最大的感受就是很难看懂, 因为注释量很少而且整个代码当中只有一个主函数没有其他的函数, 对于阅读代码并且理解代码就会十分的缓慢了,
来源: http://www.bubuko.com/infodetail-2535012.html