Przeszukiwanie listy (kontenery STL)

Question

Witam,

pewnie to prosta rzecz ale zjadła już kilka moich godzin tak, że zaczynam panicznie wyrywać włosy z głowy, proszę o pomoc. Podsyłam link do repozytorium z całym:projektem https://github.com/Blauto/2d_Game/tree/master/Arcanoid/Arcanoid

Problem polega na tym że elementy (Slaby) nie wiem jak to ująć ale nie do końca usuwają się z listy. W jednej pętli program je widzi(przy sprawdzaniu kolizji) a w innej nie (przy rysowaniu) pomimo tego ze pętle są identyczne. Zauważyłem że zmiany które wykonuje na elementach listy poprzez wskaźnik nie zapisują się (dałem specjalne ify które mają to sprawdzać )

sf::Vector2f SlabMenager::checkColision(sf::CircleShape & ball)
{
	for (it = slabList.begin(); it != slabList.end(); it++) {

		if (ball.getGlobalBounds().intersects(it->slab.getGlobalBounds())) {

			sf::Vector2f slabPosPom = it->slab.getPosition();

			it->removeLive();
			if (it->live < 1)
				slabList.erase(it);

			return slabPosPom;
		}
	}
	return sf::Vector2f(0.0f, 0.0f);
}

void Slab::removeLive()
{
	live--;
}

a to pętla która pomimo erasa w poprzedniej i tak przechodzi przez wszystkie elementy które były i rysuje wszystko 
dodałem tego ifa żeby dodatkowo sprawdzić skoro wcześniej na tych elementach odejmowałem to ta zmiana powinna być do sprawdzenia nawet jeżeli przechodzę przez wszystkie elementy i tego wcześniejszego erasa by tam miało nie być to live powinno być mniejsze od 0, ale operacji remove live tak jakby nie pamięta :P

	for (it = slabList.begin(); it != slabList.end(); it++) {

		if (it->live > 0) {
			it->update();
			it->Draw(game->window);

		}
	}

Dziękuję za pomoc!

Answer 1

Dwie instancje SlabMenager:

Game::Game()
{
	player = new Player(this, windowMode.width / 2);
	slabMenager = new SlabMenager(this); // <--- !!!!
}



void Game::run()
{
	Ball ball(this, 100.0f, 100.0f);
	list<Ball> ballList;
	ballList.push_back(ball);
	SlabMenager slabMenager(this); // <--- !!!


	while (window.isOpen()) {
		/* ... */

 

Dlaczego iterator SlabMenager::it zrobiłeś polem klasy?

 

PS. nie Menager, tylko Manager.

Comments

Szczerze? nie wiem, :) bez sensu, usunę tego iteratora z klasy (jako pole). Dziękuję za zwrócenie uwagi na Menagera. Oczywiście za komentarz też, szczerze myślałem ze zamknąłem ten temat, bo już wczoraj do tego doszedłem sprawdzając rejestry i poszczególne adresy że to są inne listy :P. Masakra jedna linijka zapomniana przy usuwaniu i tyle czasu straconego. Dziękuję za komentarz!