ALLEGRO 5 [Język C, zwiększanie tablicy]

Question

Witam, piszę Snake'a w języku C przy pomocy ALLEGRO 5 i natknąłem się na problem zwiększania węża po zjedzeniu owoca, do pewnego momentu wszystko dobrze działa, lecz potem wyrzuca błąd o wycieku pamięci. Zakładam, że źle korzystam z funkcji realloc (funkcja addNode). Byłbym wdzięczny, gdyby ktoś mógł mi wyjaśnić w czym jest problem.

Kod: https://pastebin.com/XpVkgiSB

Answer 1

void addNode(struct Snake *snake, struct Node *prev)
{
	/* ... */
	
	prev = (struct Node *)realloc(prev, snake->length * sizeof(struct Node));
}

To powoduje wyciek pamięci. Jeśli już, to tak:

void addNode(struct Snake *snake, struct Node **prev)
{
	/* ... */

	*prev = (struct Node *)realloc(*prev, snake->length * sizeof(struct Node));
}

 

Comments

I w jaki sposób to ma działać? Przecież realloc, jako pierwszy argument przyjmuje adres wskaźnika, który chce zwiększyć.

---

Przekazujesz wskaznik przez wartosc i probujesz cos do niego przypisac czyli taka sytuacja:

void f(int x)
{ x= 5; }

 

---

@wixy0, problem w tym, że prev w twojej wersji funkcji jest lokalnym wskaźnikiem, to zwiększanie pamięci ma lokalny charakter. Zatem to, co dasz w drugim parametrze funkcji, nie zostanie zwiększone. W mojej wersji trzeba podać wskaźnik na wskaźnik, dzięki czemu przypisanie adresu będzie widoczne po wyjściu z funkcji.

---

Chociaż może zamiast komplikować tak sprawy lepiej zrobić to od razu w funkcji logic. W jaki sposób odnosić się do składowych albo do tego wskaźnika, jeżeli argumentem jest dajmy na to struct Snake **snake. Np. nie wiem jak dostać się do składowej length, żeby ją zwiększyć.

---

W jaki sposób odnosić się do składowych albo do tego wskaźnika

(*snake)->składowa

 

Choć nie wiem, po co snake miałaby być wskaźnikiem na wskaźnik.

---

Muszę też modyfikować wielkość tablicy nodes, która jest składową snake.

---

W takim razie nie musisz przekazywać struktury Snake przez wskaźnik na wskaźnik. Tak jak masz teraz, jest ok.

---

To poprawiony kod: https://pastebin.com/9QvKhxvv

Korzystając z okazji, natknąłem się na błąd który może wynikać z tego w jaki sposób Allegro obsługuje wydarzenia ale nie jestem pewien. Otóż jak wąż będzie się składał z więcej niż 3 kwadratów i np. poruszając się w górę w prawie jednym momencie nacisne strzałke w lewo/prawo i w dół to zalicza to jako kolizje. Czy da radę jakoś wyeliminować ten błąd, np nie pozwalając wciskać dwóch klawiszy na raz?

---

Pisałem przecież , że nie musisz przekazywać Snake'a przez podwójny wskaźnik. Tak wystarczy:

void logic(struct Snake *snake, struct Node *fruit, struct Node **prev)
{
	//KOLIZJA Z GRANICAMI PLANSZY
	if (snake->head->y < 0)
		gameOver = 0;
 
	if (snake->head->y > H - recSize)
		gameOver = 0;
 
	if (snake->head->x < 0)
		gameOver = 0;
 
	if (snake->head->x > W - recSize)
		gameOver = 0;
 
	//ZJEDZENIE OWOCU
	if (snake->head->x == fruit->x && snake->head->y == fruit->y)
	{
		snake->length++;
		snake->nodes = (struct Node *)realloc(snake->nodes, snake->length * sizeof(struct Node));
		*prev = (struct Node *)realloc(*prev, snake->length * sizeof(struct Node));
 
		fruit->x = (rand() % (W / recSize - 1) + 1) * recSize;
		fruit->y = (rand() % (H / recSize - 1) + 1) * recSize;
	}
 
	//KOLIZJA Z OGONEM
	for (int i = 0; i < snake->length; i++)
		if (snake->head->x == snake->nodes[i].x && snake->head->y == snake->nodes[i].y)
			gameOver = 0;
}

Czy da radę jakoś wyeliminować ten błąd, np nie pozwalając wciskać dwóch klawiszy na raz?

Zapewne musisz obsłużyć zdarzenia ALLEGRO_EVENT_KEY_DOWN i ALLEGRO_EVENT_KEY_UP, czyli nie reagujesz na key-down, jeśli poprzednie naciśnięcie nie zakończyło się zdarzeniem key-up.