char * 型 内存的回收处理~

[duoshanx]

#include <iostream>
using namespace std;
class mojiretu{
public:
mojiretu(const char *n){
name = new char(strlen(n)+1);
strcpy(name,n);
}
mojiretu(char n){
name= new char(2);
name[0]=n;
name[1]='\0';
}
void show(){cout << name << endl;}
~mojiretu(){
if (name!=NULL) delete[] name;
cout << "-END-\n";
}
private :
char *name;

};

int main(){
mojiretu m1("watanabe");
m1.show();

mojiretu m2('x');
m2.show();
}
----------------
以为new char()了，所以内存一定要回收，
但回收的时候提示
HEAP CORRUPTION DETECTED：....
难道已经自动回收了吗？

请指教。

[duoshanx]

为什么呢？

[crst_zh]

[Quote=引用 5 楼 lingyin55 的回复:]
需要手动添加'\0'

引用 4 楼 duoshanx 的回复:
C/C++ code
#include <iostream>
using namespace std;
class mojiretu{
public:
mojiretu(const char *n){
name = new char[strlen(n)+1];
strcpy(name,n);
name[strlen(n)]='\0'; // <---这里也忘记加上'\0'了,再追问一句，strcpy之后，需要加'\0'吗？
}
mojiretu(char n){
name= new char[2];…
[/Quote]

不需要加'\0'，看看是strcpy的源码吧，最后一个0也是拷贝过去了的。

另外最好在析构里面这样写：
~mojiretu(){
delete[] name;
name=NULL;
cout << "-END-\n";
}

[snail_1989]

楼至的类名很古怪啊 [mojiretu]文字列（もじれつ）

[crst_zh]

strcpy()源码：

char * __cdecl strcpy(char * dst, const char * src)
{
char * cp = dst;

while( *cp++ = *src++ )
; /* Copy src over dst */

return( dst );
}

看看，先赋值了，然后再检查是不是为'\0'的，也就是说，'\0'是被复制了的。

[sszwbuaa]

被LZ骗了，应该是[],不是(),我还调试了半天。。。
name = new char[strlen(n)+1]
name= new char[2];

[woods2001]

原来是这样个错误·
楼主一点细节 没注意好~

[duoshanx]

to lingyin55
谢谢， 回答的够细，够N。

to snail_1989
这是公司的练习题，我是新入社员:)

to sszwbuaa woods2001
多谢捧场，我也忙了半天没找到原因，嘿嘿

[wanyuzhen]

new char(2),在堆上建立一个char类型空间，并把2赋值
实际上只开辟了一个char.类似构造函数

[duoshanx]

[Quote=引用 13 楼 wanyuzhen 的回复:]

new char(2),在堆上建立一个char类型空间，并把2赋值
实际上只开辟了一个char.类似构造函数
[/Quote]
我不是很明白。
可能你是说括号（）调用了构造函数，把2传给构造函数。？！

[adventurelw]

new char(2)是将char初始化为编码为2的字符，应该用delete name;

但你明明name是个字符串，所以初始化分配时应该是new char[2];然后就可以用delete [] name了。

[effective_person]

清区分
new char[2] 和 new char(2)

#include <iostream> 
using namespace std; 
class mojiretu
{ 
public: 
	mojiretu(const char *n)
	{ 
		name = new char[strlen(n)+1];  // here 
		strcpy(name,n); 
	} 

	mojiretu(char n)
	{ 
		name= new char[2];  // here 
		name[0]=n; 
		name[1]='\0'; 
	} 
	void show()
	{
		cout << name << endl;
	} 
	~mojiretu()
	{ 
		if (name!=NULL) 
			delete[]  name; 
		cout << "-END-\n"; 
	} 
private : 
	char *name; 
	
}; 

int main()
{ 
	mojiretu m1("watanabe"); 
	m1.show(); 
	
	mojiretu m2('x'); 
	m2.show(); 
} 

[duoshanx]

Thanks !!

[duoshanx]

#include <iostream>
using namespace std;
class mojiretu{
public:
	mojiretu(const char *n){
		name = new char[strlen(n)+1];
		strcpy(name,n);
		name[strlen(n)]='\0';          //<---这里也忘记加上'\0'了,再追问一句，strcpy之后，需要加'\0'吗？
	}
	mojiretu(char n){
		name= new char[2];
		name[0]=n;
		name[1]='\0';
	}
	void show(){cout << name << endl;}
	~mojiretu(){
		delete[] name;
		cout << "-END-\n";
	}
private :
	  char *name;

};

int main(){
	mojiretu m1("watanabe");
	m1.show();

	mojiretu m2('x');
	m2.show();
 
	mojiretu m3("abc");
	m3.show(); 
}

-------------------------
再追问一句，strcpy之后，需要加'\0'吗？

[lingyin55]

需要手动添加'\0'

[Quote=引用 4 楼 duoshanx 的回复:]
C/C++ code
#include <iostream>
using namespace std;
class mojiretu{
public:
mojiretu(const char *n){
name = new char[strlen(n)+1];
strcpy(name,n);
name[strlen(n)]='\0'; //<---这里也忘记加上'\0'了,再追问一句，strcpy之后，需要加'\0'吗？
}
mojiretu(char n){
name= new char[2];
name[0]=n;
name[1]='\0';
}
void …
[/Quote]