How do you clone a tool into a players inventory from a touched event?
Hi, I'm trying to make a part that when you step on it it clones a tool into the person who stepped on it's backpack. I did this code but it said attempt to index nil with Backpack.
01 | script.Parent.Touched:Connect(function(player) |
02 |
03 | local humanoid = player.Parent:FindFirstChild("Humanoid") |
04 |
05 | if humanoid ~= 0 then |
06 |
07 | local backpack = game.Players:FindFirstChild(player.Name):FindFirstChild("Backpack") |
08 |
09 | local clonedTool = script.Parent.Burger:Clone() |
10 |
11 | clonedTool.Parent = backpack |
12 |
13 | end |
14 |
15 | end) |
2 answers
There are two issues to point out within your program. The parameter that is supplied containing the Instance on the opposing contact, is merely a body part if theoretically being touched by a Character Rig. You attempt to search the Players container for this Object which won't return a proper result. Lastly, you're trying to compare a number to the Object Humanoid, this won't compare properly, so the conditional won't run the code below. I assume you're trying to check their Health, that of which is a float, so we can evaluate:
Note:
We can use the :GetPlayerFromCharacter() method of Players to uplink a Player Object with the Character derived from Hit.Parent.
01 | local Players = game:GetService("Players") |
02 |
03 | local Tool = script.Parent |
04 |
05 | local Burger = Tool.Burger |
06 |
07 | Tool.Touched:Connect(function(Hit) |
08 | local Humanoid = Hit.Parent:FindFirstChildOfClass("Humanoid") |
09 | if (Humanoid and Humanoid.Health > 0) then |
10 | local Player = Players:GetPlayerFromCharacter(Hit.Parent) |
11 | local Backpack = Player:WaitForChild("Backpack") |
12 | Burger:Clone().Parent = Backpack |
13 | end |
14 | end) |
It's best to maintain a practice of filtering for a Humanoid within a .Touched signal to ensure the chances of Hit.Parent correlating to a Character.
If this helps, don't forget to accept this answer!
Hello, Krystal_Kyle. The problem with your script is that you wrote player.Name instead of player.Parent.Name. Also, I'd call the parameter "hit" instead of "player" as it can sometimes confuse you. Try this:
1 | script.Parent.Touched:Connect(function(hit) |
2 | local player = game:GetService("Players"):GetPlayerFromCharacter(hit.Parent) |
3 |
4 | if player then |
5 | local clonedTool = script.Parent.Burger:Clone() |
6 | clonedTool.Parent = player.Backpack |
7 | end |
8 | end) |
The variable called player will get the player from its character which is hit.Parent. Then there's an if statement checking if the part hit is a player. If it is, the clonedTool's parent will get set to the player's backpack. Also, I used :GetService() as it checks if the service is existent. If it isn't, it creates the service. Please upvote and accept this answer if it helped.