1

我有一个 GUI 类 Gui：

public class Gui extends JFrame implements Runnable
{

private JPanel outer, inner;
private JLabel[][] labels = new JLabel[22][12];
private Color[][] defaultMap, map;
Thread t;
private int row, col;
private Color color;

public Gui()
{
    Container content = getContentPane();
    content.setLayout(new BorderLayout());

    setBackground(Color.BLACK);
    setSize(1000, 1000);
    setLocation(300, 0);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setResizable(false);

    defaultMap = createMap();
    draw(defaultMap);
}

public Color[][] createMap()
{
    Color[][] map = new Color[22][12];
    for (int i = 0; i < 22; i++)
    {
        for (int j = 0; j < 12; j++)
        {
            map[i][j] = Color.WHITE;
        }
    }
    for (int i = 0; i < 22; i++)
    {
        map[i][0] = Color.GRAY;
        map[i][11] = Color.GRAY;
    }
    for (int i = 0; i < 12; i++)
    {
        map[0][i] = Color.GRAY;
        map[21][i] = Color.GRAY;
    }
    return map;
}

public void draw(Color[][] map)
{

    outer = new JPanel();
    outer.setLayout(new BorderLayout());
    outer.setBackground(Color.WHITE);
    outer.setPreferredSize(new Dimension());

    inner = new JPanel();
    inner.setLayout(new GridLayout(22, 12, 2, 2));
    inner.setBackground(Color.BLACK);

    for (int i = 0; i < 22; i++)
    {
        for (int j = 0; j < 12; j++)
        {
            labels[i][j] = new JLabel();
            JLabel label = labels[i][j];
            label.setPreferredSize(new Dimension(20, 20));
            label.setBackground(map[i][j]);
            label.setOpaque(true);
            inner.add(label);
        }
    }

    add(outer);
    add(inner);
    pack();
}

public void move(int row, int col, Color color)
{
    System.out.println(row+","+col);
    map = defaultMap;
    map[row][col] = color;
    t = new Thread(this);
    t.start();
}

@Override
public void run()
{
    draw(map);
}
}

这是从我的主课中调用的，如下所示：

public static void main(String[] args)
{
    SwingUtilities.invokeLater(new Runnable()
    {
        @Override
        public void run()
        {
            try
            {
                gui = new Gui();
                gui.setVisible(true);
                gui.move(2,5,Color.GREEN);
                Thread.sleep(1000);
                gui.move(3,5,Color.GREEN);
                Thread.sleep(1000);
                gui.move(4,5,Color.GREEN);
            } catch (InterruptedException ex)
            {
                Logger.getLogger(Tetris.class.getName()).log(Level.SEVERE, null, ex);
            }
        }
    });
}

因此，当调用 gui.move() 函数时，就会发生奇怪的事情。如果有帮助，您可以忽略其余部分或使用它。但是每次都应该在 gui 的 2,5 处“添加”一个绿色块；3,5; 和 4,5；1秒后。

问题：

Gui 保持黑色一段时间，然后立即使用正确的网格和颜色重新绘制/更新，前两个块正确着色为绿色，但它在 4,5 处缺少最后一个块。同样，应该立即绘制初始的“defaultMap”，但事实并非如此，JFrame 是黑色的，直到所有内容都立即绘制在一起减去最后一个绿色块。然后每个绿色块应该一个接一个地涂上 1 秒。

有趣的部分是 Gui 中 move 方法中的 System.out.println() 位打印出我期望的行和列...它们在终端中大约一秒钟后出现。所以这告诉我事情进展顺利。但是我不确定Gui是怎么回事...

编辑：故事略有不同。经过仔细检查，我注意到最后一个绿色块在整个地图被绘制后确实会出现一秒钟，但会立即“消失”并重新绘制为白色。

4

1 回答 1

6

您正在 EDT（事件调度线程）上睡觉。你永远不应该阻止 EDT。

  • 永远不要打电话sleep或wait在 EDT
  • 将所有长时间运行的任务移动到其他线程（使用Executorsor SwingWorker）
  • 对于 UI 的重复或延迟更新，您可以使用javax.swing.Timer
  • 所有 UI 操作都应在 EDT 上执行（使用SwingUtilities.invokeLaterorjavax.swing.Timer或SwingWorker

在swing 标签 wiki中阅读更多关于 Swing 并发的信息。

于 2013-05-15T14:11:17.733 回答