Hi, I can add the commands to move the image in my current "main_loop ();" or I'll have to modify it?

EDIT 1: In the Code::Blocks the indentation was perfect, here messed up a little and I do not know the reason :X

1) Use integers local to main_loop and pass them to draw_character :

void draw_character(int x , int y) {
   al_draw_bitmap(char , x , y , 0);
}

void main_loop() {
   int cx = 10;
   int cy = 10;
   while (1) {
      /// blah
      draw_character(cx,cy);
   }
}

2) Use global integers

In your code, in your function , draw_character(), you lod the bitmap every time.
This isn't wise and will populate your game with memory leaks.

Its just some tips, of couse you would llike to load all characters sprites in the load function, and destroy all in the destroy function.

"I'm trying to figure out why it happens

http://i52.tinypic.com/dcwm7o.jpg"

EDIT 1: The first question I fix with "al_clear_to_color...." Thanks!

If I want to put three sprites to change as the character moves, I can create a simple animation of the three images and put to animate if the condition is = move? or has an easier way?

To animate, you will have to load all the char sprites, and you can put themm all in a vector, and access them by the index of the frame you want to show.
for example:

Thats the basic i think.
PS: std::vector is a c++ class, and you have to include: #include<vector>

I'm trying this way:

When I get back I'll see if I can fix it!

EDIT 1: I managed to animate, now I'm trying to change of
picture the every move!

You have to erase what you've drawn previously (by drawing the scene or just clearing to black for testing) then draw everything again.

  ...
  std::vector<ALLEGRO_BITMAP*> sprites;
  sprites.push_back(al_load_bitmap("Sprites/FFT1/21.bmp"));
  sprites.push_back(al_load_bitmap("Sprites/FFT1/22.bmp"));
  sprites.push_back(al_load_bitmap("Sprites/FFT1/23.bmp"));
  ...

I was trying to use the code that load the images off the "void" to separate the parts that loads the images for not have to load always, but after of the third move with the character... my program give error when I do that!

Do you have new code? I can't see how that's happening with your last paste.

My code...

and I tried to do:

*The indentation is correct?

The first code you posted is not good code. You load three images every time you draw your character.

Only use al_convert_mask_to_alpha once for each sprite that needs it, not every time you draw it.

The indentation is still not totally right :

  ALLEGRO_EVENT ev;
    sprites.push_back(al_load_bitmap("Sprites/FFT1/21.bmp"));

Both lines should be at the same indentation level :

   ALLEGRO_EVENT ev;
   sprites.push_back(al_load_bitmap("Sprites/FFT1/21.bmp"));

Hi... when I separate the images that I have to load of the my "void draw_character..." the my program hangs in the third movement!

**The indentation is correct in my code: blocks, but when I post here some parts stay crooked and i dont know why.**

Are you mixing tabs with spaces? Use one or the other, but never both.

Você está misturando as guias com os espaços? Use um ou outro, mas nunca ambos.

void update(void)
{
    ALLEGRO_KEYBOARD_STATE state;
    al_get_keyboard_state(&state);
    if (al_key_down(&state, ALLEGRO_KEY_DOWN)) {
        int lastFrame = sprites.size();
        frame+= 0.0;
        if((int)frame >= lastFrame)
        frame = 0;
    }
}

Don't mix ALLEGRO_EVENT_KEY_DOWN with ALLEGRO_KEYBOARD_STATE - use one or the other. In your case, I would just use the ALLEGRO_KEYBOARD_STATE, and check that during the timer event :

Não misture com ALLEGRO_EVENT_KEY_DOWN ALLEGRO_KEYBOARD_STATE - use um ou outro. No seu caso, gostaria apenas de usar o ALLEGRO_KEYBOARD_STATE, e verificar que, durante o evento timer :

   if (ev.type == ALLEGRO_EVENT_TIMER) {
      ALLEGRO_KEYBOARD_STATE state;
      al_get_keyboard_state(&state);
      if (al_key_down(&state , ALLEGRO_KEY_DOWN)) {
         ++frame;
         if ((size_t)frame >= sprites.size()) {
            frame -= (float)(sprites.size() - 1);
         }
      }
      else if (al_key_down(&state , ALLEGRO_KEY_UP)) {
         ///
      }
      else if (...) {}
   }

The reason your code is failing now is because you only correct an out of bounds 'frame' variable when the timer ticks. You could have an ALLEGRO_EVENT_KEY_DOWN event just after a timer tick incrementing frames to 3.0 and then access (int)3.0 in your sprites array, which is out of bounds.

O motivo é o seu código não é agora, porque você só corrigir uma variável fora do 'frame' limites quando o cronômetro carrapatos. Você poderia ter um evento ALLEGRO_EVENT_KEY_DOWN logo após um timer tick quadros incremento de 3,0 e, em seguida, (int) Access 3.0 em sua matriz sprites, que está fora dos limites.

Hi... Firstly, thank you! I fix the "update" as you show me , I'm thinking in a solution for animation now, when I'm moving my character... still give error in the third movement, I'm to look for what you said about "frame".

