JPanel上绘制小球无法显示在不同位置的问题求助

你的问题根源在于错误地将每个小球设计成了独立的JPanel组件，这完全违背了Swing绘制自定义图形的正确逻辑。让我一步步帮你理清问题并修复：

为什么你的代码会出现这种奇怪现象？ #

1. 布局管理器的干扰：你每次点击创建一个Ball（继承自JPanel）并添加到主面板，而主面板默认使用FlowLayout布局管理器。这个布局会自动控制组件的摆放位置，你传入的x,y根本没有被用到，所以所有Ball组件都被挤在面板左上角，看起来就像同一个位置。
2. 坐标体系混淆：Ball类的paintComponent方法里，你在(0,0)位置绘制椭圆，这个坐标是相对于Ball自身组件的左上角，而不是主面板的全局坐标。就算后续拖动窗口触发布局重新计算，每个小球也只会在自己组件的左上角显示，完全达不到“点击位置绘制小球”的效果。

正确的实现思路 #

我们不需要把每个小球做成独立组件，而是采用更高效的自定义绘制方案：

• 用一个**纯数据类Ball**来存储小球的位置、大小等信息；
• 自定义一个主面板，重写它的paintComponent方法，在这个方法里遍历所有小球，逐个绘制在对应位置；
• 鼠标点击时，创建Ball对象加入列表，然后触发主面板的重绘。

修改后的代码示例 #

1. 重构Ball类（纯数据+绘制方法） #

public class Ball {
    int x;
    int y;
    int size; // 统一用size表示小球直径，更直观

    public Ball(int x, int y, int size) {
        this.x = x;
        this.y = y;
        this.size = size;
    }

    // 提供绘制方法，接收全局的Graphics对象
    public void draw(Graphics g) {
        // 让小球圆心对准点击位置（x,y），而不是左上角，更符合直觉
        g.fillOval(x - size/2, y - size/2, size, size);
    }
}

2. 重构GUIBalls类（自定义绘制面板） #

import javax.swing.*;
import java.awt.*;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.util.ArrayList;

public class GUIBalls {
    public JFrame frame = new JFrame("Balls");
    private ArrayList<Ball> balls = new ArrayList<>();
    // 自定义绘制面板，负责所有小球的绘制逻辑
    private JPanel drawPanel = new JPanel() {
        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            // 遍历所有小球，逐个绘制在对应位置
            for (Ball ball : balls) {
                ball.draw(g);
            }
        }
    };

    public GUIBalls(){
        setFrame();
    }

    public void setFrame(){
        frame.setBounds(0,0,400,400);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); // 加上关闭程序的操作，避免后台残留
        drawPanel.setLayout(null); // 不需要布局管理器，我们自己控制绘制位置
        drawPanel.addMouseListener(new MouseAdapter() {
            // 用MouseAdapter只重写需要的方法，避免冗余代码
            @Override
            public void mousePressed(MouseEvent e) {
                int x = e.getX();
                int y = e.getY();
                createBall(x,y);
            }
        });
        frame.add(drawPanel);
        frame.setVisible(true); // 最后设置窗口可见，确保组件初始化完成
    }

    public void createBall(int x, int y){
        Ball ball = new Ball(x,y,20); // 小球直径设为20，显示更明显
        balls.add(ball);
        drawPanel.repaint(); // 触发面板重绘，刷新所有小球
    }

    public static void main(String[] args) {
        // 确保Swing组件在事件调度线程（EDT）中创建，这是Swing的规范要求
        SwingUtilities.invokeLater(() -> new GUIBalls());
    }
}

额外的优化建议 #

• 使用MouseAdapter替代MouseListener：你只需要处理鼠标点击事件，MouseAdapter允许你只重写需要的方法，代码更简洁；
• 遵循Swing线程规范：所有Swing组件的创建和操作都应该在EDT线程中进行，所以main方法里用SwingUtilities.invokeLater；
• 让小球圆心对准点击位置：原来的绘制方式会让小球左上角在点击位置，现在调整为圆心在点击位置，更符合用户操作直觉；
• 添加窗口关闭操作：避免关闭窗口后程序还在后台运行。

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