java 多线程

Question

以下代码高并发时会有啥问题，怎么优化

Answer 1

1、用了synchronized修饰了方法，又用synchronized修饰方法体，这两者是等效的，都是获得this(MyStack)的对象监视器并且临界区也是一致的，同是因为synchronized是可重入的，所以你这样用不会发生错误，但是这是不必要的；

2、可能会发生 @房管局规划部 中出现的错误，wait()可能出现假唤醒，而不满足临界条件，后续逻辑就会异常。
可以参看jdk wait()方法的注释描述：

所以，应该是这样：

synchronized( method_or_shared_object){

    while(list.size()<=0)

        wait();

    // pop something...

}

一般来说，都需要在while(condition) wait()来防止假唤醒。

Answer 2

这段代码在高并发的情况下会出现锁竞争激烈，性能低下的问题。其它的死锁什么的不会出现，不要想太多了。高并发场景建议用concurrent linked queue，分段加锁，能降低锁竞争

Answer 3

list变量每个线程进来都会new一个新的吧

Answer 4

首先synchronized修饰方法的问题

对于非static方法，其作用相当于synchronized(this):

synchronized void method(){

// method body

}

// 等价于

void method() {

   synchronized(this){

       // method body

   }

}

对于static方法，其相当于synchronized(YourClass.class):

class YourClass {

   synchronized static void method() {

       // method body

   }

   // 等价于

   static void method() {

       synchronized(YourClass.class) {

           // method body

       }

   }

}

其次关于假唤醒问题，就是@spance说的。官方docde描述是:

A thread can also wake up without being notified, interrupted, or timing out, a so-called spurious wakeup.

官方给出的解决方案是：

synchronized (obj) {

     while (<condition does not hold>)

         obj.wait();

     ... // Perform action appropriate to condition

  }

The doc