Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
why do i earn money and it shows up in the leaderboard but wont show up for another player?
So i am working a simulator game and i was working pretty well but then i hit a slump. This script works and all but when a person clicks and earn money the money wont show up for another player.
Here is the script:
01 | local toolName = "AdvancedCard" -- what tool(name) must be held to earn? |
02 | local randomizeAmount = true -- randomize cash amount? if true, cash will be randomised, if false, cash will be the same everytime |
03 | local cashStatic = 1 --if 'randomizeAmount' is FALSE, how much money should be earned by 1 click? |
04 | local cashRandomX = 15 --if 'randomizeAmount' is TRUE, what is the MINimum cash amount? |
05 | local cashRandomY = 20 --if 'randomizeAmount' is TRUE, what is the MAXimum cash amount? |
06 | local debugging = true --debugging mode? if true, info will be print to the console, if false, script is not printing anything |
07 | local leaderstatName = "Cash" --name of Cash leaderstat |
08 |
09 | ---------- |
10 |
11 | local plr = game.Players.LocalPlayer |
12 | local equip = false |
13 | local mouse = plr:GetMouse() |
14 |
15 | plr.Backpack:WaitForChild(toolName).Equipped:Connect(function() |
View all 39 lines...
Please help me fix this.
0
you need to use a remote event to change the "cash" value, it cant be changed from a localscript. DinozCreates 1070 — 6y
0
What he said, I have encountered this before WideSteal321 773 — 6y
1 answer
Log in to vote
1
You can't change a value on the server from a local script on filtering enabled games because it won't replicate to other clients. Filtering will make sure this change is not replicated. You should use a remote event or a remote function to do this, preferably a remote event.
You should try something like this:
(Please note, this is very easy to exploit, apply some good security to your remotes.)
Server:
01 | -- Script (on server) |
02 |
03 | -- Create a remote event that will be fired to update player's cash. |
04 |
05 | local RE = Instance.new("RemoteEvent", game.ReplicatedStorage)) |
06 | RE.Name = "UpdateCash" |
07 |
08 | RE.OnServerEvent:Connect(function(self, value) |
09 | self.leaderstats.Cash.Value = self.leaderstats.Cash.Value + value |
10 | end) |
Client:
01 | -- Local Script (on client) |
02 |
03 | repeat |
04 | wait() |
05 | until game:GetService("Players").LocalPlayer |
06 |
07 | local lp = game.Players.LocalPlayer |
08 | local lpM = lp:GetMouse() |
09 |
10 | local randomizeAmount = true |
11 |
12 | lp.Backpack:WaitForChild("AdvancedCard").Equipped:Connect(function() |
13 | local debounce = false |
14 | lpM.MouseButton1Down:Connect(function() |
15 | if debounce == true then |
View all 27 lines...
Sorry if I left behind any errors or bad coding habits, un-testing code and kind of rushing.
0
I wouldn't put `self`, instead I would put `player` as that is more descriptive and `self` can cause some confusion SirDerpyHerp 262 — 6y