コレクションからオブジェクトを削除するときにIllegalStateExceptionが発生する

Question

arrayListを繰り返し処理して、特定の条件に該当する項目を削除しています。

public static ArrayList<String> filteredArrayList(HashMap<String, Boolean> filters){ 

    ArrayList<String> filteredArrayList = new ArrayList<>(); 
    filteredArrayList = fullList(); 

    Iterator<String> i = filteredArrayList.iterator(); 
    while (i.hasNext()) { 
     String s = i.next(); 

     if(!(filters.get("sometextStatus")) && (s.toLowerCase().trim().contains("sometext".trim().toLowerCase()))){ 
       i.remove(); 
     } 
     if(!(filters.get("120Status")) && (s.toLowerCase().trim().contains("120".trim().toLowerCase()))){ 
       i.remove(); 
     } 
    } 
} 

は、なぜ最初の文の仕事ん：

if(!(filters.get("sometextStatus")) && (s.toLowerCase().trim().contains("sometext".trim().toLowerCase()))){ 
    i.remove(); 
} 

しかし、もう一つは失敗しますか？

if(!(filters.get("120Status")) && (s.toLowerCase().trim().contains("120".trim().toLowerCase()))){ 
    i.remove(); 
} 

私が取得しています：スレッドで

例外 "メイン" java.lang.IllegalStateException

を1秒間。

Answer 1

public static ArrayList<String> filteredArrayList(HashMap<String, Boolean> filters){ 

    ArrayList<String> filteredArrayList = fullList(); //Note, that fullList method already creates and returns array, so you don't need to create it before calling this method. 

    Iterator<String> i = filteredArrayList.iterator(); 
    while (i.hasNext()) { 
     String s = i.next(); 

     if(!(filters.get("sometextStatus")) && (s.toLowerCase().trim().contains("sometext".trim().toLowerCase()))){ 
       i.remove(); 
       continue; //We removed current element, so next if check might potentially removed already removed element! 
     } 

     if(!(filters.get("120Status")) && (s.toLowerCase().trim().contains("120".trim().toLowerCase()))){ 
       i.remove(); 
       continue; //Maybe a programmer will decide to copy-paste if checks, so it is safer to do so with continue. 
     } 
    } 
} 

更新

デバッグした後、私は

Exception in thread "main" java.lang.NullPointerException 
    at com.ca.training.jdk.loops.Main2.filteredArrayList(Main2.java:34) 
    at com.ca.training.jdk.loops.Main2.main(Main2.java:14) 

問題を発見できるという

filters.get("120Status") 

かの戻りNULL、あります！？。だから、追加ヌルチェック

Boolean status1 = filters.get("sometextStatus"); 
if((status1 != null && status1) && (s.toLowerCase().trim().contains("sometext".trim().toLowerCase()))) { 
    i.remove(); 
} 

キーワードcontinueプログラムは、新たなループパラメータでループを継続します。したがって、最初のifステートメントがtrueの場合、2番目のifチェックには行かず、次のi値でループ実行を続けます。

アドバイス

利用代わりに子クラスの基底クラスやインタフェースを、それが可能であるところはどこでも。

public static ArrayList<String> filteredArrayList(Map<String, Boolean> filters){ //Note, map 

    List<String> filteredArrayList = fullList(); //Note, list. 

Answer 2

最初のi.remove()は既に要素を削除したと思います。 2番目のチェックも一致すると、2番目のi.remove()はもう使用できなくなり、代わりに例外がスローされます。 i.remove()を呼び出したときはいつでもcontinue;を使用してください。

continueループの次の要素にジャンプします。

例外が発生した場所でコードの正確な行を投稿するのが簡単にわかります。

Answer 3

Java8を多用していた場合は、これが役立つ可能性があります。

私たちはコンテンツを確認しているので、正確な一致ではありません。trim()アイテムはありません。

数値が一致する場合は、lowerCase()も必要ありません。アイテムを削除する

シンプルラムダ：

filteredArrayList.removeIf(s -> 
       s.contains("123") 
      || s.toLowerCase().contains("asd".toLowerCase()) 
      || s.toLowerCase().contains("something else".toLowerCase()) 
      || s.toLowerCase().contains("something else".toLowerCase()) 
     ); 

sは、あなたがそれをはい反復子のアイテムですwant-として、あなたがそれを呼び出すことができ、リスト内のアイテムです。