4

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

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）...

4

3 回答 3

8

您的synchronized关键字在实例方法上。没有两个线程可以同时执行您的线程对象之一的此方法。但是，这不是您的代码所做的。每个线程在自己的实例上执行该方法。同步并没有按照您的意图进行。如果它是一种static方法，它会。

于 2012-12-05T00:12:19.687 回答
1

你的increment方法应该是static：

public static synchronized void increment() {

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

于 2012-12-05T00:15:47.510 回答
0

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

于 2012-12-05T07:01:33.213 回答