Java 中组件的重用没有正确修剪未重用的组件

Question

我有一个名为 calcResPanel 的 javax.swing.JPanel （使用带有 1 列的 java.awt.GridLayout 和不定 (0) 行)，用于接收和显示一组 BHSelectableLabel（扩展 javax.swing.JTextField），共同表示这存储在称为结果的String列表中的文本。我想我不妨给它以下行为：

1. 第一次，它只会添加新的行为
2. 接下来的时间，它会：
   1. 将已添加的尽可能多的标签的文本更改为结果中尽可能多的值的文本
   2. 如果还有任何未更改的标签，请将其删除，因为它们不是必需的。否则，根据需要添加任意数量的新标签。

这对我来说很有意义。 如果这个算法不是我应该做的，那么现在停止阅读并用更好的算法发布答案。但是，如果您同意，请告诉我我的代码做错了什么：

  int i, r, l;
  for (i=0, r = results.length(), l = calcResPanel.getComponentCount(); i < r; i++)
    if (i < l)
      ((BHSelectableLabel)calcResPanel.getComponent(i)).setText(results.get(i));
    else
      calcResPanel.add(new BHSelectableLabel(results.get(i)));
  for (;i < l; i++)//If there are excess, unused lables, remove them
    calcResPanel.remove(i);

此代码的问题在于它不一致地在 calcResPane 中留下多余的标签。如果您认为这个算法在概念上很好，那么请告诉我我的代码有什么问题导致它留下多余的标签？

回答

---

也是一个如此简单的答案。我觉得自己很聪明^^;

  int i, r, l;
  for (i=0, r = results.length(), l = calcResPanel.getComponentCount(); i < r; i++)
    if (i < l)
      ((BHSelectableLabel)calcResPanel.getComponent(i)).setText(results.get(i));
    else
      calcResPanel.add(new BHSelectableLabel(results.get(i)));
  for (;i < l; i++)//If there are excess, unused lables, remove them
    calcResPanel.remove(r);

Answer 1

for (;i < l; i++)//If there are excess, unused lables, remove them     
    calcResPanel.remove(i); 

你永远不能像这样删除，因为你会跳过每第二个项目。假设您有 5 个项目，您尝试将它们全部删除：

第一次通过循环 i = 0，因此您删除了项目 0，只剩下 1、2、3、4。

下次通过循环 i = 1 ，所以你删除了第 2 项，剩下了 1, 3, 4。

我希望你能明白这个模式。

解决方案是从末尾移除项目，一次一个。