复制构造函数问题

Question

我正在尝试了解此副本构造函数问题。 程序退出后，我遇到的问题与析构函数有关。 似乎变量char *标题没有被破坏，我认为这可能是错误的，谢谢

另一个问题是为什么当对象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;

在int你刚刚new -ed得到由传入的值覆盖temp 。

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

但是最终，您的方法在多个层面上都值得怀疑。

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

Answer 2

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

但是您的构造函数和副本构造函数都始终创建new p和new title 。 因此，您需要一直检查和删除两者。

Answer 3

尝试

*p = *temp;

代替

 p = temp;

并在破坏者中

if(title)

代替

else if(title)