Java节奏游戏按钮ActionListener无响应问题求助

你的按钮不响应ActionListener的核心原因是你根本没把实际的JButton组件添加到Game面板中，只是手动绘制了按钮的图片。另外Button类的设计存在冗余，继承了JButton却又内部维护了另一个JButton实例，完全没必要。

核心问题拆解 #

• Button类继承JButton后，又创建了独立的button实例，继承的父类实例从未被使用，属于冗余设计。
• Game类通过button.draw(g2d)手动绘制图片，而非将JButton组件添加到JPanel容器中。Swing的事件监听只对容器内的组件生效，你点击的只是绘制的像素，不是按钮组件。
• 重写paint方法而非paintComponent，不符合Swing的绘制规范。

修复方案 #

1. 重构Button类

去掉内部冗余的JButton实例，直接使用继承的JButton本身：

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

public class Button extends JButton implements ActionListener {
    public Button(int x, int y) {
        ImageIcon icon = new ImageIcon("Betterbutton.png");
        setIcon(icon);
        setBounds(x, y, 100, 100);
        addActionListener(this);
        setFocusable(false);
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        if (e.getSource() == this) {
            System.out.println("Clicked");
        }
    }
}

2. 修正Game类

不再手动绘制按钮，而是将Button组件添加到JPanel中，同时修正绘制方法：

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

public class Game extends JPanel implements Runnable, ActionListener {
    private Button[] buttons;

    public Game() {
        setLayout(null); // 配合setBounds使用空布局
        buttons = new Button[2];
        buttons[0] = new Button(100, 500);
        buttons[1] = new Button(500, 500);
        
        // 将按钮添加到面板，这是事件能触发的关键
        for (Button button : buttons) {
            add(button);
            button.addActionListener(this); // 如需在Game类监听事件，添加此行
        }
        
        new Thread(this).start();
    }

    @Override
    public void run() {
        try {
            while (true) {
                Thread.sleep(16); // 约60fps，比5ms更合理
                repaint();
            }
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }

    // 重写paintComponent而非paint，保证Swing组件正确绘制
    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        Graphics2D g2d = (Graphics2D) g;
        
        g2d.setFont(new Font("Broadway", Font.BOLD, 50));
        // 这里绘制游戏其他元素，按钮由Swing自动渲染，无需手动绘制
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        for (Button button : buttons) {
            if (e.getSource() == button) {
                System.out.println("Button clicked!");
                // 添加你的按钮点击逻辑
            }
        }
    }
}

额外提示 #

• 尽量避免setBounds和空布局，推荐使用LayoutManager管理组件位置，适配不同窗口大小。
• 线程中使用Thread.sleep(16)更接近60帧刷新频率，比5ms更高效。
• 不要忽略异常，打印异常信息方便调试。

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