Question

具有同步的简单多线程测试。我想如果它是“同步的”，其他线程会等待。我错过了什么？

public class MultithreadingCounter implements Runnable {

    static int count = 0;

    public static void main(String[] args) {
        int numThreads = 4;
        Thread[] threads = new Thread[numThreads];

        for (int i = 0; i < numThreads; i++)
            threads[i] = new Thread(new MultithreadingCounter(), i + "");

        for (int i = 0; i < numThreads; i++)
            threads[i].start();

        for (int i = 0; i < numThreads; i++)
            try {
                threads[i].join();
            } catch (Exception e) {
                e.printStackTrace();
            }
    }           

    @Override
    public void run() {
        increment();
    }

    public synchronized void increment(){
            System.out.print(Thread.currentThread().getName() + ": " + count + "\t");
            count++; // if I put this first or increment it directly in the print line, it works fine.
    }
}

我认为这会显示如下：

0: 1    2: 0    1: 2    3: 3

但它的实际输出：

0: 0    2: 0    1: 0    3: 3

和其他类似的变化。它应该按顺序显示每个增量（即0,1,2,3）......

Answer 1

您的synchronized关键字位于实例方法上。没有两个线程可以同时执行此某个线程对象的 。但是，这不是你的代码所做的。每个线程在自己的实例上执行该方法。同步不会达到你想要的目的。如果它是static方法，它会。

Answer 2

您的increment方法应为static：

public static synchronized void increment() {

现在，每个对象都在该单个实例上进行同步，但由于count是一个静态变量，因此您应该在Class对象本身上进行同步。

Answer 3

当在方法之前使用synchronized关键字时，它确保该方法一次只能由一个线程执行。它不能确保其他对象的线程安全。

Java多线程使用synchronized不是线程安全的

3 个答案: