帮忙看下这个程序,运行时报内存错误!谢谢

[simonqsm]

#include<string.h>
#include<iostream.h>
class Person{
private :
char *name;
public :
Person(){
name=new char[256];
}
Person(Person &p){
name=p.name;
}
~Person(){
delete[] name;
}
void SetName(const char* name1){
if(strlen(name1)+1<256) strcpy(name,name1);
}
void PrintName(){
cout<<name<<endl;
}
};
int main(){
Person p1;
p1.SetName("John");
p1.PrintName();
Person p2(p1);
p2.SetName("Alice");
p2.PrintName();
p1.PrintName();
}

[ttkk_2007]

Person(Person &p){
name=p.name;
}
===============
Person(Person &p){
if(this != &p){
name = new [strlen(p) + 1];
strcpy(name, p.name);
}

}

void SetName(const char* name1){
if(strlen(name1)+1 <256) strcpy(name,name1);
}
===================
这个也错了，strlen(name1)永远等于4

[baihacker]

Person(Person &p){ 
    name=new char[256]; 
    strcpy(name, p.name); 
} 

[ttkk_2007]

Person(Person &p){ 
    if(this != &p){ 
        name = new [strlen(p) + 1]; 
        strcpy(name, p.name); 
    } 

} 
//改改
Person(Person &p){ 
    if(this != &p){ 
        name = new char[strlen(p.name) + 1]; 
        strcpy(name, p.name); 
    } 

} 

[ttkk_2007]

void SetName(const char* name1){
if(strlen(name1)+1 <256) strcpy(name,name1);
}
===================
这个也错了，strlen(name1)永远等于4
======================================
收回这句话

[K行天下]

Person(Person &p){
name=p.name;
}
错了，字符串的拷贝要使用strcpy函数

[Giegie]

每次析构时，重复delete一个指针，下次再delete就有问题了

Person(Person &p){
if(this != &p){
name = new [strlen(p) + 1];
strcpy(name, p.name);
}

}
有问题

一楼的，注意是strlen，不是sizeof

[wangdeqie]

//这么写
#include <string.h> 
#include <iostream.h> 
class Person
{ 
private : 
	char *name; 
public : 
	Person()
	{ 
		name=new char[256]; 
	} 
	Person(Person &p)
	{ 
		name=new char[256]; 
		strcpy(name,p.name);
	} 
	~Person()
	{ 
		if (name!=NULL)
			delete[] name; 
	} 
	void SetName(const char* name1)
	{ 
		if(strlen(name1)+1 <256) strcpy(name,name1); 
	} 
	void PrintName()
	{ 
		cout <<name <<endl; 
	} 
}; 
int main()
{ 
	Person p1; 
	p1.SetName("John"); 
	p1.PrintName(); 
	Person p2(p1); 
	p2.SetName("Alice"); 
	p2.PrintName(); 
	p1.PrintName(); 
}

[kojie_chen]

原来是这样的啊