代码检查 整理汇总（一）

 

当下，正在实施代码检查的工作，先将检查的结果和建议列举如下：

（一）2011.08.01

1、中文注释

 所有的javaBean的属性都建议增加中文说明

2、仅保留需要的

利用eclipse生成的构造函数，除了缺省的默认构造函数，带参数的构造函数如果需要就保留，不需要就删除掉

3、方法、类名、变量名尽量避免包括数字、或者单纯的无意义的字母

4、注意JSP页面中代码的格式，尤其是JSP中javascript脚本的格式，此外，在ＪＳＰ的文件中建议不出现Ｊａｖａ代码，充分利用Ｓｔｒｕｔｓ的标签库。

5、Ａｃｔｉｏｎ中没有用到的继承自父类的方法，就不用覆写了

6、方法的注释里面，建议书写每个参数的含义，以及约束条件

7、去掉不必要的控制台输出（System.out），如果需要输出建议都采用log4j方式进行输出

8、sql语句中，用到like关键字，一定要根据每个数据库产品的要求，进行匹配。如果没有“%”，那么不要使用like

9、replace 与replaceAll的区别是什么？建议在通常情况下使用replaceAll函数，当字符串拥有转义的字符（例如，“\”）时，且不需要转义的情况下，适用replace函数

10、当使用request.getParameter("xxxxx")时，该函数本身就返回String类型，所以不需要在后面增加.toString()方法

（二）2011.08.09

1、字符串的截取，如果需要获取字符串里某个字符分割的前后部分，尽量使用substring函数。split函数执行效率相对慢很多

2、在循环体里面，尽量不声明新的变量

3、调试完毕后，意义不大的输出语句，一定删除

4、&& 与&的区别，一定要搞清楚，不能乱用：&&是逻辑与，类似“并且”；&是二进制的与运算

5、Bean的属性，需要增加注释，且注释要完整

（三）2011.08.15

1、编写代码的时候，注意逻辑顺序。举例，有一方法检验登录名唯一，通过方法传过来的变量判断是否返回布尔值，只有当真的时候才去查询数据库。如果顺序颠倒，将会带来性能上的浪费。

2、循环体内，避免创建新的对象

3、定义变量的时候，注意规则。避免使用类似[String string=""] 的小写关键词为变量的方式

4、Sql中如果多处用到从数据库中查询得到的值（例如，地区编码、机构编码等），建议首先查询出来，放到缓存中去

5、需要在程序中写死的常量，建议通过常量类来实现。方便以后的维护

6、代码格式要统一，对于“变量.属性”的，不要换行。代码行较长的，应换行，建议换行要求，屏幕大小，不需拖动滚动条