前几天业务方反馈有一个接口查询时间,超过了1.5秒,不能忍了,让我看看,我看了下逻辑,对其中的一些逻辑进行了优化,也比较简单,这里做一个记录。
1.第一个改造就是
List callSaleInfoDtos = CallSaleInfoDto.fromEntity(callSaleInfoList, new CallSaleInfoDto());
这个方法本质上还是使用了
BeanUtils.copyProperties(dto, entity);
这个方法耗时特别长,200个数据耗时将近要1秒左右,后来改成 Lambda形式,200个时间缩短到10毫秒左右
List callSaleInfoDtos = callSaleInfoList.stream().map(callSale -> {CallSaleInfoDto dto = new CallSaleInfoDto();dto.setId(callSale.getId());dto.setUserId(callSale.getUserId());dto.setName(callSale.getName());dto.setProperty(callSale.getProperty());dto.setPrivateNum(callSale.getPrivateNum());dto.setBindTel(callSale.getBindTel());dto.setWorkPhone(callSale.getWorkPhone());dto.setTeam(callSale.getTeam());dto.setLogin(callSale.getLogin());dto.setBusy(callSale.getBusy());dto.setDeleted(callSale.getDeleted());dto.setSupplier(callSale.getSupplier());dto.setAxbSupplier(callSale.getAxbSupplier());dto.setCreateTime(callSale.getCreateTime());dto.setUpdateTime(callSale.getUpdateTime());dto.setBusyChangeTime(callSale.getBusyChangeTime());dto.setRrcCno(callSale.getRrcCno());return dto;}
).collect(Collectors.toList());
还有一个:
for (int i = 0; i < callSaleInfoDtos.size(); i++) {List callSaleDetailList = callSaleService.getDetailByCCid(callSaleInfoDtos.get(i).getId());callSaleInfoDtos.get(i).setCallSaleDetailList(callSaleDetailList);
}
这里遍历了每一个值,然后去查询数据库,这样如果有200个数据,就会跟数据库有200次的IO交互,对数据库的压力很大,并且耗时也很大,改成了下面这种
List idList = callSaleInfoDtos.stream().map(s -> s.getId()).collect(Collectors.toList());
List callSaleDetailList = callSaleService.getDetailByCcIds(idList);
if (CollectionUtils.isEmpty(callSaleDetailList)) {return Response.ok(Collections.emptyList());
}
Map> callSaleDetailMap = callSaleDetailList.stream().collect(Collectors.groupingBy(CallSaleDetail::getCcId));
for (int i = 0; i < callSaleInfoDtos.size(); i++) {List detailList = callSaleDetailMap.getOrDefault(callSaleInfoDtos.get(i).getId(), Collections.emptyList());callSaleInfoDtos.get(i).setCallSaleDetailList(detailList);
}
改成了上面这种,首先拿到id列表,然后再数据库中使用in 查询一次数据库,将结果再使用Lambda 转换成Map,最后再根据map去匹配,在set进去最终的结果。
这里要注意的一点是 mysql对 in 也有要求,有的时候也会不走索引,但是这里最多就几百个,并且表中的总数据也就2000左右,问题不大。
结论:
1、改造前这个接口平均1.5秒,改造后 平均相应300毫秒,提升了5倍
那么问题来了,使用Lambda为啥会这么快呢?这值得研究一下:
java8特性lambda基本原理及性能分析:https://blog.csdn.net/heima201907/article/details/104983485
有时间看一下这篇文章,今天先写到这里