Podwójne dodawanie obiektów do vectora

Question

Witam,

Obecnie testuję dodawanie i wyświetlanie obiektów RectangleShape do/z wektora i natrafiłem na problem. Otóż po dodaniu obiektu do kontenera (klawisz spacji) okazuje się, że dodawane są dwa obiekty a nie jeden. Ktoś mi wytłumaczy dlaczego się tak dzieje i jak rozwiązać ten problem?

#include <SFML\Graphics.hpp>
#include <vector>

using namespace sf;

int main()
{
	RenderWindow window(VideoMode(640, 480), "testy");
	window.setKeyRepeatEnabled(false);

	std::vector<RectangleShape> vec;

	RectangleShape rect;

	while (window.isOpen())
	{
		Event ev;
		while (window.pollEvent(ev))
		{
			if (ev.type == Event::Closed || Keyboard::isKeyPressed(Keyboard::Escape))
				window.close();

			if (Keyboard::isKeyPressed(Keyboard::Space))
			{
				vec.push_back(rect);
				printf("%i\n", vec.size());
			}
		}

		window.clear();
		window.display();
	}
}

 

Answer 1

Jeżeli chcesz dodawać obiekty klawiszem zrób to tak:

if (ev.type == sf::Event::KeyPressed)
{
    if (ev.key.code == sf::Keyboard::Space)
    {
         ...
    }
}

W taki sposób eventy klawiszy działają tak jak wciskanie klawiszy w notatniku. Jeżeli chcesz zrobić sterowanie postaci to lepszym wyborem jest isKeyPressed, ale POZA pętlą eventów. 

Comments

Dzięki za odpowiedź. Po co dwa ify skoro można to zrobić w jednym? Owszem, jeśli zamienię ifa z pozycji 23 na:

if (ev.type == Event::KeyPressed && ev.key.code == Keyboard::Space)

program działa, jak należy - dodawany jest tylko jeden obiekt do kontenera.

Jednak nadal nie uzyskałem odpowiedzi na pytanie DLACZEGO tak się dzieje?

---

Jak będziesz miał obsługę 20 klawiszy to będziesz miał dużo nadmiarowego kodu. Odpowiedz na główne pytanie:
Metoda pollEvent w momencie wciśnięcia klawisza umieszcza po kolei zdarzenia KeyPressed, TextEntered. W czasie pierwszego jest prawdą, że klawisz spacja jest wciśnięty. W czasie drugiego też. Dlatego obiekt zostanie dodany dwa razy.

---

Mógłbyś też wyszczególnić w czasie trwania którego eventu ma nastąpić dodawanie obiektów.

if (ev.type == Event::KeyPressed)
	if (Keyboard::isKeyPressed(Keyboard::Space))

---

I takiej odpowiedzi oczekiwałem :)

Dziękuję.