今天在学习时随手编了一个小程序, 然而运行时却发现总在出错,而且每次中断位置都显示在析构函数的delete 一行。起初时时百思不得,
后来才发现问题所在,自我反思后觉得这是一个对我这样的初学者来讲很重要的问题。
问题出在Print函数上是很明显的,但到底是什么原因呢?
#include<iostream>
using namespace std;
class Person
{
public :
Person()
{
name = NULL;
cout << "Default consturctor called!\n\n";
}
Person(char *str)
{
name = new char [strlen(str) + 1];
strcpy_s(name, strlen(str) + 1, str);
cout << "Consturctor called!\n\n";
}
~Person()
{
delete[] name;//总在此处
cout << "Destructor called!\n\n";
}
friend ostream& operator << (ostream& out, Person &A)
{
out << A.name << endl;
return out;
}
friend istream& operator >> (istream& in, Person &A)
{
char temp[10];
in.getline(temp, 10);
int len = strlen(temp) + 1;
A.name = new char[len];
strcpy_s(A.name, len, temp);
return in;
}
friend void Print(Person a)
{
cout << a.name << endl;
}
private :
char *name;
};
int main()
{
Person a("mike");
//cin >> a;
Print(a);
//cout << a;
system("pause");
return 0;
}
关键在于深浅拷贝的问题,这里可以看到在Print函数中传过去的是一个对象,要命就要命在成员中有一个指针。
我们知道, C++中将对象作为参数时,并非将对象直接传过去,而是调用拷贝构造函数传过去一个对象的副本。而在函数结束后调用析
构函数。当函数中没有定义拷贝构造函数时,就会调用默认的拷贝构造函数。但不幸的是这个默认的函数采用的是“浅拷贝”,也就是说
它仅仅将指针本身的值复制过去。造成的后果是两个指针指向同一处,这样在函数结束后调用析构函数释放该处内存空间, 在程序结束后
又是释放一次,自然就导致了错误。
解决方法很简单,定义一个拷贝构造函数就行了。
Person(const Person &a)
{
name = new char[strlen(a.name) + 1];
strcpy_s(name, strlen(a.name) + 1, a.name);
cout << "Copy constructor called!\n\n";
}
由此可以得到一个教训:当成员中含指针是, 必须要自定义拷贝构造函数。