Многоуровневое наследование и foreach Java loop

Question

У меня была куча табличных представлений, в которых был дублирован код. Они использовали для прямого наследования класс AbstractView. Поэтому я заставил их наследовать AbstractListView (который в свою очередь наследовал бы от AbstractView), который был бы новым классом, содержащим все общие свойства для этих таблиц.

Поскольку я сделал, что изменение этого цикла ведет себя странно:

for (AbstractViewPanel view : registeredViews) { 
     view.modelPropertyChange(evt); 
} 

Вот ArrayList: registeredViews:

private ArrayList<AbstractViewPanel> registeredViews; 

Я запускать отладчик и registeredViews ArrayList имеет все включая таблицы. По какой-то причине он кидается на последний непосредственно унаследованный вид в коллекции и полностью пропускает их.

Все мои взгляды, используемые непосредственно наследовать:

public class someView extends AbstractViewPanel 

Теперь, так как мое изменение некоторых взглядов, как это:

public class someOtherView extends AbstractListView 

Вот AbstractListView:

public abstract class AbstractListView extends AbstractViewPanel 

Вот AbstractViewPanel

public abstract class AbstractViewPanel extends JPanel { 


public abstract void modelPropertyChange(PropertyChangeEvent evt); 

} 

Петля просто не перебирает ничего, что наследует AbstractListView, но я бы предположил, что все виды в конце имеют тип AbstractView.

UPDATE мне удалось найти обходной путь он просто включает в себя изменение цикла:

for (int i = 0; i < registeredViews.size(); i++) 
{ 
     registeredViews.get(i).modelPropertyChange(evt); 
} 

Я хотел бы знать, хотя, почему цикл Еогеасп стиль дает мне головную боль.

Answer 1

Для справки, я создал скелет sscce вашего дизайна. List<E> реализует Iterable<E>, поэтому нет a priori причина, по которой ваш foreach должен потерпеть неудачу. Вот несколько вещь, чтобы проверить:

• Убедитесь, что объекты GUI построены и манипулировать только на event dispatch thread.

• Убедитесь, что вы не с видом на ClassCastException когда экземпляр SomeOtherView извлекается из List.

• Как List неоднородно, см Bloch, Effective Java 2nd ed., пункт 29:.. Рассмотрят типизированные разнородные контейнеры»Узор упоминается here

• Добавление: бы вы не видите одновременное исключение модификации в этом случае?

  Нет, ConcurrentModificationException «не всегда означает, что объект был одновременно модифицирован различной нити.» Исключение выдается, если «список был structurally modified» за пределами итератора, с наилучшей точки зрения. Скорее всего, неудачный итератор был получен после, добавив несколько видов, но до, добавив другие. Это может произойти неожиданно, если список был создан на начальном потоке и повторен по EDT.

SSCCE:

import java.awt.EventQueue; 
import java.awt.GridLayout; 
import java.util.ArrayList; 
import java.util.List; 
import javax.swing.JFrame; 
import javax.swing.JLabel; 
import javax.swing.JPanel; 

public class Test { 

    private List<AbstractViewPanel> registeredViews; 

    public static void main(String[] args) { 
     EventQueue.invokeLater(new Runnable() { 

      @Override 
      public void run() { 
       new Test().init(); 
      } 
     }); 
    } 

    private void init() { 
     JFrame f = new JFrame("Test"); 
     f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     f.setLayout(new GridLayout(0, 1)); 

     registeredViews = new ArrayList<AbstractViewPanel>(); 
     for (int i = 0; i < 2; i++) { 
      registeredViews.add(new SomeView()); 
      registeredViews.add(new SomeOtherView()); 
     } 
     for (AbstractViewPanel view : registeredViews) { 
      f.add(view); 
     } 

     f.pack(); 
     f.setLocationRelativeTo(null); 
     f.setVisible(true); 
    } 

    abstract class AbstractViewPanel extends JPanel { 

     public AbstractViewPanel() { 
      add(new JLabel(getClass().toString())); 
     } 
    } 

    abstract class AbstractListView extends AbstractViewPanel { 
    } 

    class SomeView extends AbstractViewPanel { 
    } 

    class SomeOtherView extends AbstractListView { 
    } 
} 

Answer 2

Я считаю, что ваш modelPropertyChange изменяет registeredViews список.

Поэтому, когда вы запускаете цикл, используя счетчик i, цикл ограничивается размером списка в начале цикла.

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