Java KeyListener 没有检测到键盘输入

Question

我正在尝试修改此程序，以便它绘制城堡的图像，并且我能够使用向上和向下箭头键缩放此图像。我无法让 keylistener 工作，程序运行但它没有响应按键。任何帮助将不胜感激，谢谢。

import java.awt.*;
import java.awt.geom.*;
import java.util.*;
import javax.swing.*;
import java.net.*;
import java.awt.event.*;

public class DrawImage extends JFrame implements KeyListener {
int scale = 1;
    private Image image;
    enter code here
    public static void main(String[] args) {
        new DrawImage();
    }

    public DrawImage() {
        super("DrawImage");
        setSize(600,600);
        setVisible(true);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        Toolkit tk = Toolkit.getDefaultToolkit();
        image = tk.getImage(getURL("castle.png"));
        addKeyListener(this);
    }

    private URL getURL(String filename) {
        URL url = null;
        try {
            url = this.getClass().getResource(filename);
        }
        catch (Exception e) { }
        return url;
    }

    public void paint(Graphics g) {
        Graphics2D g2d = (Graphics2D) g;
        AffineTransform trans = new AffineTransform();
        trans.scale(scale, scale);
        System.out.println("scale: " + scale);
        g2d.setColor(Color.BLACK);
        g2d.fillRect(0, 0, getSize().width, getSize().height);
        g2d.setTransform(trans);
        g2d.drawImage(image, 0, 40, this);
        addKeyListener(this);
    }

    public void keyReleased(KeyEvent e) { }
    public void keyTyped(KeyEvent e) { }
    public void keyPressed(KeyEvent e) {
        int key = e.getKeyCode();
        if (key == KeyEvent.VK_UP) {
            scale++;
        }
        if (key == KeyEvent.VK_DOWN) {
            scale--;
        }
    }
}

Answer 1

I can't manage to get the keylistener to work, the program runs but it isn't responding to key presses.

你的问题是一个很常见的问题，在这里经常被问到，几乎总是问题是由于缺乏焦点——被监听的组件没有当前焦点，而焦点对于 KeyListener 的工作是必不可少的.

简短的回答是让正在收听的组件成为焦点。

更长更好的答案是不使用 KeyListeners，而是为这样的项目使用 Key Bindings。

---

编辑
其他问题:

• 在上面的代码中，您将 KeyListener 添加到 JFrame，即使您有充分的理由使用 KeyListener，也不应该这样做。
• 您还直接在 JFrame 中绘图，在我看来这是一个更大的问题，因为您冒着不必要的副作用的风险。
• 您要在 paint 方法中添加一个监听器，这是一个更大的问题。 paint 方法不应该被重写(通常)，如果你确实必须重写它，则永远不应该用于程序逻辑、添加或删除监听器，或进行任何非绘画 Activity 。应该只是用来画画的。
• 您应该直接在 JPanel 或 JComponent 中绘制。
• 相反，您应该在绘制 JPanel 或 JComponent 的 paintComponent(Graphics g) 覆盖中进行绘制。
• 正如上面所描述的，这种方法应该只用于绘画和绘画，并且应该尽可能快。
• 您应该在 paintComponent(Graphics g) 覆盖中调用 super.paintComponent(g)。
• 同样，绘图组件应该使用键绑定(bind)(教程 here)监听击键。本教程将解释为什么这种区别(KeyListener 与 Key Bindings)很重要。

  ---

编辑2
您永远不想像此处显示的代码那样忽略异常，因为这相当于盲目驾驶的编程:

  try {
     url = this.getClass().getResource(filename);
  } catch (Exception e) {
  }

希望这不是您的生产代码的样子，您只是为了简洁而忽略了已发布代码中的异常，但对于我们中的许多人来说，看到这就像在黑板上听到钉子一样。

---

编辑 3
有关 KeyListeners 与 Key Bindings 的更多信息:假设您的代码可以与 KeyListener 一起使用，然后假设您向 GUI 添加了任何其他可聚焦组件，并且它们以某种方式通过用户交互获得了焦点，那么您的 KeyBindings 将不再起作用。如果您使用 Key Bindings 正确完成了此操作，这将不是问题。

---

编辑4
你真的希望你的比例字段是 double 的，而不是整数。你真的不想递增和递减它，而是想将它乘以并除以某个乘数常数，例如 1.2。您还需要在更改比例时调用 repaint()。

---

编辑5
请查看两个示例程序，第一个名为 DrawImagePanelKeyListener.java，使用 KeyListener，而第二个名为 DrawImagePanelKeyBindings，使用 Key Bindings。它们都应该按预期编译、运行和运行:当您按向上或向下箭头键时，显示的图像会缩小和放大。当按下两个 JButton 时，可以看出它们行为的差异。按下按钮，看看您的按键响应会发生什么。当拥有 KeyListener 的组件失去焦点时，其 KeyListener 将停止工作，但对于使用 Key Bindings 的组件则不然。

解决这个问题的一个方法是防止所有其他组件获得焦点，但这对于大多数 GUI 来说是不切实际的或不需要的。

DrawImagePanelKeyListener.java

import java.awt.*;
import javax.imageio.ImageIO;
import javax.swing.*;
import java.io.IOException;
import java.net.*;
import java.awt.event.*;

