Динамическая память new и delete

@qppq · Регистрация: 31.08.2016

имеем след. ситуацию:

C++

char* str = new char[20];
char* ing = new char[10];
char* string = new char[20];
string = strcat(str, ing);
 
//
//
 
delete[] str;
delete[] ing;
delete[] string;

в результате str и string указывают на один и тот же объект, есть ли смысл в delete[] string;

krolligator · 09.01.2019, 15:57

Сообщение от qppq

есть ли смысл в delete[] string;

Не только смысла нет, но и неправильно, так как поведение не определено, два раза пытаешься освободить один и тот же блок. Программа может просто упасть. А так как оригинальный адрес из string потерялся после strcat, то еще и утечка памяти здесь есть, если нигде больше указатель на сохранялся.

@qppq · 10.01.2019, 09:26 **[ТС]**

стековерфлоу сообщает:

Если указатель указывает на объект, выделенный при помощи new, и для него ещё ни разу не был вызван delete, то вызывать delete можно. Также можно вызывать delete по нулевому указателю (сколько угодно раз).

В любом другом случае — нельзя.

получается если сказать

C++

1	str = NULL;

то необязательный вызов

C++

1	delete[] string;

не приведет к падению, так?

@volvo · 10.01.2019, 11:41

Тебе же написали, что у тебя будет утечка. Та память, которая была выделена под string в третьей строке, благополучно утечет, потому что после выполнения 4-ой строки у тебя string указывает вообще в другое место. На кой вообще понадобилось присваивать string-у результат выполнения операции конкатенации?

Я уж не говорю о том, что все эти пляски с Сишными строковыми функциями в Билдере выглядят убого, когда есть гораздо более мощные средства.

@qppq · 10.01.2019, 13:49 **[ТС]**

Сообщение от volvo

На кой вообще понадобилось присваивать string-у результат выполнения операции конкатенации?

C++

1	char strcat(char str1, const char *str2);

функция возвращает результат, и значит его можно присвоить, не правда ли, случаи описывать не буду

Сообщение от volvo

Та память, которая была выделена под string в третьей строке, благополучно утечет, потому что после выполнения 4-ой строки у тебя string указывает вообще в другое место

в другое? проверим

@volvo · 10.01.2019, 14:00

Сообщение от qppq

функция возвращает результат, и значит его можно присвоить, не правда ли

Првда. Но перед тем, как это делать, неплохо было бы почитать доку на функцию, не правда ли? А там сказано, что

C++

1	char * strcat ( char * destination, const char * source );

Concatenate strings
Appends a copy of the source string to the destination string. The terminating null character in destination is overwritten by the first character of source, and a null-character is included at the end of the new string formed by the concatenation of both in destination.

destination and source shall not overlap.

Parameters
destination
Pointer to the destination array, which should contain a C string, and be large enough to contain the concatenated resulting string.
source
C string to be appended. This should not overlap destination.

Return Value
destination is returned.

А вот теперь попробуйте меня убедить в необходимости перезаписать указатель, под который выделена память, если все равно вернется указатель на заполненную строку. Не проще ли, если уж ТАК понадобилось иметь еще копию указателя, было написать:

C++

1
2
3

char* str = new char[20];
char* ing = new char[10];
char* string = strcat(str, ing);

, и не парить мозги ни компилятору, ни всем остальным?

@jugu · 12.01.2019, 11:06

Функция strcat дописывает source справа к destination. Она даже не проверяет достаточность памяти в destination для хранения результирующей строки. Если строки source и destination были сформированы с помощью new[], то их обе надо удалить, используя delete[].
PS: Вместо strcat лучше использовать strncat ИМХО.

krolligator 112 / 91 / 31 Регистрация: 24.10.2018 Сообщений: 336
	09.01.2019, 15:57	2
	Сообщение от qppq есть ли смысл в delete[] string; Не только смысла нет, но и неправильно, так как поведение не определено, два раза пытаешься освободить один и тот же блок. Программа может просто упасть. А так как оригинальный адрес из string потерялся после strcat, то еще и утечка памяти здесь есть, если нигде больше указатель на сохранялся. 0

@volvo Супер-модератор 30662 / 20011 / 7827 Регистрация: 22.10.2011 Сообщений: 34,840 Записей в блоге: 6
	10.01.2019, 11:41	4
	Тебе же написали, что у тебя будет утечка. Та память, которая была выделена под string в третьей строке, благополучно утечет, потому что после выполнения 4-ой строки у тебя string указывает вообще в другое место. На кой вообще понадобилось присваивать string-у результат выполнения операции конкатенации? Я уж не говорю о том, что все эти пляски с Сишными строковыми функциями в Билдере выглядят убого, когда есть гораздо более мощные средства. 0

@jugu 550 / 369 / 137 Регистрация: 11.01.2019 Сообщений: 1,532
	12.01.2019, 11:06	7
	Функция strcat дописывает source справа к destination. Она даже не проверяет достаточность памяти в destination для хранения результирующей строки. Если строки source и destination были сформированы с помощью new[], то их обе надо удалить, используя delete[]. PS: Вместо strcat лучше использовать strncat ИМХО. 0

Опции темы