编写一个延迟加载的单例设计模式时出现这个异常Exception in thread "main" java.lang.StackOverflowError。查了一下说是栈溢出问题。有懂的可以解释一下么。

1. package com.itheima;
   
2. 
   
3. /**
   
4. * 7、 编写一个延迟加载的单例设计模式。
   
5. * @author
   
6. *
   
7. */
   
8. class single{
   
9.         
   
10.         private single(){}
    
11.         private static single s=null;
    
12.         public static single getInstance(){
    
13.              if(s==null){
    
14.                  synchronized(single.class)//同步锁
    
15.                    {
    
16.                         if (s==null){
    
17.                                 s=single.getInstance();
    
18.                         }
    
19.                     }
    
20.               }
    
21.                  return s;
    
22.         }
    
23. }
    
24. 
    
25. 
    
26. public class Test7 {
    
27.         
    
28. 
    
29.         public static void main(String[] args) {
    
30.                
    
31.            single ss=single.getInstance();
    
32.         }
    
33. 
    
34. }

复制代码

恩，学习了

happymouse 发表于 2014-8-28 09:42

7# 的说得太好了，学习了

【灰太狼】 发表于 2014-8-30 20:30
17行有问题，既然你是单例模式，创建对象为什么还要用递归呢，直接new就ok了嘛，这里是无限递归了，所以一 ...

恩恩  谢谢了

17行有问题，既然你是单例模式，创建对象为什么还要用递归呢，直接new就ok了嘛，这里是无限递归了，所以一直创建对象，一直创建到占内存都满了，死递归~

17行， s=single.getInstance();
这个是在调用类里面的静态方法，然后进去又调进去又调，
然后栈里面存了N个方法。
然后就内存满了，也就是死循环导致的内存溢出，
而且你的类名要注意规范，单词的首字母都要大写，跟成员变量区别开来以示区分。

打酱油路过，，学习学习

学习了。

happymouse 发表于 2014-8-28 09:42

回答的不错哦

   7楼正解，你不断调用自己的getInstance方法却没有跳出条件，最后就是栈溢出了。。

happymouse 发表于 2014-8-28 09:42

多谢了哈

楼上说的对，顶下！

1. /**
   
2. * 7、 编写一个延迟加载的单例设计模式。
   
3. *
   
4. * @author
   
5. *
   
6. */
   
7. class single
   
8. {
   
9. 
   
10.         private single(){}
    
11.         private static single s = null;
    
12.         public static single getInstance()
    
13.         {
    
14.                 if (s == null)
    
15.                 {
    
16.                         synchronized (single.class)// 同步锁
    
17.                         {
    
18.                                 if (s == null)
    
19.                                 {
    
20.                                         //问题出在这句，你这样是循环调用了，直到内存耗完为止
    
21.                                         //应该写出：s=new Single();
    
22.                                         s = single.getInstance();
    
23.                                 }
    
24.                         }
    
25.                 }
    
26.                 return s;
    
27.         }
    
28. }
    
29. public class Test7
    
30. {
    
31.         public static void main(String[] args)
    
32.         {
    
33.                 single ss = single.getInstance();
    
34.         }
    
35. }

复制代码

笑脸迷人 发表于 2014-8-27 21:30
最好把代码发出来，一般出现这个问题是代码里有死循环，或是递归数太大了~ ...

刚学不就  有些地方不是很懂

Franklin 发表于 2014-8-27 22:30
看看代码

求指教  

看看代码

是不是方法内部调用自己了

最好把代码发出来，一般出现这个问题是代码里有死循环，或是递归数太大了~