Legacy GM Help in coding the code given below!

[prithvidiamond]

Hello dear readers and forum viewers,

I have a problem with the below code and need help,

globalvar fruit;

randomize();

fruit = irandom_range(0,3);

if fruit = 0
{
draw_self();
draw_sprite(spr_fr_wm,0,320,150);
}


if fruit = 1
{
draw_self();
draw_sprite(spr_fr_ap,0,320,150);
}

if fruit = 2
{
draw_self();
draw_sprite(spr_fr_sb,0,320,150);
}

if fruit = 3
{
draw_self();
draw_sprite(spr_fr_rs,0,320,150);
}

basically the problem is that the sprite keeps changing rapidly, I need ONLY ONE RANDOM sprite to show up until further instructions which I am unable to code.

Please Help any assistance will be of great help!

Thank you,

prithvidiamond

 

[joakimFF]

I assume all this code is in the draw event? first of you only need to call randomize(); once at the start of the game. secondly call fruit = irandom_range(0,3); in another event that is only executed when you want to.

 

[Conbeef]

globalvar fruit;

randomize();

fruit = irandom_range(0,3);

Put this part in the create event

 

[@Alex@]

It appears that you initialize the random variable over and over. This is because the draw event is executed multiple times per second. You need to initialize when you want the sprite to be set, either the create event , a button press or whenever it needs declared.

You should look into the switch statement as that'll shorten the code and make it easier to expand.

 

[BLang]

globalvar fruit;

randomize();

fruit = irandom_range(0,3);

Put this part in the create event

And then just do

fruit = irandom_range(0,3);

whenever you wanna randomize the sprite again.

 

[prithvidiamond]

Thank you all so much I will try I all your suggestions. You people are a wonderful community!

 

[prithvidiamond]

I assume all this code is in the draw event? first of you only need to call randomize(); once at the start of the game. secondly call fruit = irandom_range(0,3); in another event that is only executed when you want to.

Nice game you have made!

 

[prithvidiamond]

Thank you all, you are all a wonderful community!
it worked, thanks to all !

 

[Drift]

Instead of this, if you are using sprites with just 1 image, you could use this:

draw_sprite(spr_fr_wm,fruit,320,150);

Otherwise, use an array to store each sprite

*There's no need to use those draw_self()