复制构造函数问题

Question

我试图理解这个复制构造函数问题。我的问题与程序退出后的析构函数有关。看来变量 char* title 没有被破坏，我认为这可能是错误的，谢谢

另一个问题是为什么当对象 x 等于 x2 时不调用赋值运算符。我正在使用 g++ 和代码块。

#include <iostream>

using namespace std;  

class myClass
{
    private:

        int x;
        int *p;
        char* title;

    public:

        myClass(int tx, int* temp, char* newtitle)
        {
            x = tx;
            cout << "int constructor called with value x = " << x << endl;

            p = new int;
            p = temp;

            title = new char [strlen(newtitle)+1];
            strcpy(title, newtitle);
        }
        myClass(const myClass& mc)
        {
            cout << "copy constructor called with value = " << mc.x << endl;
            x = mc.x;
            p = new int;
            *p = *mc.p;

            title = new char [strlen(mc.title)+1];
            strcpy(title, mc.title);
        }
        myClass & operator = (const myClass & that)
        {
            cout << "assignment called" << endl;

            if(this != &that)
            {
                x = that.x;
            }
            return *this;
        }
        ~myClass()
        {
            if (p)
            {
                cout<<"deleting p"<<endl;
                delete p;
            }
            else if(title)
            {
                cout<<"deleting title"<<endl;
                delete[] title;
            }
        }
};

int main()
{
    int pointee = 10;
    int* p = &pointee;
    myClass x(3,p,"hello");
    //myClass y = myClass(3,p);
    myClass x2 = x;
    return 0;
}

Answer 1

我在您的实际代码和一般方法中都看到了各种各样的问题。

首先，char* title 不会被删除，因为您没有删除它。这可能是一个逻辑错误：

    if (p)
    {
        cout<<"deleting p"<<endl;
        delete p;
    }
    else if(title)
    {
        cout<<"deleting title"<<endl;
        delete[] title;
    }

您可能不需要 else。你为什么把它放在那里？

接下来，您将泄漏一个 int，这里：

    p = new int;
    p = temp;

刚刚 new-ed 的 int 被传入的值 temp 覆盖。

稍后，您尝试在析构函数中删除该指针。但由于您要删除指向自动变量的指针，因此您将清理堆。这也是一个逻辑错误。解决方案：不要这样做：p = temp;

然而，最终，您的方法在多个层面上都是有问题的。

1. 为什么首先要动态分配int？只需有一个类的 int 成员即可。不要使用动态分配（例如new 和delete），除非您确实必须这样做。
2. 不要使用 char* 动态分配字符串。相反，请使用 #include 中的 std::string
3. 如果您确实需要动态分配，请不要使用原始指针。请改用智能指针。 C++ 附带了一个来自 #include 的内置 std::auto_ptr，但在其他库中还有许多其他选项，通常是更好的选择。这里比较流行的是 Boost 的智能指针。

Answer 2

如果 p 为非 NULL，则析构函数将删除 p；如果非NULL并且p为NULL，它会删除title。

但是您的构造函数和复制构造函数始终都会创建 new p 和 new title 。所以你需要一直检查并删除两者。

Answer 3

尝试

*p = *temp;

在析构函数中代替

 p = temp;

and

if(title)

代替

else if(title)