帮忙看看错在哪吧！！！高分求教啦！！！

[MashiMaro2002]

刚开始学，试着写了一个实现字符串连接的小程序，运行结果却有乱码，不知道错哪了，请各位大虾帮忙指点一二，高分求教哦！！！谢了！！！
#include <iostream.h>
#include <string.h>

class String{
public:
String(char* pN="no name")
{
pName=new char[strlen(pN)+1];
if(pName!=0)
strcpy(pName,pN);
}
String(String& s)
{
pName=new char[strlen(s.pName)+1];
if(pName!=0)
strcpy(pName,s.pName);
}
~String()
{
if(pName)
{
delete pName;
pName=0;
}
}
friend String operator +(String& s1,String& s2);
void display()
{
cout<<pName<<endl;
}
protected:
char* pName;
};

String operator +(String& s1,String& s2)
{
char* newStr=new char[strlen(s1.pName)+strlen(s2.pName)+1];
strcpy(newStr,s1.pName);
strcat(newStr,s2.pName);
return String(newStr);
}

void main()
{
String s1("aaa"),s2("bbb"),s3;
cout<<"String1 is:";
s1.display();
cout<<"String2 is:";
s2.display();
s3=s1+s2;
cout<<"String1 + String2 is:";
s3.display();
}

[GoogleGeek]

test it !

[wanglei888]

在构造函数中：
String(char* pN="no name")
{
pName=new char[strlen(pN)+1];
if(pName!=0)
strcpy(pName,pN);
}
你判断if (pName != 0) 这一句，不会按你的意愿执行，因为pName没有被正确初始化，这样改：
String(char* pN="no name") ： pName(0)
{
pName=new char[strlen(pN)+1];
if(pName!=0)
strcpy(pName,pN);
}
其他部分我没有发现问题。

[Xeroo]

你没有重载=
所以当s3=s1+s2时会出现错误
还有就是不能把char *强制转换成String可能会出错
不过不敢肯定~~~~~~权当参考

[punpuny]

问题出在s3=s1+s2这句上面，你在+这个操作符重载函数中返回的是一个临时变量，出了函数体后就会析构，这样s3中的内容当然是无效的了。

[GoogleGeek]

出现乱码，是应为你使用了不该使用的内存！
你忘记了C++中的拷贝构造函数的调用时机（机制）！
只有在定义一个类的对象并且以同类的对象作为该对象的初值时，才会调用该类的拷贝构造函数！，否则，如果没有提供相应的运算符重载的话，将直接调用compiler生成的每有什么意义的bitwisecopy function!
运行下面的程序 !Everything is OK!
----------------------------------------
// console.cpp : Defines the entry point for the console application.
//
#include "stdafx.h"
#include <iostream.h>
#include <string.h>

class String{
public:
String(char* pN="no name")
{
pName=new char[strlen(pN)+1];
if(pName!=0)
strcpy(pName,pN);
}
String(const String& s)
{
pName=new char[strlen(s.pName)+1];
if(pName!=0)
strcpy(pName,s.pName);
}
~String()
{
if(pName)
{
delete[] pName;
pName=0;
}
}
//must add the function:operator =
String operator =(String& str)
{
//first delete the pName;
delete[] pName;
int len=strlen(str.pName)+1;
pName=new char[len];
strcpy(pName,str.pName);
return *this;
}
friend String operator +(String& s1,String& s2);
void display()
{
cout<<pName<<endl;
}
protected:
char* pName;
};

String operator +(String& s1,String& s2)
{
char* newStr=new char[strlen(s1.pName)+strlen(s2.pName)+1];
strcpy(newStr,s1.pName);
strcat(newStr,s2.pName);
return String(newStr);
}

void main()
{
String s1("aaa"),s2("bbb"),s3;
cout<<"String1 is:";
s1.display();
cout<<"String2 is:";
s2.display();
//call the String String::operator =(String& str),
//not the function:String::String(const String &s)
s3=s1+s2;
cout<<"Now call the overloading operators:operator =(String& str),! "<<endl;
cout<<"String1 + String2 is:";
s3.display();
//Only when you define a object ,call the function:String::(String &s)
cout<<"Now call the function:String::(const String& str)!"<<endl;
String s=s1+s2;
cout<<"String1 + String2 is:";
s.display();
}

[GoogleGeek]

To:punpuny()
问题出在s3=s1+s2这句上面，你在+这个操作符重载函数中返回的是一个临时变量，出了函数体后就会析构，这样s3中的内容当然是无效的了。
-----------------------------------------
老大！谁告诉你除了函数体变量就立即被析构掉？搞笑！
compiler 确保在使用完了变量后，才会对变量进行析构！如果象你说得那样，不只会有多少的传值调用的函数会失败！

[punpuny]

psusong(我心飞扬)不好意思，我弄错了。
你的code基本正确，在你的代码中=的重载应该返回String&而不是String，这个看一下CString的做法就知道了。

[GoogleGeek]

To:punpuny()
谢谢！我的写法不规范！确实应该返回String&!
实际上你当初的理解对于该问题也是正确的！----你的本意应该是：同一块内存被delete两次！

[punpuny]

否则对于形如
CString s1("aaa");
CString s2;
CString &(

[MashiMaro2002]

谢谢各位大虾！！！确有收获呀！！！
^_^