Помогите найти ошибку

[Tier_Mia]

#include <iostream>
#include <string>
#include <vector>
#include <algorithm>
using namespace std;

int main()
{
	vector<char> text;
	vector<char>::iterator it;
	char symb;
	char symb1;
	char symb2;
	cout << "write the symbol you want to be replaced" << endl;
	cin >> symb1;
	cout << "write the new symbol " << endl;
	cin >> symb2;
	cout << "write the amount of the symbols " << endl;
	int amount;
	cin >> amount;
	cout << "Your string is " << endl;
	for (int i = 0; i <= amount; ++i)
	{
		cin >> symb;
		text.push_back(symb);
		
	}
	for (it = text.begin(); it != text.end(); ++it)
	{
		if (*it = symb1)
		{
			it = text.erase(it);
			it = text.insert(it, symb2);
		}
	}
	for (int i = 0; i <= amount; ++i)
	{
		cout << text[i];

	}
	return main();
	
}

Задание такое: надо заменить один символ на другой в строке. Код работает, но не совсем правильно
В первом цикле я вывожу просто строку, во втором - делаю замену, а в 3 цикле заново хочу вывести новый вектор

[Desc]

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

В первом цикле я вывожу просто строку

Наоборот, вводите символы в цикле.

[p51x]

1. erase инвалидирует итераторы
2. insert может инвалидировать итераторы
3. есть std::replace_if
4. от 0 до amount включительно amount+1 элемент

[Алексей1153]

тут всё неправильно

if (*it = symb1)
		{
			it = text.erase(it);
			it = text.insert(it, symb2);
		}

пробуй так

if (*it == symb1)
{
	*it=symb2;
}

или так (это замена всего цикла)

std::replace(text.begin(),text.end(),symb1,symb2);

[Tier_Mia]

Сообщение от Алексей1153

тут всё неправильно

if (*it = symb1)
		{
			it = text.erase(it);
			it = text.insert(it, symb2);
		}

пробуй так

if (*it == symb1)
{
	*it=symb2;
}

или так (это замена всего цикла)

std::replace(text.begin(),text.end(),symb1,symb2);

string text;
	getline(cin, text);
	char symb1;
	char symb2;
	cin >> symb1;
	cin >> symb2;
	replace(text.begin(), text.end(), symb1, symb2);
	return main();

почему сейчас не работает тогда? Я же вроде все правильно сделал, нечего не понимаю...

[Tier_Mia]

Сообщение от Алексей1153

тут всё неправильно

if (*it = symb1)
		{
			it = text.erase(it);
			it = text.insert(it, symb2);
		}

или так (это замена всего цикла)

std::replace(text.begin(),text.end(),symb1,symb2);

#include <iostream>
#include <string>
#include <vector>
#include <algorithm>
using namespace std;

int main()
{
	vector<char> text;
	vector<char>::iterator it;
	char symb;
	char symb1;
	char symb2;
	cout << "write the symbol you want to be replaced" << endl;
	cin >> symb1;
	cout << "write the new symbol " << endl;
	cin >> symb2;
	cout << "Your string is " << endl;
	for (it = text.begin(); it != text.end(); ++it)
	{
		cin >> symb;
		text.push_back(symb);
		
	}
	replace(text.begin(), text.end(), symb1, symb2);
	for (it = text.begin(); it != text.end(); ++it)
	{
		cout << *it;

	}
	return main();
	
}

пробуй так

if (*it == symb1)
{
	*it=symb2;
}

попробовал ещё так, тоже не работает

[Алексей1153]

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

почему сейчас не работает тогда? Я же вроде все правильно сделал, нечего не понимаю...

всё там прекрасно работает, вот попробуй

	std::string text="1212112";
	char symb1='2';
	char symb2='3';
	std::replace(text.begin(), text.end(), symb1, symb2);

тебе осталось только убрать рекурсивный вызов main() и стек перестанет падать :D