在 C++ 中获取引用的构造函数

Question

我正在尝试创建引用对象的构造函数。使用引用创建对象后，我需要打印两个对象的字段值。然后我必须删除第一个对象，并再次显示两个对象的字段值。我的班级 Person 看起来像这样：

class Person {   
    char* name;
    int age;
public:

    Person(){
        int size=0;
        cout << "Give length of char*" << endl;
        cin >> size;
        name = new char[size];      
        age = 0;
    }

    ~Person(){
        cout << "Destroying resources" << endl;
        delete[] name;
        delete age;
    }  

void init(char* n, int a) {
    name = n;
    age = a;
} 
}; 

这是我的实现（使用函数 show() ）。我的教授说，如果这个任务写得正确，它会返回一个错误。

#include <iostream>
using namespace std;

class Person {   
    char* name;
    int age;
public:

    Person(){
        int size=0;
        cout << "Give length of char*" << endl;
        cin >> size;
        name = new char[size];      
        age = 0;
    }

    Person(const Person& p){
        name = p.name;
        age = p.age;
    }

    ~Person(){
        cout << "Destroying resources" << endl;
        delete[] name;
        delete age;
    }  

void init(char* n, int a) {
    name = n;
    age = a;
}


void show(char* n, int a){
    cout << "Name: " << name << "," << "age: " << age << "," << endl; 
}

}; 


int main(void) {
    Person *p = new Person;  
    p->init("Mary", 25);

    p->show();

    Person &p = pRef;
    pRef->name = "Tom";
    pRef->age = 18;

    Person *p2 = new Person(pRef);

    p->show();
    p2->show();

    system("PAUSE");
    return 0;
}

Answer 1

您的复制构造函数的问题在于它只分配了 p.name:

name = p.name // Now this and p hold a pointer to the same memory

由于this 和p 现在都持有指向同一个内存位置的指针，因此第一个销毁的将释放内存，而第二个将持有指向不存在对象的指针。随后使用该指针或删除它将导致未定义的行为。解决方案是为 name 分配一个新数组，并将 p.name 的内容复制到该数组中，这样内存就不会共享了。

同样，您的 init 函数会覆盖 name，忽略内存已分配的事实（这是内存泄漏），也忽略字符串稍后将被破坏的事实（即使调用者可能希望拥有并释放它）字符串本身）。另外，我应该指出，您的 show 函数采用参数“n”，但使用“name”。你的 show 函数可能不应该带任何参数（实际上，你调用它的方式暗示它没有），因为你的类中已经存在所有需要的字段（或者你可能打算让它成为一个独立的函数这需要类的字段？）。您应该再次查看您的代码是否有其他错误。

Answer 2

首先，尝试编译你的代码（在发布到 SO 之前编译代码通常是个好主意。）它包含几个错误，编译器会显示它们。第二部分将更改以下内容：

p->init("Mary", 25);

到

{
  std::string mary("Mary");
  p->init(mary.c_str(), 25);
}

它应该在运行时给你一个错误，它会让你有机会在你的实现中发现一个问题。