Управление памятью C++

Question

Я создал эту простую функцию.

char* tmp = new char[len]; 
strncpy(tmp,str+start,len); 
int ret = atoi(tmp);  
delete []tmp; 
return ret; 

У меня проблема с управлением памятью. Когда я читаю переменную ret, значение равно null. Если я удалю инструкцию «delete [] tmp;» значение верное, но память быстро увеличивается (потому что я не освобождаю память).

Любые идеи? Thanks

Answer 1

Что вы можете сделать, так это выделить буфер из-за пределов функции и передать аргумент tmp в виде массива. Однако лучше всего выделить объект. Они разрушаются до тех пор, пока вы не используете новое ключевое слово. std::string был бы идеальным в этом сценарии.

Answer 2

Убедитесь, что tmp прекращается с '\0'

Answer 3

От человека strncpy: Функция strncpy() аналогична чем зЬгсру, за исключением того, что в большинстве п байтов ЦСИ копируются. Предупреждение. Если среди первых n байтов src нет нулевого байта, строка, помещенная в dest, не будет заканчиваться нулем.

Проверьте переменную длину str и проверить это нуль завершающего condicion на strncpy

Answer 4

Есть несколько проблем, с atoi, один из них является то, что он не имеет какой-либо проверки, что строка вы передаете действительно число. Вместо этого вместо этого вы можете использовать strtol.

Также обратите внимание, что strncpy может не прерывать строку в некоторых случаях. И чтобы вы могли выделить один дополнительный символ (len + 1) для терминатора.

Answer 5

strncpy заполняет целевой массив символом '\ 0, как только будет достигнут конец источника.

atoi ожидает нулевую завершенную c-строку, означает массив символов, который заканчивается символом '\ 0'.

Поэтому вам необходимо создать массив размером len + 1, функция strncpy будет автоматически завершать нулевой целевой массив.

Answer 6

Сложно сказать, что не так с str и len не показаны.

Эта функция может быть намного проще:

int ret = atoi(str + start);  
return ret; 

КСТАТИ ret является int и NULL обычно называют указателями.