遇到一个问题, 请大家帮我分析下,谢谢

[liujia2008]

小弟刚接触C++,今天碰到点问题,有点想不通,请大哥们帮小弟分析下,可能对于您来说非常简单的,如下:
class Node
{
public:
double *nums;
public:
Node(void);
~Node(void);
};
Node::Node(void)
{
std::cout << "Node构造函数被调用!" << std::endl;
}

Node::~Node(void)
{
std::cout << "Node析构函数被调用!" << std::endl;
}

调用如下:
map<string,Node> nodeList;
Node node;
nodeList.insert (make_pair("000",node));

system("pause");

输出结果如下:
Node构造函数被调用!
Node析构函数被调用!
Node析构函数被调用!
Node析构函数被调用!

问题:为什么这里析构函数被调用了三次?还有个问题见下面我的回复中!

[liujia2008]

还是上面那个类,扩充了一下:
class Node
{
public:
double *nums;
public:
void RemoveBuffer(void);
Node(void);
~Node(void);
};

Node::Node(void)
{
std::cout << "Node构造函数被调用!" << std::endl;
nums=NULL;
}

Node::~Node(void)
{
std::cout << "Node析构函数被调用!" << std::endl;
RemoveBuffer();
}

void Node::RemoveBuffer(void)
{
if(nums!=NULL)
{
delete [] nums;
nums=NULL;
}
}

调用:
map<string,Node> nodeList;
Node node;
node.nums =new double[3];
memset(node.nums ,0,sizeof(int)*3);
nodeList.insert (make_pair("000",node));

system("pause");

问题:
执行后为什么会导致程序崩溃?

Debug Assertion Failed
_block_type_is_valid

[M_S_D_N]

node节点被默认构造一次，复制构造两次。
复制构造不进入默认构造函数，两次复制构造导致两次析构。

崩溃是因为复制构造时没有深度拷贝指针nums，new一次，而多次delete，在第二次delete时已经是一个野指针的，崩掉。。。

[sdljgxb]

学习~~~

[waizqfor]

申请一个对象 构造一次 构造函数
复制的时候 构造了2次 不调用默认的构造函数 2次析构

所以打印出来的时候 就是调用3次析构函数

void Node::RemoveBuffer(void) 
{ 
if(nums!=NULL) 
{ 
delete [] nums; //多次的DELETE造成了程序崩溃的原因
nums=NULL; 
} 
} 

[叶落寒山]

map <string,Node> nodeList;  // 调用一次Node的构造函数
Node node;                   // 调用一次Node的析构函数
nodeList.insert (make_pair("000",node)); // make_pair("000",node) 调用一次Node的析构函数
                                               // insert 调用一次Node的析构函数
                                               // 进入函数执行时，需要创建node的临时变量，在函数退出时需要析构临时变量。这也是
                                               // 为什么在使用类作为函数的形参时，需要考虑使用类的引用的形式来进行参数的传递，
                                               // 来减少程序的开销


崩溃是因为指针nums，new一次，而多次delete，在第二次delete时，num为野指针。

[猫已经找不回了]

要么就把析构函数改为只有一个实例对象时候才delete.
用友元对象记录实例数.

[dongpy]

[Quote=引用 2 楼 M_S_D_N 的回复:]
node节点被默认构造一次，复制构造两次。
复制构造不进入默认构造函数，两次复制构造导致两次析构。

崩溃是因为复制构造时没有深度拷贝指针nums，new一次，而多次delete，在第二次delete时已经是一个野指针的，崩掉。。。
[/Quote]
UP！应该自定义拷贝构造函数。

[gzvigo]

[Quote=引用 5 楼 huayehanshan 的回复:]
C/C++ code
map <string,Node> nodeList; // 调用一次Node的构造函数 //为什么这里会有构造函数调用??Node node; // 调用一次Node的析构函数 //为什么到这里是调用的析构函数?
nodeList.insert (make_pair("000",node)); // make_pair("000",node) 调用一次Node的析构函数
// insert 调用一次Node的析构函数
// 进入函数执行时，需要创建node的临时变量，在函数退出时需…
[/Quote]


请5楼再详细解释一下，好吗。谢谢!!!看不太懂呀

[activateMan]

Node node这个声明语句时，node被构造一次，然后是make_pair("000",node)时，要用到Node的临时变量，复制构造一次，在insert时，pair对象中的node又被复制构造一次

[passionboy03]

你定义的是指针，没有分配内存
[Quote=引用 1 楼 liujia2008 的回复:]
还是上面那个类,扩充了一下:
class Node
{
public:
double *nums;
public:
void RemoveBuffer(void);
Node(void);
~Node(void);
};

Node::Node(void)
{
std::cout < < "Node构造函数被调用!" < < std::endl;
nums=NULL;
}

Node::~Node(void)
{
std::cout < < "Node析构函数被调用!" < < std::endl;
RemoveBuffer();
}

void Node::RemoveBuffer(void)
{
if(nums!=NULL)
{
…
[/Quote]

[passionboy03]

关键是定义的指针没分配内存吧
[Quote=引用 2 楼 M_S_D_N 的回复:]
node节点被默认构造一次，复制构造两次。
复制构造不进入默认构造函数，两次复制构造导致两次析构。

崩溃是因为复制构造时没有深度拷贝指针nums，new一次，而多次delete，在第二次delete时已经是一个野指针的，崩掉。。。
[/Quote]