代码走查报告审查日期 2011 -6-169:30 走查时间( 分钟)720 参考文档《详细设计说明书》和《java 编码规范》缺陷个数统计 15 参与人员康瑞伟、刘树强、杨洲走查问题列表序号问题内容影响分析 1 WaitWorkServiceImpl 类的 allotProblem 方法执行分配问题操作时,使用了 for 循环执行 sql 语句,影响性能采用 for 循环执行 sql 语句,影响数据库的执行效率 2 WaitWorkServiceImpl 类的 disuseUnAllot 方法用到的参数 array 命名不明确影响代码的可阅读性和可维护性 3 WaitWorkAction 类有被注释掉的无用代码行,影响代码质量影响代码的可阅读性和代码的质量 4 WaitWorkServiceImpl 类的 queryForPageAllot 方法 sql 查询语句字段不完整影响代码的可阅读性和可维护性 5 pleteWorkServiceImpl 类的 allotFull 方法中的参数 userId 参数名意思不明确影响代码的可阅读性和可维护性 6 MessageDisposeAction 类中的玩家可见信息添加,删除,修改方法命名混乱,没有规则影响代码的可阅读性和可维护性 7 执行 MessageDisposeAction 类的 queryProblemParticulars 查询问题详细时,属性影响系统的安全性 8 VariableDaoImpl 类中 queryVariableForVariableID 方法中定义的 int result 无意义影响代码的可阅读性和可维护性及代码的性能 9 userGroupAction 类中 updateUserGroup 方法中存在 (); 语句应该去掉影响代码的可阅读性和可维护性 10 problemManageAction 类中 insertUserListForTypeID 方法中使用 for 循环添加分配列表,应该使用批处理进影响系统的性能 11 problemManageAction 类中 updateDestributionList 方法中使用 for 循环添加分配列表,应该使用批处理进行影响系统的性能 12 public class pleteWorkServiceImpl implements pleteWorkService {…}的 public 影响系统的安全性和代码的可阅读性 13 public class WaitWorkAction extends BaseAction {…}的 public String queryForPageAllot(){...} 方影响系统的安全性 14 类缺少注释影响代码的可阅读性和可维护性 15删除 UserGroupDao 接口中的无用接口影响代码的可阅读性和可维护性 16 17 18 19 20 21 检查结果本次代码检查邀请杨洲参加,依据《详细设计说明书》和《Java 编码规范》,在检查过程中发现一部分代码的规范性方面做的还不到位,对代码编码规范要求没有做到;发现少量模块的实现方式欠缺规范和经验,没能及时考虑到系统的性能和可维护性,在后期的代码实现及BUG 修改
HLSD-CIM-代码走查报告 来自淘豆网www.taodocs.com转载请标明出处.