当不同线程中的标志更改时循环未结束 [重复]

Question

我在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 ' flag '变量的回显，程序会正确运行，但显然我不想在循环中打印任何内容。

我猜这个问题可能是Java编译器试图将空while循环优化到停止检查“flag”变量的结果？有没有人有正确完成这项工作的建议，或者一些更好的基于并发的方法，让主线程暂停，直到被按键的线程执行？

谢谢！

Answer 1

1.您需要为标志使用< code>volatile关键字，因为当我们对实例变量调用volatile时，就像告诉JVM确保访问它的线程必须将它自己的实例变量副本与存储在内存中的副本进行协调。

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

3.Volatile会使变量在一个线程中把它的变化值反映给另一个线程，但这并不妨碍竞争条件。

4.因此，在Atomic语句上使用synchronized关键字来设置标志的值。

例如：

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

Answer 2

虽然其他人提出的易失性解决方案应该可以工作，但除非你需要 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();
        }
    }

    ...

否则，您将反复浪费在主线程上的循环，一遍又一遍地检查标志的值（一般来说，如果发生这种情况足够多，可能会降低其他线程的CPU速度，并可能耗尽移动设备上的电池电量）。这正是 wait（） 所针对的情况。

Answer 3

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