3

Никак не могу понять в чём дело.

   std::string* array_str0 = new std::string[2];
   std::string* array_str1 = new std::string[2];

   array_str0[0] = "00000000";
   array_str0[1] = "111111111111111111111111";

   memmove(array_str1, array_str0, sizeof(std::string) * 2);

   delete[] array_str0;

   for (size_t i = 0; i < 2; i++)
       std::cout << array_str1[i] << "\n";

Вывод:

00000000

0000000════════════════

Если убрать delete[] array_str0, то всё работает правильно:

00000000
111111111111111111111111

С другими типами (int, char) также всё работает правильно. Кто-нибудь может объяснить почему так происходит?

1

1 ответ 1

7

Смотрите - вы просто

memmove(array_str1, array_str0, sizeof(std::string) * 2);

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

Удаляя указатель,

delete[] array_str0;

вы вызываете деструкторы строк, которые эту память освобождают. Все. Строк больше нет!

Но в array_str1 лежат указатели на уже удаленные строки, и вы обращаетесь к памяти, которая была освобождена - UB!...

Вот так - через memmove - сложные типы не копируются, только POD (plain old data, грубо - С-шные типы без указателей) - что вы сами заметили: С другими типами (int, char) также всё работает правильно.

Вам надо было копировать примерно так:

array_str1[0] = array_str0[0];
array_str1[1] = array_str0[1];

А у вас получилось взрывоопасная смесь французского (С++) с нижегородским (С)...

14
  • ага, а sizeof(std::string) * 2 - это где то 24*2 до 32*2 байт. Там чуточку больше копируется:)
    – KoVadim
    28 июл 2020 в 14:34
  • @KoVadim Ну, я упомянул "те же указатели" - что надо читать как "в том числе", но не "только"..
    – Harry
    28 июл 2020 в 14:38
  • Я бы предложил сделать std::swap вместо копирования
    – user206435
    28 июл 2020 в 14:39
  • интересно, почему sso не отработало
    – KoVadim
    28 июл 2020 в 14:39
  • 1
    Можно очень быстро и правильно - но только что-то одно :) У вас программа не проходит какие-то временные тесты? Или работает слишком медленно? И вы профилировали ее и точно знаете, что проблема именно в копировании?...
    – Harry
    28 июл 2020 в 15:33

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.