Legacy GM (SOLVED) Why wont this work

[JDSTIGER]

Having some problems with the alarms..:

//return to menu in 3 seconds when game is over
if (game_over) {
    
   alarm[4] = 120;
    
}

But the code right before it works:

 if (!game_over) {
        alarm[2] = 90;
    }
}

I could use any other code in place of the alarm & it works fine..:

if (game_over) {
    
 room_goto (rm_menu)
    
}

 

[Silver_Mantis]

You should use room_speed for your counter, it's much much easier.

I'll give you an example.

// Set alarm for 3 seconds.
alarm[0] = room_speed * 2;

You also want to use the Else Statement, instead of using two If Statements.
Like this:

if (!game_over)
{
alarm[2] = room_speed * 1;
}
else
{
alarm[4] = room_speed * 2;
}

Try that!

 

[YellowAfterlife]

You do not check whether alarm was not set yet, so may very well be re-setting it to 120 frames every frame (meaning that it won't expire for as long as game_over == true).

 

[Zerb Games]

What's up with the clickbait titles? Tell us at least something useful in your title. My god youtube is really corrupting the internet ): ^

 

[chaotic]

Your if statement keeps checking to true so it resets the alarm to 120 seconds every gamestep

//return to menu in 3 seconds when game is over
if (statement == true) {
    
   alarm[0] = 120;
   statement = false; // Gotta set to false otherwise it will keep checking true
                      // meaning it keeps resetting alarm
}

 

[bojack29]

This works too.

if (alarm[0] == -1){
     alarm[0] = 30;
}

 

[JDSTIGER]

What's up with the clickbait titles? Tell us at least something useful in your title. My god youtube is really corrupting the internet ): ^

Well, it wasn't misleading right? Also, ' My God, YouTube... ' **

Your if statement keeps checking to true so it resets the alarm to 120 seconds every gamestep

//return to menu in 3 seconds when game is over
if (statement == true) {
   
   alarm[0] = 120;
   statement = false; // Gotta set to false otherwise it will keep checking true
                      // meaning it keeps resetting alarm
}

This worked thank you!! & thanks to all who replied

 

[JDSTIGER]

You should use room_speed for your counter, it's much much easier.

I'll give you an example.

// Set alarm for 3 seconds.
alarm[0] = room_speed * 2;

You also want to use the Else Statement, instead of using two If Statements.
Like this:

if (!game_over)
{
alarm[2] = room_speed * 1;
}
else
{
alarm[4] = room_speed * 2;
}

Try that!

****Actually**** This worked better making it an Else statement, thank you!

 

[Silver_Mantis]

****Actually**** This worked better making it an Else statement, thank you!

Not a problem my friend, glad I could help.