代码走查要点规范表
评审对象 |
|
作者 |
|
|
评审人 |
|
评审工作量 |
|
|
序号 |
检查项 |
评审意见 |
||
走查前准备 |
|
|||
1 |
得到一份解释代码的最新的设计文档 |
|
||
2 |
代码都已提交 ,版本统一 |
|
||
程序结构组织 |
|
|||
1 |
所有代码的结构清晰,具有良好的结构外观和整齐 |
|
||
2 |
所有的模块(函数和外部接口)定义清晰,模块分解清楚 |
|
||
3 |
所有的功能需求都明显的覆盖 |
|
||
4 |
结构设计能够满足机能变更 |
是“机能变更”还是“功能变更” |
||
5 |
整个代码体系结构组合合理 |
|
||
6 |
所有主要的数据构造描述清楚,合理 |
|
||
7 |
模块中所有的数据结构都定义为局部的,并且通过定义好的函数进行访问 |
|
||
8 |
所有的接口模块化,因此修改时不影响其他代码模块 |
|
||
9 |
代码体系构架对空间和速度都已经进行考虑 |
|
||
10 |
提供了处理数据的策略 |
|
||
11 |
数据库操作、IO操作等是否正确关闭资源。并且必须在try -catch-finally 的finally中关闭。 |
|
||
12 |
一个业务如果进行多次数据库更新、添加、删除是否正确添加事务。 |
|
||
13 |
进行逻辑与、逻辑或判断时是否使用短路与、短路或。 |
|
||
14 |
多处使用相同代码时,应定义唯一方法或变量以供使用。 |
|
||
15 |
对象是否使用工厂获取。 |
|
||
16 |
导入类时,如果仅使用包中的几个类,应导入具体类,而不是导入整个包。 |
|
||
17 |
数组声明的时候使用 int[] index ,而不要使用 int index[]。 |
|
||
18 |
检查代码和详细设计是否一致。 |
|
||
19 |
检查类中是否有无效的代码或者是无用的代码。 |
|
||
20 |
不要使用System.out.print()输出,需要进行日志处理 |
也不能使用System.err |
||
21 |