image 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，而是在这样的项目中使用键绑定

---

编辑
其他问题：

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

  ---

编辑2
正如您的代码所示，您永远不想忽略异常，因为这相当于盲目驾驶：

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

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

---

编辑3
更多关于KeyListener vs.Key Bindings的信息：假设您的代码与KeyListener一起工作，然后假设您将任何其他可聚焦组件添加到GUI中，并且它们通过用户交互以某种方式获得了焦点，那么您的KeyBindings将不再工作。如果您使用密钥绑定正确地完成了这一点，那么这就不会是一个问题

---

编辑4
你真的希望你的比例场是一个双精度的，而不是一个整数。你真的不想增加和减少它，而是想乘以它，然后除以某个乘数常数，比如说1.2。当你改变刻度时，你也会想打电话给repaint()

---

编辑5
请查看两个示例程序，第一个名为DrawImagePanelKeyListener。java使用一个KeyListener，而第二个叫做DrawImagePanelKeyBindings，使用键绑定。它们都应该按预期编译、运行和运行：当您按下向上或向下箭头键时，显示的图像会缩小和增大。当按下两个按钮时，可以看到它们行为上的差异。按下按钮，查看按键响应发生了什么变化。当具有KeyListener的组件失去焦点时，其KeyListener将停止工作，但对于使用键绑定的组件，情况并非如此

解决这个问题的一个难题可能是阻止所有其他组件获得关注，但这对大多数GUI来说既不实际，也不理想

DrawImagePanelKeyListener。爪哇

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();
   }
}

DrawImagePanelKeyBinding。爪哇

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();
   }
}