代码评审表模板.doc

《代码评审表模板.doc》由会员上传分享，免费在线阅读，更多相关内容在教育资源-天天文库。

1、代码评审清单（CodeChecklist）版本：1.0产品/项目组名称：宅急送产品/项目名称：公共版本号：1.0被检查人签字：检查内容：_________检查人签字：___________检查日期：_____说明：类清单被检查类名模块名称产品名称检查项常规是否程序是否明确地注释了其实现功能注释格式是否正确程序单元是否承担单一职责复杂程序是否合理地分解为子程序程序中接口定义是否明晰异常处理是否符合规范包名、类名命名是否符合规范方法名命名是否符合规范变量/常量是否相关系列变量是否统一集中声明，自定义数据类型变量是否进行了必要而充分的描述变量是否按相关规范进行了命名（有意义的命名、数据类型的区分等

2、）变量是否按其命名含义承担了单一的使用目的，必要时是否清晰定义了额外变量变量引用是否正确关闭常量是否使用getter/setter方法访问是否应用良好定义的枚举类型替代了简单的字符标识或布尔变量使用UFBoolean是否使用UFBoolean构造函数进行构造处理可变String时是否未使用StringBuffer是否优先使用Java标准库提供的容器（ArrayList、HashSet、HashMap、LinkList）来处理顺序结构、集合、关联数组合堆栈队列对数组的使用场景是否正确对于public尽量谨慎使用不同作用域变量名称尽量不要相同算法是否数据结构是否合理、精简算法是否可以独立测试，是

3、否与数据库和其他算法隔离是否尽量避免了嵌套的运用复杂逻辑是否进行了必要而充分的注释控制是否代码执行路径是否清晰Switch语句是否有缺省分支控制逻辑复杂度是否合理，是否进行了必要而充分的注释每个循环体是否仅执行了单一而明确的功能与常数比较需要将常数放在比较表达式的前面布局是否代码是否良好格式化并能体现其逻辑结构设计是否程序是否可读、可扩展并健壮具体实现细节是否已尽可能的“隐藏”是否优先使用接口而不是抽象类或具体类方法参数是否在5个以内方法慎用public修饰符对于不需要子类来重载的类尽量使用final对象作用域是否过大效率是否尽量不要在循环内出现远程调用每个业务动作远程调用次数是否小于3次远

4、程调用数据传输是否有不必要的冗余数据SQL是否Sql语句小写引用字符使用单引号严禁使用select*形式的语句，必须指出具体字段严禁使用insertintotablevalues（？，？，？），必须指出具体要赋值的字段避免隐含的类型转换（不同数据类型字段相加）子查询前后必须加上括号避免在where使用’1=1’,’1=2’这种表达方式作为部分条件禁止使用视图禁止使用XXin()orXXin()(in中的元素个数不应超过500)禁止使用or超过500个禁止使用notin，建议使用notexist禁止在一条sql语句中使用3层以上的嵌套如果有多表连接时，应该有主从之分，尽量从一个表取数Where

5、子句过滤条件，索引列或过滤记录最多的条件应该放在前面字符串连接必须使用“

6、

7、”Casewhen语句中只能出现=、>=、<=以及isnull运算符左连接写法必须带”outer”关键字Sql中函数是否在指定范围内Join与on必须严格匹配问题清单问题描述预计修改日期修改日期检查人签字