[A5] Timer & Keyboard issues

[A5] Timer & Keyboard issues

Joined

Member #12,691

March 2011

Hi to all.
I am new to the forum and i'm trying to make my very first game with Allegro 5.
As it is new i can't find so much examples on the internet but with the help of the wiki i made something.
This "simple" program works like a charm exept for the fact that the scrolling of the choices in the menu is very slow. Another problem is that if i draw a filled circle at line 141, when i start the program the whole menu goes away and i have to go up and down with the keyboard 3-4 times and the menu comes back again.

Can you help me with the debugging?
Thanks in advance.

#SelectExpand
   1#include <stdio.h>
   2#include <allegro5/allegro.h>
   3#include <allegro5/allegro_font.h>
   4#include <allegro5/allegro_ttf.h>
   5#include <allegro5/allegro_audio.h>
   6#include <allegro5/allegro_acodec.h>
   7#include <allegro5/allegro_image.h>
   8#include <allegro5/allegro_primitives.h>
   9#define SCREEN_W 800
  10#define SCREEN_H 600
  11#define FPS 60
  12
  13int main()
  14{
  if(!al_init())
  {
      fprintf(stderr, "Failed to initialize Allegro.\n");
      system("pause");
      return -1;
  }
  if(!al_install_keyboard())
  {
      fprintf(stderr, "Failed to initialize the keyboard!\n");
      system("pause");
      return -1;
  }
  al_init_font_addon();
  if (!al_init_ttf_addon())
  {
      fprintf(stderr, "Failed to initialize the TTF addon!\n");
      system("pause");
      return -1;
  }
  if (!al_install_audio())
  {
      fprintf(stderr, "Failed to initialize the audio addon!\n");
      system("pause");
      return -1;
  }
  if (!al_init_acodec_addon())
  {
      fprintf(stderr, "Failed to initialize the acodec addon!\n");
      system("pause");
      return -1;
  }
  if (!al_reserve_samples(2))
  {
      fprintf(stderr, "Failed to reserve the samples!\n");
      system("pause");
      return -1;
  }
  if (!al_init_image_addon())
  {
      fprintf(stderr, "Failed to initialize image addon!\n");
      system("pause");
      return -1;
  }
  if(!al_init_primitives_addon())
  {
    fprintf(stderr, "Failed to initialize the primitives addon!\n");
    system("pause");
    return -1;
  }
  64
  ALLEGRO_FONT *font = al_load_ttf_font("pirulen.ttf",96,0 ); // load the font | Font from Larabie Free Fonts
  ALLEGRO_FONT *font2 = al_load_ttf_font("pirulen.ttf", 30,0 ); // load the font | Font from Larabie Free Fonts
  ALLEGRO_FONT *font3 = al_load_ttf_font("pirulen.ttf", 12,0 ); // load the font | Font from Larabie Free Fonts
  ALLEGRO_DISPLAY *display = NULL;
  ALLEGRO_EVENT_QUEUE *event_queue = NULL;
  ALLEGRO_SAMPLE *sample = NULL;
  ALLEGRO_BITMAP *sfondo = NULL;
  ALLEGRO_TIMER *timer = NULL;
  73
  timer = al_create_timer(1.0 / FPS);
  75
  bool redraw = true;
  77
  bool doexit = false;
  bool inizia = false;//false = iniziano i rossi; true = iniziano i gialli
  bool suoni = false;//true = attivati; false = spenti
  int index = 0;//indica su quale voce del menu sono posizionato inizialmente
  82
  //carico lo sfondo
  sfondo = al_load_bitmap("background_t_p.jpg");
  85
  //creo il display principale
  display = al_create_display(SCREEN_W,SCREEN_H);
  88
  //carico la musica
  sample = al_load_sample("around-the-world.wav");
  91
  if (!sample)
  {
      fprintf(stderr, "Failed to load sample.\n");
      system("pause");
      return -1;
  }
  98
  if (!display)
  {
      fprintf(stderr, "Failed to create display.\n");
      system("pause");
      return -1;
  }
 105
  event_queue = al_create_event_queue();
 107
  if(!event_queue)
  {
    fprintf(stderr, "failed to create event_queue!\n");
    al_destroy_display(display);
    return -1;
  }
 114
  al_register_event_source(event_queue, al_get_timer_event_source(timer));
  al_register_event_source(event_queue, al_get_keyboard_event_source());
  al_register_event_source(event_queue, al_get_display_event_source(display));
 118
  al_start_timer(timer);
 120
  while(!doexit)
  {
    ALLEGRO_EVENT ev;
    al_wait_for_event(event_queue, &ev);
 125
    if ((ev.type == ALLEGRO_EVENT_TIMER)&&(redraw))
    {
        printf("entro");
        redraw = false;
 130
        al_draw_bitmap(sfondo, 0, 0, 0);
 132
        al_draw_text(font, al_map_rgb(204,0,0), SCREEN_W/2, (SCREEN_H/8)-10,ALLEGRO_ALIGN_CENTRE, "FORZA 4");// draw the text
 134
        if (index==0) al_draw_text(font2, al_map_rgb(255,0,0), SCREEN_W/2-20, (SCREEN_H/2)-60,ALLEGRO_ALIGN_CENTRE, "GIOCA");// draw the text
        else al_draw_text(font2, al_map_rgb(0,0,0), SCREEN_W/2-20, (SCREEN_H/2)-60,ALLEGRO_ALIGN_CENTRE, "GIOCA");// draw the text
 137
        if (index==1) al_draw_text(font2, al_map_rgb(255,0,0), SCREEN_W/2-10, (SCREEN_H/2)-20,ALLEGRO_ALIGN_CENTRE, "INIZIA <    >");// draw the text
        else al_draw_text(font2, al_map_rgb(0,0,0), SCREEN_W/2-10, (SCREEN_H/2)-20,ALLEGRO_ALIGN_CENTRE, "INIZIA <    >");// draw the text
 140
        if (inizia) al_draw_text(font2, al_map_rgb(255,255,0), SCREEN_W/2+55, (SCREEN_H/2)-20,ALLEGRO_ALIGN_CENTRE, "O");// draw the text
        else al_draw_text(font2, al_map_rgb(255,0,0), SCREEN_W/2+55, (SCREEN_H/2)-20,ALLEGRO_ALIGN_CENTRE, "O");// draw the text
 143
        if (index==2)
        {
            al_draw_text(font2, al_map_rgb(255,0,0), SCREEN_W/2-70, (SCREEN_H/2)+20,ALLEGRO_ALIGN_CENTRE, "SUONI");// draw the text
 147
            if (suoni) al_draw_text(font2, al_map_rgb(255,0,0), SCREEN_W/2+50, (SCREEN_H/2)+20,ALLEGRO_ALIGN_CENTRE, "<ON>");// draw the text
            else al_draw_text(font2, al_map_rgb(255,0,0), SCREEN_W/2+50, (SCREEN_H/2)+20,ALLEGRO_ALIGN_CENTRE, "<OFF>");// draw the text
        }
        else
        {
            al_draw_text(font2, al_map_rgb(0,0,0), SCREEN_W/2-70, (SCREEN_H/2)+20,ALLEGRO_ALIGN_CENTRE, "SUONI");// draw the text
 154
            if (suoni) al_draw_text(font2, al_map_rgb(0,0,0), SCREEN_W/2+50, (SCREEN_H/2)+20,ALLEGRO_ALIGN_CENTRE, "<ON>");// draw the text
            else al_draw_text(font2, al_map_rgb(0,0,0), SCREEN_W/2+50, (SCREEN_H/2)+20,ALLEGRO_ALIGN_CENTRE, "<OFF>");// draw the text
        }
 158
        if (index==3) al_draw_text(font2, al_map_rgb(255,0,0), SCREEN_W/2-20, (SCREEN_H/2)+100,ALLEGRO_ALIGN_CENTRE, "ESCI");// draw the text
        else al_draw_text(font2, al_map_rgb(0,0,0), SCREEN_W/2-20, (SCREEN_H/2)+100,ALLEGRO_ALIGN_CENTRE, "ESCI");// draw the text
 161
        al_draw_text(font3, al_map_rgb(0,0,0), SCREEN_W-100, SCREEN_H-50,ALLEGRO_ALIGN_CENTRE, "by Joined ® 2011");// draw the text
 163
        al_flip_display();
    }
    else if(ev.type == ALLEGRO_EVENT_DISPLAY_CLOSE)
       break;
    else if (ev.type == ALLEGRO_EVENT_KEY_DOWN)
    {
        switch(ev.keyboard.keycode)
        {
          case ALLEGRO_KEY_UP:
          {
             if (index>0)
             {
                 index--;
                 redraw = true;
             }
             break;
          }
          case ALLEGRO_KEY_DOWN:
          {
             if (index<3)
             {
                 index++;
                 redraw = true;
             }
             break;
          }
          case ALLEGRO_KEY_LEFT:
          {
             if (index == 1)
             {
                 if (inizia == true)
                 {
                     inizia = false;
                     redraw = true;
                 }
             }
             else if (index == 2)
             {
                 if (suoni == true)
                 {
                     suoni = false;
                     redraw = true;
                     al_stop_samples();
                 }
             }
             break;
          }
          case ALLEGRO_KEY_RIGHT:
          {
             if (index == 1)
             {
                 if (inizia == false)
                 {
                     inizia = true;
                     redraw = true;
                 }
             }
             else if (index == 2)
             {
                 if (suoni == false)
                 {
                     suoni = true;
                     redraw = true;
                     al_play_sample(sample, 1.0, 0.0, 1.0, ALLEGRO_PLAYMODE_LOOP, NULL);
                 }
             }
             break;
          }
          case ALLEGRO_KEY_ENTER:
          {
             if (index == 3)
              doexit=true;
             break;
          }
       }
    }
  }
 241
  al_destroy_display(display);
  al_uninstall_audio();
  al_destroy_timer(timer);
  al_destroy_bitmap(sfondo);
  al_destroy_event_queue(event_queue);
 247
  return 0;
 249}

