如何复制 JButton 操作并更改其对基础对象的引用？

Question

以下示例使用 JButton、JTextField 和 JLabel 创建一个 JFrame。 按下按钮时，它会增加文本字段和标签中的值。 此示例还创建了第二个 JFrame，它是第一个 JFrame 的副本。 按钮、文本字段和标签也会被复制。

目前的问题是复制框架上的按钮仍会更新原始框架上的文本字段和标签。 “为什么”是相当明显的，因为代码对文本字段和标签进行了具体引用。

虽然这不是以最好的方式编写的，但它是我正在解决的场景的一个很好的例子。

目标是，在不进行重大重写的情况下，让复制的按钮操作更新复制的测试字段和标签而不是原始的，侵入性最小的方法是什么？

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JTextField;

class ButtonTextFieldLabel extends JFrame
{
    JButton bnt1 = new JButton("B1");
    JTextField tf1 = new JTextField("1");
    JLabel lbl1 = new JLabel("100");

    public ButtonTextFieldLabel()
      {
          super("Main Frame");
          setLayout(null);

          bnt1.setBounds(50,100,120,40);
          tf1.setBounds(300,100, 80,40);
          lbl1.setBounds(200,100,80,40);
          bnt1.addActionListener(new ListenerHolder(this));

          add(bnt1);
          add(tf1);
          add(lbl1);

          setSize(500,500);
          makeCopy(this);       
          setVisible(true);
      }

    private void makeCopy(ButtonTextFieldLabel originalObj) 
    {
          JFrame copyFrame = new JFrame();
          copyFrame.setTitle("Copy of " + originalObj.getTitle());
          copyFrame.setSize(originalObj.getSize());
          copyFrame.setLocation(originalObj.getX()+100, originalObj.getY()+100);
          copyFrame.setLayout(null);

          JButton copyBnt1 = new JButton();       
          copyBnt1.setBounds(originalObj.bnt1.getBounds());
          copyBnt1.setLabel(originalObj.bnt1.getLabel());
          copyFrame.add(copyBnt1);

          for (ActionListener al : originalObj.bnt1.getActionListeners())
          {
              copyBnt1.addActionListener(al);
          }

          JTextField copyTf1 = new JTextField();
          copyTf1.setBounds(originalObj.tf1.getBounds());
          copyTf1.setText(originalObj.tf1.getText());

          JLabel copyLbl1 = new JLabel();
          copyLbl1.setBounds(originalObj.lbl1.getBounds());
          copyLbl1.setText(originalObj.lbl1.getText());

          copyFrame.add(copyBnt1); 
          copyFrame.add(copyTf1);
          copyFrame.add(copyLbl1);

          copyFrame.setVisible(true);
    }

    public void runThis() 
    {
        tf1.setText(  Integer.toString(Integer.parseInt(tf1.getText())+1)   );
        lbl1.setText(  Integer.toString(Integer.parseInt(lbl1.getText())+1)   );
    }

} 

class ListenerHolder implements  ActionListener
{
    ButtonTextFieldLabel ph;
    public ListenerHolder(ButtonTextFieldLabel ph)
    {
        this.ph = ph;       
    }
    @Override
    public void actionPerformed(ActionEvent arg0) 
    {
        ph.runThis();

    }
}

public class TestBTL
    {
      public static void main(String[] args){
        new ButtonTextFieldLabel();
    }
}

Answer 1

您已经知道问题的原因——您正在复制原始的 ActionListener，并附上它对原始 GUI 组件的引用。整体解决方案不是复制动作侦听器，而是创建您的 GUI 来保存和维护它们自己的独特状态。一种解决方案不是尝试通过 kludge 复制组件，而是创建一个自包含的 GUI 对象来保存和更新自己的状态。如果需要，您可以使用工厂方法创建多个 GUI。

例如：

import java.awt.Dimension;
import java.awt.event.ActionEvent;

import javax.swing.*;

public class TestBtl2 {
    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> {
            createAndDisplayFrame("Frame 1").setVisible(true);
            createAndDisplayFrame("Frame 2").setVisible(true);
        });
    }

    // Factory method
    private static JFrame createAndDisplayFrame(String text) {
        BtlPanel btlPanel = new BtlPanel();
        JFrame frame = new JFrame(text);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.add(btlPanel);
        frame.pack();
        frame.setLocationByPlatform(true);
        return frame;
    }
}

class BtlPanel extends JPanel {
    private int value = 0;
    private JButton button1 = new JButton(new ButtonAction("Button 1"));
    private JLabel label1 = new JLabel("00");
    private JTextField textField1 = new JTextField("00");

    public BtlPanel() {
        textField1.setFocusable(false);
        add(button1);
        add(Box.createHorizontalStrut(20));
        add(label1);
        add(Box.createHorizontalStrut(20));
        add(textField1);

        setPreferredSize(new Dimension(300, 100));
    }

    public void incrementValue() {
        value++;
        String text = String.format("%02d", value);
        label1.setText(text);
        textField1.setText(text);
    }

    private class ButtonAction extends AbstractAction {
        public ButtonAction(String name) {
            super(name);
        }

        @Override
        public void actionPerformed(ActionEvent e) {
            incrementValue();
        }
    }

}

---

建议：

• 虽然空布局和setBounds() 在 Swing 新手看来是创建复杂 GUI 的最简单和最好的方法，但创建的 Swing GUI 越多，使用它们时遇到的困难就越严重。当 GUI 调整大小时，它们不会调整您的组件大小，它们是增强或维护的皇家女巫，放置在滚动窗格中时它们完全失败，在与原始不同的所有平台或屏幕分辨率上查看时它们看起来很糟糕.
• 查看：The Use of Multiple JFrames, Good/Bad Practice?