1.这个坑,是我今天才遇到了,自己把自己坑了,场景是我从数据库A,查出数据,转换成数据库B需要的类型进行保存,发现保存的数据全部一样。
模拟下场景:
public class TestSS { public static void main(String[] args) { List<String> list1 = new ArrayList<String>(); String str = null; List<User> list2 = new ArrayList<User>(); User user = new User(); for(int i=0;i<10;i++) { str = i+""; list1.add(str); user.setStr(str); list2.add(user); } for(String sr :list1) { System.out.print(sr+","); } System.out.println(); for(User us:list2) { System.out.print(us.getStr()+","); } } } class User{ private String str; public String getStr() { return str; } public void setStr(String str) { this.str = str; } }
输出结果如下:
很明显,list2的结果并不是我想要的。
到这里,我想有的人已经知道原因了。但还是简单说下我的分析:
看下内存堆栈图:
User对象内存地址:ad1
User对象的属性str内存地址:ad2
字符串内存地址:ad3
每次循环list1中字符串str每次都指向新的地址;
list2中对象User地址是不变的,因为它声明在循环外面,但是它的属性str每次都改变,
所以在循环结束,list2总对象的属性值都一样,以循环最后一次赋值为准。
所以应该为每次循环定义一个User对象。
这是一个坑,所以应该把User对象的声明放在循环里面。