Hi!

I would like to create a button for deleting all JLabel components on JDesktopPane. My code is shown below. The error message is shown below the code.

For instance, let's say there are 3 JLabel components (the 4th is not JLabel). In this case, only 2 of 3 JLabel components can be deleted. If I press the button second time, then the remaining JLabel is deleted. Why ALL JLabel components are not deleted after the first click on the button? Please, give me a hint of my error.

Thanks!

private void clearAll() {
        SelectablePanel mainPanel = MainClass.getSelectablePanel();
        Component[] c = mainPanel.getComponents();
        System.out.println(c.length);
        for(int i = 0; i < c.length; i++) {
            if(c[i] instanceof JLabel) {
                try {
                    System.out.println(mainPanel.getComponent(i).getName());
                    mainPanel.remove(i);
                    mainPanel.repaint();
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        }       
    }
4
null
null
java.lang.ArrayIndexOutOfBoundsException: No such child: 3
        at java.awt.Container.getComponent(Container.java:294)
        at mainClassesPack.Menu.clearAll(Menu.java:125)
        at mainClassesPack.Menu.actionPerformed(Menu.java:81)
        at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
        at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
        at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
        at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
        at javax.swing.AbstractButton.doClick(AbstractButton.java:357)
        at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:1223)
        at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:1264)
        at java.awt.Component.processMouseEvent(Component.java:6263)
        at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
        at java.awt.Component.processEvent(Component.java:6028)
        at java.awt.Container.processEvent(Container.java:2041)
        at java.awt.Component.dispatchEventImpl(Component.java:4630)
        at java.awt.Container.dispatchEventImpl(Container.java:2099)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4574)
        at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4238)
        at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
        at java.awt.Container.dispatchEventImpl(Container.java:2085)
        at java.awt.Window.dispatchEventImpl(Window.java:2478)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
        at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
        at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

try the below code

for (Component comp : mainPanel.getComponents()) {
 if(comp instanceof JLabel){
  mainPanel.remove(comp);
 }

Hi! Thanks a lot! I just included mainPanel.repaint(); to make your code fully workable.

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.