Проблема с анимацией?

У вас есть только один индекс для каждого направления движения!

Предполагая, что вы нарисовали около 3 (левая нога вперед, перекрытие, правая нога вперед), вам нужно будет увеличивать индекс со временем или движением по мере продвижения марша.

Интересно, как далеко продвинулось Allegro — десять лет назад это была в основном графика DOS (я использовал ее с DJGPP) и была заметка от автора, живущего в лондонской квартире. Как сурово я думал, только теперь я вижу, как далеко мы, провинциалы, от лондонских цен.

В ответ на ваш комментарий попробуйте что-то вроде:

#define MAXPICS 3
BITMAP *OskarWalksLeft[MAXPICS];
BITMAP *OskarWalksRight[MAXPICS];
picNumber = x % MAXPICS;
if (facingLeft) {
    draw_sprite(Buffer, OskarWalksLeft[picNumber], x, y);
} else //facingRight
    draw_sprite(Buffer, OskarWalksRight[picNumber], x, y);

или если вам не нужен новый кадр на пиксель, но каждые 3 или около того, попробуйте заменить

picNumber = x % MAXPICS;

с

picNumber = (x / 3)  % MAXPICS;

Я попытался немного привести в порядок ваш код и придумал это, у меня нет аллегро, поэтому я не смог его проверить:

#include "Player.h"
#include "Global.h"
#include <allegro.h>

Player::Player(){}

Player::~Player(){}

enum playerDir {left, right, front, back, stationary} direction;
BITMAP *Oskar[stationary][3];

void Player::Init(){
    x = 10;
    y = 10;
    velocityX = 0;
    velocityY = 0;
    speed = 1;
    picNumber = x % MAXPICS;

    char filename[256];
    const char *dirStrs[] = {"left","right","front","back"};
    for (playerDir i = left; i < stationary; (int)i = (int)i + 1)
        for (j = 0; j < 3; ++j) {
            strcpy(filename, "Character\\Oskar");
            strcat(filename, dirStrs[i]);
            strcat(filename, itoa(j));
            strcat(filename, ".bmp");
            Oskar[i][j] = load_bitmap(filename, NULL);
        }
}

void Player::Update(){Player::Controls();}
void Player::Draw(BITMAP *Buffer){
    switch (direction) {
        case left :
                draw_sprite(Buffer, OskarLeft[picNumber], x, y);
                break;
        case right :
                draw_sprite(Buffer, OskarRight[picNumber], x, y);
                break;
        case front :
                draw_sprite(Buffer, OskarFront[picNumber], x, y);
                break;
        case back :
                draw_sprite(Buffer, OskarBack[picNumber], x, y);
                break;
        default:
                draw_sprite(Buffer, OskarFront[1], x, y);
        }
}

void Player::Controls(){
    direction = stationary;
    if(key[KEY_RIGHT]){
        velocityX = speed;
        direction = right;
    } else if(key [KEY_LEFT]){
        velocityX = -speed;
        direction = left;
    } else velocityX = 0;

    if(key [KEY_UP]){
        velocityY = -speed;
        direction = front;
    } else if(key [KEY_DOWN]){
        velocityY = speed;
        direction = back;
    } else velocityY = 0;

    x += velocityX;
    y += velocityY;
 }

Кажется, вы не делаете никакой анимации. Кажется, вы просто устанавливаете определенный кадр на основе клавиши направления, которую нажимает пользователь. Вам нужна функция, которая делает что-то вроде этого:

void Player::updateFrame(float time_delta)
{
    time_to_next_frame -= time_delta;
    while (time_to_next_frame <= 0.0f)
    {
        nextFrame();
        time_to_next_frame += time_per_frame;
    }
}

Назовите его каждый кадр, передав ему количество времени, прошедшее с момента последнего кадра.

В ответ на ваши комментарии к двум другим ответам вы, кажется, не знаете, как не перейти к определенному значению.

Для этого (например, вам нужно что-то в диапазоне [0, 2] + c, где c — некоторая константа, определяющая начало вашего индекса. Например, вы можете сделать следующее:

void Player::nextFrame( int startIndex /* Changes based on your direction */) {
      static unsigned short iIndex = 0;

      // Increment and map value to the range [0, 2]
      ++iIndex;
      iIndex %= 3;


      // Note: this assumes that picNumber is a member function, else you
      // will have to pass it by reference to this function.
      picNumber = iIndex + startIndex;
}

Это должно делать то, что вы хотите (т.е. в зависимости от направления, в котором вы меняете начальный индекс, а затем он увеличивается, сопоставляя значение с диапазоном [0, 2].

Надеюсь это поможет!