为什么会两次调用析构函数

 

在类生成临时对象时，遇到一个问题，程序会两次调用析构函数

1. #include <cstring>
2. #include <iostream>
3. using namespace std;
4. 
   
5. class stack
6. {
7. public:
8.     stack(const char *value);
9.     ~stack();
10.     char * data;
11. };
12. 
    
13. stack::stack(const char *value)
14. {
15.     if(value)
16.     {
17.         data = new char[strlen(value)+1];
18.         strcpy(data,value);
19.     }
20.     else
21.     {
22.         data = new char[1];
23.         *data = '/0';
24.     }
25. }
26. 
    
27. inline stack::~stack()
28. {
29.  cout << "Destructor" << endl;
30.  //delete []data; //此处会报错
31. }
32. 
    
33. void dosth(stack pstk)
34. {
35.     cout << pstk.data << endl;
36. }
37. 
    
38. int main()
39. {
40.  stack str("iamxczhang");
41.  dosth(str);
42.  return 0;
43. }
44. 
    

查了一些资料，还不是很清楚，但是经过仔细分析后，发现原来是系统调用默认拷贝构造函数的结果。

在返回对象和按值传递参数时，要生成临时对象，生成临时对象要调用默认拷贝构造函数。

通过这个例子更让我加深了对Effective C++的理解。

只要类里有指针变量就得自己写拷贝构造函数和赋值函数，但是你确定用不着这些函数时，可以把这些函数做private声明而不去实现它，这就防止了会有人去调用它们，也防止了编译器去生成它们。

问题在于：
void dosth(stack pstk)
{
cout << pstk.data << endl;
}

用引用！
你这里有一个浅复制的问题

发这个贴子就是为了强调生成临时对象时要调用默认拷贝构造函数，因为有时候我们很容易忽略。当然如果用引用肯定不会有这个问题。