помогите создать функцию для выделения новой памяти для строки

[marenko.lilia]

Задача: Есть 2 строки. Вторую надо скопировать в первую. предварительно их передать в функцию и выделить новую память для первой строки.
Мой код работает, но, во-первых, не пойму почему не могу удалить память в функции перед тем как выделить новую? и во-вторых, в конец итоговой строки дописывает еще 9 символов "мусора". Подскажите в чем ошибка.

#include <iostream>
#include <string.h>
using namespace std;

char *newMem(char*, char*);

int main()
{
	setlocale(LC_ALL, "rus");

	char *str1 = new char [(strlen("строка 1") + 1)] = "строка 1";
	

	char *str2 = new char [(strlen(" + строка 2") + 1)] = " + строка 2";

cout << "1)" << str1 << endl;
cout << "2)" << str2 << endl << endl;

	char str[9] = "";//для копирования строки str1 перед удалением памяти 
	
	int x = 0;//управляющая переменная
	while (str1[x] != '\0')
	{
		str[x] =  str1[x];//копируем строку
		x++;
	}

	cout << "копия str: " << str << endl;

	
	str1 = newMem(str1, str2);//функция, которая перевыделит память

	x = 0;
		while (str[x] != '\0') 
		{
			str1[x] = str[x];
			x++;
		}
		int y = 0;
		while (str2[y] != '\0')
		{
			str1[x] = str2[y];
			y++;
			x++;
		}

		cout << "Итоговая строка: " << str1 << endl << endl;

return 0;
}

char *newMem(char *pstr1, char *pstr2)
{
	int lenPstr1 = strlen(pstr1) + 1;//длина строки pstr1 c '\0'
	int lenPstr2 = strlen(pstr2) + 1;


	int newSize = lenPstr1 + lenPstr2;//21 
	

	//delete  pstr1;//освобождаем память перед выделением новой памяти для 2х строк
		
	pstr1 = new char [newSize];

	return pstr1;
}

[waleri]

char *str1 = new char [(strlen("строка 1") + 1)] = "строка 1";
Это что, по вашему?
Так строки не копируются, воспользуйтесь strcpy()

[marenko.lilia]

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

char *str1 = new char [(strlen("строка 1") + 1)] = "строка 1";
Это что, по вашему?
Так строки не копируются, воспользуйтесь strcpy()

int main()
{
	setlocale(LC_ALL, "rus");

	char *str1 = new char [(strlen("строка 1") + 1)];
	strcpy(str1, "строка 1");

	char *str2 = new char [(strlen(" + строка 2") + 1)];
	strcpy(str2, " + строка 2");

сделала! ситуация не изменилась

[andreil]

А 21 и должно вывести, ибо:

	int lenPstr1 = strlen(pstr1) + 1;//длина строки pstr1 c '\0'
	int lenPstr2 = strlen(pstr2) + 1;

Зачем дважды прибавлять 1 Это делается ТОЛЬКО 1 раз!
Чуток "причесанный" код:

#include <iostream>
#include <string.h>
using namespace std;

char *newMem(char*, char*);

int main()
{
	setlocale(LC_ALL, "rus");

	char *str1 = new char [(strlen("строка 1") + 1)];
	strcpy(str1, "строка 1");

	char *str2 = new char [(strlen(" + строка 2") + 1)];
	strcpy(str2, " + строка 2");

    cout << "1)" << str1 << endl;
    cout << "2)" << str2 << endl << endl;

	char str[9] = "";//для копирования строки str1 перед удалением памяти

	int x = 0;//управляющая переменная
	while (str1[x] != '\0')
	{
		str[x++] =  str1[x];//копируем строку
	}

	cout << "копия str: " << str << endl;


	str1 = newMem(str1, str2);//функция, которая перевыделит память

	x = 0;
    while (str[x] != '\0')
    {
        str1[x++] = str[x];
    }
    int y = 0;
    while (str2[y] != '\0')
    {
        str1[x++] = str2[y++];
    }

    cout << "Итоговая строка: " << str1 << endl << endl;
    cout << strlen(str1) <<endl;

return 0;
}

char *newMem(char *pstr1, char *pstr2)
{
	int lenPstr1 = strlen(pstr1);//длина строки pstr1 c '\0'
	int lenPstr2 = strlen(pstr2);


	int newSize = lenPstr1 + lenPstr2 + 1;
    cout << "newSize=" << newSize <<endl; //20


	//delete  pstr1;//освобождаем память перед выделением новой памяти для 2х строк

	pstr1 = new char [newSize];

	return pstr1;
}

Результат:

1)строка 1
2) + строка 2

копия str: строка 1
newSize=20
Итоговая строка: строка 1 + строка 2

19

Все ОК, отработало нормально.

[andreil]

Сообщение от marenko.lilia

во-первых, не пойму почему не могу удалить память в функции перед тем как выделить новую?

В чём это выражается? У меня, при раскомментировании строчки с удалением, всё сработало нормально.
В какой среде собираете? Я использую дефолтный GCC, а у Вас, наверное MSVC какой-нибудь

[marenko.lilia]

Спасибо, andreil!
но опять же в итоге cout << strlen(str1) <<endl; не 20, как нам надо, а 31 показывает и "мусор" остался.

1)строка 1
2) + строка 2

копия str: строка 1
newSize=20
Итоговая строка: строка 1 + строка 2Нээээ\Х4V`.☻

31
Для продолжения нажмите любую клавишу . . .

[marenko.lilia]

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

В чём это выражается? У меня, при раскомментировании строчки с удалением, всё сработало нормально.
В какой среде собираете? Я использую дефолтный GCC, а у Вас, наверное MSVC какой-нибудь

да, у меня "MSV какой-нибудь" )))

после того, как ты изменил код - память удалилась нормально! Наверное причина была в том одном '\0'... сейчас буду разбираться

[andreil]

Добавь перед выводом:

str1[x] = '\0';

Это немного не правильно, но сработать должно...

[marenko.lilia]

andreil, вот я прямо беру твой код копирую, запускаю и ...


1)строка 1
2) + строка 2

копия str: строка 1
newSize=20
Итоговая строка: строка 1 + строка 2Нээээ\Х4V`.☻

31
Для продолжения нажмите любую клавишу . . .

[marenko.lilia]

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

Добавь перед выводом:

str1[x] = '\0';

Это немного не правильно, но сработать должно...

да, конечно, так все красиво показывает )))))

но это и правда немного неправильно