При копировании данных из одного массива в другой последний элемент копируется с мусорными значениями. Как это устранить?

Question

[HentaiEtoIskusstvo]

String::String() {
	m_length = 80;
	m_capacity = 160;
	m_str = new char[m_length] {};
}

void String::setStr(char* str) {
	if (m_length < strlen(str)) {
		return;
	}
	else {
		if (str != nullptr) {
			for (int i = 0; i < strlen(str); ++i) {
				m_str[i] = str[i];
			}
		}
	}
}

void String::ShrinkToFit() {
	String str;
	str.setStr(m_str);

	m_str = nullptr;
	delete[] m_str;
	
	m_str = new char[strlen(str.m_str)];
	setStr(str.m_str);
}

Comments

[Mercury13]

Код

for (int i = 0; i < strlen(str); ++i) {
        m_str[i] = str[i];

неоптимален; O(n²) при возможном O(n).

Answer 1

[devalone]

Добавь в конце строки символ окончания строки(имеет код 0). А ещё копировать лучше с помощью strncpy

Comments

[HentaiEtoIskusstvo]

Не догадался. Спасибо!

[jcmvbkbc]

лучше с помощью strncpy

devalone, помощью strncpy, как известно, лучше всего стрелять себе в ногу.

[devalone]

jcmvbkbc, почему? С помощью strcpy да. А вообще, чтоб не стрелять в ногу, нужно использовать std::string, но ТС судя во всему учится и в обучении писать велосипеды полезно.

[jcmvbkbc]

m_str = new char[strlen(str.m_str)];
HentaiEtoIskusstvo, вот это тоже надо будет поправить, если соберёшься добавлять 0 в конец.
А вообще, если уж ты всё равно хочешь скопировать всю строку, есть strdup для этого.

[jcmvbkbc]

почему?

devalone, потому что что происходит, когда строка длиннее чем n из strncpy? А что происходит, когда короче?

[devalone]

jcmvbkbc, www.cplusplus.com/reference/cstring/strncpy

[jcmvbkbc]

devalone, ну, и? Сирена в голове от этого не включается?:

If the end of the source C string (which is signaled by a null-character) is found before num characters have been copied, destination is padded with zeros until a total of num characters have been written to it.

No null-character is implicitly appended at the end of destination if source is longer than num. Thus, in this case, destination shall not be considered a null terminated C string (reading it as such would overflow).

[devalone]

jcmvbkbc, ну, если добавлять в конце символ окончания строки, то всё ок, просто не поместится строка. Согласен, для копирования конечно лучше strdup, но я бы делал вообще по другому. Я бы проверял, если строка помещается, то копировал используя, например, std::copy и добавлял ноль, а если нет, то выделял память, копировал и добавлял ноль, но это всё уже делает прекрасно std::string.

Answer 2

[Kamil Khamitov]

А ты уверен, что память не портится у тебя в setStr() копирование идёт только если длинна строки > длинны буфера, и копируешь ты всю строку целиком, попутно создавая повреждение памяти. Ну и про \0 символ тебе уже написали выше. Вообще есть strdup() но он вне стандарта.