Scripting Helpers is winding down operations and is now read-only. More info→

For some reason this script says "Attempting to index a nil value", How would I fix this?

Asked by

7 years ago

.I am making a shop gui for a game and I made a simple button that detects whether or not the player has enough money when they try to purchase an item, and if they do it gives them they item and subtracts the money from their account. The script is very basic but it gives me an error code saying "Attempting to index local "player" (a nil value).

1script.Parent.MouseButton1Click:connect(function(player)
2    local lstats = player:WaitForChild('leaderstats')
3    if lstats.Tokens.Value >= 60 then
4        lstats.Tokens.Value = lstats.Tokens.Value - 60
5    else
6        script.Parent.Text = 'Insufficient Funds!'
7    end
8end)

I'm tried doing it in both local and global scripts and added a wait feature but neither worked, how would I get this to work in a local script?

its most likely talking about the (lstats =) part. Paste the whole line from the output and i can help you further. Nikkulaos 229 — 7y

12:54:37.883 - Players.EnderGamer358.PlayerGui.StoreGui.BaseFrame.PurchaseFrame.WeaponsFrame.BasicPistolFrame.Purchase.LocalScript:2: attempt to index local 'player' (a nil value) 12:54:37.885 - Stack Begin 12:54:37.888 - Script 'Players.EnderGamer358.PlayerGui.StoreGui.BaseFrame.PurchaseFrame.WeaponsFrame.BasicPistolFrame.Purchase.LocalScript', Line 2 12:54:37.889 - Stack End EnderGamer358 79 — 7y

Also would changing player to LocalPlayer still give me the player who clicked the button? I tried it in studio and it worked, but I don't know if it would work with other players on the server EnderGamer358 79 — 7y

1 answer

Answered by

UgOsMiLy 1074

7 years ago

Edited 7 years ago

The reason that your script broke is that player is not a parameter to the RobloxScriptSignal [event] MouseButton1Click. If this is in a local script, which it should be, because all scripts that work in the PlayerGui, or StarterGui must be LocalScripts due to FilteringEnabled then you would try this method:

1local player = game.Players.LocalPlayer
2script.Parent.MouseButton1Click:connect(function()
3    local lstats = player:WaitForChild('leaderstats')
4    if lstats.Tokens.Value >= 60 then
5        lstats.Tokens.Value = lstats.Tokens.Value - 60
6    else
7        script.Parent.Text = 'Insufficient Funds!'
8    end
9end)

But that probably won't work online because of FIltering Enabled. Your best bet is to use this method of using RemoteFunctions

This is how to use the recommended method

Step 1: put a RemoteFunction in the ReplicatedStorage named HandleTransaction

Step 2: Put these in your game

Replace your script with this LocalScript and put it in the same place.

1local player = game.Players.LocalPlayer
2local rf = game.ReplicatedStorage:WaitForChild("HandleTransaction")
3 
4script.Parent.MouseButton1Click:Connect(function()
5    local transaction = rf:InvokeServer()
6    if not transaction then
7        script.Parent.Text = 'Insufficient Funds!'
8    end
9end)

Insert a Script inside the ServerScriptStorage

01local rf = game.ReplicatedStorage:WaitForChild("HandleTransaction")
02 
03function re.OnServerInvoke(player)
04    if player:FindFirstChild("leaderstats") and player.leaderstats:FindFirstChild("Tokens")
05then
06        if player.leaderstats.Tokens.Value >= 60 then
07            player.leaderstats.Tokens.Value = player.leaderstats.Tokens.Value - 60
08            -- if you want something to happen as a result of the transaction, like giving the player an item, put the code here.
09            return true
10        else
11            return false
12        end
13    end
14end

Thanks a lot, it's working now :) EnderGamer358 79 — 7y

For some reason this script says "Attempting to index a nil value", How would I fix this?

1 answer

This is how to use the recommended method

Answer this question