【Android源码 栏目提醒】:网学会员鉴于大家对Android源码 十分关注,论文会员在此为大家搜集整理了“Android版代码走查情况报告 - 讲义教程”一文,供大家参考学习
编号 密别 乐土软件 软件代码走查情况报告 共 8 页 项 目 代 号 项 目 名 称 乐土 项 目负责人 张琛 评 审 组 长 赵斌 1 项目名称 乐土 产品名称 乐土 项目负责人 张琛 主办单位 易图通科技北京有限公司 评审地点 上海 评审时间 2010年12 月3 日至 12月3 日 评审结论 代码审查组根据项目组提交的详细设计文档、审查模块的源代码、代码审查检查表确定了代码审查的范围具体抽样审查了编码中存在的与详细设计不符的错误、编码在数据、算法、逻辑和语言方面存在的错误、是否违背编码规范、编码的可读性、可测试性和可维护性等内容并同时对代码注释行进行了抽样量测。
代码走查结果表明乐土软件代码严格按照编程规范进行编写基本上做到了与代码检查表的要求一致注释行数量占程序数量达到20以上具备较好的程序易读性。
乐土软件
Android版客户端未使用第三方模块。
具体详细的抽查情况见附页。
评审组长签名 年 月 日 2 附页 代码审查总表 编号 项目名称 乐土软件 项目负责人 张琛 审查组员 赵斌、章恩伟 审查组长 赵斌 总模块数 3 审查模块数 3 百分比 100 序号 模块名称 程序员 审查时间 代码总行数 统计单号 1 X2baselib 张建永 10分钟 749 2 X2engine 张建永 5分钟 30 3 luutoo 朱峰、王冠立、孙德华 30分钟 29853 C/C代码检查表 编号DMGT-C-001 文件结构 重要性 审查项 结论 头文件和定义文件的名称是否合理 是 头文件和定义文件的目录结构是否合理 是 版权和版本声明是否完整 是 重要 头文件是否使用了 ifndef/define/endif 预处理块 是 头文件中是否只存放“声明”而不存放“定义” 是 程序的版式 重要性 审查项 结论 空行是否得体 是 代码行内的空格是否得体 是 长行拆分是否得体 是 “” 和 “” 是否各占一行并且对齐于同一列 是 重要 一行代码是否只做一件事如只定义一个变量只写一条语句。
是 3 重要 If、for、while、do等语句自占一行不论执行语句多少都要加“”。
是 重要 在定义变量或参数时是否将修饰符 和 紧靠变量名 是 注释是否清晰并且必要 否 重要 注释是否有错误或者可能导致误解 否 重要 类结构的public protected private顺序是否在所有的程序中保持一致 是 注释量是否达到一定比例 是 命名规则 重要性 审查项 结论 重要 命名规则是否与所采用的操作系统或开发工具的风格保持一致 否 标识符是否直观且可以拼读 否 标识符的长度应当符合“min-length max-information”原则 是 重要 程序中是否出现相同的局部变量和全部变量 否 类名、函数名、变量和参数、常量的书写格式是否遵循一定的规则 否 静态变量、全局变量、类的成员变量是否加前缀 是 表达式与基本语句 重要性 审查项 结论 重要 如果代码行中的运算符比较多是否已经用括号清楚地确定表达式的操作顺序 是 是否编写太复杂或者多用途的复合表达式 否 重要 是否将复合表达式与“真正的数学表达式”混淆 否 重要 是否用隐含错误的方式写if语句 例如 1将布尔变量直接与TRUE、FALSE或者1、0进行比较。
2将浮点变量用“”或“”与任何数字比较。
3将指针变量用“”或“”与NULL比较。
否 如果循环体内存在逻辑判断并且循环次数很大是否已经将逻辑判断移到循环体的外面 否 重要 Case语句的结尾是否忘了加break 否 重要 是否忘记写switch的default分支 否 重要 使用goto 语句时是否留下隐患 例如跳过了某些对象的构造、变量的初始化、重要的计算等。
否 常量 重要性 审查项 结论 是否使用含义直观的常量来表示那些将在程序中多次出现的数字或字符串 是 在C 程序中是否用const常量取代宏常量 是 重要 如果某一常量与其它常量密切相关是否在定义中包含了这种关系 是 是否误解了类中的const数据成员因为const数据成员只在某个对象生存期内是常量而对于整个类而言却是可变的。
否 函数设计 4 重要性 审查项 结论 参数的书写是否完整不要贪图省事只写参数的类型而省略参数名字。
是 参数命名、顺序是否合理 是 参数的个数是否太多 否 是否使用类型和数目不确定的参数 否 是否省略了函数返回值的类型 否 函数名字与返回值类型在语义上是否冲突 否 重要 是否将正常值和错误标志混在一起返回正常值应当用输出参数获得而错误标志用return语句返回。
否 重要 在函数体的“入口处”是否用assert对参数的有效性进行检查 是 重要 使用滥用了assert 例如混淆非法情况与错误情况后者是必然存在的并且是一定要作出处理的。
否 重要 return语句是否返回指向“栈内存”的“指针”或者“引用” 否 是否使用const提高函数的健壮性const可以强制保护函数的参数、返回值甚至函数的定义体。
“Use const whenever you need” 是 单个函数的长度应有一定的节制 否 尽量使用标准库函数和公共函数 是 不要随意定义全局变量尽量使用局部变量 是 循环、分支层次不要不必要地过多 否 所有变量在调用前必须被初始化 是 对所有的用户输入必须进行合法性检查 是 程序与环境或状态发生关系时必须主动去处理发生的意外事件 是 内存管理 重要性 审查项 结论 重要 用malloc或new申请内存之后是否立即检查指针值是否为NULL防止使用指针值为NULL的内存 是 重要 是否忘记为数组和动态内存赋初值防止将未被初始化的内存作为右值使用 否 重要 数组或指针的下标是否越界 否 重要 动态内存的申请与释放是否配对防止内存泄漏 否 重要 是否有效地处理了“内存耗尽”问题 是 重要 是否修改“指向常量的指针”的内容 否 重要 是否出现野指针例如 1指针变量没有被初始化。
2用free或delete释放了内存之后忘记将指针设置为NULL。
否 重要 是否将malloc/free 和 new/delete 混淆使用 是 重要 malloc语句是否正确无误例如字节数是否正确类型转换是否正确 是 重要 在创建与释放动态对象数组时new/delete的语句是否正确无误 是 5 C 函数的高级特性 重要性 审查项 结论 重载函数是否有二义性 否 重要 是否混淆了成员函数的重载、覆盖与隐藏 否 运算符的重载是否符合制定的编程规范 是 是否滥用内联函数例如函数体内的代码比较长函数体内出现循环。
否 重要 是否用内联函数取代了宏代码 是 类的构造函数、析构函数和赋值函数 重要性 审查项 结论 重要 是否违背编程规范而让C 编译器自动为类产生四个缺省的函数1缺省的无参数构造函数2缺省的拷贝构造函数3缺省析构函数4缺省的赋值函数。
否 重要 构造函数中是否遗漏了某些初始化工作 否 重要 是否正确地使用构造函数的初始化表 是 重要 析构函数中是否遗漏了某些清除工作 否 是否错写、错用了拷贝构造函数和赋值函数 否 重要 赋值函数一般分四个步骤1检查自赋值2释放原有内存资源3分配新的内存资源并复制内容4返回 this。
是否遗漏了重要步骤 否 重要 是否正确地编写了派生类的构造函数、析构函数、赋值函数注意事项 1派生类不可能继承基类的构造函数、析构函数、赋值函数。
2派生类的构造函数应在其初始化表里调用基类的构造函数。
3基类与派生类的析构函数应该为虚即加virtual关键字。
4在编写派生类的赋值函数时注意不要忘记对基类的数据成员重新赋值。
是 类的高级特性 重要性 审查项 结论 重要 是否违背了继承和组合的规则 1若在逻辑上B是A的“一种”并且A的所有功能和属性对B而言都有意义则允许B继承A的功能和属性。
2若在逻辑上A是B的“一部分”a part of则不允许B从A派生而是要用A和其它东西组合出B。
否 其它常见问题 重要性 审查项 结论 重要 数据类型问题 变量的数据类型有错误吗 存在不同数据类型的赋值吗 存在不同数据类型的比较吗 否 重要 变量值问题 变量的初始化或缺省值有错误吗 否 6 变量发生上溢或下溢吗 变量的精度够吗 重要 逻辑判断问题 由于精度原因导致比较无效吗 表达式中的优先级有误吗 逻辑判断结果颠倒吗 否 重要 循环问题 循环终止条件不正确吗 无法正常终止死循环吗 错误地修改循环变量吗 存在误差累积吗 否 重要 错误处理问题 忘记进行错误处理吗 错误处理程序块一直没有机会被运行 错误处理程序块本身就有毛病吗如报告的错误与实际错误不一致处理方式不正确等等。
错误处理程序块是“马后炮”吗如在被它被调用之前软件已经出错。
否 重要 文件I/O问题 对不存在的或者错误的文件进行操作吗 文件以不正确的方式打开吗 文件结束判断不正确吗 没有正确地关闭文件吗 否
上一篇:
基于android的远程视频监控系统
下一篇:
7*30m公路预应力混凝土连续梁桥上部结构设计