文档介绍：项目管理体系文件
中软国际Java项目开发
代码Review常见问题
实例分析及指南
编 撰 人：TMO
审 核 人：
批 准 人：
批准日期：2010-9-1
保密级别：机密
文档版本：
北京中软国际信息技术有限公司
版本历史
日期
版本
说明
作者
目录
第一章 综述 5
第二章 常见问题分类及实例分析 5
开发规范类 5
命名规范 5
代码格式 8
性能影响及系统稳定性相关 11
Java操作数据库 11
数据库SQL开发注意事项 15
数据库存储过程编写 18
Java编程方面 20
编程易错误区及系统健壮性 22
空指针错误的引发 22
格式化数字错误 23
字符串越界错误 23
没有克隆(clone)返回的对象 23
不必要的克隆 25
自编代码来拷贝数组 26
拷贝错误的数据 27
检查new 操作的结果是否为null 29
用== 29
混淆原子操作和非原子操作 30
在catch 块中作清除工作 31
增加不必要的catch 块 32
没有正确实现equals，hashCode，或者clone 等方法 32
Java编程不良****惯 33
Servlet中获取数据库连接问题 33
对于Exception的处理 33
有关常量的使用 34
静态变量的使用 34
未使用的变量 34
过于庞大的try块 34
最好不要让系统自己进行类型转换 37
ResourceOne开发框架使用问题 37
在普通JavaClass中获得数据库连接 37
Servlet中使用SearchDAO 38
程序调试信息 38
超长类名：使用R1Studio生成数据访问类的问题 38
代码中直接使用JDBC进行数据库操作 39
代码的分包问题 39
使用DAO操作时直接在Servlet里进行操作 40
业务逻辑问题 40
程序逻辑错误 40
综述
基础技术资源开发与管理部在颁布《六统一管理代码review办法》中，从管理的角度对中软国际所有的软件开发项目的程序代码review作了相关的要求，并且推行Jupiter这样的CodeReview协同管理工具来辅助促进和跟踪review的状态和结果。
CodeReview的基本手段还是需要技术经理通过人工检查项目成员的代码，来将各种问题扼杀在开发阶段，但是不同经验及技术水平的经理在review同一段代码所发现的问题可能相差比较大，不同的Team可能因此产生的效果也不同。本文从实际项目中抽取了一些项目中常见的Java开发所涉及的问题，进行实例分析，为各技术经理提供靶子和借鉴参考，如果审查者能够有意识地寻找特定的错误，而不是靠漫无目的的浏览代码来发现错误，那么代码审查的效果会事半功倍如能做到举一反三，则意义更大。
常见问题分类及实例分析
开发规范类
在六统一规范中颁布的《Java开发编码规范》和《Delphi开发规范》，里面都专门对于Java和Delphi语言进行开发的详细规定。下面是对一些基本要求的重申，对于代码review比较基础和关键。
命名规范
Java程序
包名
Package 的名字应该都是由小写单词组成。
具体的示例如下：
。
stmacmbas为本项目的根程序包。