Container.remove(int i) 抛出意外的 ArrayIndexOutOfBoundsException

Question

我正在尝试从 Java 容器中删除除第一个子组件之外的所有组件。以下代码记录“There are 3”并抛出 ArrayIndexOutOfBoundsException: Array index out of range: 2

int componentCount = cardPanel.getComponentCount();
logger.info("There are " + componentCount);
for (int i = 1; i < componentCount; i++) {
    cardPanel.remove(i);
}

但是，此修改后的版本工作完美：

Component[] components = cardPanel.getComponents();
logger.info("There are " + components.length);
for (int i = 1; i < components.length; i++) {
    cardPanel.remove(components[i]);
}

Container.getComponentCount() 和 Container.remove(int i) 似乎不一致容器中组件的数量。还有其他人遇到过这个问题吗？

Answer 1

当您执行 cardPanel.remove(i) 时，组件的数量正在减少。

所以你有 [0, 1, 2]，并删除索引 1 处的项目。
现在您有 [0, 2] 并删除索引 2 处的项目，这会引发 ArrayIndexOutOfBoundsException。

修改后的版本可以工作，因为它是从容器中删除实际对象，而不是从索引中删除。

试试这个

int componentCount = cardPanel.getComponentCount();
logger.info("There are " + componentCount);
for (int i = 1; i < componentCount; i++) {
    cardPanel.remove(1);
}

Answer 2

本尼的答案是正确的，但根据容器的不同，它可能效率低下。像 ArrayList 这样的东西并不喜欢多次从前面删除内容，因为它每次都会复制所有其余的数据。

这是一个替代方案：

for (int i = cardPanel.getComponentCount() - 1; i >= 1; i--)
{
    cardPanel.remove(i);
}

基本上，这是从末尾开始并朝着开始的方向努力。

它只会对大型收藏有意义，但值得了解该技术。

Answer 3

索引从 0 开始，三项表示第一项位于索引 0，第二项位于索引 1，第三项位于索引 2。

Answer 4

Container.remove(int i) 删除索引 i 处的项目

您的代码删除索引 1,2,3 处的项目

Answer 5

你想做什么？如果您想从容器中删除所有项目，那么 removeAll 是正确的选择。

如果您需要在移除时处理这些项目，那么典型的习惯用法是从头到尾，正如 Jon Skeet 所解释的那样。

除非有必要从 0 到末尾，在这种情况下，您将不断删除第 0 个元素：

for (int i = 0; i < componentCount; i++) {
    cardPanel.remove(0);
}

Answer 6

如果您想根据索引上的某些条件删除项目，最好使用显式迭代器来删除它们，同时保留您自己的索引计数。

例如，删除奇数索引处的所有对象：

    int index = 0;
    final Iterator<String> iterator = list.iterator();
    while (iterator.hasNext())
    {
        iterator.next();
        if (index % 2 != 0)
        {
            iterator.remove();
        }
        index++;
    }

虽然这有点长，但它避免了棘手的代码，并且当列表中存在相等的对象时也可以工作。