7

我在我的 Java 程序的 main 方法中运行了一个 while 循环。循环应该一直运行，直到在程序的 keyPressed 方法中将布尔标志变量设置为 true（我将程序作为 KeyListener 添加到 JFrame）。

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import javax.swing.JFrame;

public class ThreadWhile implements KeyListener {
    private boolean flag = false;

    public static void main(String[] args) {
        //Set up key listening on a dummy JFrame
        JFrame frame = new JFrame("Key Detector 9000");
        frame.setVisible(true);
        ThreadWhile tw = new ThreadWhile();
        frame.addKeyListener(tw);

        System.out.println("Looping until flag becomes true...");
        while(!tw.flag) {
            //Commenting the println out makes the loop run forever!
            System.out.println(tw.flag); 
        }
        System.out.println("Flag is true, so loop terminated.");
        System.exit(0);
    }

    public void keyPressed(KeyEvent e) {
        flag = true;
        System.out.println("flag: " + flag);
    }

    public void keyReleased(KeyEvent e) {}
    public void keyTyped(KeyEvent e) {}
}

我的理解是 keyPressed 方法在它们自己的线程中执行，所以当我按下一个键时，变量 'flag' 应该设置为 true，并且在 main 方法中运行的 while 循环应该结束。

但是，当我运行这个程序时，循环会永远运行，即使我们可以看到“flag”变量被正确设置为 true！奇怪的是，如果我在 while 循环中插入“标志”变量的快速 System.out.println 回显，程序会正常运行，但显然我不想在循环中打印任何内容。

我猜这个问题可能是Java编译器试图优化空while循环到它停止实际检查'flag'变量的结果？有没有人有关于使这项工作正常工作的建议，或者可能有一些更漂亮的基于并发的方法来使主线程暂停，直到 keyPressed 线程执行？

谢谢！

4

3 回答 3

9

您需要声明标志volatile，否则编译器可以优化您的代码并跳过标志的读取。

于 2012-07-21T21:14:15.933 回答
4

虽然volatile其他人提出的解决方案应该可以工作，但除非您需要 while 循环中的代码连续执行，否则您可能应该在同步部分中使用wait()和notify()（或notifyAll()）（以避免“忙等待”）。就像是：

public class ThreadWhile implements KeyListener {
    private boolean flag = false;
    private Object flagLock = new Object();

    public static void main(String[] args) {
        //Set up key listening on a dummy JFrame
        JFrame frame = new JFrame("Key Detector 9000");
        frame.setVisible(true);
        ThreadWhile tw = new ThreadWhile();
        frame.addKeyListener(tw);

        System.out.println("Waiting until flag becomes true...");
        synchronized (tw.flagLock) {
            while(!tw.flag)
                tw.flagLock.wait();   // Note: this suspends the thread until notification, so no "busy waiting"
        }
        System.out.println("Flag is true, so loop terminated.");
        System.exit(0);
    }

    public void keyPressed(KeyEvent e) {
        synchronized (flagLock) {
            flag = true;
            System.out.println("flag: " + flag);
            flagLock.notifyAll();
        }
    }

    ...

否则，您将在主线程上反复浪费周期检查flag一遍又一遍的值（通常，如果这种情况发生得足够多，可能会减慢其他线程的 CPU 速度，并可能耗尽移动设备上的电池）。这正是为这种情况wait()而设计的。

于 2012-07-21T21:14:46.697 回答
0

1.您需要volatile为标志使用关键字，因为当我们在实例变量上调用 volatile 时，这就像告诉 JVM 确保访问它的线程必须将其自己的该实例变量的副本与存储在记忆。

2. Volatile 还有助于防止线程中值的缓存。

3. Volatile 将使该变量在一个线程中将其更改的值反映到另一个线程，但这不会阻止竞争条件。

4.所以在设置标志值的原子语句上使用同步关键字。

例如：

public void keyPressed(KeyEvent e) {
     synchronized(this){
        flag = true;
        System.out.println("flag: " + flag);
      }
    }
于 2012-07-21T21:32:44.150 回答