IPD_PSSD_T_0075 代码评审检查表V2.1.xlsVIP

检查表 封面 文档修订记录 序号 版本号 变化状态 变更（+/-）说明 作者 日期 *变化状态：C――创建，A——增加，M——修改，D——删除，只有比较大的改动才需要版本记录 项目名称： 项目编号： 项目负责人: 检查项 执行情况 代码评审检查表 代码正确性与结构 可维护 代码是否进行了编译，并遵照项目规定消除了相应的错误和告警提示？ 代码是否进行了静态检查分析，并遵照项目规定消除了相应的错误和告警提示？ 代码实现是否正确完整的实现了设计说明书中的所有设计？ 代码是否正确地实现了项目的需求？ 是否用有意义的枚举或宏来代替无明显意义的数字？ 指针是否确保先赋值后引用？ 执行字符串拷贝操作时，是否保证字符串的长度合法，并在处理过程开始时对长度进行合法性检查？ 是否处理了被调用函数的各种可能的返回值？ 是否初始化有关变量及运行环境？确保没有未经初始化的变量被引用。 对断言的使用是否正确？没有用断言来检查最终产品肯定会出现且必须处理的错误情况。 是否为单元测试、集成测试、系统测试准备了相应的调测开关及相应的调试结果打印函数？ 代码中是否没有未被调用、执行的冗余代码？ 代码中是否不存在多余的桩程序或者测试代码，如果有是否用编译开关进行包括？ 是否不存在能通过调用外部复用组件、模块或库函数替代的代码？ 是否不存在过于复杂的模块需要重新构造或者拆分成多个程序？ 代码是否与概要设计文档及详细设计文档中的设计保持一致，并同步更新？ 所有变量的命名是否符合规范，清晰并具备良好的自注释性？ 所有被赋值的变量类型是否保持一致，如果被赋值的变量与赋值的变量类型不一致，是否有显式的类型转换？ 是否没有声明冗余的或者没有被使用的变量？ 代码是否避免了对浮点型数值的相等比较等操作？ 除数是否做了零值测试？ 所有的循环、分支判断逻辑构造是否完整、正确并且嵌套适当？ If /else 条件组合中，最可能满足的条件是否放在最前面？ 循环结束的条件逻辑是否清晰并且总是可以满足以避免死循环？ 循环控制变量、索引、数组下标在循环开始前是否已经正确的初始化？ 是否尽量已经将可以放在循环体外的变量声明放在循环体外？ 代码中是否避免了对索引变量进行操作或者依靠索引变量来退出循环？ 如果存在多重循环，是否将复杂的操作代码尽可能的放在最外层循环？ 索引、指针和下标是否经过了数组、记录或文件的边界测试？ 是否验证了导入的数据或输入的参数的正确性与完整性？ 所有输出的变量是否都被赋值？ 如果存在对文件的操作是否判断了文件是否存在？ 在程序结束的时候所有的文件和设备是否都保持了正确的状态？ 变量 算法操作 循环和分支语句 防御性编程 代码排版是否遵循了相应的编程规范，风格是否保持一致？ 代码是否实现了PPL中定义的性能目标？ 引用数组时,数组下标是否没有越界？ 对临界资源(共享内存/静态变量/全局链表/消息队列等)使用时是否进行了互斥保护？ 在过程/函数退出之前是否释放了所有需要释放的资源(内存/文件句柄等)？ 如存在是否考虑了字节对齐问题？定义的结构尽量自然对齐或使用填充字节(PADDING)。 是否考虑了字节顺序问题？ 在发送数据时，将主机序转换为网络序；接收数据时，将网络序转换为主机序。 是否没有将相等比较操作符(＝＝)误用为赋值操作符（＝）？ 代码中是否尽可能消除goto语句？ 是否正确地使用了If /else 语句？有可能的话，if语句尽量加上else分支，对没有else分支的语句要小心对待。 switch语句是否有default分支？每个case块是否都有break语句？ 是否尽量减少一个任务/进程独占CPU的时间？避免使用大循环。 分配的内存空间是否都被及时释放？ 结论 说明 说明: 1. 每个检查项在执行情况列中有“是”，“否”，“免”三个选项，对于选择免的检查项要求在说明栏中解释免于检查的原因。 2. 本检查表提供了代码检视过程中的一系列最为常用的检查项。各个开发项目组可以根据项目的实际情况来增加或删除相应的检查项。 代码是具备用易于维护的清晰充分的注释？ 说明 评审结果 1.00 2.00 3.00 4.00 5.00 6.00 7.00 8.00 9.00 10.00 11.00 12.00 13.00 14.00 15.00 16.00 17.00 18.00 19.00 20.00 21.00 22.00 1.00 2.00 1.00 2.00 3.00 1.00 2.00 3.00 4.00 5.00 1.00 2.00 3.00 4.00 5.00 6.00 7.00 8.00 9.00 10.00 1.00 2.00 3.00 4.00 5.00 6.00