Scripting Helpers is winding down operations and is now read-only. More info→
Ad
Log in to vote
1

Mistakes

Asked by 10 years ago

Could anyone tell me If I got this right? Is there a more efficient way to do this?

game.Players.PlayerAdded:connect(function(player)
  player.Chatted:connect(function(msg)
    if msg == "Heal" then
      player.Character.Humanoid.Health = player.Character.Humanoid.MaxHealth
      end
  end)
end)

game.Players.PlayerAdded:connect(function(player)
  player.Chatted:connect(function(msg)
    if msg == "Kill" then
      player.Character.Humanoid.Health = 0
      end
  end)
end)

game.Players.PlayerAdded:connect(function(player)
  player.Chatted:connect(function(msg)
    if msg == "TeleportBack" then
      player.Character.Humanoid:MoveTo(CFrame.new(0,10,0))
      end
  end)
end)
1
Bad question title. User#11893 186 — 10y

1 answer

Log in to vote
5
Answered by
MrNicNac 855 Moderation Voter
10 years ago

Connecting three events can be alleviated by using elseif.

game.Players.PlayerAdded:connect(function(player)
    player.Chatted:connect(function(msg)
        if msg == "Heal" then
            player.Character.Humanoid.Health = player.Character.Humanoid.MaxHealth
        elseif msg == "Kill" then
            player.Character:WaitForChild("Humanoid").Health = 0
        elseif msg == "TeleportBack" then
            player.Character.Torso.CFrame = CFrame.new(0,10,0)
        end
    end)
end)

0
ok thanks man. Was "player.Character.Humanoid:MoveTo(CFrame.new(0,10,0))" wrong? ConnorVIII 448 — 10y
0
player.Character.Humanoid:MoveTo(CFrame.new(0,10,0)) would make them walk to 0,10,0 Thewsomeguy 448 — 10y
Ad

Answer this question