JFrame MouseListener 不起作用

Question

我有一个子类 JFrame 的小程序。代码如下：

package complexviewer;  

import java.awt.*;  
import java.awt.event.*;    
import java.util.concurrent.*;  
import static java.util.concurrent.TimeUnit.*;  
import javax.swing.*;   

public class Applet extends JFrame {    

    // Window and applet stats  
    public static final int windowSize = 400;   
    public static final int updateDelay = 20;   

    // Instance variables   
    private ScheduledExecutorService timer = Executors.newScheduledThreadPool(1);   

    public static void main(String[] args) {    
        new Applet();   
    }   
    public Applet() {   
        // Set up window    
        super("Complex Viewing Frame"); 
        setSize(windowSize, windowSize);    
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 

        setVisible(true);

        // Add listeners    
        JPanel panel = new JPanel();    
        panel.setFocusable(true);   
        panel.addKeyListener(new KeyListener() {    
            public void keyPressed(KeyEvent key) {  
                System.out.println("Key pressed."); 
            }   
            public void keyReleased(KeyEvent key) {}    
            public void keyTyped(KeyEvent key) {}   
        }); 
        panel.addMouseListener(new MouseListener() {    
            public void mouseReleased(MouseEvent mouse) {}  
            public void mouseClicked(MouseEvent mouse) {}   
            public void mousePressed(MouseEvent mouse) {    
                System.out.println("Mouse clicked.");   
            }   
            public void mouseEntered(MouseEvent mouse) {}   
            public void mouseExited(MouseEvent mouse) {}    
        }); 
        panel.addMouseMotionListener(new MouseMotionListener() {    
            public void mouseDragged(MouseEvent mouse) {    
                System.out.println("Mouse dragged.");   
            }   
            public void mouseMoved(MouseEvent mouse) {} 
        }); 
        add(panel); 
        panel.requestFocus();   

        timer.scheduleAtFixedRate(() -> repaint(), 0, 200, MILLISECONDS);   
    }   

    public void paint(Graphics G) { 
        // Drawing  
    }   

}

当我运行它时，“按键”事件工作正常，但鼠标事件都不起作用。我在这里想念什么？我查看了其他答案，例如

• Problems With MouseListener
• Why isn't mouselistener working?
• MouseListener on JFrame
• MouseListener/KeyListener not working (JPanel)
• Unresponsive KeyListener for JFrame

此外，选择不同的组件，如 getContentPane() 似乎也不能解决问题。

Answer 1

如上所述，您应该在添加所有组件后调用setVisible(true)。其他问题：

• 不要将您的非小程序 GUI 命名为“Applet”，因为该名称与常用的核心 Java 类冲突，这会使其他审查您的工作的人非常困惑。你的类不是一个小程序，即使它是，也应该被称为别的东西。
• 你问，what structure would you suggest using besides JFrame?。我建议您很少扩展 JFrame，而是将代码用于创建（或如果需要，扩展）JPanel。这样做会增加类的灵活性，因为现在它可以在 GUI 中的任何位置使用，包括在 JFrame、另一个 JPanel、与 CardLayout 交换、在 JApplet 中......
• 永远不要直接在 JFrame 中进行绘制。
• 改为扩展 JPanel 并在其 paintComponent(...) 方法中绘制。
• 您没有显示您的绘画代码，但由于您在 Timer 中重复调用 repaint，而该计时器不执行任何其他操作，这表明您的类的状态在绘画方法中发生了更改, 永远不会发生的事情，因为您无法完全控制何时或什至何时调用绘画方法。
• 你说，According to this question, a javax.swing.Timer, which I heard was deprecated as I should use ScheduledExecutorService instead. 这是非常错误的。您应该使用 Swing Timer 来满足 Swing 计时器的需求，而不是 java.util.Timer。这些人都没有被弃用，但我在非 Swing 应用程序中使用 ScheduledExecutorService 而不是 java util Timer。
• 计时器代码应该更改类的状态并然后调用repaint()。
• 您会希望使用键绑定而不是 KeyListener。一方面，您可以避免强制 JPanel 或 JFrame 获得键盘焦点的麻烦。

Answer 2

事实证明，正如 Hovercraft Full of Eels 所建议的那样，问题在于 setVisible(true) 在 add(panel) 之前被调用。移动此语句允许所有侦听器正常工作。工作代码的可编译示例如下：

package complexviewer;  

import java.awt.*;  
import java.awt.event.*;    
import java.util.concurrent.*;  
import static java.util.concurrent.TimeUnit.*;  
import javax.swing.*;   

public class Applet extends JFrame {    

    // Window and applet stats  
    public static final int windowSize = 400;   
    public static final int updateDelay = 20;   

    // Instance variables   
    private ScheduledExecutorService timer = Executors.newScheduledThreadPool(1);   

    public static void main(String[] args) {    
        new Applet();   
    }   
    public Applet() {   
        // Set up window    
        super("Complex Viewing Frame"); 
        setSize(windowSize, windowSize);    
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 

        // Incorrect location for setVisible    

        // Add listeners    
        JPanel panel = new JPanel();    
        panel.setFocusable(true);   
        panel.addKeyListener(new KeyListener() {    
            public void keyPressed(KeyEvent key) {  
                System.out.println("Key pressed."); 
            }   
            public void keyReleased(KeyEvent key) {}    
            public void keyTyped(KeyEvent key) {}   
        }); 
        panel.addMouseListener(new MouseListener() {    
            public void mouseReleased(MouseEvent mouse) {}  
            public void mouseClicked(MouseEvent mouse) {}   
            public void mousePressed(MouseEvent mouse) {    
                System.out.println("Mouse clicked.");   
            }   
            public void mouseEntered(MouseEvent mouse) {}   
            public void mouseExited(MouseEvent mouse) {}    
        }); 
        panel.addMouseMotionListener(new MouseMotionListener() {    
            public void mouseDragged(MouseEvent mouse) {    
                System.out.println("Mouse dragged.");   
            }   
            public void mouseMoved(MouseEvent mouse) {} 
        }); 
        add(panel); 
        panel.requestFocus();   

        setVisible(true); // Correct location for setVisible    

        timer.scheduleAtFixedRate(() -> repaint(), 0, 200, MILLISECONDS);   
    }   

    public void paint(Graphics G) { 
        // Drawing  
    }   

}