点击按钮后Swing按键绑定失效，设按钮不可聚焦仍无效

核心问题 #

按键失效的根本原因是事件调度线程（EDT）被卡死：

• Swing的所有UI事件（按钮点击、按键输入）和组件绘制都依赖EDT执行
• 按钮的ActionListener.actionPerformed方法本身就在EDT中运行，你在里面直接启动Gameloop，而Gameloop的构造函数是无限死循环，直接占满了EDT，导致后续的按键事件完全无法被处理
• 把new Gameloop()移到ActionListener外时，它在主线程执行，不会阻塞EDT，所以按键功能正常

修复方案 #

1. 将游戏循环移到独立线程

绝对不能在EDT中运行无限循环，必须把游戏逻辑放到单独的线程里，避免阻塞UI事件处理。

2. 使用Swing标准绘制方式

不要用force.getGraphics()直接绘制，这是Swing的错误用法。正确做法是自定义JPanel，重写paintComponent方法完成绘制，Swing会自动调度绘制任务到EDT。

3. 修正按键绑定细节

用KeyEvent.VK_D代替字符'd'，避免大小写或输入法导致的按键识别问题。

4. 避免null布局（可选但推荐）

Swing的null布局会导致跨平台适配问题，尽量使用布局管理器。

完整修复代码 #

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

public class Main {
    public static JFrame frame = new JFrame();
    public static Rectangle player = new Rectangle(10, 10, 10, 10);
    // 自定义绘制面板
    private static GamePanel gamePanel = new GamePanel();

    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> {
            frame.setUndecorated(true);
            frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            frame.setLayout(new BorderLayout()); // 用BorderLayout代替null布局
            frame.setSize(1000, 1000);

            // 按键绑定：使用WHEN_IN_FOCUSED_WINDOW确保窗口内都能响应
            InputMap inputMap = frame.getRootPane().getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW);
            inputMap.put(KeyStroke.getKeyStroke(KeyEvent.VK_D, 0), "right");
            ActionMap actionMap = frame.getRootPane().getActionMap();
            actionMap.put("right", new RightMoveAction());

            JButton testBtn = new JButton("Start Game");
            testBtn.setFocusable(false);
            testBtn.addActionListener(e -> new GameLoop().start()); // 启动独立线程

            frame.add(testBtn, BorderLayout.NORTH);
            frame.add(gamePanel, BorderLayout.CENTER);
            frame.setVisible(true);
        });
    }

    // 自定义玩家右移动作
    static class RightMoveAction extends AbstractAction {
        @Override
        public void actionPerformed(ActionEvent e) {
            player.x += 4;
            gamePanel.repaint(); // 移动后触发重绘
        }
    }

    // 自定义绘制面板
    static class GamePanel extends JPanel {
        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            // 绘制玩家矩形
            g.drawRect(player.x, player.y, player.width, player.height);
        }
    }

    // 游戏循环放到独立线程
    private static class GameLoop extends Thread {
        @Override
        public void run() {
            while (!Thread.currentThread().isInterrupted()) {
                try {
                    Thread.sleep(16); // 约60帧的间隔，比1ms更合理
                } catch (InterruptedException e) {
                    Thread.currentThread().interrupt();
                    break;
                }
                // 如果需要游戏逻辑更新（比如AI移动），放在这里
                // 重绘任务必须放到EDT执行
                SwingUtilities.invokeLater(gamePanel::repaint);
            }
        }
    }
}

关键修改点说明 #

• 用SwingUtilities.invokeLater初始化UI，确保所有UI操作都在EDT执行
• GameLoop继承Thread，重写run方法，把循环放到独立线程
• 用GamePanel的paintComponent绘制，调用repaint()触发Swing的绘制流程
• 按键移动后主动调用repaint()更新画面
• 游戏循环中用SwingUtilities.invokeLater触发重绘，确保绘制操作在EDT执行
• 把Thread.sleep(1)改成Thread.sleep(16)，避免过度占用CPU

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