崔则赵佳佳审核报告

1、 概要部分

（1）  代码基本符合需要和规格，但是我们还有好多需要改进的地方。

（2）  设计有经过很长时间的考虑。

（3）  代码可读性还不是很好，还有一些不足。

（4）  由于我们的功能不是很完善，所以维护程度还不是很完善。

（5）  检查并且执行了。

2．设计规范部分

（1）设计遵从已知的设计模式或项目中常用的模式。

（2）有硬编码或字符串/数字等存在。

（3）代码没有依赖于某一平台，不会影响将来的移植。

（4）开发者新写的代码能否用已有的Library/SDK/Framework中的功能实现。在本项目中是否存在类似的功能可以调用而不用全部重新实现。

（5）没有无用代码清除，我们已经进行了一次又一次的修改。

3．代码规范部分

（1）修改的部分符合代码标准和风格么?

①、我们进行了很多优化，所以已经删除了很多多余的功能。

②、修改部分代码改换成有利于理解的代码，比较标准，风格也比较相符

③、修改的程序都是基于原有的理念上。

4．具体代码部分

（1）有没有对错误进行处理？对于调用的外部函数，是否检查了返回值或处理异常？

     对错误进行了处理，并进行了改进。对于调用的外部函数进行了金叉返回值和处理。

（2）参数传递有无错误，字符串的长度是字节的长度还是字符（可能是单/双字节）的长度，是以0开始计数还是以1开始计数？

     在参数传递有误的这一问题上我们进行了反复的测试，最后我们是以1开始计数的。

（3）边界条件是如何处理的？Switch语句的Default是如何处理的？循环有没可能出现死循环？

     边界条件是把所有的这些信息搜索起来，然后计算出每一个变量的这些被切碎了的区间，每一个区间的左右两边就是边界条件了。所以运用触发器让他变成死循环。

（4）有没有使用断言（Assert）来保证我们认为不变的条件真的满足？

      没有使用断言（Assert）来保证认为不变的条件真的满足。

（5）对资源的利用，是在哪里申请，在哪里释放的？有没有可能导致资源泄露（内存、文件、各种GUI资源、数据库访问的连接，等等）？有没有可能优化？

     没有对资源进行申请。没有可能导致资源泄露。没有优化，但是对新的代码和程序可以进行检查。

5、效能

（1）代码的效能（Performance）如何？最坏的情况是怎样的？

       还算稳定，现在在控制的范围内。最坏的时候是用户会在用着的时候系统崩溃。

（2）代码中，特别是循环中是否有明显可优化的部分（C++中反复创建类，C#中 string 的操作是否能用StringBuilder 来优化）？

        没有明显可优化的部分。

（3）对于系统和网络调用是否会超时？如何处理?

    这是一个小程序，不存在超时的问题。

6．可读性

代码可读性如何？有没有足够的注释？

   代码通俗易懂，没有特别复杂的句子。对于有点复杂的句子我们进行了注释。

7．可测试性

代码是否需要更新或创建新的单元测试？

还可以有针对特定领域开发（如数据库、网页、多线程等）的核查表。

在特别需要的时候会进行程序的更新，但是最基本的功能是不变的。但是在更新的前提下，我们会进行测试，看看是否能得到大众的喜欢。

 

 

                                                   【编程】赵佳佳

                                                 【复核审查】崔则