动态添加JTextField后将其删除

Question

My goal is to delete two JTextFields at the same time by clicking a JLabel.

I've created the textfields like this:

public void mouseClicked(MouseEvent e) {


            inc++;
            txtName= new JTextField();
            txtNumber = new JTextField();
            txtName.setName("txtName"+inc);
            txtNumber.setName("txtNumber" + inc);

            pnlPanel.add(txtName);
            pnlPanel.add(txtNumber);

            if(count>0){
                x+=50;
                y+=50;

                txtName.setBounds(225,6+y, 182, 27);
                txtNumber.setBounds(35, 6+y, 182, 27);
                txtName.setName(tempBox+count);
                if(pnlTxtText.getComponentCount() >9){

                    pnlPanel.setPreferredSize(new Dimension(450+y,50+y));
                    pnlPanel.add(txtStudName);
                    pnlPanel.add(txtStudentNumber);

                    frmFrame.repaint();
                    scrpPanel.revalidate();
                }
            }
            frmFrame.repaint();
        }

    });

And this is my code for removing the textfields:

public void mouseClicked(MouseEvent e) {
    int countPlace= pnlPanel.getComponentCount();
    int countOfRemaining =countPlace;
    pnlPanel.remove(--countOfRemaining);
    frmFrame.revalidate();
    pnlPanel.remove(--countOfRemaining);
    frmFrame.revalidate();
}

});

Instead of deleting the txtfields on the same row, it deletes it one by one, i dont want that. Please help me. Thank you.

Answer 1

Call pnlPanel.revalidate() before repaint()

And don't use setBounds(). Define porper LayoutManager instead.

Answer 2

Instead of this,

public void mouseClicked(MouseEvent e) {
        int countPlace= pnlPanel.getComponentCount();
        int countOfRemaining =countPlace;
        pnlPanel.remove(countOfRemaining-1);
        frmFrame.repaint();
        pnlPanel.remove(countOfRemaining-1);
        frmFrame.repaint();
    }
});

Use this

public void mouseClicked(MouseEvent e) {
        int countPlace= pnlPanel.getComponentCount();
        int countOfRemaining =countPlace;
        pnlPanel.remove(--countOfRemaining);
        frmFrame.revalidate();
        pnlPanel.remove(--countOfRemaining);
        frmFrame.revalidate();
    }
});

The above one throws an ArrayIndeOutOfBounds Exception because the count remaining variable is not being decremented after you remove a component. Hence the index goes out of bound when you try to remove the second time.

Answer 3

I recommend that you add all of your JPanels to a Map (I use HashMap) in the form of <Integer, JPanel> . Name them all in order, and then just do Map.remove(Map.size() - 1) and Map.remove(Map.size() - 2) .

You can also get the current set of Integers (the keys) by doing a Map.keySet() ;

Map<Integer, JPanel> temp = new HashMap<Integer, JPanel>();
temp.put(0, new JPanel());
temp.put(1, new JPanel());
temp.put(2, new JPanel());
temp.remove(temp.size() - 1);

Makes it much easier to maintain numerous sets of panels. In my applications I don't control them numerically, but with short names instead.