kenmasters1976

Member #8,794

July 2007

As for your program being slow I suggest you move the addon initialization code (like primitives and font addons) and put it after display creation. That should help.

Peter Wang

Member #23

April 2000

As a clarification of the previous comment, it's not so much that you should move the initialisation code after creating the display, but you should load bitmaps and fonts after creating the display. Otherwise the bitmaps will be created as 'memory' bitmaps, which are always drawn with relatively slow software routines, instead of 'video' bitmaps, which can take advantage of hardware rendering.

Edgar Reynaldo

Major Reynaldo

May 2007

Your structure is like this (psuedocode) :

while (!quit) {
   wait_for_event();
   if (is_timer_event() && redraw) {
      Redraw();
      redraw = false;
   }
   // logic
   if (stuff_changes) {redraw = true;}
}

The problem with it is that redraw may be set to true by your logic, but it has to wait until the next event fires to perform the Redraw(). You won't have to wait very long because of the timer events, but it's still a little funky.

A better layout would be :

while (!quit) {
   while (1) {
      wait_for_event();
      if (stuff_changes) {redraw = true;}
      if (is_timer_event()) {update_logic();redraw = true;}
      if (!events_in_queue) {break;}
   }
   if (redraw) {
      Redraw();
      redraw = false;
   }
}

Eagle and Allegro 5 binaries | Older Allegro 4 and 5 binaries | Allegro 5 compile guide
King Piccolo will make it so Then I could just flip it with some sweet, sweet matrix math. In either case, I’ll ride this wagon until the wheels fall off. Thanks to those that keep it running