#include<iostream>
#include<cstdlib>
#include<vector>
#include<cstring>
using namespace std;

class CDemo{
	public:
		CDemo():str(NULL){};
		
		~CDemo()
		{
			if(str)
			{
				delete[] str;
			}
		}
		char *str;
};

int main(int argc,char** argv)
{
	CDemo d1;
	d1.str=new char[32];
	strcpy(d1.str,"trend micro");
	vector<CDemo>*a1=new vector<CDemo>();
	a1->push_back(d1);
	delete a1;
	return EXIT_SUCCESS;
}
 

問題：找出以上程式錯誤的地方？（見p105-106）

分析

這個程式退出時會出問題，重複delete 同一片記憶體，程式崩潰。如果把解構函式改為如下，可以更清楚地看到這一點：

	~CDemo()
		{
			if(str)
			{
				static int i=0;
				cout<<"&CDemo"<<i++<<"="<<(int *)this<<",   str="<<(int *)str<<endl;
				delete[] str;
			}
		}

執行結果如下：

也就是說，發生了CDemo 類的兩次析構，兩次析構str 所指向的同一記憶體地址空間（兩次str 值相同=0x362e18）。

本問題歸根結底就是淺拷貝與深拷貝的問題。如果CDemo 類新增一個這樣的複製拷貝建構函式就可以解決問題：

		CDemo(const CDemo &cd)
		{
			this->str=new char[strlen(cd.str)+1];
			strcpy(str,cd.str);
		}

#include<iostream>
#include<cstdlib>
#include<vector>
#include<cstring>
using namespace std;

class CDemo{
	public:
		CDemo():str(NULL){};
		
		CDemo(const CDemo &cd)
		{
			this->str=new char[strlen(cd.str)+1];
			strcpy(str,cd.str);
		}
		
		~CDemo()
		{
			if(str)
			{
				static int i=0;
				cout<<"&CDemo"<<i++<<"="<<(int *)this<<",   str="<<(int *)str<<endl;
				delete[] str;
			}
		}
		char *str;
};

int main(int argc,char** argv)
{
	CDemo d1;
	d1.str=new char[32];
	strcpy(d1.str,"trend micro");
	vector<CDemo>*a1=new vector<CDemo>();
	a1->push_back(d1);
	delete a1;
	return EXIT_SUCCESS;
}
 

新增深複製拷貝建構函式就可以解決問題了。