C++ vector::erase

[onewho]

	void remove(Student &obj) {
		
		std::vector<Student>::iterator it;
		for (it=data.begin(); it!=data.end(); it++)
			if (strcmp(it->get_name(),obj.get_name())) {
				it = data.erase(it);
				break;
			}

вот оно :cry:

[Д_М]

Замени на it = data.erase(it). Делов-то.

После этого it указывает на следующий за удаленным элемент.
Потом делаем it++, в результате пропускаем один эл-т

[onewho]

не работает)))
к тому же мне кажется уж после этого это никак не влияет ибо break;

[Obey-Kun]

Ты его неправильно понял. Тебе надо делать так:

void remove(Student &obj) {
		
		std::vector<Student>::iterator it;
		for (it=data.begin(); it!=data.end(); it++)
			if (strcmp(it->get_name(),obj.get_name())) {
				it = data.erase(it);
                                --it;
			}

Но лучше делать через перегрузку operator== и юзать erase без написания всяких циклов.

Ещё вариант -- data.erase(std::remove_if(data.begi n(), data.end(), <сюда впихни фугнкцию-сравнивалку>), data.end());

Всё, я спать, а то уже 30 часов без сна

[onewho]

Печально но и это не работает.
И еще ребят, вам не кажется что после it = data.erase(it);
всякие -- и ++ какбы не имеют смысла?

Я блин в тупике

[Д_М]

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

И еще ребят, вам не кажется что после it = data.erase(it);
всякие -- и ++ какбы не имеют смысла?

Имеют. data::erase возвращает валидный итератор.

Я бы вообще шерстил вектор от конца к началу.

[onewho]

а это что for (it=data.begin(); it!=data.end(); it++) ?
поцаны хелп))

[Д_М]

Кажется нашел.
Не хватает оператора копирования. Вызывается автоматически сгенеренный компилятором, и при освобождении name портится память

Надо вставить

        Student& operator=(const Student& obj) {
                name = new char[strlen(obj.name)+1];
                strcpy(name,obj.name);
                for (int i=0; i<n; i++)
                        marks[i]=obj.marks[i];
        }

И еще

if (strcmp(it->get_name(),obj.get_name())) {
                                it = data.erase(it);

здесь по смыслу надо strcmp(it->get_name(),obj.get_name()) != 0

[onewho]

офигеть... Д_М ты бох.. заработало!
нет, там надо именно ==0. т.к такое значение возвращает strcmp() если строки равны

кстати и такая форма

data.erase(it);

тоже работает

и никакого присваивания... объясните ?!

[Д_М]

нет, там надо именно ==0

Да-да, описка у меня