遍历容器和组件以查找给定类的对象？

Question

我为JLabels和AbstractButtons编写了MnemonicsBuilder类。我想编写一个方便的方法setMnemonics( JFrame f )，该方法将遍历JFrame的每个子级并选择JLabels和AbstractButtons。如何获得对JFrame中包含的所有内容的访问权限？我试过了：

LinkedList<JLabel> harvestJLabels( Container c, LinkedList<JLabel> l ) {
    Component[] components = c.getComponents();
    for( Component com : components )
    {
        if( com instanceof JLabel )
        {
            l.add( (JLabel) com );
        } else if( com instanceof Container )
        {
            l.addAll( harvestJLabels( (Container) com, l ) );
        }
    }
    return l;
}

在某些情况下，这很好。在其他情况下，它会耗尽内存。我在想什么呢？有没有更好的方法来搜索子组件？我的递归有缺陷吗？这不是Swing中事物如何“包含”其他事物的图片吗？例如，Swing不是一棵有根的树吗？

JFrame
|
|\__JMenuBar
|   |
|    \__JMenu
|       |
|        \__JMenuItem
|
|\__JPanel
|   |
|   |\__JButton 
|   |
|   |\__JLabel
|   |
|   |\__ ... JCheckBoxes, other AbstractButtons, etc.

Answer 1

在这里与Tom保持一致…您的问题是，您已经传递了List将JLabels 添加到递归方法中，并且还返回了它-
因此将相同的项多次添加到您的列表中。用更政治上正确的术语来说-这List是您的累加器。

您的方法应如下所示：

public void harvestJLabels(Container c, List<JLabel> l) {
    Component[] components = c.getComponents();
    for(Component com : components) {
        if(com instanceof JLabel) {
            l.add((JLabel) com);
        } else if(com instanceof Container) {
            harvestJLabels((Container) com, l));
        }
    }
}

然后，您可以使用一个辅助方法来启动此收获：

public List<JLabel> harvestJLabels(Container c) {
    List<JLabel> jLabels = new ArrayList<JLabel>();
    harvestJLabels(c, jLabels);
    return jLabels;
}