Is this is a good way to make a cooldown?

Help and Feedback Scripting Support
#21

It will be stuck in Roblox Studio and crash it

#22

did you put it in a while loop?

#23

Again, your method is not working cuz I want a cooldown without any chrashes. If I would use it in a while loop or in a RunnService it will crash cuz untill the wait(CoolDown) ends it will print too many times “hey”

#24

This is why I would not prefer

if pressed == true then
wait()-- time you want to wait
pressed = false
end

#25

wait() has its place, but it is thread-blocking and messy. I still would recommend you go with my original advice. Whenever someone tries to use something with a cooldown, check if enough time has passed for it to be allowed. This way it does not block any threads and is only called when it is needed.

2 Likes
#26

Now I will try to use ur tick() function in a if statement

Debounce doesn't work for runservice
#27

There are a couple ways to do it but the best way is with the method @SuchASaltyLemon used.

You’ll want to do fully on the server so it’s anti-exploitable. This way basically compares the last timestamp to how much time has passed instead of using instances or waits. Another thing is to use os.clock() because from what I’ve heard it is the best function for timestamps.

local cooldown = 5
local cooldowns = {}

local function CreateCooldown(player)
	cooldowns[player.UserId] = os.clock()
end

local function CanUse(player)
	local last = cooldowns[player.UserId]
	if os.clock() - last >= cooldown then
		return true
	end
	return false
end

Tables are used so that you can store multiple player cooldowns.
Also, an even better method would to use module scripts for the cooldown table so it is accessible by multiple server scripts.

Edit: You’ll want to check if the player is actually in the table first before setting their cooldown, or else you will have an error. You can use a Player Added function and add their UserId in that way, or you can check if it is nil when you check their cooldowns and if it is, add their UserId.

3 Likes
#28

this is not working maybe because I am using it in RunService too

#29

I looked at your first post and your variable names are a bit funny. You used the method that @Fusionet and I recommended, but you have previous as the first variable, but then later Previous

#30 
debounce = false

if not debounce then
 debounce = true
 print("hey")
 wait(cooldown)
 debounce = false
end
#31

This method is not as reliable for a couple reasons.

If used on the client it is easily exploitable. Also, the wait() function is sometimes never accurate and can actually end up waiting much more than it is supposed to. For this reason the timestamp method with os.clock() is much better.

2 Likes
#32

guess the recourses i am using are a bit outdated lol

1 Like
#33

I don’t use the cooldown you mentioned, but instead use the os.time() method. Is there a difference? Does it increase performance by limiting client & server, running the cooldown on both instead of firing an event every frame.