这个代码错在哪

王鹿其

#include<iostream>
#include<string>
using namespace std;
class phone
{
public:
       
        phone(string pname)
        {
                cout << "phone的函数调用" << endl;
                p_name = pname;
        }
        ~phone()
        {
                cout << "phone的析构函数调用" << endl;
        }
        phone(const phone&p)
        {
                cout << "phone的拷贝函数调用" << endl;
        }
        string p_name;

};
class person
{
public:
        person(string name, string phonename) :m_phone(phonename)
        {
                cout << "person的构造函数调用" << endl;
                m_name = name;
                //为什么要用初始化列表，而不能直接写 m_phone.p_name=phonename;
        }
        ~person()
        {
                cout << "person的析构函数调用" << endl;
        }
        person(const person& p)
        {
                cout << "peeson的拷贝函数调用" << endl;
        }
        string m_name;
        phone m_phone;
};

void test01()
{
        person p("张三","苹果max");
        cout << p.m_name << "拿着" << p.m_phone.p_name << endl;
}

int main()
{
        test01();
        return 0;
}

isdkz

这个C++代码中的主要问题是你没有正确实现拷贝构造函数。对于类 phone 和 person ，你的拷贝构造函数没有正确复制对象的数据，这可能会导致问题。

修改后的代码如下：

1. #include<iostream>
   
2. #include<string>
   
3. using namespace std;
   
4. class phone
   
5. {
   
6. public:
   
7.     phone(string pname)
   
8.     {
   
9.         cout << "phone的函数调用" << endl;
   
10.         p_name = pname;
    
11.     }
    
12.     ~phone()
    
13.     {
    
14.         cout << "phone的析构函数调用" << endl;
    
15.     }
    
16.     phone(const phone&p)
    
17.     {
    
18.         cout << "phone的拷贝函数调用" << endl;
    
19.         p_name = p.p_name; // 这里需要复制对象的数据
    
20.     }
    
21.     string p_name;
    
22. 
    
23. };
    
24. class person
    
25. {
    
26. public:
    
27.     person(string name, string phonename) :m_phone(phonename)
    
28.     {
    
29.         cout << "person的构造函数调用" << endl;
    
30.         m_name = name;
    
31.     }
    
32.     ~person()
    
33.     {
    
34.         cout << "person的析构函数调用" << endl;
    
35.     }
    
36.     person(const person& p) : m_phone(p.m_phone) // 需要使用初始化列表来初始化m_phone
    
37.     {
    
38.         cout << "person的拷贝函数调用" << endl;
    
39.         m_name = p.m_name; // 这里需要复制对象的数据
    
40.     }
    
41.     string m_name;
    
42.     phone m_phone;
    
43. };
    
44. 
    
45. void test01()
    
46. {
    
47.     person p("张三","苹果max");
    
48.     cout << p.m_name << "拿着" << p.m_phone.p_name << endl;
    
49. }
    
50. 
    
51. int main()
    
52. {
    
53.     test01();
    
54.     return 0;
    
55. }
    

复制代码

这个修订的版本应该能够正确地复制 phone 和 person 对象。