Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
How do I make the parameter be for the player who triggered the event?
My code will turn you into a noob, but my code doesn't do so when it runs.
01 | script.Parent.Touched:connect(function(player) |
02 | player.Parent.Torso.BrickColor = BrickColor.new("Really Blue") |
03 | player.Parent.RightLeg.BrickColor = BrickColor.new("Forest Green") |
04 | player.Parent.LeftLeg.BrickColor = BrickColor.new("Forest Green") |
05 | player.Parent.RightArm.BrickColor = BrickColor.new("New Yeller") |
06 | player.Parent.LeftArm.BrickColor = BrickColor.new("New Yeller") |
07 | player.Parent.Head.BrickColor = BrickColor.new("New Yeller") |
08 | player.Parent.Shirt.Destroy() |
09 | player.Parent.Pants.Destroy() |
10 | end) |
Output result: "18:29:22.524 - RightLeg is not a valid member of Model 18:29:22.528 - Script 'Workspace.Noobifier.Head.Script', Line 3 18:29:22.531 - Stack End"
So how do I make the parameter be for the player who touched the part?
0
If you could remember to accept my answer, it helps a lot. The accept button is under my name by my answer. Thank you :) User#11440 120 — 8y
1 answer
Log in to vote
2
It's Right Leg, with a space.
Along with the arms and the rest of the legs.
In order to fix this, you have to use FindFirstChild.
Here's the following script,
01 | script.Parent.Touched:connect(function(player) |
02 | player.Parent.Torso.BrickColor = BrickColor.new("Really Blue") |
03 | player.Parent:FindFirstChild("Right Leg").BrickColor = BrickColor.new("Forest Green") |
04 | player.Parent:FindFirstChild("Left Leg").BrickColor = BrickColor.new("Forest Green") |
05 | player.Parent:FindFirstChild("Right Arm").BrickColor = BrickColor.new("New Yeller") |
06 | player.Parent:FindFirstChild("Left Arm").BrickColor = BrickColor.new("New Yeller") |
07 | player.Parent.Head.BrickColor = BrickColor.new("New Yeller") |
08 | player.Parent.Shirt.Destroy() |
09 | player.Parent.Pants.Destroy() |
10 | end) |
I would also suggest using an if statement to make sure that what touched the brick is a Humanoid.
01 | script.Parent.Touched:connect(function(player) |
02 | if not player.Parent:FindFirstChild("Humanoid") then return end |
03 | player.Parent.Torso.BrickColor = BrickColor.new("Really Blue") |
04 | player.Parent:FindFirstChild("Right Leg").BrickColor = BrickColor.new("Forest Green") |
05 | player.Parent:FindFirstChild("Left Leg").BrickColor = BrickColor.new("Forest Green") |
06 | player.Parent:FindFirstChild("Right Arm").BrickColor = BrickColor.new("New Yeller") |
07 | player.Parent:FindFirstChild("Left Arm").BrickColor = BrickColor.new("New Yeller") |
08 | player.Parent.Head.BrickColor = BrickColor.new("New Yeller") |
09 | player.Parent.Shirt.Destroy() |
10 | player.Parent.Pants.Destroy() |
11 | end) |
I hope I helped!
Good Luck!
0
Thank you for helping me, that worked! AtomicChocolate 35 — 8y
1
Instead of FindFirstChild, you could also use square brackets to index the objects, e.g. `player.Parent["Right Arm"].BrickColor`. However, both ways work. BlackJPI 2658 — 8y
0
What if the player got his legs or arms blown off for some reason? Always make sure FindFirstChild actually returns something before setting an object's properties. Link150 1355 — 8y
0
People tell me to keep it simple, then I get yelled at for not making the best script in the whole world. He said it fixed the problem, so eh :P User#11440 120 — 8y