使用Timer/TimerTask每秒重绘AWT JFrame时图形无法显示的问题

你遇到的核心问题有三个：

1. 在paint方法内重复创建Timer：每次系统触发paint（比如窗口初始化、大小改变）都会新建一个Timer，导致多个定时任务同时运行，数据更新逻辑混乱。
2. 无效的Graphics上下文：paint方法的Graphics对象是临时的，paint执行完毕后该对象会被系统回收，TimerTask里使用这个过期的Graphics根本无法在窗口上绘制内容。
3. 线程安全问题：java.util.TimerTask的run方法运行在后台线程，而AWT/Swing的绘图操作必须在**事件调度线程（EDT）**中执行，跨线程直接操作绘图上下文会导致绘图失效。

修正后的实现方案 #

1. 用JPanel作为绘图容器（推荐） #

不要直接在JFrame的paint方法里绘图，改用JPanel的paintComponent方法，它是专门用于自定义绘图的入口，且会自动处理背景刷新：

class DrawingPanel extends JPanel {
    private PhysicsController jTP;
    private int scale;

    public DrawingPanel(PhysicsController jTP, int scale) {
        this.jTP = jTP;
        this.scale = scale;
        setBackground(Color.WHITE); // 初始化背景色
    }

    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g); // 必须调用父类方法，确保组件绘制流程正确
        // 遍历绘制所有物理对象
        for (PhysicsObject obj : jTP.physicsObjectArrayList) {
            obj.drawObject(g, scale);
        }
    }
}

2. 把Timer移到类初始化逻辑中（仅创建一次） #

使用javax.swing.Timer替代java.util.Timer，它默认在EDT线程执行任务，无需额外处理线程安全：

public class MainFrame extends JFrame {
    private DrawingPanel drawingPanel;
    private PhysicsController jTP;
    private int scale;

    public MainFrame(PhysicsController jTP, int scale) {
        this.jTP = jTP;
        this.scale = scale;

        // 初始化绘图面板并添加到窗口
        drawingPanel = new DrawingPanel(jTP, scale);
        add(drawingPanel);

        // 窗口基础配置
        setSize(800, 600);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setVisible(true);

        // 初始化Swing定时器，每秒执行一次
        javax.swing.Timer updateTimer = new javax.swing.Timer(1000, e -> {
            // 1. 更新物理对象的位置数据
            for (int i = 0; i < jTP.physicsObjectArrayList.size(); i++) {
                PhysicsObject obj = jTP.physicsObjectArrayList.get(i);
                obj.setxPos(obj.getxPos() + getForceX(i) + getVelocityX(i));
                obj.setyPos(obj.getyPos() + getForceY(i) + getVelocityY(i) + getGravity(i));
            }
            // 2. 触发重绘，系统会自动在EDT调用paintComponent
            drawingPanel.repaint();
        });
        updateTimer.start(); // 启动定时器
    }

    // 以下是你的物理计算方法，根据实际实现补充
    private int getForceX(int index) { return 0; }
    private int getVelocityX(int index) { return 0; }
    private int getForceY(int index) { return 0; }
    private int getVelocityY(int index) { return 0; }
    private int getGravity(int index) { return 0; }
}

3. 保留原有的drawObject方法 #

你的drawObject方法逻辑没问题，直接保留即可：

public class PhysicsObject {
    // ... 你的成员变量（xPos, yPos, size, colors, rand等）
    public void drawObject(Graphics g, int scale){
        g.setColor(colors[rand]);
        g.fillOval(xPos, yPos, scale * size, scale * size);
    }
    // ... get/set方法
}

关键修正点说明 #

• 分离数据更新与绘图逻辑：定时器只负责更新物理对象的位置数据，绘图完全交给paintComponent方法，由系统统一调度。
• 使用Swing专属定时器：javax.swing.Timer的任务在EDT执行，避免跨线程操作GUI组件的风险。
• 避免在paint内创建资源：paint方法会被频繁调用，重复创建Timer会导致资源泄漏和逻辑混乱。

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