反復的削除時に発生するConcurrentModificationExceptionのスマートな回避

質問

次のコードのように反復中にそのコレクションを削除しようとすると、ConcurrentModificationExceptionが発生してしまいます。

Set<Integer> set = new HashSet<Integer>();
for (int i = 0 ; i < 10 ; i++) {
　set.add(i);
}
// このループで例外発生
for (Integer currValue : set) {
　if (currValue % 2 == 0) {
　　set.remove(currValue);
　}
}
for (Integer currValue : set) {
　System.out.println(currValue + " ");
}

これを回避するために、反復中に削除せず、削除対象をまず保持しておき、その後でまとめて削除するという方法です。

Set<Integer> set = new HashSet<Integer>();
Set<Integer> removeReserve = new HashSet<Integer>();
for (int i = 0 ; i < 10 ; i++) {
　set.add(i);
}
for (Integer currValue : set) {
　if (currValue % 2 == 0) {
　　removeReserve.add(currValue);
　}
}
for (Integer currValue : removeReserve) {
　set.remove(currValue);
}
for (Integer currValue : set) {
　System.out.println(currValue + " ");
}

しかしこの方法だと、ループがもう一度増えてしまい、なんだか気持ちよくありません。
そこで、もしももっとよい解決策の考え方があれば、教えて頂きたいのです。
「この方法以外ないと思う」という意見もほしいです。
その場合は、上記の解決策にしようと思います。

回答No.1

sakusaker7 · ベストアンサー

Iteratorを使うように書き換えるよりないのでは?

For-Each ループ
http://java.sun.com/j2se/1.5.0/ja/docs/ja/guide/language/foreach.html

for-each ループは好きなときに使えます。使用すればコードがわかりやすくなりますが、どこででも使用できるわけではありません。たとえば、expurgate メソッドについて考えてみましょう。現在の要素を削除するには、反復子にアクセスする必要があります。for-each ループでは反復子が隠されるため、remove メソッドを呼び出すことができません。そのため、for-each ループをフィルタリングに使用することはできません。同様に、リストや配列内の要素をたどりながら置き換えるときには for ループを使用できません。

お礼

・・・そういえばIteratorにremoveメソッドが用意されていましたね。
最近forばかり使っており、Iteratorのremoveメソッドを実際に使ったことがなかったので、その存在をすっかり忘れておりました。

おかげ様で変なコードの実装を避けることができます。
ありがとうございました。