同步方法中的并发修改异常

Question

我对我的多线程程序进行了一些修改，并开始在对象的HashTable获取ConcurrentModificationException ，因此我将对此HashTable所有访问都转换为synchronised方法，以避免对我的对象进行并发访问，这与我的期望并没有解决问题。 以下是使用的设计概述：

首先我有一个JTable显示的USB设备的一些数据，这个JTable请求实现通信，核心对象Observer （一读取和写入数据到我的设备）通过一个Observable的对象就是了中的数据，它想要的数据取决于显示哪些行，以便在用户滚动de table时更新，这个通信核心对象通过synchronized update方法获取此通知，该方法调用更新HashTable其他synchronized方法，该方法保存最后读取值以供使用后（当通过其他Observable对象读取值时，此通信类通知我的JTable类和其他对象）。 更新HashTable时，在最后一个方法中发生异常。 这是代码的简化：

以下方法是通信核心对象

public synchronized void update(Observable o, Object arg)
{
    // do some other work
    // calls the second synchronized method
    updateMonitorList();
}

private synchronized void updateMonitorList()
{
    // updates a list of monitoring addresses (no problem here)
    // m_lastValues is the HashTable that is giving me headaches
    Iterator<Parameter> itr = m_lastValues.keySet().iterator();
    while (itr.hasNext())
    {
        // removes the currently not used parameters from the HashTable
        Parameter p = itr.next(); // concurrent exception here WHY? :/
        boolean contains = false;
        for (int i = 0; i < m_monitorList.size(); i++)
        {
            if (p.equals(m_monitorList.get(i)))
            {
                contains = true;
                break;
            }
        }
        if (!contains)
        {
            m_lastValues.remove(p);
        }
    }
    // more operations with the HashTable
    for (int i = 0; i < m_monitorList.size(); i++)
    {
        // adds newly added parameters to the hashtable
        boolean contains = false;
        for (Observer key : m_requestedParameters.keySet())
        {
            if (key.equals(m_monitorList.get(i)))
            {
                contains = true;
                break;
            }
        }
        if (!contains)
        {
            m_lastValues.put(m_monitorList.getAt(i), m_monitorList.getAt(i).m_intUserSetting);
        }
    }
}

// this method is used to know if a value has changed
private synchronized boolean getParameterChanged(Parameter currentParamer)
{
    Integer v = m_lastValues.get(currentParamer);
    return v == null || v != currentParamer.m_intUserSetting;
}

我需要这种方法，因为我有多个窗口要求来自usb设备的值，并且这个通信对象负责处理，在我添加最后一个值的HashTable之前，并发没有问题。 此m_lastValues HashTable未在上面未列出的任何其他方法中使用。

有什么建议么？ 提前致谢。

[编辑]

实际上这不是一个多线程问题，它只是一个被误解的异常含义。 正如@Tudor所指出的那样，问题是while循环中的remove()方法。 这是我解决它的方式：

这段代码进入了updateMonitorList()方法，注意no Iterator was needed （第一个问题就是这样的for循环），实际上Iterator没有区别：

    ArrayList<Parameter> remove = new ArrayList<Parameter>();
    for (Parameter p : m_lastValues.keySet())
    {
        boolean contains = false;
        for (int i = 0; i < m_monitorList.size(); i++)
        {
            if (p.equals(m_monitorList.get(i)))
            {
                contains = true;
                break;
            }
        }
        if (!contains)
        {
            remove.add(p);
        }
    }
    for (int i = 0; i < remove.size(); i++)
    {
        m_lastValues.remove(remove.get(i));
    }

Answer 1

for (Parameter key : m_lastValues.keySet()) // exception this line
{
    // removes the currently not used parameters from the HashTable
}

问题出在这里。 无论同步如何，您都无法在迭代时更改集合。 实际上这是一个单线程问题。

您可以做的是切换到显式迭代器并使用remove()或保存要在单独的集合中删除的项目，然后删除它们。

Iterator<Parameter> iter = m_lastValues.keySet().iterator();
while (iter.hasNext()) {
    Parameter current = iter.next();
    for (int i = 0; i < m_monitorList.size(); i++) {
        if(current.equals(m_monitorList.get(i)) {
            iter.remove();
            break;
        }
    }
}