Is some instruction in c++ or library allegro that if somebody press Esc in function1() or in function2() then he go to important() - because I am doing this now as the example:

Did you compile with -f-important-esc-function in the compiler flags?

Firstly, you don't need to use readkey() to access the key[] array.

And why can't you just use:

if (key[KEY_ESC]) important();

edit:

Steve: Meanie.

I must use readkey() - without that my program wrong run, and I wrote bad example - It should be:

If somebody press "Esc" - then function1() should will finish and should will go to the start point - important()

If you need to use readkey() without checking what it returns, then your program probably isn't written properly. The code you have posted makes very little sense, so it's hard to tell exactly what you're trying to achieve.

But what you probably need in function1() is:

if (key[KEY_ESC]) {
    esc_was_pressed = 1;
    return;
}

And then in main():

if (esc_was_pressed) important();

Another alternative is just having the following in function1():

if (key[KEY_ESC]) {
    important();
    return;
}

Without knowing more about the program flow or what you're trying to achieve, I don't know which would be better.

Albin found: "code of horrors"

if (key[KEY_ESC]) {
    important();
    return;
}

after that program go to meni and exit - I think this is fault "return" in above code :/

Then post some REAL code that actually makes sense!!!

And please start using function names that describes what the function does. I have absolutely no idea what the hell it is you are trying to do, which makes it near-impossible to help you.

I can't give you all my code I give you a little part of my code - I had to change names to english because first they were in polish because I am Pole and I deleted 99% my code so maybe something is wrong :

If I was going to implement a menu, I'd first define an enumeration to declare the states the program can be in. eg:

enum { MENU, GAME, INSTRUCTIONS, AUTHORS, EXIT };

Then I'd add a loop that checks the state. eg:

Then, in your menu function, I'd change the state depending on what the user selects. eg:

The state will also need to be set to MENU when the run_game(), show_instruction(), and show_authors() functions exit.

not, I want that when somebody press ESC then he go to menu (first player choose option from menu for example "new game", and then he can play, but when he then press ESC he go to menu) and you only change fragment of my code which doing that same thing

I'm not going to write your program for you! That was just an EXAMPLE of another way of doing something that I EXPECTED YOU to be able to modify to your needs.

On that note: I quit. You're on your own.

And I don't want that you write code for me, my code was only example so you can't write for me. You idea is wrong.

You should not mix input from the key[] array and readkey(), they serve different purposes.
Rather than calling readkey() and then checking the key[] array, you should use the return value of readkey(), eg if ((readkey() >> 8) == KEY_ESC). Be sure to check the manual.

hm........... and everywhere: if ((readkey() >> 8)== KEY_ENTER)&&index!=3) this not compile and it is longer - this : if (key[KEY_DOWN]&&index!=3) is better and shorter

Count your closing parentheses...

It is shorter (but the readkey() version can be written more elegantly) and actually worse, depending on the situation.
Do you want to respond to the player having pressed the key? If yes, use readkey().
Do you want to respond to the key being held down right now? Then use key[].
Do not use readkey() in the second situation and do not use key[] in the first. Somewhere, sometime something will go wrong somehow.

I must use readkey() and key[] because ONLY then program good run. Without readkey at present showed option in menu when I press "->" it jumps over about several of position to lower part and it should about 1 position. And when I have readkey() without key[] - when I pressing another keys than "->" and "<-" postions change and I can't go to game when I am on 1. position in menu and press ENTER (1. position in my menu is "new game"). And when I use readkey() and key[] - EVERYTHING IS PERFECT:
for example - this runs very well because I have readkey() and key[] :

Did you even bother to read (and try to understand) what I wrote?

You're doing something wrong then. Post a sample of the code that doesn't run properly.

ok Ever so what is wrong - could you take attachment - then you will see that if I delete readkey() program runs wrong - you can press only KEY_UP, KEY_DOWN or ENTER.

Evertt

No, I can't. I can't open rar archives from this machine.

Make sure that if you do use readkey() instead of key[] you call it only once.

ok - now it is "zip" file

As I say without "readkey()" - everything is wrong.

a b: You are just going to have to trust us when we say is it wrong. Listen to the advice you get instead of trying to fight it.

I want trust you but when I delete readkey() - everything is wrong

I had some trouble with key[] and readkey also with my first project. The point is, you have to use allegro's functions for key input the way they are designed, which is a little different from what you do. Other people here can explain this very much better than I can, but look into do_dialog() and similar functions to make your UI. It has its root somewhere in the parallel existence of a keyboard buffer, which is handled by routines like readkey(), and the key[] array, which allows immediate access to the current key states.

For my needs, I'll do my next game with guichan, which is an excellent GUI library, but still in heavy development (working its way to version 0.6).

I understand but why must I use readkey() and key[] in below code (also in attachment) ? Do somebdoy know ? If I use only readkey() or only key[] program runs wrong:

