Java Swing Controller与View通信时按钮文本更新不生效问题

这个问题是Swing单线程模型+初始化顺序错误导致的UI更新失效，和你调用的repaint()方法本身没关系，核心错误有三个：

• 违反Swing线程规则：Swing所有组件的创建、修改、渲染操作必须在事件调度线程（EDT）上执行，你直接在主线程里new Window初始化UI，本身就会导致组件状态和渲染线程不同步，出现各种诡异的渲染异常。
• 窗口显示时机完全错误：你在setupView()里先调用了setVisible(true)把窗口渲染显示出来，之后才创建JPanel、添加按钮、把面板加到窗口中。窗口首次渲染时根本没有加载按钮组件，后续你直接操作bGoFile实例，虽然对象本身的text属性被修改了，但UI组件树没有完成布局校验，渲染层根本没感知到组件状态变更，自然不会更新显示。
• 无意义的同步锁阻塞EDT：actionPerformed本身就是EDT回调执行的方法，你在方法里加synchronized (this)块，很容易造成EDT线程被锁阻塞。Swing的所有重绘、事件处理都靠EDT单线程调度，一旦EDT卡住，哪怕你调用了setText触发了重绘请求，请求也会堵在队列里没法执行，界面自然不会有变化。

• 所有UI初始化逻辑全部放到EDT中执行，通过SwingUtilities.invokeLater()启动窗口
• 调整setupView()的执行顺序：先完成所有组件添加、属性配置，最后再调用setVisible(true)
• 删掉actionPerformed里无意义的synchronized块，EDT上不要做任何加锁、耗时IO操作，避免阻塞UI线程
• 修复代码里的语法错误：new JPanel要补括号写成new JPanel()

// Window类修正后的setupView方法
private void setupView() {
    setTitle("Cover pdf to img");
    setBounds(300, 90, 900, 600);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setLocationRelativeTo(null);
    setResizable(false);
    // 先加载图标、添加组件，不要提前调用setVisible
    ImageIcon icon = new ImageIcon("./src/resources/logo.jpg");
    setIconImage(icon.getImage());
    JPanel jp = new JPanel(); // 补全实例化的括号
    jp.add(bGoFile);
    add(jp);
    // 所有组件准备完毕最后再显示窗口
    setVisible(true);
}

// WindowController修正后的启动和事件处理
public class WindowController implements ActionListener {
    Window window;

    public WindowController() {
        // 把UI初始化逻辑放到EDT执行
        SwingUtilities.invokeLater(() -> {
            this.window = new Window(this);
        });
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        System.out.println("Action performed");
        switch (e.getActionCommand()) {
            case GO_FILE -> {
                // 删掉无意义的同步块，直接更新UI
                window.changeButtonName("New name");
                break;
            }
        }
    }

    // 程序启动入口示例
    public static void main(String[] args) {
        new WindowController();
    }
}

补充说明：如果后续确实要在非EDT线程更新UI，不要直接操作组件，统一用SwingUtilities.invokeLater()把UI更新逻辑抛到EDT执行即可，不需要手动调用repaint()，setText()本身会自动触发组件重绘请求。

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