Is this right way to make player teleport?

script.Parent.Touched:Connect(function(hit)
    if hit.Parent:FindFirstChild("Humanoid") then
        hit.Parent:MoveTo(script.Parent.Parent.Part.Position)
    end
end)

I have seen people use HumanoidRootPart and CFrame for teleport but is there anything wrong with making player teleport this way? it seems to work tho.