is this the right way to make this script?

local numPlayers = 0

for i, v in next, game.Players:GetPlayers() do
if v:FindFirstChild("TeamColor") == BrickColor.new("Bright red") then
numPlayers = numPlayers + 1
elseif v:FindFirstChild("TeamColor") ~= BrickColor.new("Bright red") then
    numPlayers = numPlayers - 1
end
end

if numPlayers == 0 then

local player = game.Players:GetPlayers()

local pickedPl = math.random(1, #player)
pickedPl.TeamColor = BrickColor.new("Bright red")
end -- I think..
wait(60)
a = game.Players:GetChildren()
for i = 1,#a do
    player = a[i]
    player.Neutral=true
    player.TeamColor=BrickColor.new("White")
    game.Workspace[a[i].Name].Torso.Name=("")
end

Would I put the end with the comment next to it there?

Do not only post code, or only post an explanation. Both are required to make a good question.

Do not only post code, or only post an explanation. Both are required to make a good question.

Also, you can use "game.Players.NumPlayers" for get the total players connected!