ArrayListから要素を反復して削除するときにjava.util.ConcurrentModificationExceptionを回避する方法

繰り返し処理したいArrayListがあります。それを繰り返しながら、同時に要素を削除する必要があります。明らかにこれはをスローしjava.util.ConcurrentModificationExceptionます。

この問題を処理するためのベストプラクティスは何ですか？最初にリストを複製する必要がありますか？

ループ自体ではなく、コードの別の部分の要素を削除します。

私のコードは次のようになります：

public class Test() {
    private ArrayList<A> abc = new ArrayList<A>();

    public void doStuff() {
        for (A a : abc) 
        a.doSomething();
    }

    public void removeA(A a) {
        abc.remove(a);
    }
}

a.doSomething呼び出すかもしれませんTest.removeA()。

2つのオプション：

• 削除する値のリストを作成し、ループ内でそのリストに追加してoriginalList.removeAll(valuesToRemove)、最後に呼び出します
• remove()イテレータ自体のメソッドを使用します。これは、拡張forループを使用できないことを意味することに注意してください。

2番目のオプションの例として、長さが5より大きい文字列をリストから削除します。

List<String> list = new ArrayList<String>();
...
for (Iterator<String> iterator = list.iterator(); iterator.hasNext(); ) {
    String value = iterator.next();
    if (value.length() > 5) {
        iterator.remove();
    }
}

ArrayListのJavaDocsから

このクラスのiteratorメソッドとlistIteratorメソッドによって返されるイテレータはフェイルファストです。イテレータの作成後、リストが構造的にいつでも変更された場合、イテレータ自身の削除または追加メソッド以外の方法で、イテレータはConcurrentModificationExceptionをスローします。

高度な "forループ"でリストから値を削除しようとしていますが、これは（コードで行った）トリックを適用しても不可能です。より良い方法は、ここでアドバイスされているようにイテレータレベルをコーディングすることです。

人々が伝統的なforループのアプローチをどのように提案していないのだろうか。

for( int i = 0; i < lStringList.size(); i++ )
{
    String lValue = lStringList.get( i );
    if(lValue.equals("_Not_Required"))
    {
         lStringList.remove(lValue);
         i--; 
    }  
}

これも機能します。

あなたは本当に伝統的な方法でアレイを反復するだけです

リストから要素を削除するたびに、後の要素が押し進められます。反復する要素以外の要素を変更しない限り、次のコードは機能するはずです。

public class Test(){
    private ArrayList<A> abc = new ArrayList<A>();

    public void doStuff(){
        for(int i = (abc.size() - 1); i >= 0; i--) 
            abc.get(i).doSomething();
    }

    public void removeA(A a){
        abc.remove(a);
    }
}

Java 8では、コレクションインターフェースを使用して、removeIfメソッドを呼び出すことでこれを行うことができます。

yourList.removeIf((A a) -> a.value == 2);

詳細については、こちらをご覧ください

通常の方法でループを実行しjava.util.ConcurrentModificationExceptionます。これは、アクセスされる要素に関連するエラーです。

だから試してください：

for(int i = 0; i < list.size(); i++){
    lista.get(i).action();
}

リストを繰り返しながら、要素を削除したい場合は可能です。以下の私の例を見てみましょう、

ArrayList<String>  names = new ArrayList<String>();
        names.add("abc");
        names.add("def");
        names.add("ghi");
        names.add("xyz");

上記の配列リストの名前があります。上記のリストから「def」名を削除したいのですが、

for(String name : names){
    if(name.equals("def")){
        names.remove("def");
    }
}

上記のコードは、反復中にリストを変更しているため、ConcurrentModificationException例外をスローします。

したがって、この方法でArraylistから「def」名を削除するには、

Iterator<String> itr = names.iterator();            
while(itr.hasNext()){
    String name = itr.next();
    if(name.equals("def")){
        itr.remove();
    }
}

上記のコードでは、イテレータを使用してArraylistから "def"名を削除し、配列を出力しようとすると、次の出力が表示されます。

出力：[abc、ghi、xyz]

1つのオプションは、removeAメソッドをこれに変更することです-

public void removeA(A a,Iterator<A> iterator) {
     iterator.remove(a);
     }

しかし、これはあなたが意味するであろうdoSomething()合格することができるはずiteratorにremove方法。あまり良い考えではありません。

あなたは、リストを反復処理するときの代わりに選択した要素を除去する、最初のループでは、：あなたは、2つの段階のアプローチでこれを行うことができますマークとして、それらを削除します。このため、これらの要素を簡単にコピー（浅いコピー）して別のList。

次に、反復が完了したらremoveAll、最初のリストから2番目のリストのすべての要素を実行します。

これは、別のリストを使用して削除するオブジェクトを追加し、その後stream.foreachを使用して元のリストから要素を削除する例です。

