赋值运算符“=”的重载问题

xnec

请问这个代码中：
#include<iostream.h>
#include<string.h>
class string{
        char *ptr;
public:
        string (char *s)
        {ptr=new char[strlen(s)+1];
        strcpy(ptr,s);
        }
        ~string()
        {delete ptr;}

        void print()
        {cout<<ptr<<endl;}

        string &operator=(string &;
};

string &string:perator = (string &s)
{
        if(this==&s) return *this;
        delete ptr;
        ptr=new char[strlen(s.ptr)+1];
        strcpy(ptr,s.ptr );
        return *this;
}
int main()
{
        string p1("chen";
        {
                string p2("    ";
                p2=p1;
                cout<<"p2:";
                p2.print();
        }

        cout<<"p1:";
        p1.print() ;
        return 0;
}

将string &operator=(string &;改为string operator=(string &;
将string &string:perator = (string &s)
{
        if(this==&s) return *this;
        delete ptr;
        ptr=new char[strlen(s.ptr)+1];
        strcpy(ptr,s.ptr );
        return *this;
}
改为string string:perator = (string &s)
{
        if(this==&s) return *this;
        delete ptr;
        ptr=new char[strlen(s.ptr)+1];
        strcpy(ptr,s.ptr );
        return *this;
}
运行结果为什么不一样呢？

tyc611

1. 代码里太多笑脸了，请用[code]……[/code]把它包越来
2. 看见你自己的类与标准库中的string类同名，不好的命名，容易与库冲突，建议改为如MyString的名字

xnec

1. 请问这个代码中：
   
2. #include<iostream.h>
   
3. #include<string.h>
   
4. class string{
   
5.         char *ptr;
   
6. public:
   
7.         string (char *s)
   
8.         {ptr=new char[strlen(s)+1];
   
9.         strcpy(ptr,s);
   
10.         }
    
11.         ~string()
    
12.         {delete ptr;}
    
13. 
    
14.         void print()
    
15.         {cout<<ptr<<endl;}
    
16. 
    
17.         string &operator=(string &;
    
18. };
    
19. 
    
20. string &string:perator = (string &s)
    
21. {
    
22.         if(this==&s) return *this;
    
23.         delete ptr;
    
24.         ptr=new char[strlen(s.ptr)+1];
    
25.         strcpy(ptr,s.ptr );
    
26.         return *this;
    
27. }
    
28. int main()
    
29. {
    
30.         string p1("chen";
    
31.         {
    
32.                 string p2("    ";
    
33.                 p2=p1;
    
34.                 cout<<"p2:";
    
35.                 p2.print();
    
36.         }
    
37. 
    
38.         cout<<"p1:";
    
39.         p1.print() ;
    
40.         return 0;
    
41. }
    
42. 
    
43. 将string &operator=(string &;改为string operator=(string &;
    
44. 将string &string:perator = (string &s)
    
45. {
    
46.         if(this==&s) return *this;
    
47.         delete ptr;
    
48.         ptr=new char[strlen(s.ptr)+1];
    
49.         strcpy(ptr,s.ptr );
    
50.         return *this;
    
51. }
    
52. 改为string string:perator = (string &s)
    
53. {
    
54.         if(this==&s) return *this;
    
55.         delete ptr;
    
56.         ptr=new char[strlen(s.ptr)+1];
    
57.         strcpy(ptr,s.ptr );
    
58.         return *this;
    
59. }
    
60. 运行结果为什么不一样呢？

复制代码

xnec

发表于: 2007-1-7 13:45    主题：   [引用]  [投诉] [快速回复]  


--------------------------------------------------------------------------------
跟标准库中的string类没有冲突的

tyc611

原帖由 xnec 于 2007-1-7 13:50 发表
发表于: 2007-1-7 13:45    主题：   [引用]  [投诉] [快速回复]  


--------------------------------------------------------------------------------
跟标准库中的string类没有冲突的

哎，建议你不听也罢，我是说容易出现问题！

还是回到你的问题上：
1. delete [] ptr;　释放数组
2. 你的第二个operator由于返回的是一个对象（不是引用），它会调用拷贝构造函数来构造这个临时对象。由于你没有定义拷贝构造函数，所以默认的就是浅拷贝（即新的对象只是复制那个字符指针的值，即两个对象的字符指针指向同一个字符串），而你的析构函数却要去释放那个指针。这样在对象析构时就可能释放那个指针两次，就发生错误了。