总结下java经常犯的错误
编写代码是一种艺术，认识错误是我们代码改进的重要途径之一。以下情况并非大家都能碰到过，但希望提高代码质量的人都引以为戒。以下各种情况，都是初学者经常犯的错误。
1.1 字符串没有判断是否为空
1.1.1 直接插入数据库
这种情况经常出现在服务器的java代码中，当从用户输入的表单中获取用户输入的信息时，通常认为客户端输入的信息就是我们想要的信息，直接插入到数据库。结果，数据库中出现了很多“null”的值。随便举个例子，通常在保存到数据库前，首先需要拼接sql,举例：
Java代码
String name = paramsMap.get("name");
String gender = paramsMap.get("gender");
String sql= “insert into user（name,gender） values(‘”+name+
’,’”+gender+”’)”;
[java]
String name = paramsMap.get("name");
String gender = paramsMap.get("gender");
String sql= “insert into user（name,gender） values(‘”+name+
’,’”+gender+”’)”;
String name = paramsMap.get("name");
String gender = paramsMap.get("gender");
String sql= “insert into user（name,gender） values(‘”+name+
’,’”+gender+”’)”;
正常情况，当用户填写了name和gender时，此代码没有问题。可现实并非我们想象的那样好，在这里，这里的name的取值通常有3种情况，null，“null”（通常这种情况是客户端没有处理好）和”name”,只有最后一种情况是正确的，其他的两种情况都并不是我们所想要的。也就是说，看似简单的代码，却只考虑了三分之一的情况，其他的三分之二的情况却没有考虑。结果，这就是为什么我们在上网时，为什么查看某些账单时，经常会出现null这种用户感觉很困惑的“汉字”。
正确写法：
Java代码
String name = paramsMap.get("name");
String gender = paramsMap.get("gender");
if(name==null || name=”null”){
name=””;//如果为空，设置为空,数据库不会存储值
}
String sql= “insert into user（name,gender） values(‘”+name+
’,’”+gender+”’)”;
[java]
String name = paramsMap.get("name");
String gender = paramsMap.get("gender");
if(name==null || name=”null”){
name=””;//如果为空，设置为空,数据库不会存储值
}
String sql= “insert into user（name,gender） values(‘”+name+
’,’”+gender+”’)”;
String name = paramsMap.get("name");
String gender = paramsMap.get("gender");
if(name==null || name=”null”){
name=””;//如果为空，设置为空,数据库不会存储值
}
String sql= “insert into user（name,gender） values(‘”+name+
’,’”+gender+”’)”;
当然，以上判断方法也可以写成一个公用方法，直接调用。
1.1.2 直接截取
还是以上面代码为例：如果我们想把名称里面第一个字提取出来当做姓，单独存储起来，怎么办？看以下代码：
Java代码
String name = paramsMap.get("name");
String lastName = name.substr(0,1);
String gender = paramsMap.get("gender");
String sql= “insert into user（name,last_name,gender） values(‘”+name+
’, ’”+ lastName +”’,’”+gender+”’)”;
[java]
String name = paramsMap.get("name");
String lastName = name.substr(0,1);
String gender = paramsMap.get("gender");
String sql= “insert into user（name,last_name,gender） values(‘”+name+
’, ’”+ lastName +”’,’”+gender+”’)”;
String name = paramsMap.get("name");
String lastName = name.substr(0,1);
String gender = paramsMap.get("gender");
String sql= “insert into user（name,last_name,gender） values(‘”+name+
’, ’”+ lastName +”’,’”+gender+”’)”;
看似没有什么问题，正常情况下，这段代码没有一点问题。可是，在实际的生产环境，这段代码可能在后台不停的报空指针异常。试问，当name为空时呢？或许你会说，在客户端控制为必填项，那如果这在业务上并非用户关注点呢，不是必填项呢？这里只是一个举例，程序中有很多这样类似的代码，总是认为参数是完整的，参数就是我们想象的那种格式。
正确写法：
Java代码
String name = paramsMap.get("name");
String lastName =“”;//默认为空
if(StringUtils.isNotBlank(name)){
LastName=name.substr(0,1)
}
String gender = paramsMap.get("gender");
String sql= “insert into user（name,last_name,gender） values(‘”+name+
’, ’”+ lastName +”’,’”+gender+”’)”;
[java]
String name = paramsMap.get("name");
String lastName =“”;//默认为空
if(StringUtils.isNotBlank(name)){
LastName=name.substr(0,1)
}
String gender = paramsMap.get("gender");
String sql= “insert into user（name,last_name,gender） values(‘”+name+
’, ’”+ lastName +”’,’”+gender+”’)”;
String name = paramsMap.get("name");
String lastName =“”;//默认为空
if(StringUtils.isNotBlank(name)){
LastName=name.substr(0,1)
}
String gender = paramsMap.get("gender");
String sql= “insert into user（name,last_name,gender） values(‘”+name+
’, ’”+ lastName +”’,’”+gender+”’)”;
所使用StringUtils类为org.apache.commons.lang. StringUtils
1.2 根据位置取值
在某些有规律的表单值，比如像列表，我们经常使用这种方法来获得所有用户输入的数据项，然后按照html输入的顺序，依次取值，然后再做进一步处理。然而这种代码的可维护性很差。例如：原来的顺序是姓名，性别和年龄，现在需要在年龄前面插入一列，名称是联系方式，打乱了原来的顺序。那么所有取值的代码都要把取值的顺序修改一遍，这是列表的属性比较少的时候，如果一个列表的列超过了20个，而要插入的位置是第2列，那你的悲剧就来啦。
代码举例：
Var name = arr[1];
Var gender=arr[2];
正确的写法：
不要依赖属性取值，根据键值进行取值。
1.3 ID重复
这个是在html中经常出现的错误。但是当时用extJs时，这个错误却很难排查。
推荐的方法：
在命名id时，加上前缀，例如用户管理界面，所有的ID都以USER开头。或者企业用户，则添加公司的域名。
1.4 代码堆积
这个也不能算一个错误，通常一个方法超过100行，其可读性就已经很差，但作为一个企业，稳定是大家最期待的结果，所以，前辈写好的代码，我们只在其基础上修改，增加功能，拷贝代码注释旧代码，结果一个方法
补充：软件开发 , Java ,
