java - 鼠标滚轮闪烁，更多关于原因

Question

几天前，我发布了一个关于一个程序的问题，该程序导致滚动鼠标滚轮时屏幕上的文本改变颜色。不幸的是，这是一个组合得很糟糕的问题，发布了太多特别有用的代码。

我收到了几个回复，其中之一来自用户垃圾狗，他发布了解决问题的内容(可以在本页底部找到:Window going blank during MouseWheelMotion event)，但是阅读了所有他发布的程序和执行过程中我不知道的事情我不明白为什么他的效果与我的不同。

他似乎记录了每次鼠标滚轮的移动，而我的只记录了初始移动。还有一些人评论说他们无法复制我的程序的效果，可能是因为它太大了。

下面是一个极其简化的版本，仍然能产生相同的效果(我希望)。

问题:修复处理鼠标滚轮事件时屏幕变黑的两个程序之间的根本区别是什么？

import java.awt.Color;
import java.awt.Font;
import java.awt.Graphics;
import java.awt.event.MouseWheelEvent;
import java.awt.event.MouseWheelListener;
import java.util.LinkedList;
import javax.swing.JFrame;

public class WheelPrinter implements MouseWheelListener, Runnable {

    JFrame frame;
    LinkedList colorList;
    int colorCount;

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        WheelPrinter w = new WheelPrinter();
        w.run();
    }

    public WheelPrinter() {
        frame = new JFrame();
        frame.setSize(500, 500);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.addMouseWheelListener(this);
        frame.setVisible(true);
        frame.setBackground(Color.WHITE);
        colorList = new LinkedList();
        colorList.add(Color.BLACK);
        colorList.add(Color.BLUE);
        colorList.add(Color.YELLOW);
        colorList.add(Color.GREEN);
        colorList.add(Color.PINK);
    }

    @Override
    public void mouseWheelMoved(MouseWheelEvent e) {
        colorChange();
    }

    @Override
    public void run() {
        while(true) {
            draw(frame.getGraphics());
            try {
                Thread.sleep(20);
            } catch (Exception ex) {

            }
        }
    }

    public void draw(Graphics g) {
        g.setColor(frame.getBackground());
        g.fillRect(0,0,frame.getWidth(),frame.getHeight());
        g.setFont(new Font("sansserif", Font.BOLD, 32));
        g.setColor(frame.getForeground());
        g.drawString("yes", 50, 50);
    }

    public void colorChange() {
                colorCount++;
        if (colorCount > 4) {
            colorCount = 0;
        }


        frame.setForeground((Color) colorList.get(colorCount));
    }

    }

(如果你尝试运行我的代码，请尝试用力旋转鼠标滚轮，它会变得更加明显)

Answer 1

1. while(true) { 是无限循环，没有break；

2. 使用 Swing Timer 而不是由 Thread.Sleep() 延迟的 Runnable#Thread

3. 绘制到 JPanel 或 JComponent，而不是直接绘制到 JFrame

4. 对 Swing JComponent 的所有绘制都应该在 paintComponent()

5. 更多内容请参见 2D Graphics tutorial

编辑

import java.awt.Color;
import java.awt.Dimension;
import java.awt.EventQueue;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseWheelEvent;
import java.util.LinkedList;
import java.util.Queue;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

/**
 * based on example by @trashgod
 * 
 * @see http://stackoverflow.com/a/10970892/230513
 */
public class ColorWheel extends JPanel {

    private static final int N = 32;
    private static final long serialVersionUID = 1L;
    private final Queue<Color> clut = new LinkedList<Color>();
    private final JLabel label = new JLabel();

    public ColorWheel() {
        for (int i = 0; i < N; i++) {
            clut.add(Color.getHSBColor((float) i / N, 1, 1));
        }
        //clut.add(Color.BLACK);
        //clut.add(Color.BLUE);
        //clut.add(Color.YELLOW);
        //clut.add(Color.GREEN);
        //clut.add(Color.PINK);
        label.setFont(label.getFont().deriveFont(36f));
        label.setForeground(clut.peek());
        label.setText("@see http://stackoverflow.com/a/10970892/230513");
        setBackground(Color.white);
        add(label);
        label.addMouseWheelListener(new MouseAdapter() {

            @Override
            public void mouseWheelMoved(MouseWheelEvent e) {
                label.setForeground(clut.peek());
                clut.add(clut.remove());
            }
        });
    }

    @Override
    public Dimension getPreferredSize() {
        int w = SwingUtilities.computeStringWidth(label.getFontMetrics(
                label.getFont()), label.getText());
        return new Dimension(w + 20, 80);
    }

    private void display() {
        JFrame f = new JFrame("ColorWheel");
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.add(this);
        f.pack();
        f.setLocationRelativeTo(null);
        f.setVisible(true);
    }

    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {

            @Override
            public void run() {
                new ColorWheel().display();
            }
        });
    }
}