自定义按钮拖动JFrame卡顿问题求助

我采用了类似Java自定义形状可拖动JFrame的方法来移动JFrame：

1. 编写了继承自JPanel的类，定义坐标变量：

private int pressX, pressY

2. 在mousePressed方法中记录点击位置：

pressX = e.getX(); pressY = e.getY();

3. 在mouseDragged方法中实现窗口移动：

mainFrame.setLocation((int) Math.round(mainFrame.getLocation().getX() + e.getX() - pressX), (int) Math.round(mainFrame.getLocation().getY() + e.getY() - pressY));

但拖动窗口时存在明显卡顿问题。我使用Swing库，通过一个约每2毫秒触发一次的Timer进行重绘。

编辑1： #

修改代码将坐标转换为JPanel相对坐标，但问题仍存在：

dragMe = new Draggable() { 
    private static final long serialVersionUID = 1L; 
    private Point press; 
    @Override public void mouseClicked(MouseEvent e) {} 
    @Override public void mousePressed(MouseEvent e) { 
        press = SwingUtilities.convertPoint(this, e.getPoint(), mainFrame); 
    } 
    @Override public void mouseReleased(MouseEvent e) {} 
    @Override public void mouseEntered(MouseEvent e) {} 
    @Override public void mouseExited(MouseEvent e) {} 
    @Override public void mouseDragged(MouseEvent e) { 
        Point drag = SwingUtilities.convertPoint(this, e.getPoint(), mainFrame); 
        mainFrame.setLocation((int) Math.round(mainFrame.getLocation().getX() + drag.getX() - press.getX()), 
                              (int) Math.round(mainFrame.getLocation().getY() + drag.getY() - press.getY())); 
    } 
    @Override public void mouseMoved(MouseEvent e) {} 
};

编辑2： #

遗憾的是，我创建的测试示例运行完全正常，但真实项目中问题依旧，即使将测试类复制到真实项目中仍存在卡顿：

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Point;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;
import javax.swing.Timer;
import javax.swing.WindowConstants;

public class Test extends JPanel implements ActionListener {
    private static final long serialVersionUID = 1L;
    private static JFrame frame;

    public static void main(String[] args) {
        class Draggable extends JPanel implements MouseListener, MouseMotionListener {
            private static final long serialVersionUID = 1L;
            private Point press;

            public Draggable() {
                addMouseListener(this);
                addMouseMotionListener(this);
            }

            @Override public void mouseClicked(MouseEvent e) {}
            @Override public void mousePressed(MouseEvent e) {
                press = SwingUtilities.convertPoint(this, e.getPoint(), frame);
            }
            @Override public void mouseReleased(MouseEvent e) {}
            @Override public void mouseEntered(MouseEvent e) {}
            @Override public void mouseExited(MouseEvent e) {}
            @Override public void mouseDragged(MouseEvent e) {
                Point drag = SwingUtilities.convertPoint(this, e.getPoint(), frame);
                frame.setLocation((int) Math.round(frame.getLocation().getX() + drag.getX() - press.getX()),
                                  (int) Math.round(frame.getLocation().getY() + drag.getY() - press.getY()));
            }
            @Override public void mouseMoved(MouseEvent e) {}
        }

        Test t = new Test();
        t.setBounds(0, 0, 1200, 600);
        t.setVisible(true);

        Draggable drag = new Draggable();
        drag.setBounds(24, 24, 24, 24);
        drag.setVisible(true);

        Timer repaintTimer = new Timer(2, t);

        frame = new JFrame();
        frame.setSize(1200, 600);
        frame.add(t);
        frame.add(drag);
        Dimension dim = java.awt.Toolkit.getDefaultToolkit().getScreenSize();
        frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        frame.setLocation((dim.width - frame.getWidth()) / 2, (dim.height - frame.getHeight()) / 2);
        frame.getContentPane().setLayout(null);
        frame.setAlwaysOnTop(true);
        frame.setResizable(false);

        repaintTimer.start();
        frame.setVisible(true);
        frame.requestFocus();
    }

