特定の条件に基づいてArrayListからオブジェクトを削除する

Question

特定の条件を満たす場合、JavaのArrayListから要素を削除したいと考えています。

すなわち：

for (Pulse p : pulseArray) { 
    if (p.getCurrent() == null) { 
     pulseArray.remove(p); 
    } 
} 

これが動作しない理由を私は理解することができますが、これを行うには良い方法は何ですか？

Answer 1

あなたが反復するIteratorを使用する必要がありますし、（ないリストの）イテレータのremove機能：

Iterator#remove関数はoptionnalと言われている。なお

Iterator<Pulse> iter = pulseArray.iterator(); 
while (iter.hasNext()) { 
    Pulse p = iter.next(); 
    if (p.getCurrent()==null) iter.remove(); 
} 

それはでを実現していますArrayListのイテレータ。

はこちらArrayList.javaからこの具体的な機能のコードです：あなたが反復しながら、それを使用する場合、それは例外をスローしない理由

765   public void remove() { 
766    if (lastRet < 0) 
767     throw new IllegalStateException(); 
768    checkForComodification(); 
769 
770    try { 
771     ArrayList.this.remove(lastRet); 
772     cursor = lastRet; 
773     lastRet = -1; 
774     expectedModCount = modCount; 
775    } catch (IndexOutOfBoundsException ex) { 
776     throw new ConcurrentModificationException(); 
777    } 
778   } 
779 
780   final void checkForComodification() { 
781    if (modCount != expectedModCount) 
782     throw new ConcurrentModificationException(); 
783   } 
784  } 

expectedModCount = modCount;線です。

Answer 2

コレクションのメソッドを使用して、反復処理中のコレクションを変更することはできません。ただし、イテレータ（ArrayListのイテレータを含む）の中には、イテレートする順序でメソッドを削除できるようにするremove()メソッドがあります。

Iterator<Pulse> iterator = pulseArray.iterator(); 
while (iterator.hasNext()) { 
    Pulse p = iterator.next(); 
    if (p.getCurrent() == null) { 
    iterator.remove(); 
    } 
} 

Answer 3

あなたは同じリストから要素を削除するある場合は、インデックスが乱れます。別々に以下のように少しを試してみてください：Iteratorを使用すると、あなたのイテレータを使用する代替としてのArrayListを通じて

Answer 4

、あなたはGuavaコレクションライブラリを使用することができます。 （あなたがそういったことにしている場合）これはfunctional以上であることの利点を持っていない：

Predicate<Pulse> hasCurrent = new Predicate<Pulse>() { 
    @Override public boolean apply(Pulse input) { 
    return (input.getCurrent() != null); 
    } 
}; 

pulseArray = Lists.newArrayList(Collections2.filter(pulseArray, hasCurrent)); 

Answer 5

を反復しながら、リストを変更する力を与えるだろう

for (int i=0; i < pulseArray.size(); i++) { 
    Pulse p = (Pulse)pulseArray.get(i); 
    if (p.getCurrent() == null) { 
     pulseArray.remove(p); 
     i--;//decrease the counter by one 
    } 
    } 

Answer 6

イテレータを使用する必要が。 Java 8（ストリーミングとフィルタリング機能とlambda）を使用すると、1行で実現できます。 たとえば、あなたが指定した操作が可能になるん必要なコード：

pulseArray = pulseArray.stream().filter(pulse -> pulse != null).collect(Collectors.toList()); 

Answer 7

あなたがここに、Collection::removeIf(Predicate filter)を使用することができますが、簡単な例です：

final Collection<Integer> list = new ArrayList<>(Arrays.asList(1, 2)); 
list.removeIf(value -> value < 2); 
System.out.println(list); // outputs "[2]"