一次全表审核“卡99%”故障排查：空字符串在while循环中引发的性能陷阱

今天下午听到部门内另外一个报表项目说他们全表审核 操作前端一直卡在99%，就好奇看了一下。前端页面触发全表审核 请求，过了五分钟没有响应请求失败，看后台还继续在运行。

排查问题

全表审核 是后台是异步执行，所以直接查看线程栈，IDEA里面直接点击Get Thread Dump获取线程栈信息（也可以通过jvm命令查看，visualvm也行）。看了一下全表审核确实还没执行完成，线程一直在waiting，因为代码里面是调用了join方法，会等待所有任务执行完成拿到结果。

join卡在这，说明还有执行代码的子线程，查看线程池里面的子线程列表，发现还有一个子线程在跑，根据线程栈信息跳转到对应代码行。

这个代码在平台写的jar里面。点开发现了不对劲的地方，count的值9.9个亿，心想应该是死循环了。他这个方法看注释是【统计s字符串在text字符串中出现的次数】

仔细看这个函数，问题就出在while (true)，且s字符串为空字符。

入参字符串s值为""空字符，text.indexOf(s)返回值为0
进入else条件，count用来记录出现次数的值自增+1
调用被统计字符串text的substring方法，关键点来了，i因为是0，空字符s.length()也是0。那这个方法调用就等同于text.substring(0)。substring方法形参如果为0的话，返回值是字符串对象本身。所以每次while循环执行的结果都一模一样，除了count在不停的自增。

java 复制代码

    private int count(String text, String s) {
        int count = 0;
        while (true) {
            
            int i = text.indexOf(s);
            if (i == -1) {
                break;
            } else {
                count++;
                text = text.substring(i+s.length());
            }
        }
        return count;
    }

解决方案

使用Spring框架中的StringUtil.countOccurrencesOf方法。