今天维护又反馈了一问题过来，查询试卷时报数组越界异常：
1 2017-02-28 10:45:24,827[ERROR] HttpException[10.32.111.7:60446:2D07867BE98F56D5EFFA1B1A597776AC]:/WAserver/HS851020 [com.hundsun.hsacct.core.httpresult.HandlerExceptionResolver.printHttpLog(HandlerExceptionResolver.java:156)] 2 java.lang.ArrayIndexOutOfBoundsException: 1 3 at com.hundsun.hsacct.action.paper.QryEligPaperAction.qryEligPaper(QryEligPaperAction.java:69) 4 at sun.reflect.GeneratedMethodAccessor2434.invoke(Unknown Source) 5 at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 6 at java.lang.reflect.Method.invoke(Method.java:606)
看了一下开发人员写的代码，其中有两行用逗号来作为分隔符来分割字符串，基本上已经知道了问题原因。这时，心里默默想起了墨菲定律：任何事情只要存在出错的可能性，那最后肯定会出错，没出错只是因为时机未到。大师的理论果然厉害
String answer_content = map.get("answer_content");
if (StringUtils.isNotBlank(answer_content)) {
//第一次分割,分隔符 ','
String[] stepOne = answer_content.split(",");
Map<String,String> answer = new HashMap<String,String>();
for(int i = 0; i<stepOne.length; i++)
{
//第二次分割，分隔符'、'
String[] stepTwo = stepOne[i].split("、");
//向HashMap中添加
if(answer.get(stepTwo[0])==null){
answer.put(stepTwo[0], stepTwo[1]);
}
else{
answer.put(stepTwo[0], stepTwo[1]+","+answer.get(stepTwo[0]));
}
}
paper.put(Fields.ANSWER_CONTENT, answer);
} else {
paper.put(Fields.ANSWER_CONTENT, "");
}
其实之前审核代码的时候无意瞟了一眼这里，隐约感觉这段代码里面有坑，但是测试那边没测出问题，加上这个接口业务逻辑简单，所以也没当回事，结果最后快上生产了，问题还是暴漏出来了。
问题出来了，除了觉得开发人员经验不足，还有更多的是对自己的自责，毕竟开发写的代码是通过了自己的审核，为什么轻易把问题放过去了呢？
在这里对问题做一下深入分析：
IndexOutOfBoundsException(数组越界异常) 可以说是Java代码中最常见的异常之一，还有一个是NullPointerException(空指针异常)。出现的原因：对不可控的String进行split操作然后访问其中的数据是最常见的原因。 什么叫“不可控String”：来自于我们自己的程序之外的String。什么是“可控String”：来自于我们系统内部的String，比如我们代码里面定义的常量，我们系统使用的格式固定的配置文件等等。
类似的还有StringIndexOutOfBoundsException，当你对一个长度不够的字符串进行substring操作就会抛出该异常。出现该异常的原因，还是在于使用不可控String。
作为一个有经验的程序员，当看到String.split()还有String.substring()这种代码时，一定要小心谨慎。很有可能这里已经埋下了一颗定时炸弹，暂时没出现问题只是因为时机未到。
比较好的处理方式：当我们必须要对不可控String进行split()还有substring()操作时，应当进行一下检查。如果字符串不符合格式要求，我们应当主动抛出异常，让系统维护人员第一时间知道当前系统不满足运行条件，需要进行检修，从而让故障损失降到最低。