Java Swing跨类调用remove/add方法失效 元素属性窗口不更新

问题背景 #

这是一个元素周期表展示Java程序，核心逻辑为点击元素符号时弹出新窗口展示对应元素属性：

• 公共类AtomicNumber继承JFrame、实现MouseListener接口，负责加载周期表主窗口，每个元素通过JRadioButton组件实现
• 公共类Properties继承JFrame，负责弹出元素属性展示窗口，每个元素的属性图通过JLabel承载ImageIcon对象实现

故障现象 #

点击不同元素后属性窗口的图片无法更新：

• 点击氢（H）时可正常显示氢的属性
• 点击氦（He）时窗口仍展示氢的属性，无法切换为对应选中元素的属性内容
  
  

原始问题代码 #

主类PeriodicTable #

public class PeriodicTable {
  public static void main(String[] args) {
    new AtomicNumber();
  }
}

主窗口类AtomicNumber（核心片段） #

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class AtomicNumber extends JFrame implements MouseListener {
  JRadioButton[] element;
  AtomicNumber() {
      element = new JRadioButton[119];
      element[1] = new JRadioButton("1");
      element[1].setBounds(50, 75, 50, 50);
      elementImage[1] = new ImageIcon(getClass().getClassLoader().getResource("H.png"));
      properties[1] = new JLabel();
      element[2] = new JRadioButton("2");
      element[2].setBounds(925, 75, 50, 50);
      elementImage[2] = new ImageIcon(getClass().getClassLoader().getResource("He.png"));
      // 其余元素初始化逻辑已省略
  }

  public void mousePressed(MouseEvent e) {
    new Properties();
    if (e.getSource() == element[1]) {
      new Properties().setVisible(true);
    } else if (e.getSource() == element[2]) {
      // 以下remove和add方法无法正常生效
      Properties.propertiesPanel.remove(properties[1]);
      Properties.propertiesPanel.add(properties[2]);

      Properties.propertiesPanel.revalidate();
      Properties.propertiesPanel.repaint();
      new Properties().setVisible(true);
    }
  }
}

属性窗口类Properties #

import javax.swing.ImageIcon;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class Properties extends JFrame {
  private static final long serialVersionUID = 1L;
  public JLabel[] properties;
  public JPanel propertiesPanel;

  Properties() {
    properties = new JLabel[119];
    properties[1] = new JLabel(new ImageIcon(getClass().getClassLoader().getResource("H_properties.png")));
    properties[2] = new JLabel(new ImageIcon(getClass().getClassLoader().getResource("He_properties.png")));

    propertiesPanel = new JPanel();
    propertiesPanel.add(properties[1]);
    this.add(propertiesPanel);
    this.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
    this.setResizable(false);
    this.setSize(1250, 340);
  }
}

根因分析 #

代码存在3个核心问题，导致组件操作完全不生效：

1. 实例引用逻辑错误：propertiesPanel和properties是Properties类的实例成员，不是静态成员，直接通过类名Properties.propertiesPanel调用本身就不符合Java语法规范，操作的根本不是弹出窗口内的实际组件。
2. 重复创建无效窗口实例：鼠标点击事件里连续多次执行new Properties()，每次创建的都是全新的窗口对象，每个新窗口初始化时默认都会加载氢元素的属性面板，就算修改了某一个实例的组件，后续新建的实例还是会加载默认内容，之前的修改完全不生效。
3. 跨类组件访问非法：AtomicNumber类里直接访问properties[1]、properties[2]，这两个数组是Properties类的实例成员，主窗口类中没有对应实例的引用，根本拿不到正确的组件对象，操作的是空值或者无效对象。

修复方案 #

1. 将属性窗口改为单例模式，不要每次点击都新建窗口，全局只维护一个属性窗口实例
2. 给Properties类增加公开的元素切换方法，禁止跨类直接操作内部组件
3. 点击元素时只需要调用属性窗口的切换方法，传入当前选中的元素序号，由属性窗口自身完成面板内容刷新即可

修正后的核心代码 #

Properties类修改

public class Properties extends JFrame {
  private static final long serialVersionUID = 1L;
  private JLabel[] properties;
  private JPanel propertiesPanel;
  private static Properties instance;

  // 私有构造方法，禁止外部直接创建实例
  private Properties() {
    properties = new JLabel[119];
    properties[1] = new JLabel(new ImageIcon(getClass().getClassLoader().getResource("H_properties.png")));
    properties[2] = new JLabel(new ImageIcon(getClass().getClassLoader().getResource("He_properties.png")));
    // 其余元素属性标签初始化逻辑省略

    propertiesPanel = new JPanel();
    this.add(propertiesPanel);
    this.setDefaultCloseOperation(JFrame.HIDE_ON_CLOSE);
    this.setResizable(false);
    this.setSize(1250, 340);
  }

  // 全局获取唯一属性窗口实例
  public static Properties getInstance() {
    if (instance == null) {
      instance = new Properties();
    }
    return instance;
  }

  // 公开的元素切换方法，传入元素序号即可更新展示内容
  public void showElement(int elementIndex) {
    propertiesPanel.removeAll();
    propertiesPanel.add(properties[elementIndex]);
    propertiesPanel.revalidate();
    propertiesPanel.repaint();
    this.setVisible(true);
  }
}

AtomicNumber类鼠标事件修改

public void mousePressed(MouseEvent e) {
  Properties propWindow = Properties.getInstance();
  if (e.getSource() == element[1]) {
    propWindow.showElement(1);
  } else if (e.getSource() == element[2]) {
    propWindow.showElement(2);
  }
  // 其余元素判断逻辑同理
}

内容的提问来源于stack exchange，提问作者Alanzhou