My ores are not adding money to the leaderboard?
01 | plr = game.Players.LocalPlayer |
02 | worth = game.Players.plr.leaderstats.Money.Value |
03 | function destroy(hit) --'hit' parameter is now the part that touched! |
04 | if hit.Name == "drop" then --Check the hit's name. |
05 | worth = worth+10 |
06 | hit:Destroy() |
07 | end |
08 | end |
09 |
10 | --workspace.drop.Touched:connect(destroy) |
11 | script.Parent.Touched:connect(destroy) |
Im not sure why this does not add 10 when it hits the furnace
5 answers
When you set a variable to a ValueObject's Value property, that variable becomes equivalent to what the Value was when the script executed that line of code.
By changing the variable, you are simply changing the variable. The ValueObject's value is not affected.
You need to reference the object from the variable, and index the property to set it.
Now, secondly, you are getting the Player incorrectly. Since this is a server script you cannot use game.Players.LocalPlayer. You need to get the player some other way. That way is up to you. Preferably by an 'owner' StringValue to the tycoon.
01 | local plr --This is the player. |
02 | local worth = plr:WaitForChild("leaderstats"):WaitForChild("Money") |
03 |
04 | function destroy(hit) |
05 | if hit.Name == "drop" then |
06 | worth.Value = worth.Value + 10 |
07 | hit:Destroy() |
08 | end |
09 | end |
10 |
11 | script.Parent.Touched:connect(destroy) |
I'm just gonna fix this, upvote and accept answer if worked
In a local script;
01 | local player = game.Players.LocalPlayer |
02 | local Value = 10 -- the value of the ore |
03 | local curreny = player:WaitForChild("leaderstats").Money.Value |
04 | local Furnace = script.Parent --The furnace |
05 | local Cashed = false --make debounce so the player doesn't get a lot of 10 money |
06 |
07 | Furnace.Touched:connect(function(Hit) |
08 | wait() --Dunno why |
09 | if Hit.Parent.Name == "Drop" then |
10 | Cashed = true |
11 | wait() -- Dunno why |
12 | currency = currency + Value |
13 | Hit.Parent:Destroy() |
14 | wait() -- Dunno why |
15 | Cashed = false |
16 | if Cashed then return end -- We don't want the player to get too much money |
17 | end |
18 | end) |
Try, comment, ask and explain.
0
Don't use a LocalScript!
LocalScripts don't work in workspace unless they're in a player character. Also, if FE was enabled, the local scripts would only affect the value locally, which you obviously don't want!
The next problem is you need to get the player. But which player? That's something you need to figure out, but I'll provide an example. By touching the ownerBrick, you become the owner of this tycoon.
01 | local owner |
02 |
03 | local debounce = false |
04 | local function destroy(hit) |
05 | if owner and not debounce and hit.Name == "drop" then |
06 | debounce = true |
07 | local worth = owner.leaderstats.Money |
08 | worth.Value = worth.Value+10 |
09 | hit:Destroy() |
10 | wait(.5) |
11 | debounce = false |
12 | end |
13 | end |
14 | script.Parent.Touched:Connect(destroy) |
15 |
This is the solution. You need to have the .value after the 'worth' value. Then it will work.
01 | local plr --This is the player. |
02 | local worth = plr:WaitForChild("leaderstats"):WaitForChild("Money") |
03 |
04 | function destroy(hit) |
05 | if hit.Name == "drop" then |
06 | worth.Value = worth.Value + 10 |
07 | hit:Destroy() |
08 | end |
09 | end |
10 |
11 | script.Parent.Touched:connect(destroy) |
ok so there are many errors but idk if i can completely solve ur error. i will make a few changes though.
[UPDATE!]: now that i know ur problem.. ill try again.
SCRIPT.
in that item(part) make a group then add a humanoid inside the group. then add this script inside the part.
it should look like this.
GROUP>(Humanoid, OrePart).OrePart>Script>(inside the script below)
01 | function onTouched(part) |
02 | local ore = script.Parent |
03 | local h = part.Parent:findFirstChild("Humanoid") |
04 | if (h~=nil) then |
05 | local thisplr = game.Players:findFirstChild(h.Parent.Name) |
06 | if (thisplr~=nil) then |
07 | local stats = thisplr:findFirstChild("leaderstats") |
08 | if (stats~=nil) then |
09 | local score = stats:findFirstChild("Money")-- change this to what the thing is like Money or Points |
10 | if (score~=nil) then |
11 | score.Value = score.Value + 10 -- change this to the number of cash they get |
12 | end |
13 | ore:Destroy() |
14 | end |
15 | end |
16 | end |
17 |
18 | end |
19 |
20 | script.Parent.Touched:connect(onTouched) |
MY FINAL ANSWER, HOPE IT WORKS.
0
0
0
0
0