delete出错问题

hujianghua2012

A.h:
class A{
     char *p;
public:
    A(char *str);
    ~A(){
      delete p; //调试,这里出错
   }
};

A.cpp:
A::A(char *str){
   //觉得这里可能有问题，但是不是很清楚，能解释一下吗？
   p=new char[strlen(str)];
   p=str;
}

main.cpp
int main(){
   char *str="aaaaaa";   
   A a(str);
}

liuiang

这个代码写的，让我怎么说呢？

hellioncu

   p=new char[strlen(str)];
   p=str;

cdtits

这个严格来说，不算c++代码

hujianghua2012

本来是没有p=new .....的，我想我delete了就要new一个，所以就new了一个。没有这句也是一样的问题。回复 3# hellioncu


   

hellioncu

hujianghua2012 发表于 2012-07-25 14:56
本来是没有p=new .....的，我想我delete了就要new一个，所以就new了一个。没有这句也是一样的问题。回复 3# ...

因为这句new白做了，p马上被后面的赋值修改了，delete释放的不是你new来的。
如果这样你还不知道自己的错误，那仔细看书去吧。

bruceteen

回复 2# liuiang
我只找到7个错误，当然，他一共也就9句话

class A{
     char *p;
public:
    A(char *str); // 错
    ~A(){
      delete p; // 错
   }
   // 错
   // 错
};

A::A(char *str){
   p=new char[strlen(str)]; // 错
   p=str; // 错
}

int main(){
   char *str="aaaaaa"; // 错
   A a(str);
}

Moon_Bird

bruceteen 发表于 2012-07-25 14:59
回复 2# liuiang
我只找到7个错误，当然，他一共也就9句话

你好狠 ...............哈哈   

duke3030

回复 7# bruceteen


    1. 为啥空行也算是“错”？
    2. "p = str" 和 "delete p"这两行代码不可能都错吧
    3. "p=new char[strlen(str)];"这句是错在哪里呢？
    4. “A(char *str);” 构造函数错在哪里？

求教

bruceteen

回复 9# duke3030
1. 缺少 拷贝构造 和 赋值操作。你不显式的写出来，那编译器就会给一个缺省的浅拷贝（你自己google一下什么叫浅拷贝），对你的这个例子来说浅拷贝是绝对错误的
2. 你这是字符指针的赋值，不是字符串的赋值（strcpy）；前面用的是new[]，后面就必须是delete[]（虽然某些超级烂货会骗你说无所谓）
3. strlen获取的是字符串长度，而非字符串占用的长度。字符串以‘\0'结尾，所以应该是  strlen(str)+1
4. 明显应该是 A( const char* str )