Java中的线程安全全局变量

时间:2013-05-03 14:32:44

标签: java multithreading thread-safety

我试图理解java中的线程安全机制,我需要一些帮助。我有一节课:

public class ThreadSafe {

    private Executor executor = new ScheduledThreadPoolExecutor(5);

    private long value = 0;

    public void method() {
        synchronized (this) {
            System.out.println(Thread.currentThread());
            this.value++;
        }
    }

    private synchronized long getValue() {
        return this.value;
    }

    public static void main(String... args) {
        ThreadSafe threadSafe = new ThreadSafe();
        for (int i = 0; i < 10; i++) {
            threadSafe.executor.execute(new MyThread());
        }

    }

    private static class MyThread extends Thread {

        private ThreadSafe threadSafe = new ThreadSafe();

        private AtomicBoolean shutdownInitialized = new AtomicBoolean(false);

        @Override
        public void run() {
            while (!shutdownInitialized.get()) {
                threadSafe.method();
                System.out.println(threadSafe.getValue());
            }
        }
    }

}

这里我试图使value线程安全,一次只能由一个线程访问。当我运行这个程序时,我发现即使我将它包装在value块中,synchronized上也有多个线程在运行。当然这个循环将是无限的,但它只是一个例子,我几秒后手动停止这个程序,所以我有:

2470
Thread[pool-1-thread-3,5,main]
2470
Thread[pool-1-thread-5,5,main]
2470
Thread[pool-1-thread-2,5,main]

不同的线程正在访问和更改此value。有人可以向我解释为什么会这样吗?以及如何使这个全局变量线程安全?

2 个答案:

答案 0 :(得分:10)

每个帖子都有自己的ThreadSafe,每个ThreadSafe都有自己的value。此外,synchronized方法锁定this,因此每个ThreadSafe都锁定自身 - 并且没有一个方法在线程之间共享。这称为线程局部性,它是确保线程安全的最简单方法。 :)

要进行我认为您想要的实验,您需要更改MyThread,使其构造函数采用ThreadSafe参数(而不是构造一个)。然后,让main方法创建一个ThreadSafe并在构造时将其提供给每个MyThread

答案 1 :(得分:4)

每次都获得相同的值,因为每个Runnable都有自己的ThreadSafe类实例。

如果您希望他们共享同一个班级,那么您将只需要一个ThreadSafe个实例并将其传递到您的所有工作中 - 请参阅下文。如上所述,如果您需要线程安全的共享AtomicLong,则可以使用long

此外,您的MyThread课程 extend Thread。它应该是implements Runnable。您的代码正常运行,因为Thread已经implements Runnable。如果你做myThread.interrupt()它实际上不会中断线程,因为它是调用你的run()方法的线程池线程。

以下内容可行:

ThreadSafe threadSafe = new ThreadSafe();
for (int i = 0; i < 10; i++) {
    threadSafe.executor.execute(new MyRunnable(threadSafe));
}
...
private static class MyRunnable implements Runnable {
    private final ThreadSafe threadSafe;
    public MyRunnable(ThreadSafe threadSafe) {
       this.threadSafe = threadSafe;
    }
    ...
相关问题