字符串复制函数导致“未分配指针”

Question

我有以下功能：

void stringcopy(char * to, char const * const from)
{
    int size = 1;
    while (from[size] != '\0') { ++size; }

    if (to != 0) { delete [] to; }
    to = new char[size];

    for (int i = 0; i < size; i++) { to[i] = from[i]; }
}

顾名思义，它使用动态分配复制一个字符串。

---

我认为我使用它的方式并不重要（因为我正在努力使这个函数变得健壮），但这里有一些例子：

CD::CD(char * s1, char * s2, int n, double x)
{
    stringcopy(performers, s1);
    stringcopy(label, s2);
    selections = n;
    playtime = x;
}

和

CD::CD(const CD & d)
{
    stringcopy(performers, d.performers);
    stringcopy(label, d.label);
    selections = d.selections;
    playtime = d.playtime;
}

等等

---

不幸的是，我在使用该函数时收到以下错误消息：pointer being freed was not allocated。

我认为它发生在if (to != 0) { delete [] to; }。

---

为什么这条线不能防止释放未分配的内存？

Answer 1

你在这里做什么

if (to != 0) { delete [] to; }
to = new char[size];

正在释放本地to 变量指向的内存，重新分配它并将字符串存储在那里。

这个新的本地内存地址（我们称之为to1）从不暴露给外界，因为它不是从函数返回的。函数 get 是to 地址的副本。为了解决这个问题，您需要将 to 设为双指针。或对指针的引用。