4

我正在实现一个函数,当用户单击 JButton 时,该函数传递 JList 中的选定项和 JTextField 中的值。

我正在使用几个听众。但是,当用户第二次按下按钮并产生不需要的结果时,似乎 addcartbtn 内的循环 actionPerformed 被调用了两次。当用户第三次按下时,该函数似乎被调用了三次。

   list.addListSelectionListener(new ListSelectionListener() {

        Map<String, Integer> cartlist = new HashMap<String, Integer>();

        public void valueChanged(final ListSelectionEvent e) {
            if (e.getValueIsAdjusting()) {
                System.out.println("test0");
                final ArrayList<String> cartArrayList = new ArrayList<String>();
                addcartbtn.addActionListener(new ActionListener() {

                    public void actionPerformed(final ActionEvent e2) {

                        System.out.println("test2");
                        String itemselected = "";
                        System.out.println("Index is " + e.getLastIndex());
                        String itemname = (String) hashmap.get(e.getLastIndex());
                        itemselected = itemname;

                        //System.out.println(itemselected);
                        try {
                            int insertedquantity = Integer.parseInt(quantity.getText());
                            cartlist.put(itemselected, insertedquantity);
                            //shoppingcart.revalidate();
                            String element = itemselected + " " + String.valueOf(insertedquantity);

                            cartArrayList.add(element);

                            System.out.println(element);
                            //System.out.println(counter);
                            shoppingcart.setListData(cartArrayList.toArray());
                            shoppingcart.revalidate();
                            shoppingcart.repaint();
                            System.out.println("---------");

                        } catch (NumberFormatException ex) {
                            System.out.println("Not a number!");
                        }
                    }
                });
            }
        }
    });

感谢大家的帮助!

4

2 回答 2

3

不要在 ListSelectionListener 中添加 ActionListener - 没有意义。您将无缘无故地添加许多听众。事实上,如果您希望仅在按下按钮时发生动作,我认为根本没有 ListSelectionListener 的理由。只需使用一次添加到 JButton 的 ActionListener 可能是在构造函数或设置方法中。

此外,缩进少一点可能会使您的代码更容易阅读。
编辑:我减少了您原始帖子中的代码缩进。

于 2012-05-12T10:10:20.180 回答
3

每次在 JList 中进行选择时,您都在向 addcartbtn 添加一个新的操作侦听器(如果它被命名为 addCartButton,BTW 会不会更具可读性)。侦听器只能添加一次。

于 2012-05-12T10:11:01.997 回答