1. 
   
2. <P>class SingleDemo{
   
3. private static SingleDemo s = null;
   
4. private SingleDemo(){}
   
5. private Object obj = new Object();
   
6. public static SingleDemo getIntance(){ </P>
   
7. <P>  if(s == null) {
   
8.    synchronized(SingleDemo.class) {
   
9.     if (s == null){
   
10.      s = new SingleDemo();
    
11.     }
    
12.    }
    
13.   }
    
14.   return s;
    
15. }
    
16. }</P>

复制代码

这种单例有什么缺陷？

1. private Object obj = new Object();

复制代码

首先这一行代码不知道是做什么用的
其次就算加入了2个if判断 执行效率还是不如饥汉式

如果要考虑到多线程的话你上面的代码还是不能解决问题的,如果我在synchronized块中添加Thread.sleep(10000);的话照样还是出现多线问题!所以这种方式同样是不安全的.还不如使用恶汉式:

1. class SingleDemo{
   
2. private static SingleDemo s = new SingleDemo();
   
3. private SingleDemo(){}
   
4. public static SingleDemo getIntance(){
   
5.   return s;
   
6. }
   
7. }

复制代码