Удаление элемента из вектора

@12712266 · Регистрация: 06.09.2014

Студворк — интернет-сервис помощи студентам

Вектор состоит из объектов класса schedule, требуется удалить элементы, где base.group = gr, программа вылетает на этом месте, помогите, пожалуйста, что не так?

C++
1
2
3
4
5
6
for (vector<schedule>::iterator i = base.begin(); i != base.end(); i++) {
    if ((*i).group == gr){ 
        i = base.erase(i);
        check = true;
    }
}

@Тамика · 27.11.2014, 17:59

А можете больше кода скинуть?.. Здесь всё выглядит нормально.

@Max Dark · 27.11.2014, 18:05

std::remove_if

C++
1
2
3
4
5
6
7
#include <algorithm>
 
auto nend = std::remove_if(begin(base), end(base),
    [&gr](const schedule& x) -> bool {
       return x.group == gr;
});
base.erase(nend, end(base));

Добавлено через 1 минуту
Тамика, метод erase делает итератор невалидным

@Тамика · 27.11.2014, 18:06

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

Вектор состоит из объектов класса schedule, требуется удалить элементы, где base.group = gr, программа вылетает на этом месте, помогите, пожалуйста, что не так?

Тьфу блин.
Вы ж когда делаете

C++
1
i = base.erase(i);

- Вы удаляете итератор i. А потом пытаетесь его инкрементить. А он удалён. Вот потому и вылетает.

@Max Dark · 27.11.2014, 18:07

хотя, я могу ошибаться

@12712266 · 27.11.2014, 18:09 **[ТС]**

В зависимости от принимаемого значения k, программа ищет совпадения по разным полям класса, если совпадение не найдено, выводится "В расписании нет такого поля"

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
bool deleted(vector <schedule> & base, int k){
        system("cls");
        int gr, au, number = -1;
        string ifo[3], ifo1[3];
        bool flag = true, check = false;
        cout << "Введите, что нужно удалить" << endl;
        switch (k){
            case 1: gr = checkgroup(base, 1);
                break;
            case 2: ifo[0] = checknumber(2);
                    ifo[1] = checknumber(3);
                    ifo[2] = checknumber(4);
                break;
            case 3: au = checkaudience(base, 1);
                break;
        }
        for (vector<schedule>::iterator i = base.begin(); i != base.end(); i++) {
            switch (k){
                case 1: if ((*i).group == gr){ 
                            i = base.erase(i);
                            check = true;
                        }
                    break;
                case 2: if (ifo[0] == (*i).name.firstname && ifo[1] == (*i).name.surname && ifo[2] == (*i).name.patronymic){
                            i = base.erase(i);
                            check = true;
                        }
                    break;
                case 3: if (au == (*i).audience){
                            i = base.erase(i);
                            check = true;
                        }
                    break;
            }
            number++;
        }
        if (check){
            cout << "Удаление выполнено успешко" << endl;
        } else cout << "В расписании нет такого поля" << endl;
        return check;
    }

@Тамика · 27.11.2014, 18:10

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

хотя, я могу ошибаться

Почему. Всё верно. Я тут тестовый код написала, который иммитирует ошибку ТС. Надеюсь, это моё сообщение не удалят, якобы оно мимо темы.

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
#include <iostream>
#include <cstdlib>
#include <vector>
#include <algorithm>
 
struct sh
{
public:
    int first;
    int second;
};
 
 
int main()
{
    std::vector<sh> v;
    v.resize(5);
    v[0].first = 1;
    v[1].first = 2;
    v[2].first = 3;
    v[3].first = 4;
    v[4].first = 5;
 
    v[0].second = 1;
    v[1].second = 2;
    v[2].second = 3;
    v[3].second = 4;
    v[4].second = 5;
 
    int cmp = 3;
 
    for (auto i = v.begin(); i != v.end(); ++i)
    {
        std::cout << (*i).first << " " << (*i).second << "\n";
        if ((*i).first == cmp) v.erase(std::remove_if(v.begin(), v.end(), [cmp](const sh& item) { return item.first == cmp; }), v.end());
    }
 
    for (auto i = v.begin(); i != v.end(); ++i)
        std::cout << (*i).first << " " << (*i).second << "\n";
 
    system("pause");
    
    return(0);
}

@12712266 · 27.11.2014, 18:24 **[ТС]**

Спасибо, пойду попробую

Добавлено через 13 минут

Сообщение от Тамика

Я тут тестовый код написала, который иммитирует ошибку ТС

Почему-то на ваш код, компилятор выдает 39 ошибок

@Max Dark · 27.11.2014, 18:48

12712266, очевидно, у Вас компилятор не поддерживает С++11
для MinGW/GCC 4.7+ нужно добавлять ключ -std=c++11
Можно написать класс функтора и использовать его вместо лямбда-функции

@12712266 · 27.11.2014, 19:09 **[ТС]**

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

для MinGW/GCC 4.7+ нужно добавлять ключ -std=c++11
Можно написать класс функтора и использовать его вместо лямбда-функции

Можете, пожалуйста, подробнее объяснить?

@Тамика · 27.11.2014, 22:55

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

Почему-то на ваш код, компилятор выдает 39 ошибок

А либы есть нужные?

@gray_fox · 27.11.2014, 23:54

12712266, просто не надо делать инкремент после удаления

C++
1
2
3
4
5
6
7
8
for (vector<schedule>::iterator i = base.begin(); i != base.end(); ) {
    if (i->group == gr){ 
        i = base.erase(i);
        check = true;
    } else {
        ++i;
    }
}

@TheCalligrapher · 28.11.2014, 00:02

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

что не так?

Ну представьте себе, что вектор содержит ровно один элемент и у него выполняется 'group == gr'. Т.е. его надо удалить. Что сделает ваша программа? Она выполнит 'i = base.erase(i);'. Поэле этого итератор 'i' будет указывать на 'end()' контенера. А вы к такому итератору смело применяете 'i++'. Разумеется, в результате все накрывается медным тазом.

Если вы выполнили удаление, то '++' к итератору делать не надо. См. код у gray_fox. Это, вообще-то, идиоматическая структура цикла последовательного прохода с удалением, которую все должны знать наизусть.

P.S. В ситуациях, когда нет разницы, префиксный '++' предпочтительнее постфиксного. Возмите в привычку писать '++i', а не 'i++'.

Новые блоги и статьи Все статьи Все блоги /
изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?	сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .
Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .

@Тамика Котовчанин 942 / 482 / 200 Регистрация: 16.02.2010 Сообщений: 3,338 Записей в блоге: 35
	27.11.2014, 17:59
	А можете больше кода скинуть?.. Здесь всё выглядит нормально. 0

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,793 Записей в блоге: 4
	27.11.2014, 18:07
	хотя, я могу ошибаться 0

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,793 Записей в блоге: 4
	27.11.2014, 18:48
	12712266, очевидно, у Вас компилятор не поддерживает С++11 для MinGW/GCC 4.7+ нужно добавлять ключ -std=c++11 Можно написать класс функтора и использовать его вместо лямбда-функции 1

Удаление элемента из вектора

Решение