输入1或2时JPanel无法绘制及移动正方形的问题求助

核心问题原因 #

你的代码存在多个违反Swing编程规范的严重问题，导致UI无法正常更新：

• 阻塞事件调度线程（EDT）：Swing的UI更新和事件处理都依赖EDT，你在main方法里写了死循环，又在paintComponent里用Scanner阻塞线程，直接导致EDT无法处理重绘请求。
• paintComponent方法滥用：这个方法是系统用来触发绘制的回调，不能在里面写业务逻辑（比如输入循环、新建窗口），每次调用都会重新初始化变量，绘制逻辑无法保留状态。
• UI组件初始化顺序错误：先调用frame.setVisible(true)再添加DrawSquare，组件无法正确挂载到窗口。
• 状态未持久化：正方形的位置x是paintComponent的局部变量，修改后无法保留，重绘时又回到初始值。

修正后的代码 #

主类 App.java #

import javax.swing.*;
import java.util.Scanner;

public class App {
    public static void main(String[] args) {
        // 确保Swing组件在EDT中初始化
        SwingUtilities.invokeLater(() -> {
            JFrame frame = new JFrame("test");
            frame.setSize(1000, 1000);
            frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            
            DrawSquare drawPanel = new DrawSquare();
            frame.add(drawPanel);
            
            // 先添加组件再设为可见
            frame.setVisible(true);
            
            // 启动单独线程处理控制台输入，避免阻塞EDT
            new Thread(() -> {
                Scanner scanner = new Scanner(System.in);
                while (true) {
                    System.out.println("Enter command 1=draw square 2= move square");
                    int input = scanner.nextInt();
                    System.out.println("Next step: " + input);
                    
                    if (input == 2) {
                        drawPanel.moveSquare();
                    } else if (input == 1) {
                        drawPanel.setDrawSquare(true);
                    }
                    // 触发重绘
                    drawPanel.repaint();
                }
            }).start();
        });
    }
}

绘制面板类 DrawSquare.java #

import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JPanel;

public class DrawSquare extends JPanel {
    // 成员变量保存正方形状态和位置
    private int squareX = 90;
    private boolean shouldDrawSquare = false;

    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g); // 先调用父类方法清空面板
        g.setColor(Color.RED);
        
        if (shouldDrawSquare) {
            // 绘制正方形，使用成员变量的位置
            g.fillRect(squareX, squareX, 100, 100);
        }
    }

    // 移动正方形的方法
    public void moveSquare() {
        squareX += 10; // 每次移动10像素，更明显
        shouldDrawSquare = true; // 移动后自动显示正方形
    }

    // 设置是否绘制正方形
    public void setDrawSquare(boolean draw) {
        shouldDrawSquare = draw;
    }
}

关键修正点 #

• 使用EDT初始化UI：通过SwingUtilities.invokeLater确保所有Swing组件在EDT中创建，符合Swing单线程规则。
• 单独线程处理输入：控制台输入放在独立线程中，不会阻塞EDT，保证UI能响应重绘请求。
• 状态持久化：把正方形的位置和绘制状态作为DrawSquare的成员变量，修改后能保留，重绘时使用最新值。
• 正确使用paintComponent：只在该方法中做绘制逻辑，不包含任何业务或阻塞代码，依赖成员变量的状态来决定绘制内容。
• 组件初始化顺序：先添加DrawSquare到窗口，再设置窗口可见，确保组件能正确渲染。

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