0

我在一个线程中有以下方法，它可以 100% 工作，但是一旦我删除System.out.println("here");代码，它就会停止做它正在做的事情。没有错误，它只是看起来什么都不做。它的作用是使图像的颜色变亮，其中的调试线变亮，没有它，它不会变亮。为什么会这样？

线程类：

package pocketshop.threads;

import com.jogamp.opencl.CLBuffer;
import java.awt.Container;
import java.awt.image.BufferedImage;
import java.nio.FloatBuffer;
import pocketshop.Canvas;
import pocketshop.graphics.CL;
import pocketshop.graphics.Preview;

/**
 *
 * @author Ryan
 */
public class AdjustThread extends Thread {

    protected float amount = 0;
    protected CLBuffer<FloatBuffer> buffer;
    protected String adjustment;
    protected Container parent;

    public AdjustThread(Container parent, String adjustment) {
        this.parent = parent;
        this.adjustment = adjustment;
    }
    public void setAmount(float amount){
        this.amount = amount;
    }

    public CLBuffer<FloatBuffer> getBuffer() {
        return buffer;
    }

    public void run() {
        float cAmount = 0;
        while(true){
            System.out.println("here");
            if(cAmount != this.amount){
                cAmount = this.amount;
                CL.start(adjustment, this.amount);
                buffer = CL.getBuffer();

                float[] pixels = CL.getPixels();
                BufferedImage newimage = new BufferedImage(Canvas.image.getWidth(), Canvas.image.getHeight(), BufferedImage.TYPE_INT_RGB);
                buffer.getBuffer().get(pixels).rewind();
                newimage.getRaster().setPixels(0, 0, Canvas.image.getWidth(), Canvas.image.getHeight(), pixels);
                Preview.setImage(newimage);
                Canvas.preview = Preview.getImage();
                parent.repaint();
            }
        }
    }
}

以及对话框相关代码：

package pocketshop.dialogs;

import java.awt.image.BufferedImage;
import pocketshop.Canvas;
import pocketshop.graphics.adjustments.Contrast;
import pocketshop.threads.AdjustThread;

/**
 *
 * @author Ryan
 */
public class BrightnessContrastDialog extends javax.swing.JDialog {

    AdjustThread adj;

    /**
     * Creates new form BrightnessContrastDialog
     */
    public BrightnessContrastDialog(java.awt.Frame parent, boolean modal) {
        super(parent, modal);
        initComponents();
        adj = new AdjustThread(this.getParent(), "Brightness");
        adj.start();
    }

    // Run everytime the JSlider moves
    private void sldBrightnessStateChanged(javax.swing.event.ChangeEvent evt) {                                           
        float val = sldBrightness.getValue();
        txtBrightness.setText("" + (int) val);
        adj.setAmount(val);
    }
}
4

1 回答 1

1

被this.amount声明为volatile?

我认为这个字段是由于另一个线程中的分配而改变的。如果它没有被声明为volatile，那么没有理由假设运行上述run()方法的线程会观察到它的变化。在您第一次分配this.amount给cAmount之后，它们在此后保持相等 - 从这个线程的角度来看，无论如何。

一旦您为我们阐明了声明的限定符this.amount并显示在其他地方更改的代码片段，我们可以帮助指定恢复所需行为所需的适当同步设备。

至于为什么调用PrintStream#println()似乎在这里有所作为，它可能不仅会导致延迟，而且还可能会在内存可见性边缘发生之前发生，这使得更改this.amount对该线程可见。这是一个很大的挥手，但我认为在确定特定副作用的根本原因之前，这里有更大的问题需要先解决。

于 2012-11-18T00:36:44.060 回答