Каждый раз, когда я выполняю это ... я получаю сообщение об ошибке:
char * var= new char[256];
var= "hola mundo\0";
delete var;
я получаю ошибку:
malloc: *** ошибка для объекта 0x10007fd20: освобождаемый указатель не был выделен
*** установить точку останова в malloc_error_break на
Я не хочу использовать std :: string или std :: vector ... потому что я использую в нескольких функциях char * в качестве параметра, и я хотел бы иметь возможность его уничтожить.
Когда у вас есть var= "hola mundo\0";, который присваивается строке в памяти, которую резервирует компилятор. Вызов delete по этому поводу - неопределенное поведение. Кроме того, есть утечка оригинального var, поскольку это никогда не deleted. Вы должны сделать что-то вроде использования strcpy, чтобы скопировать вашу строку в var, или использовать std::string, чтобы не беспокоиться о ее освобождении. Пример: std::string var = "hola mundo";
Значение указателя, полученное вами от new, теряется, когда вы назначаете что-то еще для var. Исходное содержимое var исчезло. Затем вы пытаетесь удалить что-то еще.
Во-первых, присвоение строкового литерала переменной char* не копирует строковые данные в выделенную вами память. Вместо этого он просто переназначает указатель, чтобы var больше не указывал на выделенную вами память. Это означает, что выделенная вами память теряется и никогда не освобождается должным образом, и это означает, что вы используете delete с указателем, который вы никогда не выделяли, что является недопустимым.
Во-вторых, вы выделяете массив с помощью new char[], поэтому вам нужно использовать delete [] вместо просто delete.
В-третьих, строковые литералы автоматически включают нулевой терминатор, поэтому вам не нужно добавлять лишний:
var = "hola mundo";
Наконец, если вы можете использовать C ++ 11 или C ++ 14, вам не следует использовать new и delete напрямую. В более ранних версиях C ++ во многих случаях можно по-прежнему избегать new и delete. В этом случае, поскольку уже существует тип std::string, вы должны просто использовать его независимо от того, какую версию C ++ вы используете. Ваши три строки кода следует заменить на:
std::string var = "hola mundo";
Это проще и безопаснее.
std::string как для передачи char* данных, так и для управления char* буферами, заполненными библиотекой. Если вы имеете в виду, что вы написали функции, которые принимают char*, и этим функциям необходимо освободить переданный буфер, тогда это может создать проблему для std::string, но это также плохая идея по нескольким причинам.
- person bames53; 13.06.2014
unique_ptr<char[]> buf(foo());, и вы можете даже захотеть немедленно скопировать данные в более разумный тип, например std::string. Например. unique_ptr<char[]> buf(foo()); std::string copy = &buf[0];.
- person bames53; 14.06.2014
delete, delete[], custom_library_deallocator(), free() и т. Д. Вы должны прочитать документацию библиотеки.
- person bames53; 14.06.2014
strcpy(var, "hola mundo");вместо assign иdelete[] var;вместоdelete- person keltar schedule 12.06.2014strcpyилиstrncpy,strcatилиstrncat'. - person Joe schedule 12.06.2014std::string::c_str()? - person The Paramagnetic Croissant schedule 12.06.2014std::string::c_str()возвращаемое значение -const char*, его нельзя изменять (даже с const-cast, это нарушит внутреннюю структуру строк, такую как длина и емкость), поэтому это не всегда подстановочная замена. - person keltar schedule 12.06.2014&str[0]. Это можно изменить. - person The Paramagnetic Croissant schedule 12.06.2014length()после изменения строки? - person keltar schedule 13.06.2014resize()это сначала , только потом копируете. - person The Paramagnetic Croissant schedule 13.06.2014