关于多线程的一个面试题，欢迎大家来讨论

一个被锁的老帖
===========================================================================
下面的代码在绝大部分时间内都运行得很正常，请问在什么情况下会出现问题？问题的根源在哪里？

package com.heyongjin.test;

import java.util.LinkedList;

public class Stack {

	LinkedList list = new LinkedList();

	public synchronized void push(Object x) {
		synchronized (list) {
			list.addLast(x);
			notify();
		}
	}

	public synchronized Object pop() throws Exception {  
		synchronized (list) {
			if (list.size() <= 0) {
				wait();
			}
			return list.removeLast();
		}
	}
}

答：这里有可能会死锁：当外界调用pop(),但堆栈中没有元素，即list.size() <= 0，Stack进入停滞状态（wait()）（注意此时list被锁定了 ，但Stack不会被锁定），此时外界仍然可以进入push()（因为wait()不会锁定Stack），但此时在push()中调用list.addLast()时，因为list 被锁定，所以无法调用。
==================================================================
答案又了，但是怎么修改程序才能避免死锁呢？？

为什么方法已经同步了，还要再同步list呢？去掉其中一个即可

楼上正解，类外线程锁this，类内线程锁list

list都空了,为什么还要wait?

同问

一个丑陋的类

Thinking in java 好像有，是wait和notify那一节吧

package com.heyongjin.test;

import java.util.LinkedList;

public class Stack {

	LinkedList list = new LinkedList();

	public void push(Object x) {
		synchronized (list) {
			list.addLast(x);
			list.notifyAll();
		}
	}

	public Object pop() throws Exception {  
		synchronized (list) {
			while(list.size() <= 0) {
				list.wait();
			}
			return list.removeLast();
		}
	}
}

不过用while会多判断一次吧
这样也可以

public Object pop() throws Exception {  
	synchronized (list) {
                Object result= null;
		if(list.size() >0) {
		     result= list.removeLast();

		}else{
		     list.wait();
                }
                return result;
	}
}

把空的判断挪到锁定对象的代码外面来

public synchronized Object pop() throws Exception { 
       if (list.size() <= 0) {   
                wait();   
       } else {    
       synchronized (list) {           
            return list.removeLast();   
       }   
       }
} 

双锁......
有两个锁对像时就有可能死锁

把锁对像变成同一个就可以避免双锁.

应该是pop方法的目的是要返回一个非空的元素，所以空的时候就要等待。