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来保存和维护自己的唯一状态。一种解决方案不是尝试通过合并复制组件，而是创建一个包含并更新其自身状态的独立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?

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

1 个答案: