SFML баг в программе

@maxm · Регистрация: 17.07.2014

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

Помогите с проблемкой.
Ниже исходники прикрепил, проект VS.
Проблема в чем: поставьте много много башень и посмотрите, как зависают пули, которые не долетают до цели, которую уже уничтожили.
Я делаю так:
для каждой пули делаю ссылку на обьект типа ВРАГ, и каждый раз когда рисую, проверяю, жив ли он. Если уже убит, а пуля не долетела, то удалить пулю из листа и не отображать.
Но они почему-то отображаются, то есть условие (it->chosen_enemy) всегда true.
Почему если я удаляю врагов из листа типа враги, ссылки на удаленные обьекты не становятся NULL ???

@maxm · 24.11.2015, 19:43 **[ТС]**

Исправил проверкой при удалении врага: в цикле все пули, которые указывают на него, автоматически получают nullptr. Проблемка еще есть одна маленькая:
Иногда пули если поставить много башень стреляют куда то в небо. Кому не лень, пожалуйста попробуйте.
И да, обновленный main.cpp

@maxm · 24.11.2015, 20:01 **[ТС]**

Вот поймал кадр как летит вверх влево. Почему такое может быть?
И вообще только что вылетела программа после того кк другая пуля тоже ушла в неизвестном направлении... Типа за пределы памяти обращаюсь..
Откуда эти ошибки???

@maxm · 24.11.2015, 20:10 **[ТС]**

Вот такое вылетает после где то минуты игры. Пишет проблема в строчке где я рисую пули

Croessmah · 24.11.2015, 21:40

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

Вот такое вылетает после где то минуты игры.

у меня сразу segfault при установке башни.

Добавлено через 10 минут
такс, это исправил, добавил chosen_enemy(0) в список инициализации конструктора класса Tower.

Добавлено через 4 минуты
Дальнейший просмотр кода выявил UB:
В цикле при проходе списка enemies, есть такой фрагмент:

C++

                for (list<Bullet>::iterator it_b = bullets.begin(); it_b != bullets.end(); it_b++)
                    if (it_b->chosen_enemy == &(*it)) it_b->chosen_enemy = nullptr;
                enemies.erase(it);//Элемент, на который был "нацелен" итератор, удален, куда теперь "нацелен" итератор it?
                break;

Исправил на:

C++

                for (list<Bullet>::iterator it_b = bullets.begin(); it_b != bullets.end(); it_b++)
                    if (it_b->chosen_enemy == &(*it)) it_b->chosen_enemy = nullptr;
                it=enemies.erase(it); //it теперу будет "нацелен" на элемент, следующий за удаленным. Чтобы его обработать
                --it ;//Идем к предыдущему элементу, тогда
                break;//после итеррации итератор опять будет на этот элемент "нацелен"

После этой правки пули стали летать как надо и вылеты прекратились. Потестите, может еще косяки есть.
А так респект за старания, обычно здесь выкладывают задания "напишите игру", а не "помогите найти ошибку в моей игре"

Добавлено через 2 минуты
Вот почему то при запуске этой игры у меня дросселя начинают свистеть в системнике

Добавлено через 39 минут
Ой, исправил еще пару ошибок такого же плана, появились баги

@maxm · 24.11.2015, 21:49 **[ТС]**

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

Ой, исправил еще пару ошибок такого же плана, появились баги

Вот же блин, есть правило: работает - не трогай !!!

А так то спасибо
Не знал от такой конструкции

Добавлено через 3 минуты
А что дает в этом случае такая правка? Почему без нее не работает то?

@maxm · 24.11.2015, 22:14 **[ТС]**

А, ну и да.. вот вам готовая, поиграть на досуге )))
Сяду за тетрис пока

Croessmah · 24.11.2015, 22:20

Проблема в том, что весь код в куче, мест, где может быть зарыта ошибка - тоже.
Сейчас имеем одну большую портянку, плюс ко всему форматирование хромает сильно и имена переменных (особенно итераторов, не разберешь где какой it).
Разбейте код на функции, выполняющие свои задачи, тогда можно более эффективно работать с кодом, да и код сам станет меньше и нагляднее

@maxm · 25.11.2015, 20:40 **[ТС]**

Да а как разбить тут?
Так ведь прийдеться для функций передавать кучу параметров, например делаю то же перемещение башни по карте - там используются все переменные практически, нужно передать параметров 10 штук.

Или можно обьявить все глобальными вне main-функц. и после их обьявления обьявлять функции ?

@maxm 63 / 35 / 25 Регистрация: 17.07.2014 Сообщений: 457
	24.11.2015, 20:01 [ТС]	3
	Вот поймал кадр как летит вверх влево. Почему такое может быть? И вообще только что вылетела программа после того кк другая пуля тоже ушла в неизвестном направлении... Типа за пределы памяти обращаюсь.. Откуда эти ошибки??? Миниатюры 0

@maxm 63 / 35 / 25 Регистрация: 17.07.2014 Сообщений: 457
	24.11.2015, 20:10 [ТС]	4
	Вот такое вылетает после где то минуты игры. Пишет проблема в строчке где я рисую пули Миниатюры 0

@maxm 63 / 35 / 25 Регистрация: 17.07.2014 Сообщений: 457
	24.11.2015, 21:49 [ТС]	6
	Сообщение от Croessmah Ой, исправил еще пару ошибок такого же плана, появились баги Вот же блин, есть правило: работает - не трогай !!! А так то спасибо Не знал от такой конструкции Добавлено через 3 минуты А что дает в этом случае такая правка? Почему без нее не работает то? 0

Croessmah Неэпический 17870 / 10635 / 2054 Регистрация: 27.09.2012 Сообщений: 26,737 Записей в блоге: 1
	24.11.2015, 22:20	8
	Проблема в том, что весь код в куче, мест, где может быть зарыта ошибка - тоже. Сейчас имеем одну большую портянку, плюс ко всему форматирование хромает сильно и имена переменных (особенно итераторов, не разберешь где какой it). Разбейте код на функции, выполняющие свои задачи, тогда можно более эффективно работать с кодом, да и код сам станет меньше и нагляднее 0

@maxm 63 / 35 / 25 Регистрация: 17.07.2014 Сообщений: 457
	25.11.2015, 20:40 [ТС]	9
	Да а как разбить тут? Так ведь прийдеться для функций передавать кучу параметров, например делаю то же перемещение башни по карте - там используются все переменные практически, нужно передать параметров 10 штук. Или можно обьявить все глобальными вне main-функц. и после их обьявления обьявлять функции ? 0

SFML баг в программе

Решение