Вложенные итераторы

Question

Я пытаюсь создать игру Space Invaders и разработал этот раздел кода, чтобы все инопланетяне выпадали, когда один из них попал на край экрана, однако я получаю довольно неприятное сообщение об ошибке когда он попадает в строку Enemy de = downwardIterator.next();. Игра по-прежнему работает так, как должна, но она отключает эти сообщения об ошибках в фоновом режиме, когда она попадает в эту строку. Вот код:

Iterator<Enemy> iterator = enemyList.iterator(); 
    while (iterator.hasNext()) { 
     Enemy e = iterator.next(); 

     if(e!=null && e.isActive()){ 
      e.move(); 
      e.draw(g); 
      if(e.edgeHit()){ 
       Iterator<Enemy> downwardIterator = enemyList.iterator(); 
       while (iterator.hasNext()) { 
        Enemy de = downwardIterator.next(); 
        de.dropPosition(); 
        de.changeDirection(); 
       } 
      } 
     } 
     else{ 
      iterator.remove(); 
     } 
    } 

e.move просто заставляет врагов двигаться влево и вправо. e.edgeHit обнаруживает, когда любой враг достигает края экрана игры.

Это полный след, когда она попадает, что линия:

Exception in thread "AWT-EventQueue-0" java.util.NoSuchElementException 
    at java.util.ArrayList$Itr.next(ArrayList.java:834) 
    at GamePanel.paintComponent(GamePanel.java:96) 
    at javax.swing.JComponent.paint(JComponent.java:1054) 
    at javax.swing.JComponent.paintToOffscreen(JComponent.java:5219) 
    at javax.swing.RepaintManager$PaintManager.paintDoubleBuffered(RepaintManager.java:1529) 
    at javax.swing.RepaintManager$PaintManager.paint(RepaintManager.java:1452) 
    at javax.swing.RepaintManager.paint(RepaintManager.java:1249) 
    at javax.swing.JComponent._paintImmediately(JComponent.java:5167) 
    at javax.swing.JComponent.paintImmediately(JComponent.java:4978) 
    at javax.swing.RepaintManager$3.run(RepaintManager.java:808) 
    at javax.swing.RepaintManager$3.run(RepaintManager.java:796) 
    at java.security.AccessController.doPrivileged(Native Method) 
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76) 
    at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:796) 
    at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:769) 
    at javax.swing.RepaintManager.prePaintDirtyRegions(RepaintManager.java:718) 
    at javax.swing.RepaintManager.access$1100(RepaintManager.java:62) 
    at javax.swing.RepaintManager$ProcessingRunnable.run(RepaintManager.java:1677) 
    at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:251) 
    at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:733) 
    at java.awt.EventQueue.access$200(EventQueue.java:103) 
    at java.awt.EventQueue$3.run(EventQueue.java:694) 
    at java.awt.EventQueue$3.run(EventQueue.java:692) 
    at java.security.AccessController.doPrivileged(Native Method) 
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76) 
    at java.awt.EventQueue.dispatchEvent(EventQueue.java:703) 
    at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:242) 
    at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:161) 
    at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:150) 
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:146) 
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:138) 
    at java.awt.EventDispatchThread.run(EventDispatchThread.java:91) 

Answer 1

Ваш второй цикл итератора неверен.

Iterator<Enemy> downwardIterator = enemyList.iterator(); 
while (iterator.hasNext()) { 
    // ... 

должен быть

Iterator<Enemy> downwardIterator = enemyList.iterator(); 
while (downwardIterator.hasNext()) { 
    // ... 

Редактировать

Вы могли бы предпочесть for-each loop (который использует скрытый итератор) как

for (Enemy de : enemyList) { 
    // ... 

Answer 2

В случае, если вы хотите объяснить, вы вызываете метод .next() объект г downwardIterator и ваш цикл, пока не проверяют правильный объект итератора, чтобы увидеть, если есть какие-либо элементы остается:

Iterator<Enemy> downwardIterator = enemyList.iterator(); 
while (iterator.hasNext()) { // Check downwardIterator instead of iterator 
    Enemy de = downwardIterator.next(); 
    de.dropPosition(); 
    de.changeDirection(); 
} 

Кроме того, когда вашим первым, если условие выполнено, вы можете использовать расширенные цикл для перебора над вашим противником, если вражеский список не изменяется.