今天在做项目的时候突然发现了系统的一个bug,当数据库使用longtext的时候,后端使用string类型接收,然后使用arraylist存储数据,然后就发生oom了,java.lang.OutOfMemoryError: Java heap space,我的本地环境中是没有问题的,但是发布到线上的uat环境中就出现了oom,可能是线上的java参数配置的问题,但是更多的还是代码层面的问题,下面我还原一下具体情况。
1.出现oom的代码如下
NewsPageCondition newsPageCondition = new NewsPageCondition();
Long userId = SessionContext.getSessionUserInfo().getUserId();
newsPageCondition.setUserId(userId);
newsPageCondition.setReadFlag(ReadFlag.UNREAD);
UserExt userExt = userService.getUserExt(userId);
List<News> newsList = dao.find(News.class, newsPageCondition);
ArrayList<NewsDTO> list = CollUtil.newArrayList();
for (News news : newsList) {
NewsDTO newsDto = new NewsDTO();
BeanUtil.copyProperties(news, newsDto);
if (EmptyUtil.isNotEmpty(userExt)==true){
newsDto.setHeadPic(userExt.getHeadPic());
}
list.add(newsDto);
}
return list;
oom具体原因:NewsDTO中存储了一个headPic,是一个String类型,它对应数据库中的longtext,longtext的内容实在太多,无法提供给大家看,大概长度是16w-60w
可以看出图片地址是非常的长,也就是longtext非常的长,而如果每条记录都有这么一个数据,那么整个arraylist的大小可想而知,我的业务中,其实所有数据的头像都是一样的,所以说上面这个代码其实很冗余,当然这个代码不是我写的,我也是中途接手的,所以后来就将代码改为如下不会出现oom的版本
2.不会oom的版本
将list改为map,然后把头像字符串单独存储,那么就存储一份,而之前上面的就不知存储一份,而是看循环执行多少次,再次重申,因为我的业务中头像其实都是登陆用户的头像,所以都是同一张,没必要每个对象都存储一次,所以直接提出来。
Map<String,Object> map = new HashMap<>();
NewsPageCondition newsPageCondition = new NewsPageCondition();
Long userId = SessionContext.getSessionUserInfo().getUserId();
newsPageCondition.setUserId(userId);
newsPageCondition.setReadFlag(ReadFlag.UNREAD);
UserExt userExt = userService.getUserExt(userId);
if (EmptyUtil.isNotEmpty(userExt)==true){
map.put("headPic",userExt.getHeadPic());
}
List<News> newsList = dao.find(News.class, newsPageCondition);
ArrayList<NewsDTO> list = CollUtil.newArrayList();
for (News news : newsList) {
NewsDTO newsDto = new NewsDTO();
BeanUtil.copyProperties(news, newsDto);
list.add(newsDto);
}
map.put("list",newsList);
return map;
跟大家分享一下出现oom的一种情景。