读书人

为何报这个错

发布时间: 2013-07-04 11:45:33 作者: rapoo

为什么报这个错
代码:

#include <iostream>

using namespace std;
class Singleton
{
protected:
Singleton() {cout<<"con of Singleton"<<endl;}
~Singleton(){cout<<"dcon of Singleton"<<endl;}
//Singleton(const Singleton& s);///只定义,不实现

public:
static Singleton * getInstance()
{
if(_instance==0)
{
_instance = new Singleton();
}
return _instance;
}
///other class functions...
void show()
{
cout<<"This is a Singleton !"<<endl;
}

private:
static Singleton * _instance;
///other class members ...

class Proxy
{
public:
~Proxy()
{
if(_instance!=0)
{
delete _instance;
_instance = 0;
}
}
};
static Proxy pt;
};
///静态成员的初始化
Singleton* Singleton::_instance = 0;//new Singleton();
Singleton::Proxy Singleton::pt;

int main()
{
Singleton *s=Singleton::getInstance();
s->show();
Singleton *a=Singleton::getInstance();
a->show();
cout<<(a == s)<<endl;
Singleton c(*s);///compile error: 'Singlenton::~Singlenton() is protected'
c.show();
return 0;
}


我想实现一个单例模式,写了上面的代码。
问题是:
Singleton c(*s);///compile error: 'Singlenton::~Singlenton() is protected'
这里为什么报错'Singlenton::~Singlenton() is protected?
我理解的是这里应该调用复制构造函数,我注释掉了,所以编译器应该合成一个,
然后生成了一个instance 。
然后在main运行完之后析构的时候报错Singlenton::~Singlenton() is protected'才可以理解。

------解决方案--------------------


此处会调用析构函数,因为main函数内的自动变量c是要回收的,编译器会自动插入调用析构函数的语句。

引用:
难道是这里调用了析构函数???
Singleton c(*s);///compile error: 'Singlenton::~Singlenton() is protected'

[解决办法]
析构函数要放在public下面。
只看过构造函数放在私有和保护下,不让随便构造。没看过不让析构的。
[解决办法]
引用:
Quote: 引用:

此处会调用析构函数,因为main函数内的自动变量c是要回收的,编译器会自动插入调用析构函数的语句。

Quote: 引用:

难道是这里调用了析构函数???
Singleton c(*s);///compile error: 'Singlenton::~Singlenton() is protected'


嗯,这个我知道。
但是自动调用析构函数应该在return 之后吧,
至少也要在c.show();之后吧??

你这是编译错误,跟运行在c.show();之后有什么关系,编译器看到你有个局部对象,就自动添加了析构代码,然后发现你的析构居然是个保护类型,所以反馈错误,你要防止别人随便析构,这里的别人也包括了编译器。
[解决办法]
就算只有一个对象最后也是要释放的!
[解决办法]
堆栈变量的析构是在return之前插入的,但是你觉得编译器在哪个位置提示你友好一点呢?难道在return那里?这个是编译器人性化的表现

读书人网 >C++

热点推荐