Allegro 5/SFML - problem z eventami (Snake)

Question

Witam, robiąc prostą wersję Snake'a zauważyłem pewien bug, który pojawia się również w wersji użytkownika emSon (https://forum.pasja-informatyki.pl/110509/c-sfml-snake-new-version). Prawdopodobnie chodzi o to, w jaki sposób obsługiwane są eventy w Allegro czy SFML. Otóż chodzi o to, że gdy Snake jest większy od stanu początkowego i poruszając się, np. w dół naciśnie się szybko lewo/prawo i w górę to zalicza to jako kolizję. Jest możliwość pozbycia się tego buga?

Kod dla Allegro 5: https://pastebin.com/9kvZeyUm

Answer 1

Prawdopodobnie chodzi o to, w jaki sposób obsługiwane są eventy w Allegro czy SFML 

Allegro nie ma pojęcia o Twojej logice gry. Wciskasz klawisz to leci event. Jeśli logika narzuca pewne tempo gry to powinieneś obsłużyć przypadek wielokrotnych wciśnięć w trakcie jednej "tury". Na szybko dodając flage changedDirection, która pilnowałaby, żeby zmienić kierunek dało się tylko raz w ciągu tury. Mógłbyś też zrobić to porządniej i w eventach klawiszy ustawiać jedynie numer wciśniętego klawisza a faktyczną zmianę kierunku robić w ALLEGRO_EVENT_TIMER

Patrząc na Twój kod dziwne wydaje mi się wywołanie funkcji logic poza żadnym ifem - czyli dla dowolnego eventu. Oznacza to, że przeliczasz logikę kilkakrotnie na jedno poruszenie sie węża.

while(gameOver)

też jest dzwine :D

No i strasznie upośledzona tablica dynamiczna, która jest zwiększana o 1 za każdym razem zamiast hurtem np podwajając długość.

Comments

Tak, zmieniłem parę rzeczy.

Co do logiki, funkcja ta jest teraz wewnątrz if'a (draw && !gameOver).

while(gameOver) to teraz while(!gameOver)

A co do zwiększania o 1 tablicy dynamicznej. Wydaje mi się, że nie ma co marnować pamięci a w ten sposób zawsze mam jej tyle ile potrzebuje.

---

Ze zwiększaniem tablicy jest taki problem, że generalnie przy zwiększeniu musisz ją całą przepisać w inne miejsce :D U Ciebie sytuacje troszke łagodzi realloc, który czasem powiększa zaalokowany blok w miejscu zamiast przepisywać całości w inne miejsce. Nie jestem ekspertem C, ale wydaje mi się, że to jak często się to uda jest troche losowe i nie polegałbym na tym za bardzo.

---

Czyli zapewne, żeby nie zostawiać śmieci w razie przepisania bloku w inne miejsce, trzeba by usuwać wskaźnik wskazujący na starą pozycję.

---

Śmieciami się nie przejmuj. Przejmowałbym się czasem potrzebnym na przepisaniem  całej tablicy w nowe miejsce.  U Ciebie akurat jest ona mała, ale mimo wszystko wygląda to źle. Może źle to opisuje, więc napiszę co robi realloc pod spodem. Mamy taką tablice

	int n = 10;
	int* tab = malloc(sizeof(int)*n);

Chcemy ją zwiększyć o 1:

	tab = realloc(tab, sizeof(int)*(n + 1));

i ten realloc pod spodem albo faktycznie zwiększa nasz obecny blok, albo będzie musiał przepisać całość w nowe miejsce w ten sposób:

	int* new_tab = malloc(sizeof(int)*(n + 1));
	for (int i = 0; i < n; i++)
	{
		new_tab[i] = tab[i];
	}
	free(tab);
	tab = new_tab;

I dlatego zwiększanie tablicy generalnie jest mało-wydajne i jeśli już zwiększasz to hurtem a nie po 1 elemencie.