如此使用单例模式！

最近拿到一个项目，大致扫了一下，发现大多数的代码都是如下的结构：

public class User {
	private String userName;
	
	public String getUserName() {
		return userName;
	}
	
	public void setUserName(String userName) {
		this.userName = userName;
	}
}

public class UserRule { 
	private static UserRule instance = null; 
	
	public static UserRule getInstance() { 
		if (instance == null) { 
			instance = new UserRule(); 
		} 
		return instance; 
	}
	
	public User AddUser(String userName) {
		// add a new user operations
	}
} 

先不说问题，来看看这样做的一些可取之处：
1、设计简单。系统的设计在这样的结构下其实就是贫血类的设计，剩下的就是在Rule类中增加相应的方法。
2、各个模块的开发可以同时进行。各人根据要求定义好模块相应的类，剩下的就是缺啥补啥了。

然后说问题：
1、首先该单例实现就有问题了，这样的实现并不能保证线程安全。不过好在其实这个UserRule本身也没有字段之类的东西，不单例其实也没多大影响。所以我想这个单例丢在这里估计只是为了写代码的时候方便点。
2、这种做法应对需求变化的结果就是不断的在UserRule中添加新的满足需求的方法，最后天知道这个方法到底干嘛的。
3、我想这样的系统应该性能上很差了，可奇怪客户用到现在居然没一点性能上的反馈。不过性能这块只是我的一个感觉，还没有对这样的系统做一个压力测试，无法提供实际情况。

不知道还有没有其他的缺点，欢迎各位来分析分析。

评论

28 楼 czqyr 2009-01-22  

这样不是单例吗？那单例应该是什么样子呢？

27 楼 amonlei 2009-01-22  

longtengsoft 写道

如果在多线程的环境中我想会出现性能问题和安全问题.
很容易出现竞争问题,而且如果一旦出现竞争就会出现数据丢失的风险!

数据丢失会有，性能如何有？

26 楼 amonlei 2009-01-22  

luanma 写道

    public static [b]synchronized[/b] UserRule getInstance() {   
        if (instance == null) {   
            instance = new UserRule();   
        }   
        return instance;   
    }  

多线程的情况下，加上synchronized就可以了。

基本，不需要

25 楼 lcllcl987 2009-01-22  

这是单例吗？这能做到单例吗？
我在任何地方如果：
UserRule userRule = new UserRule();
你如何保证UserRule只有一个instance？
连单例都没搞明白， 就别瞎折腾了。

24 楼 LargeBean 2009-01-22  

zlt2000 写道

其实直接这样就可以了

public class UserRule {    
    private static UserRule instance = new UserRule();
       
    public static UserRule getInstance() {    
        return instance;    
    }   
}

需要

     private UserRule(){}

23 楼 阳光雨露 2009-01-22  

单例前面好像加一个子synchronized就行了,这样就线程安全了呀

22 楼 arthurln 2009-01-22  

这年头都有DI了，还有人自己写Singleton

21 楼 zlt2000 2009-01-22  

其实直接这样就可以了

public class UserRule {    
    private static UserRule instance = new UserRule();
       
    public static UserRule getInstance() {    
        return instance;    
    }   
}

20 楼 narry 2009-01-22  

public class User {  
    private String userName;  
      
    public String getUserName() {  
        return userName;  
    }  
      
    public void setUserName(String userName) {  
        this.userName = userName;  
    }  
}  
 
public class UserRule {   
   private UserRule (){}
       
   static class SingletonHolder {
    static volatile UserRule userRule =   new UserRule();
  }

    public static UserRule getInstance() {   
         return SingletonHolder.userRule;
    }  
      
    public User AddUser(String userName) {  
        // add a new user operations  
    }  
}  

19 楼 longtengsoft 2009-01-22  

如果在多线程的环境中我想会出现性能问题和安全问题.
很容易出现竞争问题,而且如果一旦出现竞争就会出现数据丢失的风险!

18 楼 agiha 2009-01-22  

luanma 写道

    public static [b]synchronized[/b] UserRule getInstance() {   
        if (instance == null) {   
            instance = new UserRule();   
        }   
        return instance;   
    }  

多线程的情况下，加上synchronized就可以了。

你这个浪费性能。还是你楼上的实现好。

17 楼 edi 2009-01-22  

skzr.org 写道

评定再次没事找事的帖子，鉴定完毕

,浪费时间，看了这么无聊的帖子。

16 楼 luanma 2009-01-22  

zozoh 写道

#     public static UserRule getIn<script type="text/javascript" src="http://www.iteye.com/javascripts/tinymce/themes/advanced/langs/zh.js"></script>stance() {  
#         if (instance == null) {  
#             instance<script type="text/javascript" src="http://www.iteye.com/javascripts/tinymce/plugins/javaeye/langs/zh.js"></script> = new UserRule();  
#         }  
#         return instance;  
#     }

应该写成

public static UserRule getInstance() {   
	if (instance == null) {
		synchronized(UserRule.class){
			if (instance == null){
				instance = new UserRule();						
			}
		}
	}   
	return instance;   
}

以保证多个线程访问的安全

这个代码看上去也忒复杂了，而且有线程风险，性能上没有提高。

15 楼 luanma 2009-01-22  

    private UserRule() {}
    public static [b]synchronized[/b] UserRule getInstance() {   
        if (instance == null) {   
            instance = new UserRule();   
        }   
        return instance;   
    }  

多线程的情况下，加上synchronized就可以了。

哦，还有就是构造函数一定要定义为 private。

14 楼 skzr.org 2009-01-22  

评定再次没事找事的帖子，鉴定完毕

13 楼 amonlei 2009-01-21  

怎么就看出性能问题了？我还觉得性能提高了捏，虽然只是一点点

12 楼 xmpp 2009-01-21  

除单例实现有点问题，其它很好啊。。简洁到楼上的每个人都能看得懂。

11 楼 whaosoft 2009-01-21  

晕 这是做的什么呀

10 楼 wangchao_0625 2009-01-21  

在UserRule类中
应有

private UserRule(){}

顺便问一下lz,什么情景下用到的单例模式

9 楼 zozoh 2009-01-21  

#     public static UserRule getInstance() {  
#         if (instance == null) {  
#             instance = new UserRule();  
#         }  
#         return instance;  
#     }

应该写成
public static UserRule getInstance() {  
if (instance == null) {
synchronized(UserRule.class){
if (instance == null){
instance = new UserRule();
}
}
}  
return instance;  
}

以保证多个线程访问的安全