java-为什么synchronized块没有给出正确的循环结果

时间:2013-09-22 07:29:31

标签: java multithreading

我编写了一个代码,创建了几个线程并启动它。我使用synchronized block锁定了对象上的监视器。我希望创建的第一个线程应该锁定对象并完成其工作。然后任何其他对象都可以输入它。

但它没有发生,程序在下面。

class ThreadCreationDemo implements Runnable
{
    public void run()
    {
        synchronized(this)
        {
            for(int i=0;i<10;i++)
            {   
                System.out.println("i: "+i+" thread: "+Thread.currentThread().getName()+" threadgroup: "+Thread.currentThread().getThreadGroup()+" "+Thread.holdsLock(this));
                try {
                   Thread.sleep(1000);
                }
                catch(Exception e)
                {   
                    System.out.println(e.toString());   
                }           
            }
        }
    }

    public static void main(String args[])
    {
         Thread t[]=new Thread[5];

         for(int i=0;i<5;i++)
         {
             t[i]=new Thread(new ThreadCreationDemo());
             t[i].start();
         }
    }
}

我希望结果应该是这样的。

首先,在线程名称(如线程0)下打印i = 0到9的所有值 然后线程1等。

但输出如下:

i: 0 thread: Thread-1
i: 0 thread: Thread-3
i: 0 thread: Thread-2
i: 0 thread: Thread-0
i: 0 thread: Thread-4
i: 1 thread: Thread-1
i: 1 thread: Thread-4
i: 1 thread: Thread-3
i: 1 thread: Thread-0
i: 1 thread: Thread-2
i: 2 thread: Thread-1
i: 2 thread: Thread-3
i: 2 thread: Thread-2
i: 2 thread: Thread-0
i: 2 thread: Thread-4
i: 3 thread: Thread-1
i: 3 thread: Thread-3
i: 3 thread: Thread-0
i: 3 thread: Thread-4
i: 3 thread: Thread-2
i: 4 thread: Thread-1
i: 4 thread: Thread-3
i: 4 thread: Thread-2
i: 4 thread: Thread-4
i: 4 thread: Thread-0
i: 5 thread: Thread-1
i: 5 thread: Thread-4
i: 5 thread: Thread-3

2 个答案:

答案 0 :(得分:6)

问题是你每次都在创建新对象:new ThreadCreationDemo()

因此所有线程都获得对不同对象的锁定,因此锁定将失败。

答案 1 :(得分:3)

您正在进行同步

synchronized(this)

换句话说,每个实例都锁定自身。你没有锁定共享对象。

解决方案是锁定所有类实例共享的静态对象。例如

synchronized (ThreadCreationDemo.class) {...}

或者,在创建Thread时,传入每个Thread可以同步的共享对象引用。

new ThreadCreationDemo(new Object());
...
public ThreadCreationDemo(Object o) {
    this.lock = o
}

public void run () {
    synchronized (lock) {
        ...
    }
}