a b, use a boolean value to tell whenever a key has been pressed or not, and when pressing a key check if a key has already been pressed and if, to not execute the action again, i think this is your problem right?

I did not have time to check this, just understand the keyhasbeenpressed part. .

bool keyhasbeenpressed; << this.

int meni()
{
while(go!=0)
{
clear_to_color(bufor,makecol(255,255,255));
blit(menu,bufor,0,0,0,0,menu->w,menu->h);

masked_blit(frame,bufor, 0,0, 80,table[index],frame->w,frame->h);

blit(bufor,screen,0,0,0,0,SCREEN_W,SCREEN_H);
readkey(); << remove this.

if (key[KEY_DOWN]&&index!=3&&keyhasbeenpressed==false)
{
keyhasbeenpressed=true;
index++;
}
else if (key[KEY_DOWN]&&index==3&&keyhasbeenpressed==false)
{
keyhasbeenpressed=true;
index=0;
}
else if (key[KEY_UP]&&index!=0&&keyhasbeenpressed==false)
{
keyhasbeenpressed=true;
index--;
}
else if (key[KEY_UP]&&index==0&&keyhasbeenpressed==false)
{
keyhasbeenpressed=true;
index=3;
}
else if(key[KEY_ENTER]&&index==3&&keyhasbeenpressed==false)
{return 0;keyhasbeenpressed=true;}
else if(key[KEY_ENTER]&&index==0&&keyhasbeenpressed==false)
{start();keyhasbeenpressed=true;}
else if(key[KEY_ENTER]&&index==1&&keyhasbeenpressed==false)
{instruction();keyhasbeenpressed=true;}
else if(key[KEY_ENTER]&&index==2&&keyhasbeenpressed==false)
{autor();keyhasbeenpressed=true;}

if(!key[KEY_UP]&&!key[KEY_ENTER]&&!key[KEY_DOWN])keyhasbeenpressed=false;
}
}

Use allegro dialogs to do your menu. They have all this UP DOWN stuff already implemented, so you don't need to worry about all that. You also get your callback functions to react on button presses, and you can control your menu with keyboard and mouse. Don't try to re-do all this, it's already contained in allegro.

heheh no

This is my simple menu and frame- when you press KEY_DOWn/UP frame is going down/up, and if you press ENTER you go on another map, but without readkey() it is going strange:
http://img146.imageshack.us/img146/2143/menusx6.png
http://img462.imageshack.us/img462/1453/bvu8.jpg

i can't really tell what's so wrong with the pictures as i don't know whats right... but are you sure that you think what i posted was wrong??

Here's the proper way of doing the code you posted with readkey() alone (which is what you should be using instead of key[]):

   int k = readkey()>>8;
 
   if (k==KEY_DOWN) {
      ...
   } else if (k == KEY_UP]) {
      ...
   } else if (k == KEY_ENTER]) {
      ...
   }

As others have said, using a DIALOG is probably a better way to do this.

Evert now it is good running - thanks - but why in readkey()>>8; is ">>8" ?

And what is "dialog" - I don't know english very well so mayby I don't understand - in my dictionary dialog=conversation ?

Read the Allegro manual entry for readkey(). Check your favourite C book for information on the bit-shift operators << and >>.

Normally short for "dialog box", the things that pop up on your computer screen where you press "Ok" or "Cancel". In this specific case a dialog object, which is part of Allegro's GUI system. Again, see the manual.

If I good remember:
a= b >> 3

a 0000 1111
b 0000 0001

but "readkey()>>8" - is strange........

and how can we good write this:

readkey();
        while (!key[KEY_ESC]&&!key[KEY_ENTER])
        {
          readkey();
        }

Read the manual for the keyboard section.. please.

nooooooo - your idea with readkey()>>8 is bad I can't declare int k = readkey()>>8; as a global variable because frame wrong moves:
it is also in attachment

I can't write readkey()>>8 in each place in my program because it is going wrong:

Everything is wrong, previously how I wrote readkey() and key[] - everything was okey, it was no problems !! I am angry - sorry, but I am angry.......
Manual don't like me.......

Oh how right he was.

LennyLen if you are so intelligent, say me why readkey()>>8 in two programs UP wrong runs.

Do as Evert (and others) said. READ THE MANUAL.

I readed but it isn't simply when I can't good english........

If you can't understand the english well enough then ok. But if you can, then you don't seem to be trying to understand or think as to why it is not working based on what you read.

edit: nvm

Now I use English translator 3 but it isn't very good.

I gave more parentheses, because I thought that this can their fault

Do it like this:

k = readkey() >> 8;

if ((k == KEY_DOWN) && index!=3)
         {
               
               index++;         
         } 
if ((k == KEY_DOWN) && index == 3)
         {
               
               index=0;         
         }

Etc..

When you call readkey it waits for a key to be pressed, if you call it again it waits for another key to be pressed. So you only want to call it once, then check which key was pressed and deal with all those conditions.

Does that help?

We have in menu:

"new game" index 0

"instructions" index 1

"autor" index 2