@SuppressWarnings("serial")
public class DrawImagePanelKeyListener extends JPanel implements KeyListener {
   public static final String IMAGE_PATH = "https://duke.kenai.com/"
         + "nyanya/.Midsize/NyaNya.jpg.png";
   private static final double MULTIPLIER = 1.2;
   double scale = 1.0;
   private Image image;
   private Dimension initialSize = new Dimension(0, 0);

   public static void main(String[] args) {
      SwingUtilities.invokeLater(new Runnable() {
         public void run() {
            DrawImagePanelKeyListener drawImage = new DrawImagePanelKeyListener();
            drawImage.add(new JButton("Foo"));
            JFrame frame = new JFrame("Draw Image");
            frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            frame.add(drawImage);
            frame.pack();
            frame.setLocationRelativeTo(null);
            frame.setVisible(true);
         }
      });
   }

   public DrawImagePanelKeyListener() {
      setFocusable(true);
      requestFocusInWindow();
      URL imgURL;
      try {
         imgURL = new URL(IMAGE_PATH);
         image = ImageIO.read(imgURL);
         initialSize = new Dimension(image.getWidth(this),
               image.getHeight(this));
         addKeyListener(this);
         setVisible(true);
      } catch (MalformedURLException e) {
         e.printStackTrace();
      } catch (IOException e) {
         e.printStackTrace();
      }
   }

   @Override
   public Dimension getPreferredSize() {
      return initialSize;
   }

   @Override
   protected void paintComponent(Graphics g) {
      super.paintComponent(g);
      g.setColor(Color.BLACK);
      g.fillRect(0, 0, getSize().width, getSize().height);
      Graphics2D g2d = (Graphics2D) g.create();
      g2d.scale(scale, scale);
      g2d.drawImage(image, 0, 0, this);
   }

   public void keyReleased(KeyEvent e) {
   }

   public void keyTyped(KeyEvent e) {
   }

   public void keyPressed(KeyEvent e) {
      int key = e.getKeyCode();
      if (key == KeyEvent.VK_UP) {
         scale *= MULTIPLIER;
      }
      if (key == KeyEvent.VK_DOWN) {
         scale /= MULTIPLIER;
      }
      repaint();
   }
}

DrawImagePanelKeyBindings.java

import java.awt.*;
import javax.imageio.ImageIO;
import javax.swing.*;
import java.io.IOException;
import java.net.*;
import java.awt.event.*;

@SuppressWarnings("serial")
public class DrawImagePanelKeyBindings extends JPanel {
   public static final String IMAGE_PATH = "https://duke.kenai.com/"
         + "nyanya/.Midsize/NyaNya.jpg.png";
   private static final double MULTIPLIER = 1.2;
   double scale = 1.0;
   private Image image;
   private Dimension initialSize = new Dimension(0, 0);

   public static void main(String[] args) {
      SwingUtilities.invokeLater(new Runnable() {
         public void run() {
            DrawImagePanelKeyBindings drawImage = new DrawImagePanelKeyBindings();
            drawImage.add(new JButton("Foo"));
            JFrame frame = new JFrame("Draw Image");
            frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            frame.add(drawImage);
            frame.pack();
            frame.setLocationRelativeTo(null);
            frame.setVisible(true);
         }
      });
   }

   public DrawImagePanelKeyBindings() {
      setBindings();
      URL imgURL;
      try {
         imgURL = new URL(IMAGE_PATH);
         image = ImageIO.read(imgURL);
         initialSize = new Dimension(image.getWidth(this),
               image.getHeight(this));
         setVisible(true);
      } catch (MalformedURLException e) {
         e.printStackTrace();
      } catch (IOException e) {
         e.printStackTrace();
      }
   }

   private void setBindings() {
      int condition = WHEN_IN_FOCUSED_WINDOW;
      InputMap inputMap = getInputMap(condition);
      ActionMap actionMap = getActionMap();

      final KeyStroke[] keyStrokes = {
            KeyStroke.getKeyStroke(KeyEvent.VK_UP, 0),
            KeyStroke.getKeyStroke(KeyEvent.VK_DOWN, 0)
      };
      for (final KeyStroke keyStroke : keyStrokes) {
         inputMap.put(keyStroke, keyStroke.toString());
         actionMap.put(keyStroke.toString(), new AbstractAction() {
            @Override
            public void actionPerformed(ActionEvent evt) {
               myKeyPressed(keyStroke.getKeyCode());
            }
         });
      }
   }

   @Override
   public Dimension getPreferredSize() {
      return initialSize;
   }

   @Override
   protected void paintComponent(Graphics g) {
      super.paintComponent(g);
      g.setColor(Color.BLACK);
      g.fillRect(0, 0, getSize().width, getSize().height);
      Graphics2D g2d = (Graphics2D) g.create();
      g2d.scale(scale, scale);
      g2d.drawImage(image, 0, 0, this);
   }

   public void myKeyPressed(int key) {
      if (key == KeyEvent.VK_UP) {
         scale *= MULTIPLIER;
      }
      if (key == KeyEvent.VK_DOWN) {
         scale /= MULTIPLIER;
      }
      repaint();
   }
}