A better way to write this could be in many ways:
A simple one time coloring:
1 | local player = game:GetService( "Players" ).LocalPlayer |
2 | player:WaitForChild( "Character" ).Torso.BrickColor = BrickColor.new( "Really red" ) |
or a on join module:
1 | local player = game:GetService( "Players" ).LocalPlayer |
2 | game:GetService( "Players" ).PlayerAdded:connect( function () |
3 | player:WaitForChild( "Character" ).Torso.BrickColor = BrickColor.new( "Really red" ) |
or a module to change the color when it's changed:
01 | local player = game:GetService( "Players" ).LocalPlayer |
02 | local wfc = game.WaitForChild |
03 | local char = wfc(player, "Character" ) |
04 | char:FindFirstChild( "Torso" ).Changed:connect( function () |
05 | if char.Torso.BrickColor = = BrickColor.new( "Really red" ) then |
08 | char.Torso.BrickColor = BrickColor.new( "Really red" ) |