JAVA - 反復リストから非イテレータ要素を削除する

Question

このiteratorループを実行中に問題が発生します。このiteratorループでは、this.envの各要素を調べますが、このリスト内では、そのリストの別の要素を削除します。上記の反復リストの項目を削除しようとすると、私はこのエラーを受け取ります：java.util.ConcurrentModificationExceptionこれは、iterator.remove（）を使用せずに反復リストを変更したことが原因であると理解しています。

コード：

public void envActions(IOHandler ioHandler, PlayerClass player){ 
    Iterator<WorldElement> worldElementIterator = this.env.iterator(); 
    while(worldElementIterator.hasNext()){ 
     WorldElement worldElement = worldElementIterator.next(); 
    //for(WorldElement worldElement:this.env){ 
     if(worldElement instanceof EntityClass){ 
      EntityClass entity=(EntityClass) worldElement; 
      if(entity.nature.contains("hostile")){ 
       MonsterClass mEntity=(MonsterClass) entity; 
       if(!(mEntity.attacks.size()*(Math.random()+0.25)>=mEntity.attacks.size())){ 
        Double followerNum = (Math.random()*player.followers.size()); 
        Integer followerNumInt=followerNum.intValue(); 
        if(followerNumInt<2){ 
         PlayerClass target=player; 
         Double attacknumD=mEntity.attacks.size()*Math.random(); 
         Integer attacknum= attacknumD.intValue(); 
         Integer playerarmor=player.getArmorValue(); 
         int enemydamage=mEntity.attacks.get(attacknum).getDamage()*(1-(playerarmor/1000)); 
         target.health=target.health-enemydamage; 
         ioHandler.printToConsole("\nThe "+mEntity.name+" attacked you with "+mEntity.attacks.get(attacknum).getAttack()+" and did "+mEntity.attacks.get(attacknum).getDamage()+" damage! you have "+player.health+" health left!"); 
        } else { 
         FriendlyCreatureClass target=player.followers.get(followerNumInt); 
         Double attacknumD=mEntity.attacks.size()*Math.random(); 
         Integer attacknum= attacknumD.intValue(); 
         int enemydamage=mEntity.attacks.get(attacknum).getDamage(); 
         target.health=target.health-enemydamage; 
         if(!target.isAlive()){ 
          ioHandler.printToConsole("\nThe " + mEntity.name + " attacked " + target.name + " with " + mEntity.attacks.get(attacknum).getAttack() + " and did " + mEntity.attacks.get(attacknum).getDamage() + " damage! " + target.name+" died! Farewell "+target.name+"."); 
          target.died(ioHandler, this, player, true); 
          //>>>> THIS IS WHERE I WOUlD LIKE TO REMOVE 'target' FROM THE env LIST <<<< 
         } else { 
          ioHandler.printToConsole("\nThe "+mEntity.name+" attacked "+target.name+" with "+mEntity.attacks.get(attacknum).getAttack()+" and did "+mEntity.attacks.get(attacknum).getDamage()+" damage! "+target.name+" has "+target.health+" health left!"); 
         } 
        } 
       } 

任意の提案は大歓迎しているが、私は、唯一のJava/Androidので初心者ですから、自分のコーディングスキルを持つ慈悲を持っていてください！

ありがとうございます！

Answer 1

Iteratorを何らかの理由で使用していない場合は、コレクションを反復処理するときにSetで削除する要素を収集し、ループの後でSetを反復処理して要素を元のコレクションから削除できます。

これは、小規模なコレクションでのみ有効です。

HashSet toDelete = new HashSet(); 

for (...) { 
    if (...) 
     toDelete.add(item); 
} 
// end for 

foreach (item in toDelete) { 
    // delete from original collection 
} 

それ以外の場合は、イテレータでremove（）を呼び出すことができます。

Answer 2

これは、toDeleteリストを追加して、反復アイテムが既にリストにあるかどうかをチェックしている場合はそれをスキップし、その後削除することで動作すると思います。

私はバグを完全にチェックしていませんが、今のところは問題ありません。 ワーキングコード：高速な応答のための

public void envActions(IOHandler ioHandler, PlayerClass player){ 
    List<WorldElement> toDelete=new ArrayList<>(); 
    Iterator<WorldElement> worldElementIterator = this.env.iterator(); 
    while(worldElementIterator.hasNext()){ 
     WorldElement worldElement = worldElementIterator.next(); 
    //for(WorldElement worldElement:this.env){ 
     if(worldElement instanceof EntityClass && !toDelete.contains(worldElement)){ 
      EntityClass entity=(EntityClass) worldElement; 
      if(entity.nature.contains("hostile")){ 
       MonsterClass mEntity=(MonsterClass) entity; 
       if(!(mEntity.attacks.size()*(Math.random()+0.25)>=mEntity.attacks.size())){ 
        Double followerNum = (Math.random()*player.followers.size()); 
        Integer followerNumInt=followerNum.intValue(); 
        if(followerNumInt<2){ 
         PlayerClass target=player; 
         Double attacknumD=mEntity.attacks.size()*Math.random(); 
         Integer attacknum= attacknumD.intValue(); 
         Integer playerarmor=player.getArmorValue(); 
         int enemydamage=mEntity.attacks.get(attacknum).getDamage()*(1-(playerarmor/1000)); 
         target.health=target.health-enemydamage; 
         ioHandler.printToConsole("\nThe "+mEntity.name+" attacked you with "+mEntity.attacks.get(attacknum).getAttack()+" and did "+mEntity.attacks.get(attacknum).getDamage()+" damage! you have "+player.health+" health left!"); 
        } else { 
         FriendlyCreatureClass target=player.followers.get(followerNumInt); 
         Double attacknumD=mEntity.attacks.size()*Math.random(); 
         Integer attacknum= attacknumD.intValue(); 
         int enemydamage=mEntity.attacks.get(attacknum).getDamage(); 
         target.health=target.health-enemydamage; 
         if(!target.isAlive()){ 
          ioHandler.printToConsole("\nThe " + mEntity.name + " attacked " + target.name + " with " + mEntity.attacks.get(attacknum).getAttack() + " and did " + mEntity.attacks.get(attacknum).getDamage() + " damage! " + target.name + " died! Farewell " + target.name + "."); 
          target.died(ioHandler, this, player, false); 
          toDelete.add(target); 
         } else { 
          ioHandler.printToConsole("\nThe "+mEntity.name+" attacked "+target.name+" with "+mEntity.attacks.get(attacknum).getAttack()+" and did "+mEntity.attacks.get(attacknum).getDamage()+" damage! "+target.name+" has "+target.health+" health left!"); 
         } 
        } 
       } 
    for(WorldElement worldElement:toDelete){ 
     this.env.remove(worldElement); 
    } 
    return; 

ありがとう！