在线等~~~菜鸟问题 刚才发错地方了

[yj19888239]

编译和连接都没问题~~~
就是运行出错
~~那个图我传不上来
就是个对话框
终止~重试~忽略

代码如下：
#include <iostream>
#include <cstring>
using namespace std;

class mystring;
mystring operator+(mystring& string1, mystring& string2);
ostream& operator < <(ostream& outs, mystring& string);

class mystring
{
public:
mystring(const char *s)
{
str=new char[strlen(s)+1];
strcpy(str,s);
}

mystring():max(100)
{
str=new char[max+1];
str[0]='\0';
}


~mystring()
{
delete [] str;
}

friend mystring operator+(mystring& string1, mystring& string2);
friend ostream& operator < <(ostream& outs, mystring& string);

private:
char *str;
int max;
};

mystring operator+(mystring& string1, mystring& string2)
{
mystring temp;
strcpy(temp.str,string1.str);
strcat(temp.str,string2.str);

return temp;
}

ostream& operator < <(ostream& outs, mystring& string)
{
outs < <string.str;
return outs;
}


int main()
{

mystring a,b("yang"),c("jie");

a=b+c;

cout < <a;

return 0;

}

谁来看看啊

[lingyin55]

缺少重载operator=

[rainID]

只扫了一眼，估计就是没有重载=，所以是按位拷贝，析构的时候会重复delete

[adventurelw]

有动态分配以及复制构造过程（重载+的返回对象一定会调用复制构造函数），却没有定义相关的复制构造函数，会导致析构的时候出问题。
同时，因为一体的关系，还需要
定义重载=。

[mengde007]

#include <iostream> 
#include <cstring> 
using namespace std; 

class mystring; 
mystring operator+(mystring& string1, mystring& string2); 
ostream& operator <<(ostream& outs, mystring& string); 

class mystring 
{ 
public: 
	mystring(const char *s):max(strlen(s))
{ 
str=new char[strlen(s)+1]; 
strcpy(str,s); 
} 
mystring(const mystring& my)
{
	str=new char[strlen(my.str)+1];
	strcpy(str,my.str);

}
mystring& operator=(const mystring& a)
{
	if(this==&a)
		return *this;
	delete []str;
	str=new char[strlen(a.str+1)];
	strcpy(str,a.str);
	return *this;
}
mystring():max(100) 
{ 
str=new char[max+1]; 
str[0]='\0'; 
} 


~mystring() 
{ 
delete [] str; 
} 

friend mystring operator+(mystring& string1, mystring& string2); 
friend ostream& operator <<(ostream& outs, mystring& string); 

private: 
char *str; 
int max; 
}; 

mystring operator+(mystring& string1, mystring& string2) 
{ 

char* str1=new char[strlen(string1.str)+strlen(string2.str)+1];
strcpy(str1,string1.str); 
strcat(str1,string2.str); 
mystring temp(str1); 
delete str1;
return temp; 
} 

ostream& operator <<(ostream& outs, mystring& string) 
{ 
outs <<string.str; 
return outs; 
} 


int main() 
{ 

mystring a,b("yang"),c("jie"); 

a=b+c; 

cout <<a; 

return 0; 

} 

[lpf000]

return temp; //返回其拷贝的临时对象，系统合成的拷贝构造函数只会进行简单的复制运算temp.str=临时对象的str;
你需要自己定义拷贝构造函数 进行深拷贝

[aime99]

mystring operator+(mystring& string1, mystring& string2)
{
mystring temp;
strcpy(temp.str,string1.str);
strcat(temp.str,string2.str);

return temp;
}

a = b + c；

没有赋值运算符重载，a直接按位拷贝temp，会导致temp那个对象别释放两次

[T技术沙龙]

没看完，出去一下，现在占个位，待会回来看

[yj19888239]

9楼的代码还是出错

到底是缺少什么东西啊？
是复制构造函数?还是赋值构造函数？或者是重载=？

[papaofdoudou]

#include <iostream>
#include <cstring>
using namespace std;

class mystring;
mystring operator+(mystring& string1, mystring& string2);
ostream& operator <<(ostream& outs, mystring& string);

