下列代码有哪些错误?
#include 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 a1->push_back(d1); delete a1; return 0; } 编译能通过 解答:默认的拷贝构造只是成员拷贝,于是两个对象str的值相同,两次析构delete了相同的str,解决办法是弄一个拷贝构造函数,里边strdup一下str。其实这也就是浅拷贝和深拷贝的区别。 须改成:
#include using namespace std; class CDemo{ public: CDemo():str(NULL){}; CDemo(const CDemo &temp)
{
str = new char[strlen(temp.str) + 1];
strcpy(str, temp.str);
}
~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 a1->push_back(d1); delete a1; return 0; } 其实还有人觉得疑惑,为什么push_back的函数原型为: void push_back(const _Ty& _Val) 参数是引用传递的,那为什么还会调用拷贝构造函数呢,这个秘密在于push_back内部还调用了另一个函数insert(),而insert()函数则调用了拷贝构造函数: void push_back(const _Ty& _Val) { // insert element at end if (size() < capacity())
#if _HAS_ITERATOR_DEBUGGING { // room at end, construct it there _Orphan_range(_Mylast, _Mylast); _Mylast = _Ufill(_Mylast, 1, _Val); }
#else /* _HAS_ITERATOR_DEBUGGING */ _Mylast = _Ufill(_Mylast, 1, _Val); #endif /* _HAS_ITERATOR_DEBUGGING */
else insert(end(), _Val); }
因此上面的程序在不重载CDemo类的拷贝构造函数时会崩溃。 你的第一个例子里面,delete a1只是把对象内存回收,而不会把对象里面的指针指向的内存回收,所以不会发生错误,你要举得例子不好。 |