在 C++ 中深度复制字符串的正确算法 [关闭]

Question

这是我在复制构造函数中深度复制字符串的代码 那么逻辑是否正确？？我认为可能存在内存泄漏，我正在尝试修复该问题。m.pStatus 已初始化

class Monkey
{
public:

// insert your code here
Monkey();
Monkey(const Monkey& m);
Monkey& operator=(const Monkey& m);
~Monkey();
// accessors
int getX();
int getY();
char *getStatus();
void deepCopy(const Monkey& m);
// global variables (incremented each creation or destruction)
static int numStringsCreated;
static int numStringsDestroyed;

private:
        // Do not change this data
    int x;
    int y;
    char *pStatus;

};

void Monkey::deepCopy(const Monkey& m)
{
    
    if (m.pStatus)
    {
        // allocate memory for our copy
        pStatus= new char[sizeof(m.pStatus)];
 
        for (int i{ 0 }; i < sizeof(m.pStatus); ++i)
            pStatus[i] = m.pStatus[i];
    }
    else
        pStatus = nullptr;
}


    Monkey::Monkey(const Monkey& m)
    {
        deepCopy(m);
        numStringsCreated++;
    }

Answer 1

避免使用sizeof(pStatus)。除非变量是固定大小的数组，否则它将计算指针的大小，而不是字符串的长度。

strlen 和 strcpy 用于经典 C 字符串长度和复制操作。

这更接近你想要的。它确保它清除任何先前的分配，并通过在删除旧的之前先分配副本来隐式处理进行自分配的情况（例如m.deepCopy(m)）。你可以做一个额外的优化说if (m.pStatus == pStatus)并完全跳过复制操作。

我的工作假设 pStatus 被初始化为 null 或在构造函数中有效的东西。

void Monkey::deepCopy(const Monkey& m)
{
    char* psz = nullptr;
    if (m.pStatus)
    {
        size_t len = strlen(m.pStatus);
        psz = new char[len+1]; // +1 for null char
        strcpy(pStatus, m.pStatus);
    }

    delete [] pStatus;
    pStatus = psz;      
}

但是，如果 pStatus 是 std::string，则您不需要执行任何此类分配操作，并且您可能不需要重载赋值运算符或复制构造函数。