class mystring
{
public:
mystring(const char *s)
{
str=new char[strlen(s)+1];
strcpy(str,s);
}

mystring()
{
max=100;
str=new char[max+1];
str[0]='\0';
}

mystring(const mystring &strr)
{
str=new char[strlen(strr.str)+1];
strcpy(str,strr.str);
}
mystring& operator=(const mystring &strr)
{
delete []str;
str=new char[strlen(strr.str)+1];
strcpy(str,strr.str);
return *this;

}

~mystring()
{
delete [] str;
}

friend mystring operator+(mystring& string1, mystring& string2);
friend ostream& operator <<(ostream& outs, mystring& string);

private:
char *str;
int max;
};

mystring operator+(mystring& string1, mystring& string2)
{
mystring temp;

strcpy(temp.str,string1.str);
strcat(temp.str,string2.str);

return temp;
}

ostream& operator <<(ostream& outs, mystring& string)
{
outs <<string.str;
return outs;
}


int main()
{

mystring a,b("yang"),c("jie");

a=b+c;
cout <<a<<endl;
return 0;

}

[papaofdoudou]

最好两个都要重载，重载operator=和copy constructor，都需要调用：
mystring(const mystring &strr)
{
str=new char[strlen(strr.str)+1];
strcpy(str,strr.str);
}

mystring& operator=(const mystring &strr)
{
if(this==&strr)
return *this; //上面的的忘了这句自赋值情况，逻辑不完整，不过不会出错，最好加上这句。
delete []str;
str=new char[strlen(strr.str)+1];
strcpy(str,strr.str);
return *this;

}

[ldming]

去掉析构函数就可以了 具体什么原因 待定 ！！

[cpp_for_food]

在没有定义operator=和copy constructor的情况下，当遇到这个类有对operator=和copy constructor的调用时，编译器就会调用系统默认的自动生成的这个类的operator=和copy constructor，在这种情况下operator=只是对这个类中的成员变量执行标准的简单赋值操作，即mystring1=mystring2(mystring1.str=mystring2.str), 这样在mystring2对象析构时mystring2.str就会被delete两次了；copy constructor的情况类似。

[bbb332]

缺少重载operator=
，，

[chenxu_ustc]

[Quote=引用 11 楼 aime99 的回复:]
mystring operator+(mystring& string1, mystring& string2)
{
mystring temp;
strcpy(temp.str,string1.str);
strcat(temp.str,string2.str);

return temp;
}

a = b + c；

没有赋值运算符重载，a直接按位拷贝temp，会导致temp那个对象别释放两次
[/Quote]
正解

[T技术沙龙]

[Quote=引用 14 楼 tugouxp 的回复:]
#include <iostream>
#include <cstring>
using namespace std;

class mystring;
mystring operator+(mystring& string1, mystring& string2);
ostream& operator < <(ostream& outs, mystring& string);

class mystring
{
public:
mystring(const char *s)
{
str=new char[strlen(s)+1];
strcpy(str,s);
}

mystring()
{
max=100;
str=new char[max+1];
str[0]='\0';
}

mystrin…
[/Quote]

[T技术沙龙]

晚了，已经有正确答案了

[yj19888239]

分给14楼吧~~~
我回去看了下书~~~貌似这样的

[papaofdoudou]

[Quote=引用 19 楼 chenxu_ustc 的回复:]
引用 11 楼 aime99 的回复:
mystring operator+(mystring& string1, mystring& string2)
{
mystring temp;
strcpy(temp.str,string1.str);
strcat(temp.str,string2.str);

return temp;
}

a = b + c；

没有赋值运算符重载，a直接按位拷贝temp，会导致temp那个对象别释放两次

正解
[/Quote]
非正解，一定要定义赋值运算符重载的，而且copy constructor也必须要定义 11楼和19楼理解有误，
return temp 的时候会调用copy constructor，因为是按值返回，
而运行a=b+c的时候会调用operator=,这是因为此时a已经构造完毕了，成为完整的对象，要对此对象赋值，当然要调用operator=了

[papaofdoudou]

如果将
mystring a,b("yang"),c("jie");

a=b+c;
改写成：
mystring b("yang"),c("jie");

mystring a=b+c;
则就不需要调用operator=的，也就不需要定义了；
这个时候只需要一个copy constructor就行了；
但是针对楼主目前的代码，缺少copy constrctor和 operator=任何一个都是不可取的。

[koulongfei]

恩,学习.