    @Override public void actionPerformed(ActionEvent e) {
        repaint();
    }

    @Override public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.setColor(Color.BLACK);
        g.fillRect(24, 24, 24, 24);
    }
}

恳请协助排查该卡顿问题的原因及解决方法。

你的测试代码正常但真实项目卡顿，核心问题大概率出在真实项目的额外负载或Swing事件线程阻塞上，结合你的场景，我整理了几个关键排查方向和解决办法：

1. 先调整高频重绘Timer的间隔 #

2毫秒的Timer意味着每秒要触发500次重绘，这对Swing来说是极大的性能负担——Swing的所有UI操作都在事件派发线程（EDT）上执行，高频重绘会占据大量EDT资源，导致拖动事件的处理被延迟，直接表现为卡顿。

• 解决办法：
  • 把Timer间隔调整到16-33毫秒（对应60-30帧/秒），这是视觉流畅的最低要求，同时不会过度占用EDT。
  • 如果界面不需要持续重绘，完全可以去掉Timer，改用repaint()按需触发（比如只有当界面元素变化时才调用）。

2. 检查真实项目中的EDT阻塞 #

拖动卡顿的常见原因是EDT被耗时操作阻塞了——比如在EDT里执行数据库查询、文件IO、复杂计算等，导致鼠标拖动事件无法及时处理。

• 排查方法：
  • 在关键代码处用SwingUtilities.isEventDispatchThread()检查是否在EDT执行了耗时操作。
  • 用VisualVM等工具分析线程状态，看EDT是否长时间处于阻塞状态。
• 解决办法：
  • 所有耗时操作都放到单独的后台线程（比如SwingWorker）中执行，绝对不要占用EDT。

3. 优化拖动逻辑的性能 #

你的拖动代码里用了Math.round()和多次getLocation()调用，单次开销虽小，但高频触发时会累积性能损耗：

// 原代码
mainFrame.setLocation((int) Math.round(mainFrame.getLocation().getX() + e.getX() - pressX), 
                      (int) Math.round(mainFrame.getLocation().getY() + e.getY() - pressY));

• 优化建议：
  • 直接用整数运算代替浮点运算，因为窗口位置本身是整数：

    // 优化后
    Point currentLoc = mainFrame.getLocation();
    int newX = currentLoc.x + (e.getX() - pressX);
    int newY = currentLoc.y + (e.getY() - pressY);
    mainFrame.setLocation(newX, newY);
    

  • 更简洁的方式是直接记录屏幕坐标，避免坐标转换：

    private Point pressScreen;
    private Point windowStartLoc;
    
    @Override
    public void mousePressed(MouseEvent e) {
        pressScreen = e.getLocationOnScreen();
        windowStartLoc = mainFrame.getLocation();
    }
    
    @Override
    public void mouseDragged(MouseEvent e) {
        Point currentScreen = e.getLocationOnScreen();
        int newX = windowStartLoc.x + (currentScreen.x - pressScreen.x);
        int newY = windowStartLoc.y + (currentScreen.y - pressScreen.y);
        mainFrame.setLocation(newX, newY);
    }
    

4. 检查真实项目中的UI组件负载 #

真实项目可能包含大量复杂组件（比如表格、树、自定义绘制元素），这些组件的重绘会消耗更多资源。拖动窗口时，系统需要同时处理窗口移动和组件重绘，容易导致卡顿。

• 解决办法：
  • 拖动时临时禁用非必要组件的重绘：在mousePressed时调用component.setIgnoreRepaint(true)，mouseReleased时恢复为false。
  • 优化自定义组件的paintComponent方法，避免重复计算、减少不必要的绘制操作（比如只绘制变化的区域）。

5. 系统层面的排查点 #

有时候卡顿可能和系统设置有关：

• 检查是否开启了系统窗口拖动特效（比如Windows的“拖动时显示窗口内容”），老旧硬件或性能不足的机器会因此卡顿，可尝试关闭特效测试。
• 确保使用较新的Java版本，新版本Swing在UI性能上有不少优化。

内容的提问来源于stack exchange，提问作者Gigi Bayte 2