JFrame添加KeyEvent后老虎图像无法移动问题排查

我来帮你排查下几个关键问题，这应该能解决老虎按方向键不移动的问题：

1. 先填图标变量的作用域坑 #

你在PacMan的构造函数里，重新声明了tigerLeft和tigerRight作为局部变量，这就导致类的成员变量始终是null——调用moveRight()或moveLeft()时设置图标完全无效，不信你看控制台打印的tigerRight肯定是null。修改构造函数，直接给类成员变量赋值：

public PacMan() throws IOException {
    // 去掉前面的ImageIcon，直接给类里的成员变量赋值
    tigerLeft = new ImageIcon(new ImageIcon("textures/tigerLeft.png").getImage().getScaledInstance(60, 40, Image.SCALE_DEFAULT));
    tigerRight = new ImageIcon(new ImageIcon("textures/tigerRight.png").getImage().getScaledInstance(60, 40, Image.SCALE_DEFAULT));
    pacManImage.setIcon(tigerRight);
    pacManImage.setVisible(true);
}

2. 解决布局管理器锁死组件位置的问题 #

JPanel默认用FlowLayout，这种布局会自动接管所有组件的位置和大小，你调用setLocation()或者setBounds()完全是白搭，布局管理器会强行把组件拉回它认为正确的位置。解决办法是禁用JPanel的布局管理器，用绝对定位：

在UI类的构造函数里，给panel加一行：

panel.setLayout(null); // 关闭自动布局，手动控制组件位置

同时，初始化PacMan后要调用initialDraw()设置初始位置，现在布局是手动模式了，得自己指定初始坐标：

this.PacMan = new PacMan();
PacMan.initialDraw(); // 把老虎放到初始的(570,320)位置

另外，移动方法里别用getLocationOnScreen()，这个取的是屏幕全局坐标，组件在容器里的位置应该用自身的xCoords和yCoords来维护，修改移动方法更靠谱：

public void moveRight() {
    System.out.println("here: " + tigerRight);
    xCoords += 2;
    pacManImage.setIcon(tigerRight);
    pacManImage.setLocation(xCoords, yCoords);
    pacManImage.repaint();
}

public void moveLeft() {
    xCoords -= 2;
    pacManImage.setIcon(tigerLeft);
    pacManImage.setLocation(xCoords, yCoords);
    pacManImage.repaint();
}

public void moveUp() {
    yCoords -= 2;
    pacManImage.setLocation(xCoords, yCoords);
    pacManImage.repaint();
}

public void moveDown() {
    yCoords += 2;
    pacManImage.setLocation(xCoords, yCoords);
    pacManImage.repaint();
}

3. 修正键盘监听的焦点问题 #

你给UI类加了KeyListener，但实际承载老虎的是JPanel，焦点大概率不在UI上，导致键盘事件根本传不到你的监听方法里。应该把KeyListener加到panel上，并且确保panel能获取焦点：

在UI构造函数里调整这几行：

// 把键盘监听加到panel上，而不是UI本身
panel.addKeyListener(this);
panel.setFocusable(true); // 允许panel获取焦点

然后在窗口显示后，主动让panel获取焦点：

frame.setVisible(true);
panel.requestFocusInWindow(); // 强制让panel拿到焦点，这样键盘事件才能被捕获

额外小建议：用KeyBindings替代KeyListener #

KeyListener对焦点太敏感，一旦焦点不小心跑到别的组件上，就完全失效了。Swing的KeyBindings更可靠，即使窗口在前台，不管焦点在哪都能响应事件。比如给panel绑定右键的例子：

// 绑定右键事件
panel.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(KeyStroke.getKeyStroke(KeyEvent.VK_RIGHT, 0), "moveRight");
panel.getActionMap().put("moveRight", new AbstractAction() {
    @Override
    public void actionPerformed(ActionEvent e) {
        PacMan.moveRight();
    }
});

你可以用同样的方式绑定左、上、下方向键，这样体验会更稳定。

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