EDIT1: When this party is separated of the function that draw give the error, but a time loaded the images should be stored without the need to reload... I think!

...
    sprites.push_back(al_load_bitmap("Sprites/FFT1/21.bmp"));
    sprites.push_back(al_load_bitmap("Sprites/FFT1/22.bmp"));
    sprites.push_back(al_load_bitmap("Sprites/FFT1/23.bmp"));
...

EDIT 2: I think this is wrong, but it's the only way I could make it work.

Don't load during update. Only load your images once.

            ++frame;
            if ((size_t)frame >= sprites.size()) {
                frame -= (float)(sprites.size() - 1);
            }

Let me make a correction - I gave you the wrong code earlier - it should be :

   frame += inc;// inc must be in the range [0.0 , sprites.size() )
   if (frame >= (float)sprites.size()) {
      frame -= (float)sprites.size();
   }

The 'inc' variable is a floating point increment to let you change the frame by less than one. Set it to whatever works for you.

Hi, I updated the code how you said, but still give the same error.

http://i53.tinypic.com/2irkkrs.jpg

I can't keep guessing what is wrong. Either post your latest full code, or learn how to use a debugger.

You're probably accessing an element outside the boundaries of the vector again. Check for that first.

My last code:

/* Test */

int cx = 10;
int cy = 10;
int inc = 0.0;
float frame = 0;
std::vector<ALLEGRO_BITMAP*> sprites;

Draw Character:

void draw_character()
{
    al_convert_mask_to_alpha(sprites[(int)frame], (al_map_rgb(255, 0, 255)));
    al_draw_bitmap(sprites[(int)frame], cx, cy, 0);

}

Update:

Main loop:

Main:

      case ALLEGRO_EVENT_KEY_DOWN:
          if (ev.keyboard.keycode == ALLEGRO_KEY_UP) {
              cx -= 5;
              cy -= 5;
              frame++;
              break;
          }

You're still mixing ALLEGRO_EVENT_KEY_DOWN with al_key_down, and you're not checking the bounds of your sprites vector when you modify 'frame'.

Also, you're still converting your bitmap every time you draw it - only convert it once when you load it.

Você ainda está com a mistura ALLEGRO_EVENT_KEY_DOWN al_key_down, e você não está verificando os limites do seu vetor sprites quando você modifica 'frame'.

Além disso, você ainda converter o bitmap de cada vez que desenhá-la - só convertê-lo uma vez quando você carregá-lo.

It might help simplify things if you tried out my basic animation class :
animation class

Then your setup code would look like this :

And you would use it like this :

   if (ev.type == ALLEGRO_EVENT_TIMER) {
      anime.AdvanceFrameTime(1.0/FPS);
      redraw = true;
   }

//  ......

   if (redraw) {
      clear_to_color(al_map_rgb(0,0,0));
      anime.Draw(character_x , character_y);
      al_flip_display();
      redraw = false;
   }

Well... I'm trying to use the animation class but I no have slightest idea where will every thing.
The setup code will inside of main?

Yes, the setup code would go in main, after you initialize allegro and the image addon and create a display.

Hi, again
http://www.allegro.cc/forums/thread/607529/920236#target

...
      try {
         frames = new ALLEGRO_BITMAP*[framecount];
      }
      catch {
         frames = 0;
         return false;
      }
      num_frames = framecount;
   }
...

That would be "catch(Animation) {..." ?

No, it should have been :

      try {
         frames = new ALLEGRO_BITMAP*[framecount];
      }
      catch (...) {
         frames = 0;
         return false;
      }
      num_frames = framecount;

Theoretically it would be something more or less so?

Not quite - redraw goes outside of the event loop, and you shouldn't draw during your logic (you do it in the ALLEGRO_EVENT_KEY_CHAR check). Always separate logic from drawing.

Não é bem assim - redesenhar vai para fora do ciclo de eventos, e você não deve chamar durante a sua lógica (que fazê-lo na seleção ALLEGRO_EVENT_KEY_CHAR). Sempre lógica separada do desenho.

Now is almost 04:00 in Brazil and I'm totally lost, sleep will not let me think straight, I go to bed and when I wake up I try to solve it, thanks a lot!

Let me know how it goes.

Here, my last code:

    while (!quit) {
        if(redraw) {
            al_clear_to_color(al_map_rgb(0,0,0));
            anime.Draw(cx , cy);
            break;
        }
        al_flip_display();
        redraw = false;
    }

Without that break statement, you will be stuck in that loop forever, because quit is always false. With the break statement, you only clear the screen and draw the animation, but don't flip the display.

Your drawing code goes inside your game loop.

@@"

edit1: So I do not need to do a "if(redraw).." ?

You don't have to use a redraw flag, but it makes it clearer. Also, it probably looks better if you only redraw when the timer goes off.

The part of animation is correct?

Edit 1: I can to put the "event timer" and then redraw or is better I to put the timer in a "case" in the end?

Game loop :

Look: