这是个什么情况...

[OMG001]

#include<iostream>
using namespace std;
class String
{
public:
String(char *p){ch=p;cout<<"initialization constructor is called\n";};
String(){ch=new char();ch="abcd";cout<<"constructor called"<<endl;}
~String(){delete ch;cout<<"destructor is called\n"<<endl;};
private:
char *ch;
};
int main()
{
String *q=new String;
delete q;
return 0;
}




在运行结束后，会弹出一个中止XXX的XXX（我也说不清楚，各位弄回去跑一下试试就知道了）....谁解释下为啥子？

[书虫]

String(){ch=new char();ch="abcd";cout<<"constructor called"<<endl;}
你虽然什么了一个字节的内存，ch指向该内存，但是ch="abcd";这句使你的ch指针又指向了"abcd"常量，"abcd"常量是存储在数据段中的，你delete的不是你申请的堆内存，而是常量的内存，肯定出错咯！

[zuiyuezhou888]

#include<iostream>
using namespace std;
class String
{
public:
	String(char *p){ch=p;cout<<"initialization constructor is called\n";};
	String(){char tmp[4]={'a','b','c','d'};ch=tmp;cout<<"constructor called"<<endl;}
	~String(){cout<<"destructor is called\n"<<endl;};
private:
	char *ch;
};
int main()
{
	String *q=new String;
	delete q;
	getchar();
	return 0;
}

问题出在这
ch=new char();ch="abcd";
ch是个指针

[超gogo]

[Quote=引用 1 楼 happymawolf 的回复:]

String(){ch=new char();ch="abcd";cout<<"constructor called"<<endl;}
你虽然什么了一个字节的内存，ch指向该内存，但是ch="abcd";这句使你的ch指针又指向了"abcd"常量，"abcd"常量是存储在数据段中的，你delete的不是你申请的堆内存，而是常量的内存，肯定出错咯！
[/Quote]
正解

[OMG001]

那如果我是想在构造函数中申请一个空间来存储一个字符串，而该串在默认情况为abcd呢？

[OMG001]

收到~谢谢各位

[pathuang68]

ch=new char(); ch="abcd";
这样的写法要注意了。

1. ch只有一个byte的空间。看楼主的写法是想让它装下4bytes的内容；
所以最好这样写：
char *ch = new char[strlen("abcd") + 1];
2. 其次，看楼主的意思是想将字符串abcd赋给ch。所以请这么写：
strcpy(ch, "abcd");

[pathuang68]

按照楼主的意图实现的代码：

#include<iostream>
using namespace std;
class String
{
public:
	String(char *p)
	{
		ch = new char[strlen(p) + 1];
		strcpy(ch, p);
		cout<<"initialization constructor is called\n";
	}
	String()
	{
		char *s = "abcd";
		ch = new char[strlen(s) + 1];
		strcpy(ch, s);
		cout<<"constructor called"<<endl;
	}
	~String()
	{
		delete ch;
		cout<<"destructor is called\n"<<endl;
	}
private:
  char *ch;
};

int main(int argc, char* argv[])
{
	String *a = new String();
	String *b = new String("hello, world!");
	delete a;
	delete b;

	return 0;
}