成员函数返回对象还是引用

andy_hqj

首先附上代码
1、str.h
class String
{
        char * data;

public:
        String(char * str);
        String(String &s);
        ~String();
        String & operator=(String &s);
        String operator+(String &s);
};

2、str.cpp
#include <iostream>
#include "str.h"
#include <string.h>
using namespace std;

String::String(char *str)
{
        data=new char[strlen(str)+1];
        strcpy(data, str);
}

String::String(String &s)
{
        data=new char[strlen(s.data)+1];
        strcpy(data, s.data);
}
String::~String()
{
        delete [] data;
}
String & String:: operator=(String &s)
{
        if(this == &s)
                return *this;
        delete [] this->data;
        this->data = new char[strlen(s.data)+1];
        strcpy(this->data,s.data);
        return *this;
}
String String:: operator+(String &s)
{
        String tmp( "" );
        delete [] tmp.data ;
        tmp.data = new char[strlen(data)+strlen(s.data)+1];
        strcpy(tmp.data, data);
        strcat(tmp.data, s.data);
        return tmp;
}


3、测试函数 main.cpp
#include <iostream>
#include "str.h"
using namespace std;
int main()
{
        String s("hello" );
        String s1("world" );

        String s3("" );
        s3 = s + s1;
        return 0;
}


错误：
1、String s3("" );
        s3 = s + s1;
编译错误为：
main.cpp: In function ‘int main()’:
main.cpp:10: 错误：no match 为 ‘operator=’ 在 ‘s3 = String:: operator+(String& ) (((String& ) (& s1)))’ 中
str.h:9: 附注：备选为： String& String:: operator=(String& )
2、如果写成 String s3=s+s1；
编译错误为：
main.cpp:9: 错误：对 ‘String::String(String)’ 的调用没有匹配的函数
str.h:7: 附注：备选为： String::String(String& )
str.h:6: 附注：         String::String(char*)


但如果把重载+法运算符号的返回值改为引用，这两种方法都没问题(当然会提示返回局部变量地址)

那位帮忙解释一下，这两个错误的原因，我的环境为as5 gcc

bruceteen

一个 const修饰 也没有

folklore

回复 2# bruceteen


    **好， 我没看出什么错误， **看出来没？
难道
operator+
可
const CLASS:perator+
才行？

好像没什么道理啊。

bruceteen

folklore 发表于 2013-11-06 08:31
回复 2# bruceteen

s+s1 是个临时String，不能被作为 String & operator=(String &s) 的参数，除非参数改为 const String& s

1. class String
   
2. {
   
3. public:
   
4.     String( const char* str );
   
5.     String( const String& s );
   
6.     ~String();
   
7.     String& operator=( const String& s );
   
8.     String operator+( const String& s ) const;
   
9. private:
   
10.     char* data_;
    
11. };
    
12. 
    
13. #include <cstring>
    
14. 
    
15. String::String( const char* str )
    
16. {
    
17.     data_ = new char[strlen(str)+1];
    
18.     strcpy( data_, str );
    
19. }
    
20. 
    
21. String::String( const String& s )
    
22. {
    
23.     data_ = new char[strlen(s.data_)+1];
    
24.     strcpy( data_, s.data_ );
    
25. }
    
26. String::~String()
    
27. {
    
28.     delete[] data_;
    
29. }
    
30. String& String::operator=( const String& s )
    
31. {
    
32.     if( this == &s )
    
33.         return *this;
    
34.     delete[] data_;
    
35.     data_ = new char[strlen(s.data_)+1];
    
36.     strcpy( data_, s.data_ );
    
37.     return *this;
    
38. }
    
39. String String:: operator+( const String& s ) const
    
40. {
    
41.     String tmp( "" );
    
42.     delete[] tmp.data_;
    
43.     tmp.data_ = new char[strlen(data_)+strlen(s.data_)+1];
    
44.     strcpy( tmp.data_, data_ );
    
45.     strcat( tmp.data_, s.data_ );
    
46.     return tmp;
    
47. }
    
48. 
    
49. int main()
    
50. {
    
51.     String s("hello" );
    
52.     String s1("world" );
    
53. 
    
54.     String s3("");
    
55.     s3 = s + s1;
    
56. 
    
57.     return 0;
    
58. }

复制代码

folklore

回复 4# bruceteen


    有道理， 要改

1. String& String::operator=( const String& s )

复制代码

就行了。

写过很多operator+ =， 都自动加上const了， 中毒太深～～

andy_hqj

回复 4# bruceteen


如果把拷贝构造函数中的const去掉  编译时提示第10行，也就是  s3=s+s1；
main.cpp:10: 错误：对 ‘String::String(String)’ 的调用没有匹配的函数
str.h:9: 附注：备选为： String::String(String&)
str.h:8: 附注：         String::String(char*)

这里不是应该调用赋值运算符重载函数吗？这么会找构造函数的？

如果拷贝构造函数参数加上const 就能顺利通过，可执行的时候打印也没看到调用构造函数.
如果改成这样 String s3=s+s1； 同样没看到拷贝构造函数调用。这是为什么啊?(我在构造函数和拷贝构造中都加了打印)


   

myworkstation

回复 6# andy_hqj


    由于你的重载operator +的函数内部使用了String tmp( "" );而""这样的直接字符字面量实际上是const char*类型，它无法隐式转换为char*也无法隐式转换为string&，实际上可以转换为rvalue的char*，但此例中参数的char*以及string&都是lvalue。

cokeboL

本人过去、现在以及将来都不认识楼主，且自古以来与楼主无利益关系。楼主表述之事与本人无关.

andy_hqj

回复 7# myworkstation


那你的意思是实际报错的是 String  tmp("")；？
那String s("hello" );String s1("world" );这两句为什么没报这个错误呢？
如果是这句报错，那也应该是报 String(const char *)没有匹配的函数吧？



   

myworkstation

回复 9# andy_hqj


    你用的哪个编译器？假如我说的问题在你的编译器不存在，单独分析两次赋值：String s3=s+s1；这种方式叫copy initialize，需要调用copy构造函数，而String s3;s3=s+s1;这样的方式调用的是operator = ，当你把copy构造函数String::String( const String& s )中参数的const去掉变成String::String( String& s )，第一种方式肯定有问题String s3=s+s1,因为s+s1是个临时对象无法转换为lvalue引用。而第二种方式由于你重载的操作符String String:: operator+( const String& s ) const,返回的结果是个String，那么return tmp;是有问题的，因为tmp要copy到函数返回值中，而这个copy无法进行（函数返回值赋值处也有一个临时对象，原因同上临时对象无法转换为lvalue引用，所以operator+调用是失败的，错误提示没有准确的提示到)，你改成String& String:: operator+( const String& s )这样的原型应该就不会出这个错误了。