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

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

排查问题

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

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

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

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

  1. 入参字符串s值为""空字符,text.indexOf(s)返回值为0
  2. 进入else条件,count用来记录出现次数的值自增+1
  3. 调用被统计字符串textsubstring方法,关键点来了,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方法。

相关推荐
折哥的程序人生 · 物流技术专研2 小时前
第4篇:Lambda 简化策略模式(Java 8+)
java·设计模式·策略模式·函数式编程·lambda·代码简化·扩充系列
researcher-Jiang2 小时前
高性能计算之OpenMP——超算习堂学习1
android·java·学习
西门吹-禅4 小时前
java springboot N+1问题
java·开发语言·spring boot
DLYSB_4 小时前
生命通道:如何用 HIS 医疗系统直连网络声光终端,打造“零延误”的危急值响应网关?
java·网络·数据库·报警灯
weixin_BYSJ19874 小时前
SpringBoot + MySQL 乒乓球运动员信息管理系统项目实战--附源码04954
java·javascript·spring boot·python·django·flask·php
AI小码4 小时前
LLM 应用的缓存工程:当每次 API 调用都在燃烧成本
java·人工智能·spring·计算机·llm·编程·api
Hui Baby5 小时前
Spring Security
java·后端·spring
IT笔记5 小时前
【Rust】Rust Match 模式匹配详解
java·开发语言·rust
我才是银古6 小时前
构建 Java GIS 工具库:从碎片化 API 到统一抽象的设计实践
java·gis·geotools·gdal·esri
TPBoreas6 小时前
配置信息防泄露方案:.env 环境隔离详解(dotenv-java)
java·开发语言