墨菲定律与 IndexOutOfBoundsException(数组越界异常)

今天维护又反馈了一问题过来，查询试卷时报数组越界异常：

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)] 
java.lang.ArrayIndexOutOfBoundsException: 1
    at com.hundsun.hsacct.action.paper.QryEligPaperAction.qryEligPaper(QryEligPaperAction.java:69)
    at sun.reflect.GeneratedMethodAccessor2434.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    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()操作时，应当进行一下检查。如果字符串不符合格式要求，我们应当主动抛出异常，让系统维护人员第一时间知道当前系统不满足运行条件，需要进行检修，从而让故障损失降到最低。