Use a single state variable. In your if (redraw) {/*...*/}, check for the state there, and then call map_edit (which would be more appropriately named draw_map_editor_intro).

Use uma única variável de estado. Em sua if (redraw) {/*...*/}, para verificar o estado lá, e depois chamar map_edit (o que seria mais apropriadamente chamado draw_map_editor_intro).

Please start using consistent indentation in your code. It is really hard to read without it.

Por favor, começar a usar o recuo consistente em seu código. É realmente difícil de ler sem ele.

Don't check for the event type inside a block that has already checked the event type. You did this in your code, and the second if will never be executed :

Não verificar o tipo de evento dentro de um bloco que já verifiquei o tipo de evento. Você fez isso no seu código, eo segundo, se nunca vai ser executado:

if (ev.type == ALLEGRO_EVENT_KEY_CHAR) {
   if (ev.type == ALLEGRO_EVENT_KEY_DOWN) {

   }
}

You're missing a break statement in case 0 of your switch (state) code :

Você está perdendo uma instrução break no caso de 0 a chave do código (estado):

                       case 0 :
                          state = STARTNEWGAME;
                          printf("teste...");
                          /// MISSING BREAK STATEMENT HERE!

I reformatted and fixed your code. I took out the bad if statements, added break statements where needed, and fixed all the indentation problems. If I have to reformat your code again, I will no longer help you. So, here it is :

Eu reformatado e fixa seu código. Tirei o ruim se as declarações, acrescentou quebrar declarações, quando necessário, e corrigiu todos os problemas de recuo. Se eu tiver que formatar o seu código novamente, eu não vou mais te ajudar. Então, aqui está :

Your code is getting really complex because of all the states it has to handle. I would suggest moving your code into a class. It will allow you to separate handling of different states in your code very easily :

Seu código está ficando muito complexo, porque de todos os estados tem que segurar. Eu sugiro mover o seu código em uma classe. Ele permitirá que você separe tratamento de estados diferentes no seu código muito facilmente:

See how much cleaner that is?

Veja como muito mais limpo que é isso?

You have to define the functions declared in the class, or you will get undefined references during linking.

Você tem que definir as funções declaradas na classe, ou você vai ter referências indefinidas durante a ligação.

As an example :

void DrawPressKey() {
   al_clear_to_color(al_map_rgb(0,0,0));
   al_draw_bitmap(remakelogo, 150, 150, 0);
   al_draw_bitmap(credits, 120, 451, 0);
   myflasher.Draw();
   al_flip_display();
}

So now you will need to add your ALLEGRO_BITMAP*'s to your class, as well as your Flasher. That means you will need to load and free the ALLEGRO_BITMAP*'s in your class as well. You might want to load / free the ALLEGRO_BITMAP*'s based on your GameState too.

Então, agora você terá que adicionar o teu * ALLEGRO_BITMAP "para sua classe, bem como o seu pisca-pisca. Isso significa que você vai precisar carregar e livre é a * ALLEGRO_BITMAP "em sua classe também. Você pode querer carregar / grátis * ALLEGRO_BITMAP é baseado em seu estado de jogo também.

No, just add a Flasher object to your Game class.

Não, basta adicionar um objeto de pisca-pisca para a classe Game.

Did you load the bitmap?

The only reason al_draw_bitmap would fail is if you passed it NULL, or if you passed it an ALLEGRO_BITMAP* that was destroyed with al_destroy_bitmap.

Let's see (all) your code again.

O al_draw_bitmap única razão pela qual não é se você passou NULL, ou se você passou um * ALLEGRO_BITMAP que foi destruído com al_destroy_bitmap.

Vamos ver (tudo) o código novamente.

The problem is that you define your Flasher myflasher before you load your ALLEGRO_BITMAP* that you pass to it. So you are passing an uninitialized ALLEGRO_BITMAP* to the Flasher constructor.

Solution 1 :
Don't create your myflasher object until after you initialize Allegro and load your ALLEGRO_BITMAP*. Then you would have to pass a reference to your myflasher object to your game object, and so you would have to add a Flasher* to your Game class.

Solution 2 :
Add a Flasher object to your Game class and initialize it in the Game class constructor initialization list (Don't create your game object until after Allegro is initialized and the flasher's ALLEGRO_BITMAP* is loaded).

Solution 3 :
Add a Flasher* to your Game class. Create your Flasher myflasher object after initializing Allegro and loading the necessary ALLEGRO_BITMAP. Set the Game::Flasher* to the address of your myflasher object.

O problema é que você define o seu myflasher Flasher antes de carregar o * ALLEGRO_BITMAP que você passar para ele. Então você está passando uma * ALLEGRO_BITMAP inicializada para o construtor Flasher.

Solução 1:
Não crie o seu objeto myflasher até depois de inicializar Allegro e carregar seu ALLEGRO_BITMAP *. Então você teria que passar uma referência ao seu objeto myflasher para o objeto do jogo, e assim você teria que adicionar um * pisca-pisca para a classe Game.

Solução 2:
Adicione um objeto Flasher para sua classe Game e inicializá-lo na lista de inicialização do construtor da classe Game (Não crie o seu objeto de jogo até depois Allegro é inicializado e * o pisca-pisca da ALLEGRO_BITMAP é carregado).

Solução 3:
Adicionar Flasher * a sua classe Game. Crie seu objeto myflasher Flasher Allegro após inicializar e carregar o ALLEGRO_BITMAP necessário. Defina o jogo:: Flasher * para o endereço do seu objeto myflasher.

An example of option 3 :

class Game {
private :
   Flasher* flasher;
   GameState state;
   bool redraw;
   bool quit;

public :
   Game() : flasher(0) , state(PRESSKEY) , redraw(true) , quit(false) {}

   void SetFlasher(Flasher* f) {flasher = f;}

///..... Same as before
};

And to use it :

// in main

// initialize allegro, load bitmaps

Flasher myflasher(bmp , 200 , 200 , 0.5);
Game game;
game.SetFlasher(&myflasher);

///..... Same as before

Huh? That's odd. Show all of your code again.

Actually, I bet the problem is inside the definition of your Game class methods. You need to refer to it as 'flasher' now, not 'myflasher', because the member is named 'flasher' :

   void CheckPressKeyInput(ALLEGRO_EVENT &ev) {
      if (ev.type == ALLEGRO_EVENT_TIMER) {
         flasher->Update(1.0/FPS);
      } else if (ev.type == ALLEGRO_EVENT_KEY_DOWN) {
        if (ev.keyboard.keycode == ALLEGRO_KEY_ENTER);
         state = SHOWSTART;
       }
    }

   void DrawPressKey() {
      al_clear_to_color(al_map_rgb(0,0,0));
      al_draw_bitmap(remakelogo, 150, 150, 0);
      al_draw_bitmap(credits, 120, 451, 0);
      flasher->Draw();
      al_flip_display();
   }

Yeah, that was the problem - you're referring to it as 'myflasher.' instead of 'flasher->'. See my edit in the last post.