Scripting Helpers is winding down operations and is now read-only. More info→

Choosing the rest of the players?

Asked by

9 years ago

I recently asked a question about a bad argument error when choosing a random player. Now, I want a way to implement a gui into the rest of the players.

Go to: to see this previous question.

This is the current code that has to be updated with this feature:

01local playerstorage = game:GetService("Players")
02local storage = game:GetService("ServerStorage")
03local guifolder = storage.Guis
04local players = {}
05 
06wait (40)
07    for _, player in pairs(game.Players:GetPlayers()) do
08        if player and player.Character then
09            local humanoid = player.Character:WaitForChild("Humanoid")
10            if humanoid and humanoid.Health > 0 then
11                table.insert(players, player)
12            end
13        end
14    end
15    if #players >= 2 then
16        local alien = players[math.random(1, #players)]
17        local alieng = guifolder.Alien
18        local alieng2 = alieng:Clone()
19        alieng2.Parent = alien.PlayerGui
20        alien = script
21        --More code
22else
23    print '2 players needed!'
24 
25    end
26    wait(0.1)
 View all 26 lines...

2 answers

Answered by

ImageLabel 1541

9 years ago

You can use table.remove when choosing alien so that it returns the removed value, while at the same time nullifying it from the table players.

Therefore, whenever you iterate through players later on in your code, it will returned everyone but the removed value, alien.

01local playerstorage = game:GetService("Players")
02local storage = game:GetService("ServerStorage")
03local guifolder = storage.Guis
04local players = {}
05 
06wait (40)
07for _, player in pairs(game.Players:GetPlayers()) do
08  if player and player.Character then
09    local humanoid = player.Character:WaitForChild("Humanoid")
10    if humanoid and humanoid.Health > 0 then
11      table.insert(players, player)
12    end
13  end
14end
15 
16if #players >= 2 then
17  -- remove random value from table
18  local alien = table.remove(players, math.random(#players)
19  local alieng = guifolder.Alien
20  local alieng2 = alieng:Clone()
21  alieng2.Parent = alien.PlayerGui
22 
23  for _, player in pairs(players) do
24    print(player)
25    -- alien was removed, so this will return
26    -- the list of all values inside of table `players`
27    -- except alien.
28  end
29  --More code
30else
31  print '2 players needed!'
32end
 View all 32 lines...

Also, in your first generic for loop, you are iterating through the table returned by players:GetPlayers, which only returns player objects.. therefore, you do not need to check if they have a character, because that's a given.

1for _, player in pairs(game.Players:GetPlayers()) do
2  local humanoid = player.Character:WaitForChild("Humanoid")
3  if humanoid and humanoid.Health > 0 then
4    table.insert(players, player)
5  end
6end

Thank you! fight4money -2 — 9y

Answered by

saenae 318

9 years ago

It seems as if all you need is one more loop that just ignores the alien.

for i, v in pairs(game.Players:GetChildren()) do if v.PlayerGui:findFirstChild("Alien") == nil then -- Add Gui end end

Choosing the rest of the players?

2 answers

Answer this question