c++重载的问题

Question

#include <iostream.h>
#include <cstring>
class Mystring
{
public:
void output();
void Mystring::create(char *s);
friend Mystring operator +(Mystring &b,Mystring &c);//重载+号运算符
private:
char *str;//指针字符型私有成员
};
int main()
{
char a1[100],a2[100],a3[100];
Mystring a,b,c;
a.add();
cout<<"Please input the string of b"<<endl;
cin.getline(a1,100);
b.create(a1);
cout<<endl;
cout<<"Please input the string of c"<<endl;
cin.getline(a2,100);
c.create(a2);
cout<<endl;
a=b+c;
a.output();
cout<<endl;
return 0;
}

void Mystring::create(char *s)//定义Mystring的create成员函数
{
str=new char[strlen(s)+1];//运用到析构函数，分配动态字符型数组内存空间，
strcpy(str,s);
}

Mystring operator +(Mystring &b,Mystring &c)//对Mystring类的+运算符函数进行重载
{
Mystring a;
a.str=strcat(b.str,c.str);
return a;
}
void Mystring::output()
{
cout<<"the string of a is"<<endl;
cout<<str<<endl;
}
在自学C++，想做一个a,b,c重载的加法运算符，下面的是错误提示，求解答，谢谢

Answer 1

楼主大体都是正确的只有一处小错误和一处不足的地方已经改正过来
class Mystring
{
public:
void output();
void Mystring::create(char *s);
friend Mystring operator +(Mystring &b,Mystring &c);//重载+号运算符
private:
char *str;//指针字符型私有成员
};
int main()
{
char a1[100],a2[100],a3[100];
Mystring a,b,c;
// a.add();//注意这里!!!
cout<<"Please input the string of b"<<endl;
cin.getline(a1,100);
b.create(a1);
cout<<endl;
cout<<"Please input the string of c"<<endl;
cin.getline(a2,100);
c.create(a2);
cout<<endl;
a=b+c;
a.output();
b.output();
c.output();
cout<<endl;
return 0;
}

void Mystring::create(char *s)//定义Mystring的create成员函数
{
str=new char[strlen(s)+1];//运用到析构函数，分配动态字符型数组内存空间，
strcpy(str,s);
}

Mystring operator +(Mystring &b,Mystring &c)//对Mystring类的+运算符函数进行重载
{
Mystring a;
a.str=new char[strlen(b.str)+strlen(c.str)+1];//给a动态分配内存空间
strcpy(a.str,b.str);//保证b不被修改
strcat(a.str,c.str);
return a;
}
void Mystring::output()
{
cout<<"the string of a is"<<endl;
cout<<str<<endl;
}
谢谢~

Answer 2

首先有个问题
主函数中的a.add()是干什么用的？
这个函数不要的话，整个程序是可以运行的。
只是在vs中需要把#include<iostream.h>改成#include<iostream>

不过这个程序存在一个比较严重的问题,
就是在如下这段代码中：
Mystring operator +(Mystring &b,Mystring &c)//对Mystring类的+运算符函数进行重载
{
Mystring a;
a.str=strcat(b.str,c.str);
return a;
}

按照这种写法，b中的字符串会因此发生变化。

字符串的拷贝顺序应该是先把b中的字符串拷贝到a中，
然后再把c中的字符串追加拷贝到a中

Answer 3

char a1[100],a2[100],a3[100];
Mystring a,b,c;
a.add();

你的add（）在哪？

Answer 4

对于加法“+”而言，一般是不应改变两个加数的值的，即a = b + c;表达式中，b和c的值不应由于做了加法运算而改变。所以，在你的代码中：
Mystring operator +(Mystring &b,Mystring &c)//对Mystring类的+运算符函数进行重载
{
Mystring a;
a.str=strcat(b.str,c.str);
return a;
}
strcat(b.str,c.str);实质上是把c的字符串给加到b后了，即b的值已经发生了改变，问题就在这儿。
应该改为：
Mystring operator +(Mystring &b,Mystring &c)
{
Mystring a;
strcpy(a.str,b.str);
strcat(a.str,c.str);
return a;
}