02号张广旭 13号郭佳鑫
张广旭:代码编写
郭佳鑫:窗体、代码检查
1.概要部分
(1)代码能符合需求和规格说明么?
基本功能已经实现了,但是还有一些bug
(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是如何处理的?循环有没有可能出现死循环?
C#代码,它含有三个循环,分别是for、while、foreach循环。 switch case 语句是一个条件选择语句,找到相同的case值做为入口,执行后面的程序;若所有的case都不满足,则找default入口;若未找到则退出整个switch语句。 所以default只是一个备用的入口,有没有都无所谓。没有出现死循环
(4)有没有使用断言(Assert)来保证我们认为不变的条件真的满足?
没有使用
(5)对资源的利用,是在哪里申请,在哪里释放的?有没有可能导致资源泄露(内存、文件、各种GUI资源、数据库访问的连接,等等)?有没有可能优化?
在使用的时候就申请,不用的时候释放。 因为代码中还有一些小bug,可能会导致资源泄露。还有优化的可能。
(6)数据结构中是否有无用的元素?
没有无用的元素
5.效能
(1)代码的效能(Performance)如何?最坏的情况是怎样的?
代码基本上可以运行,窗体的抽签功能有时运行不正常
(2)代码中,特别是循环中是否有明显可优化的部分(C++中反复创建类,C#中 string 的操作是否能用StringBuilder 来优化)?
没有。
(3)对于系统和网络调用是否会超时?如何处理?
没有
6.可读性
代码可读性如何?有没有足够的注释?
代码里面有挺多的注释,通俗易懂,在维护时比较容易看懂
7.可测试性
代码是否需要更新或创建新的单元测试?
需要完善,里面的功能过于简单,还可以尝试增加一些新的功能