编程常犯错误
? ? ? ?编写代码是一种艺术，认识错误是我们代码改进的重要途径之一。以下情况并非大家都能碰到过，但希望提高代码质量的人都引以为戒。以下各种情况，都是初学者经常犯的错误。
1.1?????? 字符串没有判断是否为空
1.1.1????????直接插入数据库
这种情况经常出现在服务器的java代码中，当从用户输入的表单中获取用户输入的信息时，通常认为客户端输入的信息就是我们想要的信息，直接插入到数据库。结果，数据库中出现了很多“null”的值。随便举个例子，通常在保存到数据库前，首先需要拼接sql,举例：
?
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行，其可读性就已经很差，但作为一个企业，稳定是大家最期待的结果，所以，前辈写好的代码，我们只在其基础上修改，增加功能，拷贝代码注释旧代码，结果一个方法越来越臃肿，由几百行扩大到几千行，甚至上万行。出现问题，推卸责任，说代码之前就是这样写的。
正确方法：
如果是新增加功能，单独出一个方法写功能，在原来的方法里只需要一个调用或一个判断即可，而不要直接在原来的方法里面写代码。
如果原来的方法已经很臃肿，拆分可以独立出来的代码，然后再做修改。
1.5?????? 太信赖客户端
比如时间和日期格式。
或许，这是按模块分配任务的悲剧，如果一个人既写客户端，又写服务器端代码，那么他可以把那个模块运行的很好。可是开发只占20%，我们却要花80%的时间来修改一个模块。那个人开发完，拍拍屁股走人，另一批人就要花80%时间来修改那个人留下的bug。其中有些隐藏较深的bug就是客户端信赖。比如：
举例1：日期，在客户端通常是选择的，格式都是固定的，而且也是正确的，在服务器端只需要获取保存即可。
举例2：某些日期是显示给用户看的，默认为当前时间，是不能改变的，在服务器端也是只需获取保存即可。
以上只是最常见的两个例子，系统发布后是怎样的呢？某些日期并不合法的日期格式也给保存到数据库。默认的当前时间会出现在当前时间之后。提交的数据也是千差百错。这是因为，用户不一定按照你的思路出牌。他不使用你的日期控件，或者还没有把你用的JS加载完毕，就提交了数据。或者用户本机的时间是错误的，你默认显示的当前日期也就是错误的。或者用户使用一些表单提交工具提交数据。
所以在程序中，服务器段校验比不可少，不但要校验数据格式，还要做逻辑校验，比如默认时间不能比当前日期还晚。