How to add delay before button can be pressed / code is executed again.

[tiagosparkz]

Just want the code thats under “public void OnPress ()” to have delay of 1 second before being able to be executed again.

For reference It raises the integer of totalMoney by 1 every time it’s pressed, then it changes to a string to be shown in a text onscreen.

Tried following this " Shot Delay between button press. c# - #2 by robertbu " but I’m new to this so don’t fully understand how to incorporate it into my own code. ( Hence the random delay float. )

[zulo3d]

float changeTime=0;

public void OnPress()
{
    if (Time.time>changeTime+1)
    {
        totalMoney=totalMoney+1;
        changeTime=Time.time;
    }
}

[Kurt-Dekker]

Cooldown timers, gun bullet intervals, shot spacing, rate of fire:

Fire Rate Issues

This is commonly called a “cooldown timer.” You can google for examples and tutorials, but the simplest way is this: have a private float gunHeat; variable In Update(), if that variable is greater than zero, subtract Time.deltaTime from it (this cools down the gunHeat) if (gunHeat > 0) { gunHeat -= Time.deltaTime; } When you go to fire: — A. only fire if that gunHeat variable is zero or less. — B. set that variable to the time (in seconds) you want to have between shots. if (User…

GunHeat (gunheat) spawning shooting rate of fire:

Spawning after amount of time without spamming

Don’t use a Timer… that’s super overkill. Instead, use just a float. I like to call my float “gunHeat.” private float gunHeat; private const float TimeBetweenShots = 0.25f; // seconds Now… in Update(): // cool the gun if (gunHeat > 0) { gunHeat -= Time.deltaTime; } // is the player asking to shoot? if (PlayerAskedToShoot) { // can we shoot yet? if (gunHeat <= 0) { // heat the gun up so we have to wait a bit before shooting again gunHeat = TimeBetweenShots; // DO…

[tiagosparkz]

@zulo3d

float changeTime=0;

public void OnPress()
{
    if (Time.time>changeTime+1)
    {
        totalMoney=totalMoney+1;
        changeTime=Time.time;
    }
}

This actually works almost perfectly, it seems to add an issue where at the start of the game ( For about 3 seconds ) pressing the button does nothing.
And after some testing you can click fast enough to add 2 “money” instead of being able to do just 1, any ideas?

Some extra code in there to change the colour of the button to indicate that you can’t press it was added but works fine.

Edit:
This also comes up, even though I’ve assigned it?

[Kurt-Dekker]

@tiagosparkz

This also comes up, even though I’ve assigned it?

That just means you haven’t actually found what is null. You’re still on Step 1.

How to fix a NullReferenceException error

https://forum.unity.com/threads/how-to-fix-a-nullreferenceexception-error.1230297/

Three steps to success:

• Identify what is null ← any other action taken before this step is WASTED TIME
• Identify why it is null
• Fix that

[zulo3d]

The slight initial delay is a result of ‘delay’ being initialized to 0. You could initialize ‘delay’ to -2 to remove the initial delay. Personally I wouldn’t worry about it as I don’t see a user hitting your ‘Add Money’ button within a second of your application starting up.

Money being added twice within a second is probably a result of how you’re updating/viewing your money text. Is it being updated elsewhere with some sort of delay?. Or are you viewing the money variable directly in the inspector? Is it possible that your Unity editor is also being delayed for some reason?.

Or have you changed the timeScale value?.

[tiagosparkz]

@Kurt-Dekker

That just means you haven’t actually found what is null. You’re still on Step 1.

How to fix a NullReferenceException error

https://forum.unity.com/threads/how-to-fix-a-nullreferenceexception-error.1230297/

Three steps to success:

• Identify what is null ← any other action taken before this step is WASTED TIME
• Identify why it is null
• Fix that

Tried to resolve this issue by using the debug.log to find what was returning as null, as expected the variable noPressColour was returning as null, so I assigned it on game start. Now not returning as null on the debug log


Although this worked after testing the button didn’t turn red, and I started writing this post but stopped halfway to figure out the problem myself. And it took me a while but the script was assigned to both the canvas and the button, and the script had all of the required components assigned on the button but not canvas which caused it to not work.
Point is thank you for mentioning that issue to me, would have ignored it otherwise.

That’s the full code, I also tried what @zulo3d suggested on the delay, but even increasing the delay to an extortionate number doesn’t seem the fix the small delay at the start. Not a big deal since as he mentioned nobody would realistically press the button that quick into the game but any fix or advice for it would be appreciated, thanks.

• The issue for being able to double click seems to still be around, any help?

This is also the first project I’m trying to take “seriously” and I’m generally new to this, sorry for the hassle.

[Kurt-Dekker]

@tiagosparkz

so I assigned it on game start.

If so, then why are you calling GameObject.Find() and wiping out that setting!!!

Remember the first rule of GameObject.Find():

Do not use GameObject.Find();

More information: https://starmanta.gitbooks.io/unitytipsredux/content/first-question.html

More information: https://discussions.unity.com/t/899843/12

In general, DO NOT use Find-like or GetComponent/AddComponent-like methods unless there truly is no other way, eg, dynamic runtime discovery of arbitrary objects. These mechanisms are for extremely-advanced use ONLY.
If something is built into your scene or prefab, make a script and drag the reference(s) in. That will let you experience the highest rate of The Unity Way™ success of accessing things in your game.

[MelvMay]

If you’re posting code on these forums, please use code-tags and not screenshots of code otherwise nobody will be able to copy/paste and refer to it.