"end game" index 3

Richard your idea is also wrong - this see that your idea shoud be good but isn't - very strange

the problem is that I can't do int k = readkey()>>8; global variable

I'm going to recommend that you look at my Game Creation Articles and try to understand how to build a working game. Most of them have more code than English and I think they will help you once you understand them.

http://www.reflectedgames.com/create.html

Richard but I don't understand why I can't do int k = readkey()>>8; global variable

Think about it..

You can't call a function like that. And even if you could, it would only work ONCE.

yyyy ?? I have this program in attachment .... and I do not know about what already completely it walks

if you put k = readkey() >> 8 outside of any functions, then how are you going to call it? It is not called automatically, and so k will not be correct.

You need to call it each time you want to read 1 new key.

You're asking why you can't you do int k = readkey()>>8;? Seriously? I don't think bad English or the manual are the problems here...

No, your implementation is. I'm not going to spoon-feed you, but I will give you one more hint (even if Richard already gave the exact same advice above): don't declare it as a global variable; you want to keep calling readkey, don't you?
Think before you type.

Aside, I realise this is probably a reflection of your English not being very good, but I will tell you this: I have been programming for more than fifteen years, and programming in C with Allegro for nine years. I do not take kindly to a novice user getting angry and calling code examples I give "bad" because he fails to implement it properly. I seriously suggest that you invest some time on that attitude.

Feel free to ask any question related to the manual. Feel free todirectly ask the friendly Polish regulars for advice is Polish if you need to.

1. This is a bit uncomfortable declaring in every function this - do you judge ?

2. And when I want that programme will stop as long as I will press ANY(WHATEVER) key then I use only: "readkey()" yes ??

2. I change:

  readkey();
        while (!key[KEY_ESC]&&!key[KEY_ENTER])
        {
          readkey();
        }

on that:

 int k = readkey()>>8;  
        while (k!=KEY_ESC&&k!=KEY_ENTER)
        {
          k = readkey()>>8; 
        }

okay ??

Please think about what we have said, and what you have read in the manual. That will answer your question.. Ask some polish people here if the english is too difficult.

Another hint: you only need to make a very small change to the readkey() call in your original code.

I'm not quite sure what you're question is, but technically if you wanted k to be a global variable, you could declare int k; globally and the do k = readkey()>>8; when you need to (albeit I don't see any point in making it into a global variable, and using k as a name for a global variable is a horrible idea).

Evert: "you only need to make a very small change to the readkey() call in your original code." - what do you mean ??

I'm not going to explain more than that. I cannot say more without writing down the actual code, and I want you to do that yourself.
Read what has been written in this thread and try to understand it and think about it.

int k = readkey()>>8;
while (k!=KEY_ESC&&k!=KEY_ENTER)
{
k = readkey()>>8;
}

this code runs very very well - so....... ? ? ?

I'm completely lost now.
If the code you have now works, what is your question?

EDIT: even if your English is not very good, you can explain a bit better what you mean.

He's asking you if he changed the right part.

I don't know. Last I knew, he wanted to do something like the pseudo-code I posted here.
Why he would need a while loop at all is beyond me, because I don't think I understand what he's asking for.

It's simple. He has a menu. He wants the UP and DOWN keys to change the current selection, and ENTER to accept it. [I'm assuming] ESC exits the menu.

yes, yes LennyLen

But now I must give question - is it possible because I don't know: Now I have 6 functions, during the game (not in menu but in game) player will press keys many times - and I must write every time when player should press key if this key isn't ESC because if it is ESC then game should exit. Is it possible write as a global that if player press ESC game exit and I can't check it in each place when player should press some key for example: "enter" or "->" or another ??

Richard Phipps already answered that question.

Do it like this:

bool gameRunning = true;
...
int main()
{
  //Initialization stuff goes here
  while(gameRunning)
  {
    //Main part of your game

    if(k == KEY_ESC) gameRunning = false;
  }
  //De-init stuff goes here
  return 0;
}

That is if you want the game to exit when ESC is pressed.

Kauhiz I made that but when I press ESC - nothing do:

Did you even look at the code you wrote!? Obviously it's not going to quit, because you have while(go!=0) in your meni() function, and go isn't set to anything inside that loop . So you have an infinite loop, and it never gets to the point where it'd check to see if it should quit...

This is basic stuff that has nothing to do with allegro! If you expect to get any help on this forum in the future you have to at least TRY to solve your problems yourself, because otherwise people will get frustrated and just give up! Just like Evert said, you don't need a while-loop there! Do you just hit some random keys on your keyboard and then compile to see if it works?

I try made what you said and I change my code ...... Firstly it was:

This loop is infinite on purpose. It must be infinite (only when somebody is in index 3 and press enter will be exit and now I want also make that if somebody press ESC program will be exit ) so just therefore I ask is it possible that if somebody press ESC program will exit. This program is also example because I have bigger program and more functions, for example I have function (I will must change there readkey() and key[]):