带有迭代器的 ConcurrentModificationException

Question

我一直在编写一个程序，遇到了几个我能够妥善解决的问题。但是，我的程序抛出一个 ConcurrentModificationException，我不确定我能做什么。

我正在使用 NetBeans 8.0.2 编写代码以创建 Java 桌面应用程序（按照我教授的要求）。该计划的目的是管理酒店。所以，它有一些部分，如 "Customers"、"Staff" 和 "Booking"（给我带来问题的那个）。

在每个部分，我都有一个使用 DefaultTableModel 的 JTable。在每一部分中，我都使用硬盘驱动器上的文件来使更改持久化。我编码了新的 customer/staff 类。现在，我正在尝试编写预订和取消预订方法的代码。在我到达 "unbooking" 部分之前，一切都进行得很顺利。

在 table 和模型之后我有一个 ArrayList。在这个特定的部分，我有三个：

一个为整体table
一个只包含免费房间
只有一个已经预订的房间。

预订部分很好，我能够修改 table、文件和 ArrayList 而没有任何错误，而且我的程序实际上做了它应该做的。但我无法让取消预订部分工作。基本上应该是booking的反面。

我将 post 我的代码的一小部分，但如果您需要了解其他任何信息或需要我的代码的更多部分，我很乐意分享它。

我的代码：

public class GestionInstalaciones extends javax.swing.JFrame {
    private final String ruta = System.getProperties().getProperty("user.dir");
    private final File archivo = new File (ruta+"\Instalaciones.txt");
    private final DefaultTableModel modelo = new DefaultTableModel();
    private final ArrayList contenidoInstalaciones;
    private final ArrayList contenidoInstalacionesOcupadas;
    private final ArrayList contenidoInstalacionesLibres;
    public GestionInstalaciones() {
       initComponents ();
       contenidoInstalaciones = new ArrayList();
       contenidoInstalacionesOcupadas = new ArrayList();
       contenidoInstalacionesLibres = new ArrayList();
       //Añadimos las columnas a la tabla.
       modelo.addColumn ("Tipo");
       modelo.addColumn ("Nombre Instalacion");
       modelo.addColumn ("NIF del Ocupante");
       cargarTabla();
}

private void cargarTabla(){
    this.contenidoInstalaciones.clear();
    FileReader fr = null;
    BufferedReader br;
    String tipo;
    String nombre;
    String NIFocupante;
    String[] partes;
    String linea;

    try{
        fr = new FileReader(archivo);
        br = new BufferedReader(fr);

        while ((linea=br.readLine())!=null) {
            //Adding info to general ArrayList
            this.contenidoInstalaciones.add(linea);
            //Splitting line into 3 components.
            partes = linea.split(",",3);
            tipo = partes[0];
            nombre = partes[1];
            NIFocupante = partes[2];

            //Skipping header.
            if ( tipo.equals( "Tipo" )) { continue; }

            //Añadimos la fila al modelo.
            modelo.addRow(partes);
        }
        TablaInstalaciones.setModel(modelo);
    }
    //Capturamos excepciones y cerramos fichero.
    catch(IOException e) {}
    finally { try { if ( null != fr ) { fr.close(); } } catch (IOException e2){ } }
}//end cargarTabla()

private void botonLiberarInstalacionActionPerformed(java.awt.event.ActionEvent evt) {                                                        
    Object linea;
    int contador=0;
    String aux; 
    String tiposATrabajar = "";
    String[] tiposAInsertar;
    Iterator instalacionesOcupadas;

    //Cleaning of already booked ArrayList.
    //this.contenidoInstalacionesOcupadas.clear();

    instalacionesOcupadas = contenidoInstalacionesOcupadas.iterator();
    this.comboTipoALiberar.removeAllItems();
    this.comboTipoALiberar.addItem("Seleccione");

    //Reading the general Table.
    for (int z = 0; z < TablaInstalaciones.getRowCount() ; z++) {
        //The booking parameter is on the 3rd place.
        if(!TablaInstalaciones.getValueAt(z,2).equals("")){
            //Putting the line into the ArrayList for booked rooms..
            linea = TablaInstalaciones.getValueAt(z,0) + "," + TablaInstalaciones.getValueAt(z,1) + "," + TablaInstalaciones.getValueAt(z,2);
            this.contenidoInstalacionesOcupadas.add(linea);
            contador++;
        }
    }

    **//Reading the booked ArrayList to put the right values on the combobox related. 
      //===> THIS LINE IS GIVING THE ERROR !!!
    while(instalacionesOcupadas.hasNext()) {** 
        aux = instalacionesOcupadas.next().toString().split(",",3)[0];
        //Checking to add only 1 kind of each room type.
        if(!tiposATrabajar.contains(aux)); {
            if (tiposATrabajar.equals("")) { tiposATrabajar=aux; }
            else { tiposATrabajar = tiposATrabajar + "," + aux; }
        }
    }      
    //
    tiposAInsertar = tiposATrabajar.split(",");
    //Adding the type into the combobox.
    for (String elemento: tiposAInsertar){ this.comboTipoALiberar.addItem(elemento.replace(",","")); }

}

Answer 1

ConcurrentModificationException 当您在主动循环通过所述链时对元素链（列表）进行更改时会发生。

我经常遇到它并且总是通过反向循环遍历列表来解决它，这样你就可以向后循环并且你可以根据需要更改或删除元素而不会弄乱你正在循环的链（列表）结束了

我会为您提供代码，但我发现很难阅读您的代码，所以现在只能这样做了。

希望这对您有所帮助，祝您好运！

Answer 2

如果您正在迭代的 Collection 的内容自您上次使用迭代器以来已发生更改，则在您尝试使用或重用它时将出现异常。创建一个新的 - 作为一般规则，don't reuse an iterator.

你是：

创建迭代器
修改列表
修改列表后使用迭代器←错误！

您应该在创建迭代器之前修改列表。

此外，您应该尝试 minimize the scope of local variables。

相对于：

Iterator<String> someIterator = strings.iterator();
while (someIterator.hasNext()) {
    doSomething();
}

你可能应该这样做：

for (Iterator<String> iter = strings.iterator(); iter.hasNext();) {
    doSomething();
}

当然，如果您（如您所说）不需要修改列表，请使用 for-each 循环：

for (String s : strings) {
    doSomething();
}

一些不相关的点：

你为什么要写 extends java.awt.JFrame 这样的东西？ Import 并使用 JFrame 代替。
在需要时声明并初始化变量。如果你在你的 for 循环前面初始化了 Iterator only 你就不会遇到这个问题。
使用generics！非泛型 Collection 只会在运行时出错，而泛型会给出编译错误。

带有迭代器的 ConcurrentModificationException

ConcurrentModificationException with Iterators

java

arraylist

concurrency

concurrentmodification