Android码走查情况报告.doc

编号 密别： 乐土软件 软件代码走查情况报告 共 8 页 项 目 代 号 项 目 名 称 乐土 项 目负责人 张琛 评 审 组 长 赵斌 项目名称 乐土 产品名称 乐土 项目负责人 张琛 主办单位 易图通科技（北京）有限公司 评审地点 上海 评审时间 2010年12 月3 日至 12月3 日 评审结论： 代码审查组根据项目组提交的详细设计文档、审查模块的源代码、代码审查检查表，确定了代码审查的范围，具体抽样审查了编码中存在的与详细设计不符的错误、编码在数据、算法、逻辑和语言方面存在的错误、是否违背编码规范、编码的可读性、可测试性和可维护性等内容；并同时对代码注释行进行了抽样量测。 代码走查结果表明：乐土软件代码严格按照编程规范进行编写，基本上做到了与代码检查表的要求一致，注释行数量占程序数量达到20％以上，具备较好的程序易读性。乐土软件（Android版客户端）未使用第三方模块。 具体详细的抽查情况见附页。 评审组长签名 年 月 日 附页： 代码审查总表 编号： 项目名称 乐土软件 项目负责人 张琛 审查组员 赵斌、章恩伟 审查组长 赵斌 总模块数 3 审查模块数 3 百分比 100％ 序号 模块名称 程序员 审查时间 代码总行数 统计单号 1 X2baselib 张建永 10分钟 749 2 X2engine 张建永 5分钟 30 3 luutoo 朱峰、王冠立、孙德华 30分钟 29853 C++/C代码检查表 编号：DMGT-C-001 文件结构 重要性 审查项 结论 头文件和定义文件的名称是否合理？ 是 头文件和定义文件的目录结构是否合理？ 是 版权和版本声明是否完整？ 是 重要 头文件是否使用了 ifndef/define/endif 预处理块? 是 头文件中是否只存放“声明”而不存放“定义” 是 程序的版式 重要性 审查项 结论 空行是否得体？ 是 代码行内的空格是否得体？ 是 长行拆分是否得体？ 是 “ ” 和 “ ” 是否各占一行并且对齐于同一列？ 是 重要 一行代码是否只做一件事？如只定义一个变量，只写一条语句。 是 重要 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数据成员只在某个对象生存期内是常量，而对于整个类而言却是可变的。 否 函数设计 重要性 审查项 结论 参数的书写是否完整？不要贪图省事只写参数的类型而省略参数名字。 是 参数命名、顺序是否合理？ 是 参数的个数是否太多？ 否 是否使用类型和数目不确定的参数？ 否 是否省略了