在做项目的时候,遇到过一个很容易忽略的问题。就是自己感觉逻辑明明没错的,但是为什么输出结果跟自己想象的有点出入呢?比如下面的代码:(先自己考虑会输出什么,再试试代码会输出什么)
public class ListTest {
public static void main(String[] arg0){
List<User> userList=new ArrayList<User>();
User user=new User();
for(int i=0;i<10;i++){
user.setUserName(i+" ");
userList.add(user);
}
System.out.println(userList.size());
for(int i=0;i<10;i++){
System.out.println(userList.get(i).getUserName());
}
}
}
class User{
private String userName;
public User(){
}
public User(String userName){
this.userName=userName;
}
public String getUserName() {
return userName;
}
public void setUserName(String userName) {
this.userName = userName;
}
}
输出结果如下:
你会发现所有的结果怎么都是最后一个数的重复呢?
我们回去看上面的代码:我们new了一个User对象,就只是new了一个唯一的引用,我们将对象放入list时,list就只是添加了一个对象的引用而已,而我们只是简单的set对象进行对对象的值的改变,但是其引用还是唯一的,list又只是对于对象唯一ID的引用。所以结果就显而易见的。
要改正这个代码,只需在循环中不断new一个User对象就可以了,使其引用不唯一。
下面是改进的代码:
public class ListTest {
public static void main(String[] arg0){
List<User> userList=new ArrayList<User>();
for(int i=0;i<10;i++){
User user=new User();
user.setUserName(i+" ");
userList.add(user);
}
System.out.println(userList.size());
for(int i=0;i<10;i++){
System.out.println(userList.get(i).getUserName());
}
}
}
class User{
private String userName;
public User(){
}
public User(String userName){
this.userName=userName;
}
public String getUserName() {
return userName;
}
public void setUserName(String userName) {
this.userName = userName;
}
}