今天在学习时随手编了一个小程序, 然而运行时却发现总在出错,而且每次的断电都显示在析构函数的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"; }由此可以得到一个教训:当成员中含指针是, 必须要自定义拷贝构造函数。
新闻热点
疑难解答