单件模式释放空间

[bitxinhai]

在实现单件模式时，由于一般不会释放单件空间，往往都会造成内存的泄漏
我想了一个方法，就是使用一个类GC，在Singleton中声明一个GC的静态对象将单件的空间释放！！
但是还有点错误！！
请大虾看一下哪里有错？？
或者提供一种好的释放单件内存的方法！！
谢谢！！！

#include<iostream.h>
#include<crtdbg.h>
class GC
{
public:
GC();
~GC();

protected:
private:
};//垃圾回收类

class Singleton
{
public:
friend class GC;
static Singleton* GetInstance()
{

if(instance == NULL)
{
static GC gc;
instance = new Singleton;
}
return instance;
}
~Singleton()
{
if(instance != NULL)
{
delete instance;
instance = NULL;
}
}

void Display()
{
cout<<"Singleton!"<<endl;
}
protected:
Singleton(){}
Singleton(const Singleton& other){}
Singleton& operator=(const Singleton& other){}
private:
static Singleton* instance;

};

Singleton* Singleton::instance = NULL;


GC::GC()
{
cout<<"Con GC!"<<endl;
}
GC::~GC()
{
Singleton* pInstance = Singleton::GetInstance();
if (pInstance != NULL)
{//回收空间
delete pInstance;
pInstance = NULL;
cout<<"Delete Singleton::instance in GC!"<<endl;
}
}

void main()
{
Singleton* pInstance = Singleton::GetInstance();
pInstance->Display();

_CrtDumpMemoryLeaks();

}

[taodm]

去看《modern c++ design》不用闭门造车

[就呆在云上]

#include <iostream> 
#include <crtdbg.h> 
using namespace std;
class GC 
{ 
public: 
GC(); 
~GC(); 

protected: 
private: 
};//垃圾回收类 

class Singleton 
{ 
public: 
friend class GC; 
static Singleton* GetInstance() 
{ 

        if(instance == NULL) 
{ 
static GC gc; 
instance = new Singleton; 
} 
return instance; 
} 
~Singleton() 
{ 
if(instance != NULL) 
{ 
delete instance; 
instance = NULL; 
} 
} 

void Display() 
{ 
cout <<"Singleton!" <<endl; 
} 
protected: 
Singleton(){} 
Singleton(const Singleton& other){} 
Singleton& operator=(const Singleton& other){} 
private: 
static Singleton* instance; 

}; 

Singleton* Singleton::instance = NULL; 


GC::GC() 
{ 
cout <<"Con GC!" <<endl; 
} 
GC::~GC() 
{ 
  Singleton* pInstance = Singleton::GetInstance(); 
if (pInstance != NULL) 
{//回收空间 
delete pInstance; 
pInstance = NULL; 
cout <<"Delete Singleton::instance in GC!" <<endl; 
} 
} 

void main() 
{ 
Singleton* pInstance = Singleton::GetInstance(); 
pInstance->Display(); 

_CrtDumpMemoryLeaks(); 

}

完全没有问题啊，编译和运行都没有问题：

[Zark]

既然已经是singleton了，要考虑释放做什么？

只有一个占用内存的对象活在整个程序期中，到了程序结束，系统回收了所有空间，那个singleton占用的空间当然也就收回了，所以也就没有必要考虑这件事了．

[bitxinhai]

[Quote=引用 2 楼 ztz0223 的回复:]
C/C++ code#include <iostream>
#include <crtdbg.h>
using namespace std;
class GC
{
public:
GC();
~GC();

protected:
private:
};//垃圾回收类

class Singleton
{
public:
friend class GC;
static Singleton* GetInstance()
{

if(instance == NULL)
{
static GC gc;
instance = new Singleton;
}
return instance;
}
~Singleton()
{
if(instance != NULL…
[/Quote]


你调试一下就可以看出泄漏的空间了！！！！

[xdkui]

[Quote=引用楼主 bitxinhai 的帖子:]
在实现单件模式时，由于一般不会释放单件空间，往往都会造成内存的泄漏
我想了一个方法，就是使用一个类GC，在Singleton中声明一个GC的静态对象将单件的空间释放！！
但是还有点错误！！
请大虾看一下哪里有错？？
或者提供一种好的释放单件内存的方法！！
谢谢！！！

#include <iostream.h>
#include <crtdbg.h>
class GC
{
public:
GC();
~GC();

protected:
private:
};//垃圾回收类

cl…
[/Quote]

1. 可以用个全局auto_ptr来装你的singleton指针, 系统退出时会自动释放, 没有泄露
2. 可以声明一个全局空间来容纳你的对象, 全局数据段,不在heap里,也没有泄露. 但这样singleton对象内部分配的资源不会释放. 可以加个Release之类的函数
3. 像上面说的,这个singleton只有一个, 不会随着程序运行, 而消耗内存量增加. 不算真正内存泄露,无所谓

[星羽]

    ~Singleton() 
    { 
        if(instance != NULL) 
        { 
            delete instance; 
            instance = NULL; 
        } 
    } 

- -b

你这个是干嘛呢，无限调用了。。。。

[lsmdiao0812]

[Quote=引用 6 楼 Chiyer 的回复:]
C/C++ code
~Singleton()
{
if(instance != NULL)
{
delete instance;
instance = NULL;
}
}

- -b

你这个是干嘛呢，无限调用了。。。。
[/Quote]

?????????????????????????

[jia_xiaoxin]

楼主的GC也是静态的，何时删除不好掌握。而且使用友元，对封装来说就不怎么好了。
可以用引用计数来删除

class Singleton 
{ 
private:
	static Singleton * instance;
	//引用计数
	static int count;
	Singleton()
	{
		count++;
	} 
public: 
	static Singleton* GetInstance()
	{
		if(count == 0)
			instance = new Singleton();
		return instance;
	}
	~Singleton() 
	{ 
		cout <<"Singleton is deleted!" <<endl;
	} 
	void Display() 
	{ 
		cout <<"Singleton!" <<endl; 
	} 
	void Destroy()
	{
		count--;
		if(count == 0)
			delete instance;
	}
}; 

int Singleton::count = 0;
Singleton * Singleton::instance = NULL;

void main() 
{ 
	Singleton* pInstance = Singleton::GetInstance(); 
	pInstance->Display(); 
	pInstance->Destroy();
	_CrtDumpMemoryLeaks(); 
	
}

[airtrack]

static Singleton* GetInstance()
{
static Singleton obj;
return &obj;
}
就够了。

[星羽]

推荐一篇文章

http://redwolf.blog.51cto.com/427621/102348
由“单件模式”引发的思考及解决方案

[xtjextje]

[Quote=引用 7 楼 lsmdiao0812 的回复:]
引用 6 楼 Chiyer 的回复:
C/C++ code
~Singleton()
{
if(instance != NULL)
{
delete instance;
instance = NULL;
}
}

- -b

你这个是干嘛呢，无限调用了。。。。


?????????????????????????
[/Quote]

怎么会是无限调用,其实是根本就不会调用.
程序根本都不用运行到它的虚构函数.

原因 1 instance静态的变量,在程序退出的时间.程序保证会销毁 instance 这个变量.但不会销毁这个变量所指的由new生成的空间.而在程序退出后.系统会回收由这个程序申请的空间.

[xtjextje]

补充一点.其实单件模式没有必要自己去释放空间.系统会回收.
但如果你一定要自己回收.不要系统去回收这内存.可以有一个解决的方案

那就是 申请的空间用 auto_ptr 来存储.