private ObservableList<CustomerTableEntry> customersTableViewItems = FXCollections.observableArrayList();
...
private void removeOutdatedRowsElementsFromCustomerView()
{
    ObjectProperty<TimeStamp> currentTimestamp = new SimpleObjectProperty<>(TimeStamp.getCurrentTime());
    long diff;
    long diffSeconds;
    List<Object> objectsToRemove = new ArrayList<>();
    for(CustomerTableEntry item: customersTableViewItems) {
        diff = currentTimestamp.getValue().getTime() - item.timestamp.getValue().getTime();
        diffSeconds = diff / 1000 % 60;
        if(diffSeconds > 10) {
            // Element has been idle for too long, meaning no communication, hence remove it
            System.out.printf("- Idle element [%s] - will be removed\n", item.getUserName());
            objectsToRemove.add(item);
        }
    }
    objectsToRemove.stream().forEach(o -> customersTableViewItems.remove(o));
}

For eachループを使用する代わりに、通常のforループを使用します。たとえば、次のコードは、java.util.ConcurrentModificationExceptionを指定せずに配列リストのすべての要素を削除します。ユースケースに応じてループの条件を変更できます。

   for(int i=0;i<abc.size();i++)  {

          e.remove(i);
        }

次のように簡単な処理を行います。

for (Object object: (ArrayList<String>) list.clone()) {
    list.remove(object);
}

ストリームを使用した代替のJava 8ソリューション：

        theList = theList.stream()
            .filter(element -> !shouldBeRemoved(element))
            .collect(Collectors.toList());

Java 7では、代わりにGuavaを使用できます。

        theList = FluentIterable.from(theList)
            .filter(new Predicate<String>() {
                @Override
                public boolean apply(String element) {
                    return !shouldBeRemoved(element);
                }
            })
            .toImmutableList();

Guavaの例では不変のリストが生成されることに注意してください。

ArrayListの代わりにCopyOnWriteArrayListを使用することもできます。これは、JDK 1.5以降の最新の推奨アプローチです。

私の場合、受け入れられた回答が機能していません。例外は停止しますが、リストに不整合が生じます。次の解決策は私にとって完璧に機能しています。

List<String> list = new ArrayList<>();
List<String> itemsToRemove = new ArrayList<>();

for (String value: list) {
   if (value.length() > 5) { // your condition
       itemsToRemove.add(value);
   }
}
list.removeAll(itemsToRemove);

このコードでは、削除するアイテムを別のリストに追加し、list.removeAllメソッドを使用して必要なアイテムをすべて削除しました。

「最初にリストを複製する必要がありますか？」

これが最も簡単な解決策であり、クローンから削除し、削除後にクローンをコピーして戻します。

私のrummikubゲームの例：

SuppressWarnings("unchecked")
public void removeStones() {
  ArrayList<Stone> clone = (ArrayList<Stone>) stones.clone();
  // remove the stones moved to the table
  for (Stone stone : stones) {
      if (stone.isOnTable()) {
         clone.remove(stone);
      }
  }
  stones = (ArrayList<Stone>) clone.clone();
  sortStones();
}

リストからすべての要素を削除することが目的である場合は、各項目を反復処理してから、次を呼び出します。

list.clear()

私は遅れて到着しますが、この解決策はシンプルでエレガントだと思うので、これに答えます。

List<String> listFixed = new ArrayList<String>();
List<String> dynamicList = new ArrayList<String>();

public void fillingList() {
    listFixed.add("Andrea");
    listFixed.add("Susana");
    listFixed.add("Oscar");
    listFixed.add("Valeria");
    listFixed.add("Kathy");
    listFixed.add("Laura");
    listFixed.add("Ana");
    listFixed.add("Becker");
    listFixed.add("Abraham");
    dynamicList.addAll(listFixed);
}

public void updatingListFixed() {
    for (String newList : dynamicList) {
        if (!listFixed.contains(newList)) {
            listFixed.add(newList);
        }
    }

    //this is for add elements if you want eraser also 

    String removeRegister="";
    for (String fixedList : listFixed) {
        if (!dynamicList.contains(fixedList)) {
            removeResgister = fixedList;
        }
    }
    fixedList.remove(removeRegister);
}

これはすべて、1つのリストから別のリストへの更新用であり、1つのリストからすべてを作成できます。メソッドの更新では、両方のリストを確認して、リスト間に要素を消去または追加できます。これは、両方のリストが常に同じサイズであることを意味します

配列リストの代わりにイテレーターを使用する

型が一致するセットをイテレータに変換する

そして次の要素に移動して削除します

Iterator<Insured> itr = insuredSet.iterator();
while (itr.hasNext()) { 
    itr.next();
    itr.remove();
}

要素を削除するにはインデックスが必要なので、次への移動はここで重要です。

についての何

import java.util.Collections;

List<A> abc = Collections.synchronizedList(new ArrayList<>());

ArrayList.remove（A）ステートメントの後にブレークを追加するだけです