ICode9

精准搜索请尝试: 精确搜索
首页 > 其他分享> 文章详细

一个留了两个星期的bug,基础忘光了

2020-11-06 19:00:59  阅读:219  来源: 互联网

标签:星期 ScaleAndEnterpriseDTO 忘光 dto dtoList copyDtoList o2 copyDto bug


List<ScaleAndEnterpriseDTO> copyDtoList = new ArrayList<>();
        dtoList.sort((o1, o2) -> o2.getCumulativeInvestmentMoney().compareTo(o1.getCumulativeInvestmentMoney()));
        for (int i = 0; i < dtoList.size(); i++) {
            ScaleAndEnterpriseDTO dto = dtoList.get(i);
            dto.setCumulativeNumber(i + 1);
            dto.setCumulativeProvinceName(getMonthLink(dto.getCumulativeProvinceNumber(), month, dto.getCumulativeProvinceName()));
            dto.setBalanceProvinceName(getMonthLink(dto.getBalanceProvinceNumber(), month, dto.getBalanceProvinceName()));
            dto.setCustomerProvinceName(getMonthLink(dto.getCustomerProvinceNumber(), month, dto.getCustomerProvinceName()));
            // 注意不能直接往copyDtoList.add(dto),因为这样会引用的同一个对象,在dtoList里面对这个dto做了修改,copyDtoList里面的这个dto也是同样被修改了
            ScaleAndEnterpriseDTO copyDto = new ScaleAndEnterpriseDTO();
            BeanUtil.copyProperties(dto, copyDto, false);
            copyDtoList.add(copyDto);
        }

        dtoList.sort((o1, o2) -> o2.getInvestmentBalance().compareTo(o1.getInvestmentBalance()));
        for (int i = 0; i < dtoList.size(); i++) {
            ScaleAndEnterpriseDTO copyDto = copyDtoList.get(i);
            ScaleAndEnterpriseDTO dto = dtoList.get(i);
            System.out.println(dto);
            copyDto.setBalanceNumber(i + 1);
            copyDto.setBalanceProvinceName(dto.getBalanceProvinceName());
            copyDto.setBalanceGroup(dto.getBalanceGroup());
            copyDto.setInvestmentBalance(dto.getInvestmentBalance());
            copyDto.setBalanceProvinceNumber(dto.getBalanceProvinceNumber());
//            System.out.println(copyDto);
        }

        dtoList.sort((o1, o2) -> Long.compare(o2.getCustomerPeopleNumber(), o1.getCustomerPeopleNumber()));
        for (int i = 0; i < dtoList.size(); i++) {
            ScaleAndEnterpriseDTO copyDto = copyDtoList.get(i);
            ScaleAndEnterpriseDTO dto = dtoList.get(i);
            copyDto.setCustomerGroup(dto.getCustomerGroup());
            copyDto.setCustomerNumber(i + 1);
            copyDto.setCustomerProvinceName(dto.getCustomerProvinceName());
            copyDto.setCustomerPeopleNumber(dto.getCustomerPeopleNumber());
            copyDto.setCustomerProvinceNumber(dto.getCustomerProvinceNumber());
        }

当时发现在dtoList第二次排序的时候,会出现属性重复的问题,然后debug发现这个问题之后,一直想不通为什么会这样。明明没有对dto做任何set,但是在dtoList的第二次排序之后循环到第二次,dto就会被改变。

今天突然想到之前有过一次这种写法:

List<DTO> copyDtoList = dtoList;

这个当时就注意到引用的问题,然后换成了

List<DTO> copyDtoList = new ArrayList<>(dtoList);

 

标签:星期,ScaleAndEnterpriseDTO,忘光,dto,dtoList,copyDtoList,o2,copyDto,bug
来源: https://www.cnblogs.com/woyujiezhen/p/13938486.html

本站声明: 1. iCode9 技术分享网(下文简称本站)提供的所有内容,仅供技术学习、探讨和分享;
2. 关于本站的所有留言、评论、转载及引用,纯属内容发起人的个人观点,与本站观点和立场无关;
3. 关于本站的所有言论和文字,纯属内容发起人的个人观点,与本站观点和立场无关;
4. 本站文章均是网友提供,不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属;如您发现该文章侵犯了您的权益,可联系我们第一时间进行删除;
5. 本站为非盈利性的个人网站,所有内容不会用来进行牟利,也不会利用任何形式的广告来间接获益,纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。

专注分享技术,共同学习,共同进步。侵权联系[81616952@qq.com]

Copyright (C)ICode9.com, All Rights Reserved.

ICode9版权所有