《代码评审检查表》由会员分享，可在线阅读，更多相关《代码评审检查表（7页珍藏版）》请在金锄头文库上搜索。
1、Java代码审查检查表文件标识：卜PR-CODE-项目名称检查日期文件编号、名称:填写人编号问题是否不适用BU数变量,Auribute,和常量声明缺陷(VC)1变量和常量的命名是否与约定保持一致？2是否存在容易混淆的相似的变量和属性名？3变量和属性是否书写正确？4变量和属性是否被正确的初始化？5非局部变量是否能用局部变量替换？6所有的for循环的控制变量是否都在循环顶部被声明？7是否有应该命名为常量的文字常量？8变量和属性是否可以用常量替换？9属性是否可以用本地变量？10所有的属性是否都有正确的访问限制符11是否有静态属性应该是非静态或vice-versa?方法定义缺陷(FD)1方法名的描述方
2、法是否与命名约定一致？2每个方法的参数值在使用之前是否都作了检查？3对于每一个方法，它是否都返回了正确的值？4每种方法是否都有正确的访冋限制符(private, protected,5静态方法是否应该为非静态或vice-versa?类定义缺陷(CD)1每一个类是否都有正确的构造函数和析构函数？ 12在子类中是否有应该放到父类中的通用成员？3类的继承层次是否能被简化？数据引用缺陷(DR)1对于每一个数组引用，下标值是否在定义的范围内？ J2对于对象和数组引用，是否组确定其值应为非空？计算/数值缺陷(CN)1是否存在不冋类型数据之间的混合计算？2在计算中是否存在上溢或下溢的可能？ 13关于数值计算
3、的顺序和优先级的假设是否正确？ 14是否用了括号来避免模糊不清？口比较/关系缺陷(CR)1对每一个布尔测试，正确条件是否被检查？2比较操作符是否正确？ J3布尔表达式是否通过内部否定操作进行了简化4每个布尔表达式是否都正确？ 口5比较操作是否存在不引人注意的副作用？6&是否被不小心替换为&? ”是否被不小心替换为流程控制缺陷（CF）1对于每一个循环：是否选用了最佳的循环结构？2所有的循环是否都能结束？3如果一个循环有多个出口，是否每个出口都有必要并且得到正确处4理?itch声明是否都有default条件？5是否所有的case-switch-break对应关系都已更正并加上批注？6是否named
4、 break叙述都跳到正确的地方？7循环和分支的嵌套是否过深？是否正确？8是否有if嵌套可以转换程switch嵌套？9空控制叙述是否都正确，并加上括号及批注？ 110所有的异常是否都得到了正确的处理11每一个方法在是否都结束？输入输出缺陷（10）1文件在被使用之前是否都被打开？2输入对象的属性是否与使用的文件一致？:3文件在被使用之后是否都被关闭？计算/数值缺陷（CN）1文本中是否有拼写和语法上的错误？2所有的I/O异常处理的是否合理？模块间接口缺陷1方法调用的参数的数量，顺序,类型和值是否与该方法声明一致？2度量单位是否一致（如：公分 vs.公尺）？3如果对象或数组被传递，它们是否改变？是否
5、被调用方法正确改变？注释缺陷（CM）1每一个方法，类和文件是否都有适当的头注释？2每一个属性，变量和常量的声明是否都有注释？3每个类和方法的潜在行为是否都有用简易的语言进行解释？4方法和类的头注释是否和它们的功能保持一致？5注释和代码是否保持一致？6注释对于理解代码是否有帮助？7代码中的注释是否充分？8代码中的注释是否过多？布局和封包缺陷（LP）1代码布局格式和缩排标准是否前后一致？2对于每一个方法，它的代码量是否都不超过60行？3对于每一个编译模块，它的代码量是否都不超过600行？模块性缺陷（MO）1模块（方法,类）之间是否具有低偶合性？ 口2每个模块（方法,类）自身是否具有高聚合性？3是否
6、存在重复的代码，它的功能可以通过调用其它方法实现？4Java类库的使用是否适时适地？存储器使用缺陷（SU）1数组是否足够大？2数组和对象不再使用之后，它们的引用是否被赋为空值？性能缺1陷（PE）可选1是否有更好的数据结构和算法可以米用？2测试安排是否合理，使易于通过的且代价低廉的测试优先于代价较高 且通过频率较低的测试？3是否可以通过对数值进行一次计算并将结果保存来减少对它重新计 算带来的消耗？4每一个计算出并保存了的结果是否都被应用？5计算是否能被移到循环之外？6在循环内是否有不需要的测试？7短循环是否可以取消？8对同一个数据进行操作的两个循环是否可以合并成一个？其他结论：厂通过厂有条件通过不通过说明:CODE-CHECKLIST-YYYYMMDD备注版本：V1.1
