Why am I getting a lot of "MatchTags" even though I said only do once?

Here Is the code:

players = {} -- Only one player which is me whenever I test the game, so it should do only once right???

for _, player in pairs(game.Players:GetPlayers()) do
    -- Teleport Players
    local spawns = game.Workspace:WaitForChild("Spawns"):GetChildren()
    local spawnsBil = #spawns
    for _, player in pairs(players) do
        if player and player.Character and #spawns > 0 then
            for i = 1, #players do
                local torso = players[i].Character:WaitForChild("UpperTorso")
                local spawnIndex = math.random(1, spawnsBil)
                local Spawn = spawns[spawnIndex]
                if Spawn and torso then
                    table.remove(spawns, spawnIndex)
                    torso.CFrame = CFrame.new(Spawn.Position + Vector3.new(0,3,0))
                -- HERE IS THE CODE THAT GIVES THE PLAYER MATCHTAGS.
                    local matchTag = Instance.new("StringValue")
                    matchTag.Name = "MatchTag"
                    matchTag.Parent = players[i].Character
                -- ENDS HERE.           
                end
            end
        end
    end -- End of this for loop
    print("Teleport?")
end

The script should only give me one MatchTag because I'm the only one in the game but it gives me like 20 MatchTags instead. Any ideas??? thanks.