Allegro 5 - problem z implementacją delta time i timestep

Question

Witam, chcąc zaimplementować delta time i timestep natknąłem się na problem z niestałymi wartościami timestep. Otóż, gdy przytrzymuje jakiś klawisz ruchu, if po Inpucie jest pomijany przez co wartość timestep jest dużo mniejsza, co z kolei wpływa na movement gracza (nie jest on już płynny).

Tak wygląda główna pętla gry.

void Run()
{
	float time;

	while (!gameOver)
	{
		time = (float)al_get_time() * 1000.0;
		timeStep = time - lastFrameTime;
		lastFrameTime = time;

		Input();

		if (draw && al_is_event_queue_empty(eventQueue))
		{
			game->draw = false;

			UpdateSnake(timeStep, direction);
			Logic();
			Draw();

			al_flip_display();
			al_clear_to_color(al_map_rgb(0, 0, 0));
		}
	}
}

A tak wygląda Input()

void Input()
{
	ALLEGRO_EVENT events;
	al_wait_for_eventeventQueue, &events);

	switch (events.type)
	{
		case ALLEGRO_EVENT_TIMER:
		{
			game->draw = true;

			switch (keys)
			{
				case UP:
					direction = (Position) { 0, -1 };
					break;

				case DOWN:
					direction = (Position) { 0, 1 };
					break;

				case LEFT:
					direction = (Position) { -1, 0 };
					break;

				case RIGHT:
					direction = (Position) { 1, 0 };
					break;
				}
			}

			break;
		}

		case ALLEGRO_EVENT_KEY_DOWN:
		{
			if (events.keyboard.keycode == ALLEGRO_KEY_UP || events.keyboard.keycode == ALLEGRO_KEY_W)
				game->keys = UP;
	
			if (events.keyboard.keycode == ALLEGRO_KEY_DOWN || events.keyboard.keycode == ALLEGRO_KEY_S)
				game->keys = DOWN;
					
			if (events.keyboard.keycode == ALLEGRO_KEY_LEFT || events.keyboard.keycode == ALLEGRO_KEY_A)
				game->keys = LEFT;

			if (events.keyboard.keycode == ALLEGRO_KEY_RIGHT || events.keyboard.keycode == ALLEGRO_KEY_D)
				game->keys = RIGHT;

			break;
		}

		case ALLEGRO_EVENT_DISPLAY_CLOSE:
		{
			gameOver = true;
			break;
		}
	}
}

 

Answer 1

Jestem niewyspany (gorąca noc, wyjący wiatrak) i nie piłem jeszcze kawy.
Wydaje mi się, że zmienna lastFrameTime powinna być aktualizowana podczas rysowania.
 

void Run()
{
    float time = (float)al_get_time() * 1000.0;
    lastFrameTime = time;
 
    while (!gameOver)
    {
        time = (float)al_get_time() * 1000.0;
        timeStep = time - lastFrameTime;
 
        Input();
 
        if (draw && al_is_event_queue_empty(eventQueue))
        {
            lastFrameTime = time;
            game->draw = false;
 
            UpdateSnake(timeStep, direction);
            Logic();
            Draw();
 
            al_flip_display();
            al_clear_to_color(al_map_rgb(0, 0, 0));
        }
    }
}

Comments

Niestety to raczej nie w tym rzecz, mam wrażenie że to coś z obsługą eventów w Allegro. Jeśli chcesz możesz zobaczyć jak to wygląda (najlepiej jest przemieścić się do jednej krawędzi i przemieszczając się do drugiej przytrzymać klawisz, i przegrać): https://www.mediafire.com/file/b3mg9cetm54w6zf/Snake.rar/file

Mogę również udostępnić cały kod lub projekt w VS 2017.

---

Ewentualnie mógłbyś spróbować obsłużyć wszystkie eventy z kolejki przed rysowaniem.

while (!al_is_event_queue_empty(eventQueue)) {
    Input()
}
if (draw) {
    // rysowanie
}

 

---

Jeżeli podzielisz się kodem to spróbuję później zdebugować problem.

---

Kod jest kompilowany w C.

Skoro wysyłam już kod to skorzystam z okazji, mógłbyś ocenić czy mój kod się do czegoś nadaje? Nie znam żadnych patternów dla C, więc nie wiem czy tak powinno się pisać. Jakieś rady byłyby mile widziane :)

https://www.mediafire.com/file/ncuvoag2gwkmc2v/Kod-Snake-Allegro5.rar/file

---

Dopiero zauważyłem, że wysłałem wersje ze złym main.c, poprawiam.

#include "Game.h"

int main(void)
{
	Game GAME;

	Setup(&GAME);
	Run(&GAME);
	Free(&GAME);

	return EXIT_SUCCESS;
}

 

---

Testowałem na Ubuntu.
Liczenie delty w warunku if (draw && ...) naprawiło mi problem z szarpaniem podczas trzymania klawisza. Jednak sterowanie ciągle było "ociężałe". Zastosowałem drugi pomysł który ci zaproponowałem i wszystkie moje problemy znikły.

void Run(Game *game)
{
    float time = (float)al_get_time() * 1000.0;
    lastFrameTime = time;

    while (!game->gameOver)
    {
        while (!al_is_event_queue_empty(game->eventQueue)) 
            Input(game);

        if (game->draw)
        {
            time = (float)al_get_time() * 1000.0;
            timeStep = time - lastFrameTime;
            lastFrameTime = time;

            /* reszta kodu */
        }
    }
} 

Na pierwszy rzut oka kod ładnie się prezentuje. Widzę, że część logiki jeszcze nie jest zaimplementowana. Brakuje mi sprawdzania błędów podczas wczytywania zasobów.
Później poszukam czy nie ma problemów z zarządzaniem zasobami.

---

Udało mi się znaleźć na forum Allegro algorytm interpolacji napisany przez jednego z twórców tej biblioteki i w nim była jedna różnica, otóż:

while (al_get_next_event(eventQueue, &events))

Wszystko było w takim while zamiast al_wait_for_event().

Przesyłam zaaktualizowany plik Game.c i folder res: https://www.mediafire.com/file/fw8eiygtt0xlnf7/updated.rar/file