今天在做一个项目的时候,发现了一个问题,我直接放代码:
@Override
public void populate(ProductModel source, StockData stockData) throws ConversionException {try {
ArrayList<FutureStockLevelData> list = new ArrayList<FutureStockLevelData>();
FutureStockLevelData futureStockLevelData = new FutureStockLevelData();
List<StockLevelModel>stocklevels=
acerchemFutureStockservice.getStockLevelModel(source.getCode());
for (StockLevelModel stockLevelModel2 : stocklevels) {//远期库存数量
Integer preOrder = stockLevelModel2.getPreOrder();
//远期库存时间
Integer preOrderReleaseDay = stockLevelModel2.getPreOrderReleaseDay();
if (null!=preOrderReleaseDay&&null!=preOrder) {
//添加到data
futureStockLevelData.setFutureStockLevel(preOrder);
futureStockLevelData.setReleaseDay(preOrderReleaseDay);
list.add(futureStockLevelData);
}
}
stockData.setFutureStock(list);
} catch (Exception e) {
e.printStackTrace();
}
}
当我断点调试的时候就会发现一个很严重的问题,那就是后一个futureStockLevelData实体会不断的覆盖前一个实体,最后导致list集合里面的内容都是一样的!后来我在网上查了,看到有人说是数据类型的区别:Integer是值类型,而上面是实体futureStockLevelData是引用类型!这两者的区别博客园的大牛们都写了很多文章,也很详细,其中我就注意到了这两句:引用类型变量的赋值只复制对象的引用,而不复制对象本身。而将一个值类型变量赋给另一个值类型变量时,将复制包含的值。
我这里的实体是引用类型,也就是说我在for循环赋值的时候,一直都在同一个对象赋值add给List集合。所以不断的给List添加同一个对象,自然会被不断覆盖前有的值,而值类型却没有
所以我就意识到解决这个bug其实只需要把for循环外面定义的实体类转移到for循环内部就可以了,每添加一个实体对象就new一个出来保证不同;其正确的代码如下:
@Override
public void populate(ProductModel source, StockData stockData) throws ConversionException {
try {
ArrayList<FutureStockLevelData> list = new ArrayList<FutureStockLevelData>();
List<StockLevelModel> stocklevels = acerchemFutureStockservice.getStockLevelModel(source.getCode());
for (StockLevelModel stockLevelModel2 : stocklevels) {
//远期库存数量
Integer preOrder = stockLevelModel2.getPreOrder();
//远期库存时间
Integer preOrderReleaseDay = stockLevelModel2.getPreOrderReleaseDay();
if (null!=preOrderReleaseDay&&null!=preOrder) {
//添加到data
FutureStockLevelData futureStockLevelData = new FutureStockLevelData();
futureStockLevelData.setFutureStockLevel(preOrder);
futureStockLevelData.setReleaseDay(preOrderReleaseDay);
list.add(futureStockLevelData);
}
}
stockData.setFutureStock(list);
} catch (Exception e) {
e.printStackTrace();
}
}
----------个人笔记,为了提高自己.