Как исправить ошибку «Buffer is too small» C++?

Question

[Данил Васькевич]

Работаю с классами. Нужно сделать конкатенацию двух char массивов с пробелом между ними. Написал вот такой код :

String String::concat(const String& other)
{
	String result;
	
	char space[3]{ "  \0" };
	strcat_s(result.string_, strlen(string_) + strlen(space) , string_);
	strcat_s(result.string_, strlen(string_) + strlen(space), space);
	strcat_s(result.string_, strlen(result.string_) + strlen(other.string_) + strlen(space), other.string_);

	return result;
}

Во время первой конкатенации все идет хорошо, но вторая вызывает исключение "Buffer is too small". Когда я стараюсь исправить ошибку чистой(delete[]) массива result и переопределением его наново. В таком случае также вызывается ошибка но уже другая — "String is not null terminated".
Помогите пожалуйста.

Comments

[Евгений Шатунов]

Данил Васькевич , покажи полностью все объявление типа String.

[Данил Васькевич]

class String
{
private:
char* string_ = nullptr;

public:
// default ctor
String() { // default
size_t size = 80;
string_ = new char[80]{""};
}

// ctor with any string size
String(const size_t size) {
string_ = new char[size];
}

// ctor with string from user
String(const char* string) {
delete[] string_;
size_t size = strlen(string);
strcpy_s(string_, size, string);
}

// copy ctor
String(const String &str) {
size_t size = strlen(str.string_) + 1;
strcpy_s(string_, size, str.string_);
}

// keyboard input method
void inputString();

// print string
void print() const;

String concat(const String& other);
};

[Александр Ананьев]

Данил Васькевич, ну видимо вам не хватает 80 символов. Выделяйте столько памяти, сколько нужно для новой строки.

[Евгений Шатунов]

Данил Васькевич, не забывай ставить упоминания. Только так человек (я), которому ты отвечаешь, сможет узнать что ты ему ответил.
Странно что в вопросе ты обрамил код соответствующим тегом, а в комментарии - нет. Будет лучше если ты отредактируешь свой комментарий и приведешь код в читабельный вид.

По коду, пока только то, что я могу увидеть.

size_t size = 80;
string_ = new char[80]{""};

Какой смысл в size если она не используется?
Зачем в конструкторе по умолчанию выделять какую-то память, когда ты не знаешь как пользователь дальше будет распоряжаться объектом?

String(const char* string) {
delete[] string_;
size_t size = strlen(string);
strcpy_s(string_, size, string);
}

Этот код не работает. delete[] string_; в этом месте будет означать delete[] nullptr;, т.е. нонсенс. Оператор delete[] ничего не освободит. В это время, strcpy_s(string_, size, string); будет означать strcpy_s(nullptr, size, string);, что приведет к падению.

String(const String &str) {
size_t size = strlen(str.string_) + 1;
strcpy_s(string_, size, str.string_);
}

Этот код тоже падает, тоже по тем же причинам.

При этом, у тебя в коде нет функций resize() и reserve(), которые и должны управлять подконтрольным строке буфером памяти.
Твой concat() должен быть примерно таким:

String String::concat(const String& other) const
{
  String result;
  
  result.reserve( length() + other.length() + 1 );
  result = *this;
  result += ' ';
  result += other;
  
  return result;
}

Answer 1

[Drottarutarnum]

Преобразуй массивы в строки (в гугле c++ char to string)
А после result = string1 + " " + string2

Comments

[Александр Ананьев]

Человек написал свой класс String, поэтому стандартный его вряд-ли устроит.

Answer 2

[none7]

Ошибка в том, что все сишные строки они null-termanated и strlen этот '\0' не учитывает при подсчёте длины строки. В первой строке оно работает по ошибке, так как не должно быть там strlen(space). Вообще это совершенно дурной способ использования функции strcat_s, ведь второй параметр это не длина копируемой строки, а максимально возможная длина буфера, который 80 байт. В таком виде очень даже возможно переполнение буфера.

Comments

[Евгений Шатунов]

none7 , приведи, пожалуйста, ссылки на документацию, где сказано что стандартная функция strlen() при своей работе игнорирует терминальный символ.

[none7]

Евгений Шатунов, я имел ввиду, что в строке strlen("a") == 1 при этом sizeof("a") == 2. Это общеизвестный факт, что при подсчёте длины строки, нулевой символ не учитывается(игнорируется). Поэтому при высчитывает буфера под строку необходимо добавлять к сумме чисел полученных из strlen(), ещё один байт.

[Евгений Шатунов]

none7, тогда тебе нужно именно так в ответе и написать. Пока твой ответ вызывает только обозначенный мной вопрос. Обрати внимание на точность моего вопроса.
Сейчас в своем ответе ты написал что strlen() вообще игнорирует терминальный символ в своей работе. Но функция его не учитывает только в своем результате, что и отражено в документации функции.

Никаких общеизвестных фактов на этот счет не существует. Единственным твоим источником информации должна быть только документация.