Slow movement over background

agonvs

Hi all,

Any ideas why the ball moves so slowly over this background?

#SelectExpand
   1#include<allegro5/allegro.h>
   2#include<allegro5/allegro_primitives.h>
   3#include<allegro5/allegro_image.h>
   4#include<iostream>
   5using namespace std;
   6
   7void render()
   8{
cout << endl;
  10}
  11
  12enum directions {STILL, NORTH, NORTHEAST, EAST, SOUTHEAST, SOUTH, SOUTHWEST, WEST, NORTHWEST};
  13enum buttons {NONE,ZERO,ONE,EIGHT,NINE};
  14
  15int main()
  16{
  17
float gameTime = 0;
int frames = 0;
int gameFPS = 0;
if(!(al_init()))
  cout << "Couldn't initialize Allegro." << cout;
if(!(al_install_joystick()))
  cout << "Couldn't install joystick." << endl;
al_init_image_addon();
ALLEGRO_EVENT_QUEUE *event_queue = NULL;
event_queue = al_create_event_queue();
if(!event_queue)
  cout << "al_create_event_queue failed." << endl;
ALLEGRO_TIMER *timer;
timer = al_create_timer(1.0/60);
al_register_event_source(event_queue, al_get_timer_event_source(timer));
al_start_timer(timer);
gameTime = al_current_time();
al_register_event_source(event_queue, al_get_joystick_event_source());
ALLEGRO_JOYSTICK* joy = al_get_joystick(0);
float ballposX=0;
float ballposY=0;
ALLEGRO_JOYSTICK_STATE jst;
directions dir = STILL;
buttons button = NONE;
bool render = false;
ALLEGRO_DISPLAY *display = NULL;
ALLEGRO_BITMAP *image = NULL;
ALLEGRO_BITMAP *texture = NULL;
texture = al_load_bitmap("texture.jpg");
image = al_load_bitmap("goldenball.png");
display = al_create_display(1280,720);
bool alive = true;
while(alive)
{
  ALLEGRO_EVENT ev;
  al_wait_for_event(event_queue, &ev);
  al_get_joystick_state(joy, &jst);
  if (ev.type == ALLEGRO_EVENT_JOYSTICK_AXIS)
  {
    if ((jst.stick[2].axis[0] == 0) && (jst.stick[2].axis[1] == 0))
      dir = STILL;
    if ((jst.stick[2].axis[0] == 0) && (jst.stick[2].axis[1] == -1))
      dir = NORTH;
    if ((jst.stick[2].axis[0] == 1) && (jst.stick[2].axis[1] == -1))
      dir = NORTHEAST;
    if ((jst.stick[2].axis[0] == 1) && (jst.stick[2].axis[1] == 0))
      dir = EAST;
    if ((jst.stick[2].axis[0] == 1) && (jst.stick[2].axis[1] == 1))
      dir = SOUTHEAST;
    if ((jst.stick[2].axis[0] == 0) && (jst.stick[2].axis[1] == 1))
      dir = SOUTH;
    if ((jst.stick[2].axis[0] == -1) && (jst.stick[2].axis[1] == 1))
      dir = SOUTHWEST;
    if ((jst.stick[2].axis[0] == -1) && (jst.stick[2].axis[1] == 0))
      dir = WEST;
    if ((jst.stick[2].axis[0] == -1) && (jst.stick[2].axis[1] == -1))
      dir = NORTHWEST;
  
  }
  if (ev.type == ALLEGRO_EVENT_JOYSTICK_BUTTON_DOWN)
  {
    if (jst.button[0])
      button = ZERO;
    if (jst.button[1])
      button = ONE;
    if (jst.button[8])
      button = EIGHT;
    if (jst.button[9])
      button = NINE;
  }
  else if (ev.type == ALLEGRO_EVENT_TIMER)
  {
    render = true;
    frames++;
    if (al_current_time() - gameTime >= 1)
    {
      gameTime = al_current_time();
      gameFPS = frames;
      frames = 0;
    }
  }
  if (render && al_is_event_queue_empty(event_queue))
  {
    render = false;
    switch (dir)
    {
      case NORTH:
        ballposY -= 3;
        break;
      case NORTHEAST:
        ballposY -= 2;
        ballposX += 2;
        break;
      case EAST:
        ballposX += 3;
        break;
      case SOUTHEAST:
        ballposY += 2;
        ballposX += 2;
        break;
      case SOUTH:
        ballposY += 3;
        break;
      case SOUTHWEST:
        ballposX -= 2;
        ballposY += 2;
        break;
      case WEST:
        ballposX -= 3;
        break;
      case NORTHWEST:
        ballposY -= 2;
        ballposX -= 2;
        break;
    }
    switch (button)
    {
      case ZERO:
        if (jst.button[0])
          image = al_load_bitmap("goldenball.png");
        break;
      case ONE:
        if (jst.button[1])
          image = al_load_bitmap("silverball.png");
        break;
      case EIGHT:
        if (jst.button[8])
        break;
      case NINE:
        if (jst.button[9])
        alive = false;
        break;
    }
 151
 152
 153// BTW I've tried this with a grid of 128x128 tiles. Both JPEG and pcx with identical results.
    al_draw_bitmap(image, ballposX, ballposY, 0);
    al_flip_display();
    al_clear_to_color(al_map_rgb(0,0,0));
    al_draw_bitmap(texture, 0,0,0);    
  }
}
al_destroy_bitmap(image);
al_destroy_timer(timer);
al_destroy_event_queue(event_queue);
return 0;
 164}

Elias

[Edit] ah, read the code wrong

Try loading the bitmaps after the display is created.

agonvs

Read it wrong? You mean me? How so?

Edgar Reynaldo

Also, don't start your timer until you enter your main loop. You're doing it before loading bitmaps and creating the display which takes time, which means timer events will pile up before you get started.

And you are also loading your golden and silver ball bitmaps every time the button is pressed and a frame is rendered. That is a big memory leak. You should only load them once and then use them later by assignment.

ALLEGRO_BITMAP* goldenball = al_load_bitmap("goldenball.png");
ALLEGRO_BITMAP* silverball = al_load_bitmap("silverball.png");
if (!goldenball || !silverball) {
   return 1;
}
ALLEGRO_BITMAP* ball = goldenball;

//...

if (ev.type == ALLEGRO_EVENT_JOYSTICK_BUTTON_DOWN) {
//
   if (button[0]) {ball = goldenball;}
   if (button[1]) {ball = silverball;}

Chris Katko

agonvs said:

Read it wrong? You mean me? How so?

No, he means what he originally wrote doesn't apply upon further inspection because he misread your code.

Thread #615215. Printed from Allegro.cc