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

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关键字时，它确保该方法一次只能由一个线程执行。 它不能确保其他对象的线程安全。