Помогите Проблема с потоками (ConcurrentModificationException)

[saharin94]

В общем есть плагин, который не даёт использовать нескольким игрокам болк одновременно (защита от кражи в ресурсообменниках). Всё бы хорошо, но периодически в консоли вылетает ConcurrentModificationException. Слышал об операторе synchronized, но не могу ума приложить где и как правильно (что в аргумент передавать) его использовать.

Фрагмент кода плагина:

public class DoubleInteract implements Listener {
  
  private HashMap<Block, Player>lockedBlocks;
  
  private void initCloseInvListener(){
    main.protocolManager.addPacketListener(new PacketAdapter(this.main, ConnectionSide.CLIENT_SIDE, ListenerPriority.HIGHEST, new Integer[]{Integer.valueOf(101)}){
      @Override
      public void onPacketReceiving(PacketEvent e){
         if(! config.doubleInteractEnable) return;
         final Player player = e.getPlayer();
         if(player == null) return;
         //synchronized(lockedBlocks){
           removePlayerLock(player); // удаляем из списка. Здесь вылетает исключение.
         //}
      }
    });
  }
  
   private void removePlayerLock(Player player){
     if(lockedBlocks.containsValue(player)){
       for(Block checkBlock : lockedBlocks.keySet()){
         if(player == lockedBlocks.get(checkBlock)){
           lockedBlocks.remove(checkBlock);
         }
       }
     }    
   }
  
  @EventHandler(priority = EventPriority.HIGH, ignoreCancelled = true)
  public void onPlayerInteract(PlayerInteractEvent e) {
  if(e.getAction() != Action.RIGHT_CLICK_BLOCK) return;
  lockedBlocks.put(block, player); // вносим в список
  }

}

 

[Den_Abr]

       for(Block checkBlock : lockedBlocks.keySet()){
         if(player == lockedBlocks.get(checkBlock)){
           lockedBlocks.remove(checkBlock);
         }
       }

}

Вы не можете удалять элементы коллекции во время её итерации

 

[serega6531]

Можно сделать это, используя Iterator.delete().

 

[saharin94]

Можно сделать это, используя Iterator.delete().

И где его нужно использовать?

 

[serega6531]

И где его нужно использовать?

Нужно делать итерацию через итератор, а не через for( : ), и в нужный момент использовать iter.remove().

 

[ptnk]

Нужно делать итерацию через итератор, а не через for( : ), и в нужный момент использовать iter.remove().

Я тебе скажу по секрету, что в for( : ) идёт обход тоже через итератор.

 

[serega6531]

Я тебе скажу по секрету, что в for( : ) идёт обход тоже через итератор.

Я и не спорю, но обращаться к нему нельзя же.

 

[ptnk]

Я и не спорю, но обращаться к нему нельзя же.

Я всего лишь указал на неточность формулировки.

 

[Reality_SC]

Создай новый пустой список до итерирования, и заполняй его ключами в цикле тем, что хочешь удалить.
После цикла сделаешь ещё один цикл удаления.

        ArrayList<Block> removeThem = new ArrayList<>();
        for(Block checkBlock : lockedBlocks.keySet())
            if(player == lockedBlocks.get(checkBlock))
                removeThem.add(checkBlock);
        for(Block r : removeThem)
            lockedBlocks.remove(r);

 

[serega6531]

Создай новый пустой список до итерирования, и заполняй его ключами в цикле тем, что хочешь удалить.
После цикла сделаешь ещё один цикл удаления.

        ArrayList<Block> removeThem = new ArrayList<>();
        for(Block checkBlock : lockedBlocks.keySet())
            if(player == lockedBlocks.get(checkBlock))
                removeThem.add(checkBlock);
        for(Block r : removeThem)
            lockedBlocks.remove(r);

Зачем такие костыли?

 

[Shevchik]

Создай новый пустой список до итерирования, и заполняй его ключами в цикле тем, что хочешь удалить.
После цикла сделаешь ещё один цикл удаления.

        ArrayList<Block> removeThem = new ArrayList<>();
        for(Block checkBlock : lockedBlocks.keySet())
            if(player == lockedBlocks.get(checkBlock))
                removeThem.add(checkBlock);
        for(Block r : removeThem)
            lockedBlocks.remove(r);

Да вы просто мастер вредных советов.

 

[Reality_SC]

Да вы просто мастер вредных советов.

Да, наверное Почему бы и нет

 

[kirill2011s]

В крайнем случае можно перебирать созданием счетчика:

for (int i = 0; i < size; i++) {
И тут перебор с удалением.
}

 

[saharin94]

В крайнем случае можно перебирать созданием счетчика:

for (int i = 0; i < size; i++) {
И тут перебор с удалением.
}

:facepalm:

 

[JustBlender]

:facepalm:

А почему бы и нет?

 

[Shevchik]

А почему бы и нет?

Ну ну. Попробуй.

 

[JustBlender]

Ну ну. Попробуй.

Если я это и использовал с листами, проблем не было никаких.

 

[Shevchik]

Если я это и использовал с листами, проблем не было никаких.

И где ты тут лист видишь?

 

[JustBlender]

И где ты тут лист видишь?

Да йобана, не заметил.