Создайте глубокую копию массива C++

Question

Я пытаюсь решить некоторые проблемы в своей программе, и, похоже, проблема связана с моим конструктором копирования или с моим деструктором. Я получаю исключение памяти.

любая помощь мне оценили Спасибо

ArrayStorage::ArrayStorage(const ArrayStorage &a):readArray(a.readArray),arraysize(a.arraysize) 
{ 
    readArray = new string[arraysize]; //create the array 

    memcpy (readArray,a.readArray,sizeof(string)*arraysize);//Copy the values of bytes from the location pointed at by the souce and destination. 
} 

ArrayStorage::~ArrayStorage(void) 
{ 
    delete[](readArray);//deconstuctor to delete the array. 
} 

это было бы лучше, чтобы скопировать массив, кроме тетсру:

for (int i = 0 ; i < arraysize ; i ++) 
    { 
     readArray[i] = a.readArray[i]; 
    } 

Answer 1

Вы не можете просто memcpy случайных объектов, вам действительно нужно скопировать их с помощью операторов копирования.

string скорее всего содержит указатель на хранилище, выделенное кучей. Если вы скопируете его поразрядным, вызов деструктора в исходной строке аннулирует данные «скопированной» строки.

Используйте это как std::copy, чтобы сделать это правильно.

#include <algorithm> 
... 
std::copy(a.readArray, a.readArray+arraysize, readArray); 

Answer 2

Я не советую вам скопировать струны так, как вы делать. Поскольку строка содержит ссылку на кучную память, вы на самом деле копируете указатели, и поэтому строки в обоих массивах разделяют память. Это не очень C++ - ish и довольно опасно. Я бы посоветовал вам использовать оператор присваивания или конструкторы копирования для строк (да, сделайте цикл).

Answer 3

Строки также имеют динамическую память, поэтому вам придется пройти через каждую строку и сделать ее копию.

Исправление будет копировать каждую строку внутри вашего массива вместо memcopy. Исключение составляют две разные строки, имеющие один и тот же указатель на кусок памяти, и оба пытаются освободить его.

Answer 4

Этот код не имеет никакого смысла. Сначала вы инициализируете своих членов значениями, переданными объектом для копирования, через список инициализаторов. Затем вы выделяете память для одних и тех же членов и копируете все снова.

Скорее всего, вы копируете ненужные данные в неинициализированный указатель. Избавиться от списка инициализаторов :readArray(a.readArray),arraysize(a.arraysize).