Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Gettting, attempt to index local 'Player' (a nil value) error?
01 | local Handle = script.Parent:WaitForChild("Handle") |
02 |
03 | Handle.Touched:Connect(function(plr) |
04 | local Player = game.Players:GetPlayerFromCharacter(plr.Parent) |
05 | if Player.Character.Humanoid then |
06 | if Player.Character ~= game.Players.LocalPlayer then |
07 | Player.Character.Humanoid.Health = 0 |
08 | for _,i in pairs(Player.Character:GetChildren()) do |
09 | if i.BrickColor then |
10 | i.BrickColor = BrickColor.Random() |
11 | end |
12 | end |
13 | end |
14 | end |
15 | end) |
Getting the error "Players.NAME.Backpack.Taser.LocalScript:5: attempt to index local 'Player' (a nil value)" error. I don't get why
1 answer
Log in to vote
0
The reason for this is quite simple. When GetPlayerFromCharacter was called on line 4, you continued to use it despite it returning nil, so the script saw it like nil.Character. In order to fix this, simply check if the player exists.
01 | local Handle = script.Parent.Handle |
02 |
03 | Handle.Touched:Connect(function(part) |
04 | local Player = game.Players:GetPlayerFromCharacter(part.Parent) |
05 |
06 | if Player then -- if Player exists |
07 | if Player.Character ~= script.Parent.Parent then |
08 | -- Assuming this is under a Tool, script.Parent.Parent is the character if the script is directly under tool |
09 | Player.Character.Humanoid.Health = 0 |
10 |
11 | for _,i in pairs(Player.Character:GetChildren()) do |
12 | if i:IsA("BasePart") then -- If it is a part, it has a BrickColor property. |
13 | i.BrickColor = BrickColor.Random() |
14 | end |
15 | end |
16 | end |
17 | end |
18 | end) |
