c++ - 分配动态分配的指针数组

Question

在筛选了这里的许多帖子后，遇到了很多麻烦。一切都可以编译，但是在这个函数中我在这里遇到了崩溃，它应该动态地将一个数组的地址分配到这个指针数组中。我看到发布了一两个内存地址，所以我不确定为什么它会在这中间崩溃。

string *copyArray(string ptrArray[],int sizeArray)
   {
    string **dynamString = new string*[sizeArray];
    int i;

    for (i=0;i<=sizeArray;++i)
        {
         (*dynamString[i]) = ptrArray[i];
         cout << dynamString[i];
        }
    return *dynamString;

}

从主要我有：

string *arrPtr;

和函数调用

arrPtr = copyArray(arrayOfStrings, arraySize);

Answer 1

for (i=0;i<=sizeArray;++i)

访问数组后面的元素会产生未定义的行为。元素的索引从0到sizeArray - 1。另一个问题是您分配了指针数组：

string **dynamString = new string*[sizeArray];

然后您将取消对这些指针的引用，尽管它们尚未指向任何对象：

(*dynamString[i]) = ptrArray[i];

这也会导致未定义的行为。如果你想创建一个深拷贝，你也应该为每个对象分配内存：

for (i = 0; i < sizeArray; ++i)
{
    dynamString[i] = new std::string(ptrArray[i]);
    cout << *dynamString[i];
}

但是，您应该尽可能避免使用 C 样式的数组，而是更喜欢 STL 容器。在这种情况下，它可能很简洁std::vector<std::string>，并且它的构造函数与您的函数执行相同的操作（只是以更安全和更合理的方式，没有可能的内存泄漏）：

std::vector<std::string> myStrings(arrayOfStrings, arrayOfStrings + arraySize);

Answer 2

好的，我在这里修好了。我的指针语法不正确。这是正确的语法。

dynamString[i] = &ptrArray[i];