如何从ArrayList中删除元素？

Question

我已将数据添加到ArrayList ，现在想要更新该列表，从中删除一些元素。

我在CartEntry类型的ArrayList中有类似1,2,3,4的元素。

代码：

ArrayList<CartEntry> items = new ArrayList<CartEntry>();

public void remove(int pId)
{
    System.out.println(items.size());

    for(CartEntry ce : items)
    {
        if(ce.getpId() == pId)
        {
            items.remove(ce);
            //System.out.println(items.get(1));             
        }
    }   
    items.add(new CartEntry(pId));
}

CartEntry代码：

public long getpId() {
    return pId;
}

构造函数：

public CartEntry(long pId) {
    super();
    this.pId = pId;     
}

当我尝试这段代码时，它给了我一个错误：

java.util.ConcurrentModificationException
    at java.util.ArrayList$Itr.checkForComodification(Unknown Source)
    at java.util.ArrayList$Itr.next(Unknown Source)

这里pId是指定应该从项目中删除项目的参数。 假设我要删除包含2个数据的项目，那么我将要做什么？

Answer 1

您正面临ConcurrentModificationException因为您一次在同一list上执行两个操作。 即循环和删除相同的时间。

为了避免这种情况，请使用Iterator，它可以保证您安全地从列表中删除元素。

一个简单的例子看起来像

Iterator<CartEntry> it = list.iterator();
    while (it.hasNext()) {
        if (it.next().getpId() == pId) {
            it.remove();
            break;
        }
    }

Answer 2

您的代码至少有两个问题：

1. 你叫remove您遍历集合，这将导致ConcurrentModificationException ，如果你继续在迭代后remove 。

   有两种方法可以解决这个问题：

   • 找到要删除的对象后停止迭代（只需添加break或return ） 或
   • 从增强的for循环切换到使用Iterator 及其remove方法 。

2. 你在remove方法中添加一个元素，这可能不是你想要的。

所以我会使用这段代码（这假设在列表中只有一个具有给定id的CartEntry ）：

public void remove(int pId)
{
    for(CartEntry ce : items)
    {
        if(ce.getpId() == pId)
        {
            items.remove(ce);
            return;
        }
    }   
}

如果具有唯一ID的假设不正确，那么您将需要使用Iterator方法：

public void remove(int pId)
{
    Iterator<CartEntry> it = items.iterator();
    while(it.hasNext())
    {
        CartEntry ce = it.next();
        if(ce.getpId() == pId)
        {
            it.remove();
        }
    }   
}

Answer 3

你已经创建了一个carEntry类型的Arraylist。 所以你需要创建一个CarEntry类型的迭代器

Iterator<CarEntry> it =  items.iterator();    
while(it.hasNext())
{
   if(it.next().getPId == PId)
   it.remove();
}

Answer 4

在CartEntry中实现.equals然后使用ArrayList.remove（CartEntry）或循环遍历数组列表，找到具有某些条件的项，标记索引，并调用ArrayList.remove（index） - 循环之后

Answer 5

尝试，

public void remove(int pId){

Iterator<CartEntry> it = items.iterator();

 while(it.hasNext()) {
    CartEntry entry = it.next();
    if (entry.getpId() == pId) {
        it.remove();
    }
  }
}

Answer 6

迭代于Iterable<E>或raw Iterable的子类型的Expression的enhanced-for （或for each循环），基本等效于以下形式：

  for (I #i = Expression.iterator(); #i.hasNext(); ) {   
      VariableIdentifiers_opt TargetType Identifier = (TargetType) #i.next();
         Statement
   }

这在jls 14.14.2中有明确说明。 增强的for语句部分。

对于您的上下文， Expression是一个ArrayList 。 ArrayList的迭代器方法返回的迭代器是快速失败的：如果在创建迭代器之后的任何时候对列表进行结构修改，除了通过迭代器自己的remove或add方法之外，迭代器将抛出ConcurrentModificationException 。

使用Iterator代替并使用它自己的remove()方法：

  Iterator<E>iterator = list.iterator();
  while(iterator.hasNext())
    if(iterator.next().equals(E))
      iterator.remove();