Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Why is my script removing ten when i told it to only remove five, how is that even possible?
server script there is nothing that is changing the value xept this i have checked many times my script:
01 | local tool = game.ReplicatedStorage.Tools.Sign |
02 | local remote = game.ReplicatedStorage.Tools.Buy_Sign |
03 | print("runned") |
04 | -------------------------------------------------- |
05 | remote.OnServerEvent:Connect(function(player,Vorth) |
06 | ------------------------------------------------ |
07 | local coins = player:WaitForChild("leaderstats").Coins |
08 | ------------------------------------------------- |
09 | --remove coins |
10 | coins.Value = coins.Value - 5 |
11 |
12 | --Move tool to the player and set id. |
13 | tool.Parent = player.Backpack |
14 | end |
0
Your event can be firing twicee Fad99 286 — 5y
0
It is using a text button Freddan2006YT 88 — 5y
0
print (coins.Value - 5) asdfghjk9019 225 — 5y
0
yea it must be running twice cheetahjad -4 — 5y
View all comments (4 more)
0
It print's -5 Freddan2006YT 88 — 5y
0
It is Not running twice i know that Freddan2006YT 88 — 5y
0
whats the result of printing asdfghjk9019 225 — 5y
0
Thankyou everyone for trying to help i really apreciate it! Freddan2006YT 88 — 5y
2 answers
Log in to vote
2
Hello, Freddan2006YT!
I think the only thing missing on your script is a debounce variable, it prevents the code from running twice on a small time period
I've added it to your script
01 | local tool = game.ReplicatedStorage.Tools.Sign |
02 | local remote = game.ReplicatedStorage.Tools.Buy_Sign |
03 |
04 | local deb = false --Variable used for debounce |
05 |
06 | print("runned") |
07 | -------------------------------------------------- |
08 | remote.OnServerEvent:Connect(function(player,Vorth) |
09 | if(deb == false)then -- Lets code execute if variable "deb" is false |
10 | deb = true -- Prevents code from running again |
11 | ------------------------------------------------ |
12 | local coins = player:WaitForChild("leaderstats").Coins |
13 | ------------------------------------------------- |
14 | --remove coins |
15 | coins.Value = coins.Value - 5 |
View all 22 lines...
Edit¹: Also, this prevents hackers/exploiters from flooding their money change(or others)
Edit²: It would be good if you add something to check if player money is less than 5, so the player would not be able to have negative money
If I helped, please upvote and mark answer as accepted
0
It dosen't even remove any coins now or add the item Freddan2006YT 88 — 5y
0
hmmm Freddan2006YT 88 — 5y
0
I really don't know why, do it prints any errors on developer console(F9)? Leamir 3138 — 5y
0
Because deb is nil and will never equal false User#24403 69 — 5y
1
@incapaxx I've just changed it, I hope it works now(thanks for the hint) Leamir 3138 — 5y
Log in to vote
0
I have kinda fixed it I'm very confused why I need to do this to make it work someone explain if you know why
01 | local tool = game.ReplicatedStorage.Tools.Sign |
02 | local remote = game.ReplicatedStorage.Tools.Buy_Sign |
03 |
04 | local deb --Variable used for debounce |
05 | -------------------------------------------------- |
06 | remote.OnServerEvent:Connect(function(player) |
07 | if not deb then -- Lets code execute if variable "deb" is false |
08 | deb = true -- Prevents code from running again |
09 | ------------------------------------------------ |
10 | local coins = player:WaitForChild("leaderstats").Coins |
11 | ------------------------------------------------- |
12 | --remove coins |
13 | coins.Value = coins.Value - 5 |
14 | tool.Parent = player.Backpack |
15 | wait(0.5)--Wait 0.5 before making code able to run again |
16 | deb = false |
17 | coins.Value = coins.Value +5 |
18 | end |
19 | end) |
0
I'm gonna accept you becuse tried to help me :) Freddan2006YT 88 — 5y
0
You need to do because sometimes roblox glitches and fires a client/server 2x Leamir 3138 — 5y