java - Swing 中两个线程上的同步块(synchronized block)不起作用

Question

在概述问题之前，让我先向您描述一下问题的背景。我目前正在编写一个游戏引擎关卡编辑器，并且正在研究将充当用户交互以构建其关卡的屏幕的类。我想让屏幕与编辑器的大小成比例。

当我开始调整屏幕大小并同时在屏幕上绘图时，就会出现相关问题。我从一个线程进行绘制，同时从另一个线程(EDT)编辑我正在绘制的原始像素数组的大小。我知道这是一个很大的禁忌，因此很自然地，由于没有适当的安全措施，我在调整大小时偶尔会遇到 IndexOutOfBounds 异常。

我的想法是，我可以在调整大小代码和绘图代码上添加同步块(synchronized block)。这样，就不会有冲突，问题就可以避免。然而，同步被完全忽略了。我仍然遇到同样的错误，而且我真的很困惑为什么它不起作用。以下是两种感兴趣的方法:

public void setPixel(int r, int g, int b, int x, int y) {
    synchronized (pixels){
        System.out.println("Start Draw...");
        int color = (r << 16) | (g << 8) | b;
        pixels[y * screenWidth + x] = color;
        System.out.println("End Draw...");
    }
}


@Override
public void componentResized(ComponentEvent e) {
        synchronized (pixels) {
            System.out.println("Start resize");
            int width = e.getComponent().getWidth();
            int height = e.getComponent().getHeight();
            float aspectRatio = 4 / 3f;
            if (width > height) {
                width = (int) (height * aspectRatio);
            } else if (height > width) {
                height = width;
            }
            if (width < 0 || height < 0) {
                width = 1;
                height = 1;
            }
            this.screenWidth = width;
            this.screenHeight = height;
            image = new BufferedImage(screenWidth, screenHeight, BufferedImage.TYPE_INT_RGB);
            pixels = ((DataBufferInt) image.getRaster().getDataBuffer()).getData();
            System.out.println("End Resize");
        }
    }

我不知道这是否重要(它不应该对吗？)，但我的屏幕类扩展了 AWT Canvas。此外，它还是其父组件的监听器，因此当调整其大小时，它会触发一个事件，触发调用 componentResized。无论如何，谢谢您，感谢任何帮助。

编辑:我的绘图代码可以在下面找到。

new Thread(new Runnable(){
    @Override
    public void run() {
        while(true) {
            for (int y = 0; y < screen.getHeight(); y++) {
                for(int x = 0; x < screen.getWidth(); x++){
                    int r = (int) (Math.random() * 255);
                    int g = (int) (Math.random() * 255);
                    int b = (int) (Math.random() * 255);
                    screen.setPixel(r, g, b, x, y);
                }
            }
        }
    }
}).start();

Answer 1

我还能想到一些别的东西。不仅仅是问题评论中的内容。

让我先修改setPixel方法:

public void setPixel(int r, int g, int b, int x, int y) {
    System.out.println("Called with: " + x + ", " + y); //Just added a print statement.
    synchronized (mySyncGuard){
        System.out.println("Start Draw...");
        int color = (r << 16) | (g << 8) | b;
        pixels[y * screenWidth + x] = color;
        System.out.println("End Draw...");
    }
}

其中mySyncGuard是用作同步保护的最终属性setPixel和componentResized .

现在想象一下以下场景:

有一个循环调用 setPixel method:该循环调用从 x = 0 开始的方法和y = 0高达 x < screenWidth和y < screenHeight !就像下面这样:

for (int x = 0; x < screenWidth; ++x)
    for (int y = 0; y < screenHeight; ++y) {
        int r = calculateNextR(),
            g = calculateNextG(),
            b = calculateNextB();
        setPixel(r, g, b, x, y);
    }

其中calculateNextR()、calculateNextG()和calculateNextB()分别是产生下一个红色、绿色和蓝色分量的方法。

现在，例如，让 screenWidth 为 200，screenHeight 也为 200，并在某个时刻将大小调整为 100x100。

现在的问题是，当组件的大小即将调整为 100、100 时，x 和 y 分别为 150 和 150。

我们的定制setPixel现在调用 x==150 且 y==150 的方法。 但是在我们添加显示 x 和 y 值的打印语句之前，componentResized设法被调用并获取同步保护属性mySyncGuard的锁!所以componentResized现在正在执行将图像大小更改为 100x100 的工作，这反过来又将 像素 数组更改为比之前的数据数组更小的。

同时，setPixel现在打印“Called with 150, 150”并在同步块(synchronized block)处等待以获取mySyncGuard的锁，因为componentResized当前已获取它并将图像更改为 100x100 并相应地更改 像素 数组。

所以现在像素数组变小了，componentResized完成调整大小，最后setPixel可以获得mySyncGuard的锁。

现在的问题是:数组像素在位置 150,150 处被遍历，而其实际大小为 100x100。那么就这样吧! IndexOutOfBoundsException...

结论:

1. 我们需要您的更多代码来确定问题所在。
2. 变化的变量(例如screenWidth、screenHeight、image和pixels)需要在各处同步，不仅在 setPixel 内和componentResized方法。例如，如果您的情况类似于我刚才描述的场景，那么不幸的是 for 循环也应该位于 synchronized (mySyncGuard) 中。阻止。
3. 这不适合评论。如果您发布更多代码，那么我们可能会告诉您出了什么问题(如果不需要，我可能会删除这个答案)。