Не понимаю почему неправильно работает for_each по списку с удалением элемента

@JVStalin · Регистрация: 15.10.2017

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

Пишу тут потихоньку код для диплома, и столкнулся с такой проблемой:
При попытке уничтожить последний или предпоследний блок всё срабатывает, но при уничтожении более ближних блоков вылетает вот такая ошибка

Не понимаю почему неправильно работает for_each по списку с удалением элемента

Сам только начал разбираться со списками и векторами, до этого такую же прогу делал на js, но там немного попроще(значительно проще

)

C++
1
2
3
4
5
6
7
8
9
        for (int i = 0; i < bricks.size(); i++)
        {
            if (b.checkHit((*its)->x, (*its)->y, (*its)->w, (*its)->h, (*its)->health) == true) {
                if (((*its)->health) > 1) (*its)->health--;
                else (*its)->DeleteBrick(*its,bricks);
            }
            (*its)->update();
            its++;
        }

@nonedark2008 · 26.01.2019, 00:41

JVStalin, какой тип у bricks, и что происходит внутри DeleteBrick?
Предположу, что ты удаляешь элемент в контейнере, на который ссылается итератор, а потом пытаешься что-то с этим итератором делать.

Не по теме:

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

код для диплома

Обнищали нынче дипломы... Правда можно было арканоид в качестве дипломной работы написать?

@JVStalin · 26.01.2019, 01:01 **[ТС]**

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

Обнищали нынче дипломы... Правда можно было арканоид в качестве дипломной работы написать?

Ну, диплом я сдаю в колледже, а про его реализацию - не всё так просто как может быть. Обычно у нас все сдают самые простые АИСК'и, ну а я решил сделать Арканоид, так как он ни чем не хуже.
А что на счёт удаления, то да, я удаляю контейнер, но после этого я ничего с этим же итератором...
То есть я получается к уже не существующему итератору добавил 1(ну, переместил на следующий объект)?

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

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

какой тип у bricks, и что происходит внутри DeleteBrick?

bricks является списком, принимающим ссылку на класс Brick.
А вот что происходит в функции DeleteBrick:

C++
1
2
3
4
5
6
void DeleteBrick(Brick* b, std::list<Brick*>& cl) {
        if (!b) return;
        std::list<Brick*>::iterator i = find(cl.begin(), cl.end(), b);
        if (i != cl.end()) cl.erase(i);
        delete b;
    } //Удаление данного блока из списка

Verevkin · 26.01.2019, 01:10

Вангую: цикл от конца к началу быть должон.

@JVStalin · 26.01.2019, 01:21 **[ТС]**

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

Вангую: цикл от конца к началу быть должон.

Сработало, но как так?

Verevkin · 26.01.2019, 01:29

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

Сработало, но как так?

Магия!

Название: 36796bc6b2cd47f91dd62d524352fbf0.jpg
Просмотров: 27

Размер: 14.3 Кб

@Max Dark · 26.01.2019, 01:44

JVStalin, std::list::erase
После удаления элемента в DeleteBrick итератор its в вызывающей функции становится невалидным.
То есть после удаления Вам нужно его обновить.
В добавок Вы пытаетесь обратиться к методу удаленного объекта:(*its)->update();

Я бы переписал цикл примерно так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
std::list<Brick*>::iterator its = bricks.begin();
while (its != bricks.end())
{
    Brick* &current = *its;
    if (b.checkHit(current->x, current->y, current->w, current->h, current->health)) {
        if ((current->health--) <= 1)
        {
            delete current;
            its = bricks.erase(its);
            continue;
        }
    }
    (*its)->update();
    its++;
}

Я не знаю, что выполняет метод update
Если его нужно вызывать и для стираемого блока, то явно до удаления из памяти...

Новые блоги и статьи Все статьи Все блоги /
сукцессия микоризы: основная теория в виде двух уравнений. 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 считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .

Не понимаю почему неправильно работает for_each по списку с удалением элемента

Решение

Решение

Verevkin Злостный нарушитель 10254 / 5680 / 1262 Регистрация: 12.03.2015 Сообщений: 26,326
	26.01.2019, 01:10
	Сообщение было отмечено JVStalin как решение Решение Вангую: цикл от конца к началу быть должон. 1