《代码评审表模板.doc》由会员分享，可在线阅读，更多相关《代码评审表模板.doc（3页珍藏版）》请在金锄头文库上搜索。
1、代码评审清单（Code Checklist）版本：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 * 形式的语句，必须指出具体字段严禁使用insert into table values（？，？，？），必须指出具体要赋值的字段避免隐含的类型转换（不同数据类型字段相加）子查询前后必须加上括号避免在where使用1=1,1=2这种表达方式作为部分条件禁止使用视图禁止使用XX in () or XX in ()(in中的元素个数不应超过500)禁止使用or超过500个禁止使用not in，建议使用not exist禁止在一条sql语句中使用3层以上的嵌套如果有多表连接时，应该有主从之分，尽量从一个表取数Where子句过滤条件，索引列或过滤记录最多的条件应该放在前面字符串连接必须使用“|”Case when语句中只能出现=、=、=以及is null运算符左连接写法必须带”outer”关键字Sql中函数是否在指定范围内Join与on必须严格匹配问题清单问题描述预计修改日期修改日期检查人签字
