为什么我的ArrayList包含添加到列表中的最后一项的N个副本?

时间:2013-11-07 18:11:59

标签: java list arraylist static

我正在向ArrayList添加三个不同的对象,但该列表包含我添加的最后一个对象的三个副本。

例如:

for (Foo f : list) {
  System.out.println(f.getValue());
}    

预期:

0
1
2

实际值:

2
2
2

我犯了什么错误?

注意:这是针对本网站上出现的众多类似问题的规范Q& A.

5 个答案:

答案 0 :(得分:121)

此问题有两个典型原因:

  • 您存储在列表中的对象使用的静态字段

  • 意外地将相同的对象添加到列表中

静态字段

如果列表中的对象将数据存储在静态字段中,则列表中的每个对象看起来都是相同的,因为它们具有相同的值。考虑下面的课程:

public class Foo {
  private static int value; 
  //      ^^^^^^------------ - Here's the problem!

  public Foo(int value) {
    this.value = value;
  }

  public int getValue() {
    return value;
  }
}

在该示例中,只有一个int valueFoo的所有实例之间共享,因为它被声明为static。 (参见"Understanding Class Members"教程。)

如果您使用以下代码将多个Foo对象添加到列表中,则每个实例都会在调用3时返回getValue()

for (int i = 0; i < 4; i++) {      
  list.add(new Foo(i));
}

解决方案很简单 - 不要对类中的字段使用static关键字,除非您确实希望在该类的每个实例之间共享值。

添加相同对象

如果将临时变量添加到列表中,则每次循环时都必须创建要添加的对象的新实例。请考虑以下错误的代码段:

List<Foo> list = new ArrayList<Foo>();    
Foo tmp = new Foo();

for (int i = 0; i < 3; i++) {
  tmp.setValue(i);
  list.add(tmp);
}

这里,tmp对象是在循环外部构造的。因此,相同的对象实例将被添加到列表中三次。该实例将保留值2,因为这是在上次调用setValue()期间传递的值。

要解决此问题,只需在循环内移动对象构造:

List<Foo> list = new ArrayList<Foo>();        

for (int i = 0; i < 3; i++) {
  Foo tmp = new Foo(); // <-- fresh instance!
  tmp.setValue(i);
  list.add(tmp);
}

答案 1 :(得分:6)

您的问题是static类型,每次循环迭代时都需要重新初始化。如果你处于循环中,最好将具体初始化保留在循环中。

List<Object> objects = new ArrayList<>(); 

for (int i = 0; i < length_you_want; i++) {
    SomeStaticClass myStaticObject = new SomeStaticClass();
    myStaticObject.tag = i;
    // Do stuff with myStaticObject
    objects.add(myStaticClass);
}

而不是:

List<Object> objects = new ArrayList<>(); 

SomeStaticClass myStaticObject = new SomeStaticClass();
for (int i = 0; i < length; i++) {
    myStaticObject.tag = i;
    // Do stuff with myStaticObject
    objects.add(myStaticClass);
    // This will duplicate the last item "length" times
}

此处tagSomeStaticClass中的变量,用于检查上述代码段的有效性;你可以根据你的用例进行其他实现。

答案 2 :(得分:4)

日历实例遇到同样的问题。

错误的代码:

Calendar myCalendar = Calendar.getInstance();

for (int days = 0; days < daysPerWeek; days++) {
    myCalendar.add(Calendar.DAY_OF_YEAR, 1);

    // In the next line lies the error
    Calendar newCal = myCalendar;
    calendarList.add(newCal);
}

您必须创建日历的新对象,可以使用calendar.clone();

来完成
Calendar myCalendar = Calendar.getInstance();

for (int days = 0; days < daysPerWeek; days++) {
    myCalendar.add(Calendar.DAY_OF_YEAR, 1);

    // RIGHT WAY
    Calendar newCal = (Calendar) myCalendar.clone();
    calendarList.add(newCal);

}

答案 3 :(得分:3)

每次将对象添加到ArrayList时,请确保添加新对象且尚未使用对象。发生的事情是,当您添加相同的1个对象副本时,该相同的对象将添加到ArrayList中的不同位置。当您对其中一个进行更改时,由于反复添加相同的副本,所有副本都会受到影响。 例如, 假设你有一个像这样的ArrayList:

ArrayList<Card> list = new ArrayList<Card>();
Card c = new Card();

现在,如果您将此卡片c添加到列表中,它将被添加没有问题。它将保存在位置0.但是,当您在列表中保存相同的卡片c时,它将保存在位置1.因此请记住,您将相同的1个对象添加到列表中的两个不同位置。现在,如果您对Card对象c进行更改,位于0和1的列表中的对象也将反映该更改,因为它们是同一个对象。

一种解决方案是在Card类中创建一个接受另一个Card对象的构造函数。然后在该构造函数中,您可以设置如下属性:

public Card(Card c){
this.property1 = c.getProperty1();
this.property2 = c.getProperty2(); 
... //add all the properties that you have in this class Card this way
}

并且假设您拥有相同的1份卡片,因此在添加新对象时,您可以这样做:

list.add(new Card(nameOfTheCardObjectThatYouWantADifferentCopyOf));

答案 4 :(得分:2)

使用相同的引用而不是使用新的引用也可能导致后果。

 List<Foo> list = new ArrayList<Foo>();        

 setdata();
......

public void setdata(int i) {
  Foo temp = new Foo();
  tmp.setValue(i);
  list.add(tmp);
}

代替:

List<Foo> list = new ArrayList<Foo>(); 
Foo temp = new Foo();       
setdata();
......

public void setdata(int i) {
  tmp.setValue(i);
  list.add(tmp);
}