1

我正试图围绕 C++ 中的复制赋值运算符，我想知道如果对象为空，是否有办法创建一个新实例。

class Person {
public:
    Person(string name) { pName_ = new string(name); }
    ~Person() { delete pName_; }
    Person(const Person& rhs) {
        if (pName_ != NULL) delete pName_;
        pName_ = new string(*(rhs.pName()));
    }
    Person& operator=(const Person& rhs) {
        cout << "begin copy..." << endl;
        if (this == NULL) {
            Person* p = new Person(rhs);
            cout << "end copy null..." << endl;
            return *p; // Not working?
        }
        delete pName_;
        pName_ = new string(*(rhs.pName()));
        cout << "end copy..." << endl;
        return *this;
    };
    string* pName() const { return pName_; }
    void printName() { cout << *pName_ << endl; }
private:
    string* pName_;
};
int main() {
    Person *a = new Person("Alex");
    Person *b = new Person("Becky");
    Person *c = NULL;
    *b = *a; // copy works
    *c = *a; // copy doesn't
    if (a != NULL) a->printName(); // Alex
    if (a != NULL) delete a;
    if (b != NULL) b->printName(); // Alex
    if (b != NULL) delete b;
    if (c != NULL) c->printName(); // NULL
    if (c != NULL) delete c;
    return 0;
}

这是输出：

begin copy...
end copy...
begin copy...
end copy null...
Alex
Alex

有什么我可以在我的代码中更改以使其工作的东西，还是我什至不应该尝试尝试的东西？

4

1 回答 1

0

您显示的代码有各种错误，需要重写。

您正在通过取消引用的 NULL 指针调用未正确实现的复制赋值运算符，这是未定义的行为。复制赋值运算符被定义为仅使用有效对象调用，并且必须将值从一个对象复制到另一个对象，并返回对复制到的对象的引用。你没有那样做。

this要回答您的问题，否，复制赋值运算符在为 NULL时无法创建新对象。对于初学者，如果this为 NULL，那么您的代码中有一个需要修复的错误。但是即使this是 NULL，操作员也没有办法更新调用者的指针（假设首先使用了一个指针）以指向一个新的对象实例。this只是一个编译器生成的参数，包含被调用对象的内存地址，它不是调用者用于调用运算符的原始指针。因此，如果操作员确实在堆上分配了一个新对象，那么当操作员退出时，该对象将被泄漏，因为调用者将没有指向该对象的指针来稍后释放它。

您的代码的正确版本应该看起来更像这样：

class Person {
public:
    Person(const string &name) : pName_(name) {}

    Person& operator=(const Person& rhs) {
        // if you remove these output statements then
        // this entire operator can be removed...
        cout << "begin copy..." << endl;
        if (this != &rhs) {
            pName_ = rhs.pName();
            cout << "non self copied..." << endl;
        }
        cout << "end copy..." << endl;
        return *this;
    }

    string pName() const {
        return pName_;
    }

    void printName() const {
        cout << pName_ << endl;
    }

private:
    string pName_;
};

int main() {
    Person *a = new Person("Alex");
    Person *b = new Person("Becky");
    Person *c = NULL;

    *b = *a;
    //*c = *a; UNDEFINED BEHAVIOR! 

    if (a) {
        a->printName();
        delete a;
    }

    if (b) {
        b->printName();
        delete b;
    }

    if (c) {
        c->printName();
        delete c;
    }

    return 0;
}
于 2018-03-19T07:25:08.803 回答