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

真菜

原帖由 jonasan 于 2009-1-10 21:28 发表
楼主应该谦虚一点，我觉得你应该高兴学到了一种新的模式，而不是觉得那个engineer sb. 他觉得你不懂OOP那是他的问题，不能说别人说你什么你就不活了，是不？

我早就知道Singleton这种Stupid模式了

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

ilex

原帖由 真菜 于 2009-1-10 19:20 发表

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

别人review你的代码，你应该感到欣慰呀！
可以少泄露BUG到测试那边。

唉.......

gamania

年轻就是好

chary8088

SB senior提的建议还是有价值的，，不过说你不懂OOP，他也太扯淡了。。。

chary8088

使用Singleton还是应该的。。。
至于麻烦吗，，，还是根据实际情况处理。。。

没本

我倒是认为直接用静态变量比用singleton直观多了。

emacsnw

原帖由 真菜 于 2009-1-10 06:08 发表

我早就知道Singleton这种SB模式了

一直看到这个回帖，不得不说一句，楼主真是无知者无畏。

真菜

原帖由 emacsnw 于 2009-1-11 12:36 发表



一直看到这个回帖，不得不说一句，楼主真是无知者无畏。

Singleton确实解决了保证只有一个实例的问题，但是它解决的只是一个很容易克服的问题，却引来了很多复杂的问题。
第一：原先的调用是"remoteServer->LoadInfo()"，现在却要改为"RemoteServer * instance = RemoteServer.GetInstance(); instance->LoadInfo()"，所有的地方都要这么啰嗦，你不累？
第二：如何解决实例化Singleton顺序的问题，估计你从来只是写些理想环境中的TOY CODE，没在实际的复杂环境经历过，我实话实说，告罪了 。
第三：在多线程中的问题，你自己去上网用"Singleton 多线程"去搜索吧。

附录：我解决保证只有一个实例的问题的简单方法，实际上这只是迎合某些自以为掌握Design Pattern的engineer的，remoteServer本身就是一个全局对象，难道要获取服务器信息的人连这点都不知道吗，还要自己去重新new RemoteServer()，再调用LoadInfo()方法，STUPID!

1. 
   
2. RemoteServer::RemoteServer()
   
3. {
   
4.        static int count;
   
5.        assert(count++ == 0);
   
6. }
   

复制代码

PS：也许你痴长我几岁，但是请别在这地方装C++大牛了，KISS，understand？

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

真菜

如果真的按照那个“senior”工程师的要求，系统中所有的global object都要用Singleton模式，在C++中怎么解决如下的问题：
要求：
1、初始化RemoteServer之前，需要先初始化Config对象
2、初始化ClientMonitor对象之前，需要先初始化RemoteServer对象
3、其他的依赖关系...
如果不用Singleton，只需要：

1. 
   
2. int main()
   
3. {
   
4.      config = new Config();
   
5.      remoteServer = new RemoteServer();
   
6.      clientMonitor = new ClientMonitor();
   
7. }
   

复制代码

一切不就结了吗？
真的担心自己会实例化RemoteServer两次的化，这里有个简单的方法：

1. 
   
2. RemoteServer::RemoteServer()
   
3. {
   
4.        static int count;
   
5.        allege(count++ == 0);
   
6. }
   

复制代码

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

converse

楼主,不同意可以,没必要人身攻击,这属于态度问题了.