尽管访问的索引在范围内，但 ArrayList 索引超出范围

Question

我收到ArrayList 的索引越界异常。在异常点， size() 和 index 在打印时似乎很好。下面是相关代码。我有一个自定义表模型，单击按钮时会刷新。

public class CallingClass
{
    public void buttonClicked()
    {
        new Thread(new Runnable()
        {
            public void run()
            {                   
                dataTableModel.refresh();
            }
        }).start();     
    }
}

public class DataTableModel extends AbstractTableModel
{
    protected ArrayList<Object> data;   

    public DataTableModel()
    {
        data = new ArrayList<Object>();
    }

    public Object getValueAt(int modelRow, int modelColumn)
    {
         // Throws Index Out of Bounds Exception though a println here shows modelRow value within 0 to data.size()-1
         // DataObject is just an interface to support getValue method
        return ((DataObject) data.get(modelRow)).getValue();
    }

    public void refresh()
    {
        reloadData();
        fireTableDataChanged();
    }

    protected void reloadData()
    {
        ArrayList<TextMessage> messageList = jmsConnection.getMessageList();

        data.clear();

        try
        {
            for(int i=0; i<messageList.size(); ++i)
            {
                data.add(new MyDataObject(messageList.get(i).getJMSMessageID()));
            }
        }
        catch(Exception e)
        {
        }
    }
}

如您所见，DataTableModel.refresh() 在新线程中运行。对fireTableDataChanged 的调用可能会导致eventQueue 线程重新绘制表格，进而调用getValueAt 方法。尽管modelRow 的值小于data.size()，但这里还是会抛出异常。不确定如何存在竞争条件，因为 Runnable 线程在调用 fireTableDataChanged 之后调用 reloadData（它重新填充数据对象）。因此，当 eventQueue 线程调用 getValueAt 时，data 对象应该是稳定的。另请注意，在可运行线程出现之前，刷新按钮处于禁用状态，以避免重复调用刷新。

下面是异常信息

Exception in thread "AWT-EventQueue-0" java.lang.IndexOutOfBoundsException: Index: 3, Size: 10
    at java.util.ArrayList.rangeCheck(Unknown Source)
    at java.util.ArrayList.get(Unknown Source)
    at datatablemodels.DataTableModel.getValueAt(DataTableModel.java:70)

如果我稍微更改 reloadData 代码，异常就会消失（至少还不能重现）。我的猜测是，它只是减少了异常的可能性。

protected void reloadData()
{
    ArrayList<TextMessage> messageList = jmsConnection.getMessageList();

    ArrayList<Object> tempdata = new ArrayList<Object>();

    try
    {
        for(int i=0; i<messageList.size(); ++i)
        {
            tempdata.add(new MyDataObject(messageList.get(i).getJMSMessageID()));
        }
    }
    catch(Exception e)
    {
    }

    data = tempdata ;
}

我更感兴趣的是了解这里发生了什么。我确实有其他解决方案，例如 synchronized、CopyOnWriteArrayList 等。

Answer 1

由于您没有同步，因此您无法保证在 refresh() 方法中所做的数据更改（在其自己的线程中运行）将在运行 getValueAt() 方法的其他线程中可见（如果有的话）。例如，getValueAt() 方法或其中的一部分可能会在 reloadData() 方法中的 data.clear() 调用和数据表的重新填充“之间”运行。

并且当您放入 println() 调用时，它们可能会在 data.clear() 调用之前或重新填充数据之后运行，即使 data.get() 调用没有。

底线是，每当数据被多个线程访问时，就像这里一样，您必须以一种或另一种方式同步对数据的访问。