Играя с массив символов

Question

Это безопасно сделать что-то вроде этого:

char* charArray = new char[10]; 
strcat(charArray, "qwertyuiop"); 
charArray[3] = '\0'; 
delete [] charArray; 

Будет ли все быть удалены? Или что после \0 не будет? Я не знаю, оставил ли я мусор.

EDIT: должно быть strcpy

Answer 1

Если вы хотите написать strcpy вместо strcat, то, что является безопасным и правильным. Но, похоже, у вас есть неправильное представление о delete [] charArray. Он не удаляет символы, он удаляет память, на которую указывает charArray. В памяти даже после delete [] charArrayможет содержать эти символы, но это не гарантировано.

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

Answer 2

Нет, это прекрасно, весь массив удален. delete не смотрит на то, на что указывает указатель, на который вы указываете. Если вы согласитесь с new с delete и new[] с delete[], то необходимый объем памяти будет освобожден.

(Но не рекомендуется использовать std::string вместо символьных массивов, которые будут избежать много ошибок, как тот, который вы там пишете мимо конца вашего массива.)

Answer 3

Помимо того, что new[] для POD -types не нуль инициализирует массив, и strcat записывает завершающий '\0' за конец выделенной области, все выглядит нормально: в частности, delete удалит весь блок.

Причина для записи за конец выделенного блока заключается в том, что для 10-символьной строки "qwertyuiop" требуется 11 байтов для хранения.

Answer 4

Оператор delete[] ничего не знает о том, что хранится в буфере (в том числе, является ли оно строкой или нет), поэтому он удалит все 10 символов. Однако ваш вызов strcat переполняет конец массива (поскольку строки C имеют нулевой байт в качестве терминатора), что может сломать удаление на вашей платформе и вообще небезопасно.

Answer 5

delete[] освобождает память, выделенную после уничтожения объектов внутри (что ничего не делает для char). Он не заботится о содержимом, то есть освобождает столько объектов, сколько было выделено.

Обратите внимание, что использование strcat() зависит от нулевого символа, чтобы найти конец строки и что память, возвращаемая с new char[n], неинициализирована. Вы хотите, чтобы начать вместе с

*charArray = 0; 

... и вы могли бы хотеть рассмотреть strncat() или, еще лучше, не использовать это вообще, а использовать std::string.