How to allow player to buy a gun?
Not gonna lie, my language is C#, not Lua, so I've just yolo'd this project and I need help with something. I'm making a game (obv), and I need to make it where the player can buy this AR when they click it. Here's my code that isn't working:
01 | Player = game.Players.LocalPlayer |
02 |
03 | script.Parent.ClickDetector.MouseClick:Connect(function() |
04 |
05 | if Player.LeaderBoard.Money.Value >= 500 then |
06 | Player.LeaderBoard.Money.Value = Player.LeaderBoard.Money.Value - 500 |
07 |
08 | game.ReplicatedStorage.M18:clone().Parent = Player:WaitForChild("Backpack") |
09 | else |
10 |
11 | end |
12 | end) |
Please help.
1 answer
Hello.
Problem:
- You used
LocalPlayerin a ServerScript. This will not work asLocalPlayeronly works onLocalScripts.
Solution:
- Use the
playerargument ofClickDetector.MouseClickinstead ofLocalPlayer.
Recommendations:
Use
Clone()with an uppercase "C" asclone()is deprecated. It is never a good idea to use deprecated functions.You do not have to use
WaitForChild()on the player's backpack as it will be loaded in.Use
GetService()when indexing services as it checks if the service is existent. If it isn't it creates the service.GetService()is also useful if someone renamed the services.GetService()is the only way to create services.You do not need the
elsefor the if statement on line 02.
Fixed Script:
1 | script.Parent.ClickDetector.MouseClick:Connect(function(Player) |
2 | if Player.LeaderBoard.Money.Value >= 500 then |
3 | Player.LeaderBoard.Money.Value = Player.LeaderBoard.Money.Value - 500 |
4 | game:GetService("ReplicatedStorage").M18:Clone().Parent = Player.Backpack |
5 | end |
6 | end) |
Also, put local before declaring a variable unless you're using a global variable. Local variables are faster and more efficient than global ones though.
0
0
0
0
0