今天看到一个帖子,很早的,但是很有意义,是围绕着一段代码查错展开的,帖子地址是:http://www.iteye.com/topic/81152,里面的代码是
乍一看我除了觉的有点别扭,确实没找到什么大的问题,后来一个一个回复看下来,让我受益匪浅,现在我在总结下我从这段代码中学到的知识。
1.synchronized method(){} == synchronized(this){}
首先要理解的是synchronized这个关键字,最关键的是它锁的是对象,而不是方法或代码段,一个线程只有拿到了这个对象的锁,才能执行这个对象的方法或代码段。这里的this指的就是调用这个方法的对象。
2.synchronized(obj){} 注意这个obj要是共享资源,一般情况下是类成员变量,就像上面的代码中的,绝对不能定义在要同步的方法中,比如:
这样是没有意义的。
3.避免虚假唤醒,wait()方法一般放在while循环中,不要放在if中。
4.wait()方法使得当前线程陷入等待,并且释放this的lock(release the monitor of this);notify()方法会随机唤醒在this上等待的线程,并再次获得lock(own the monitor of this)
这两点很重要,也是上面代码错误的原因,上面的代码中每个方法都获得了两个锁(this和list),而pop只释放了this锁,造成push函数无法获得list锁,最终导致死循环的发生。
要想释放list锁,必须显示调用list.wait(),而一但调用了wait方法线程就进入等待状态,无法执行下面的语句。
class Stack
{
LinkedList list = new LinkedList();
public synchronized void push(Object x)
{ synchronized(list)
{ list.addLast( x );
notify();
}
}
public synchronized Object pop()
{ synchronized(list)
{ while( list.size() <= 0 )
wait();
return list.removeLast();
}
}
}
乍一看我除了觉的有点别扭,确实没找到什么大的问题,后来一个一个回复看下来,让我受益匪浅,现在我在总结下我从这段代码中学到的知识。
1.synchronized method(){} == synchronized(this){}
首先要理解的是synchronized这个关键字,最关键的是它锁的是对象,而不是方法或代码段,一个线程只有拿到了这个对象的锁,才能执行这个对象的方法或代码段。这里的this指的就是调用这个方法的对象。
2.synchronized(obj){} 注意这个obj要是共享资源,一般情况下是类成员变量,就像上面的代码中的,绝对不能定义在要同步的方法中,比如:
public void push(Object x) {
LinkedList list = new LinkedList();
synchronized(list)
{ list.addLast( x );
System.out.println("list notify");
list.notify();
System.out.println("this notify");
notify();
}
}
这样是没有意义的。
3.避免虚假唤醒,wait()方法一般放在while循环中,不要放在if中。
4.wait()方法使得当前线程陷入等待,并且释放this的lock(release the monitor of this);notify()方法会随机唤醒在this上等待的线程,并再次获得lock(own the monitor of this)
这两点很重要,也是上面代码错误的原因,上面的代码中每个方法都获得了两个锁(this和list),而pop只释放了this锁,造成push函数无法获得list锁,最终导致死循环的发生。
要想释放list锁,必须显示调用list.wait(),而一但调用了wait方法线程就进入等待状态,无法执行下面的语句。