My gate way system is not working

[tardy holly]

For some context up, the move of gate is working fine and i will use tween service later
The problem is on the cooldown but i don't know whats wrong, can someone anaylse whats wrong?

local system = button.Parent
local gate = system.Gate.Gate
local cooldown = system.Cooldown

button.Touched:Connect(function(hit)
    local humanoid = hit.Parent:FindFirstChild("Humanoid")
    if humanoid then
        if cooldown.Value <= 0 then
            cooldown.Value = 60
            local x = gate.Position.X
            local y = 6.689
            local z = gate.Position.Z
            gate.Position = Vector3.new(x,y,z)
            print("succesful!")
        end
    end
    while true do
        if cooldown.Value ~= 0 then
            task.wait(1)
            print(cooldown.Value)
            cooldown.Value = cooldown.Value - 1
        else
            local x = gate.Position.X
            local y = 15
            local z = gate.Position.Z
            gate.Position = Vector3.new(x,y,z)
            break
        end
    end
end)```

[tardy holly]

the cooldown is not taking the second and also going to the negtaive

[junior bluff]

Do you really need the cooldown instance? You can handle it cleanly inside the script without any object instances

What u think radish

[runic knoll]

I don't know if this is the fix but try changing the
if cooldown.Value ~= 0 then
to
if cooldown.Value <= 0 then
I don't think it's the fix but right now you're checking if the cooldown is directly 0. And since there can be a floating number(like for example, 0.000000000013 or something) it can mess up a little(Unless you're using an IntValue as the cooldown instead of Numbervalue). Other than that, I don't really know what else

[junior bluff]

C'mon, that's not it. What may be happening is that in case a touch event occurs multiple times, the while loops are conflicting

It's a conflicting issue, flags like timer running is needed.

[junior bluff]

Also he's most probably using IntValue.

He prolly went to sleep after posting this.

[runic knoll]

Well if that's the case then I think you can just add debounce?

[runic knoll]

ae

[lapis sinew]

60 seconds

not 1 second

[lapis sinew]

this is true

[junior bluff]

Wassup, how ur sell tool game going

[lapis sinew]

this is also true

[lapis sinew]

but task.wait(1) is 1 second not 1 frame and they didn't wait long enough

aka wait 60 seconds

[runic knoll]

Hang on lemme just try and copy all this so I know what went wrong

[junior bluff]

No he's back

Let him explain

[runic knoll]

oh ok

[tardy holly]

i always used a intvalue i didn't know i had use a num value

[junior bluff]

No it's okay.

That ain't the issue

[lapis sinew]

this is also true most of the time but they're counting in integers there so it doesn't apply this time

[tardy holly]

also yeah i could made it inside the script

[lapis sinew]

it so happens that lua while true do if cooldown.Value ~= 0 then task.wait(1) print(cooldown.Value) cooldown.Value = cooldown.Value - 1 else should block all zeros including the first one no matter how many copies of the loop are running

but yeah <=0 better practice just in case

[tardy holly]

ok i will try that

[lapis sinew]

wot use <= and remove the 1 from task.wait(1) to make it take only one second, coz atm it is 60 seconds

[tardy holly]

k

[tardy holly]

no if i replace the 1 it just won't take one second

task.wait() is 0 second or near it

[lapis sinew]

yeah you said 'not taking the second' so you want it to wait 1 second right?

[tardy holly]

yeah

i want it to take a second

[lapis sinew]

how many times is this loop supposed to repeat until 0?

[tardy holly]

i plan on added rebirth so its non define

[lapis sinew]

    if humanoid then
        if cooldown.Value <= 0 then
            cooldown.Value = 60``` why 60

[tardy holly]

just a placeholder

[runic knoll]

All you really need is just to add debounce value at the top actually

[tardy holly]

should i put the cooldown to another script?

[tardy holly]

what was debounce again?

[runic knoll]

Basically a cooldown, if the debounce is false then set it to true after you pressed on the button and then set it back to false after the cooldown is done, so you can't press on it multiple times until the cooldown is done

[south plank]

** You are now Level 3! **

[tardy holly]

bruh, i didn't even needed it i just had to use a numvalue instead of intValue