因为没有用Singleton模式，被人批不懂OOP！

真菜

最近参加了一个网络监控的项目，在客户机上监视用户的操作，将非法操作上传到一台指定的服务器上，使用c++开发，我的任务是编写和服务器交互的部分，我的想法很简单，构造一个类提供对服务器存取信息的操作，如下所示(这里做了很多简化)：

1. 
   
2. class RemoteServer {
   
3.       in_addr serverAddr;
   
4. 
   
5. public:
   
6.       RemoteServer();
   
7.       void StoreInfo();
   
8.       void LoadInfo();
   
9. };
   
10. extern RemoteServer * remoteServer;
    

复制代码

这个类只会有一个实例，在main函数里面初始化：

1. 
   
2. int main()
   
3. {
   
4.      读取配置信息；
   
5.      remoteServer = new RemoteServer();
   
6. }
   

复制代码

注意，在main函数里面需要先读取配置信息，才能初始化RemoteServer。当需要获取RemoteServer相关信息时，直接引用remoteServer这个全局对象即可：

1. 
   
2. void needAccessRemoteServer()
   
3. {
   
4.       remoteServer->LoadInfo();
   
5. }
   

复制代码

我的想法就是这么简单，可是公司里一个senior工程师review我的代码时，说我没有用Singleton模式，还是用C的思维来写C++程序，我日死，后来把代码按书上的要求改成下面这样子算过关了：

1. 
   
2. class RemoteServer {  
   
3.     static RemoteServer *instance;  
   
4. public:  
   
5.     static RemoteServer* GetInstance()  
   
6.     {  
   
7.         if (instance == NULL)
   
8.             instance = new RemoteServer();
   
9.         return instance;
   
10.     }
    
11.     void LoadInfo();
    
12.     void StoreInfo();
    
13. private:
    
14.      RemoteSever();
    
15. };
    

复制代码

当需要获取RemoteServer信息的时候，得先GetInstance

1. 
   
2. void needAccessRemoteServer()
   
3. {
   
4.       RemoteServer instance = RemoteServer::GetInstance();
   
5.       instance->LoadInfo();
   
6. }
   

复制代码

个人觉得这个Singleton真TMD够愚蠢的，解决防止多个实例的问题，带来了更多的问题：
1、原来调用RemoteServer方法的地方，只要:

1. 
   
2.       remoteServer->LoadInfo();
   

复制代码

一行代码就可以了，现在要改为两行：

1. 
   
2.       RemoteServer instance = RemoteServer::GetInstance();
   
3.       instance->LoadInfo();
   

复制代码

2、RemoteServer必须在“读取配置信息”的操作后初始化，使用Singleton模式怎样保证正确的初始化顺序，这个资深工程师给了我一个很tricky的建议，这里不提了。
3、后来看了网上的有些文章，还提到了多线程、autoptr等问题。

这么简单的问题被搞这么复杂，犯两次实例化的错误的概率本身就很小，你真的担心会实例化两次RemoteServer，只要在RemoteServer的构造函数里面加一个static变量判断下，不就可以了嘛：

1. 
   
2. RemoteServer::RemoteServer()
   
3. {
   
4.        static int count;
   
5.       
   
6.        if (count++ > 0) {
   
7.             fprintf("只能存在一个RemoteServer实例\n");
   
8.             exit(0);
   
9.        }
   
10.        ....
    
11. }
    

复制代码

如果系统存在两个全局对象，一般来说就是Fatal Error，开发的时候打印一个错误信息，然后程序退出不就得了，尽管是在运行时才捕获这样的错误，但这样的错误很容易修正。

[ 本帖最后由 真菜 于 2009-1-11 14:31 编辑 ]

converse

为了模式而模式的想法确实不可取...

真菜

大家平时也喜欢用Singleton吗？如果让你来写这个RemoteServer，会用Singleton吗？如果是的话，那我就真的太菜了，无语。

duanjigang

不懂模式的路过。。。
大学时有个同学算法特别强，别人看到他解决了问题，总是很惊讶：呀。。。你这个是动态规划算法。。咦。。这个是分治算法。。。。。。
他很懵：我也不知道这个叫啥算法。。就这么解决的。

converse

如果那个类对象是只有一个实例的话,会倾向于用,也不一定.但是因为不用这个模式,就说人不懂O**就不好了.

converse

活说回来,你最后那个加static变量作判断的想法不好,还不如就用singleton了呢.

一般都是把singleton实现出来,如果需要实现该类为singlton就从这个类继承下来就好了,也不麻烦.

真菜

原帖由 converse 于 2009-1-10 19:17 发表
如果那个类对象是只有一个实例的话,会倾向于用,也不一定.但是因为不用这个模式,就说人不懂O**就不好了.

谢谢，为什么是那个senior工程师review我的代码

[ 本帖最后由 真菜 于 2009-1-12 13:26 编辑 ]

tyc611

Singletom模式一般可以用C++写一个模板，直接用就行了，所以也不麻烦的说

namiii

singleton模式主要是用来方便的存取全局对象吧。
什么防止实例化两次真是无稽之谈。
开发大型的系统，singleton模式还是很方便的。

jonasan

楼主应该谦虚一点，我觉得你应该高兴学到了一种新的模式，而不是觉得那个engineer sb. 他觉得你不懂OOP那是他的问题，不能说别人说你什么你就不活了，是不？