Player Requirement Script

Help and Feedback Scripting Support
1

What I’m trying to create here is a simple script that displays a message when there is an insufficient number of players on the server and organizes a round whenever there are more than two players on at once.

This, however, does not seem to work for some reason. The first part of the script where it checks for one or less players on the server works just fine, though it seems to go downhill from there; continuing on with the statement even after another player joins.

I’ve had no errors in the output thus far and I’m not really sure what the issue is - any help would be appreciated.

Here is a simplified version of the script:

local Players = game:GetService("Players")
local RoundPlayers = Players:GetPlayers()

while true do
	while #RoundPlayers <= 1 do
		print("One.")
		wait(4)
	end
	if #RoundPlayers >= 2 then
		print("Two.")
		wait(5)
	end
end

I experienced the same issues with this simplified script as I do with the full one, so it can’t have anything to do with that. Again, any help would be much appreciated: thanks!

2

do Players.NumPlayers
so do:

wait(2)
while Players.NumPlayers == 1 do
print("1")
wait(1)
end

if u have any questions then feel free to ask

3

Players.NumPlayers is Deprecated
you should use Players:GetPlayers Instead.

4

The reason why your original script wasn’t working as intended was because you stored the number of players outside of the loop, which never changed.

If you keep checking the amount of players inside the loop, you are then able to get the latest number of players at the moment that loop is running. :slight_smile:

2 Likes
5

I was already using Players:GetPlayers - issue is that the script itself just doesn’t seem to work.

6

Oh, that makes a lot more sense - I’ll try that right now.

7

To get it to work as you intend, there are a couple ways of doing this:

  1. Using :GetPlayers() inside the loop.
local Players = game:GetService("Players")
local RoundPlayers = Players:GetPlayers()

while true do
	while #RoundPlayers <= 1 do
		print("One.")
		wait(4)
		RoundPlayers = Players:GetPlayers()
	end
	if #RoundPlayers >= 2 then
		print("Two.")
		wait(5)
		RoundPlayers = Players:GetPlayers()
	end
end
  1. Using PlayerAdded outside the loop and updating a variable that the loop can reference.
local Players = game:GetService("Players")
local RoundPlayers = Players:GetPlayers()

local function updatePlayerCount()
	RoundPlayers = Players:GetPlayers()
end

Players.PlayerAdded:Connect(updatePlayerCount)
Players.PlayerRemoving:Connect(updatePlayerCount)

while true do
	while #RoundPlayers <= 1 do
		print("One.")
		wait(4)
	end
	if #RoundPlayers >= 2 then
		print("Two.")
		wait(5)
	end
end
8

Thanks a lot, worked just fine for me!

1 Like
9 
local Players = game:GetService("Players")
local RoundPlayers = Players:GetPlayers()

local function updatePlayerCount()
	RoundPlayers = Players:GetPlayers()
end

Players.PlayerAdded:Connect(updatePlayerCount)
Players.PlayerRemoving:Connect(updatePlayerCount)

while task.wait() do
	if #RoundPlayers <= 1 then
		print("One.")
		task.wait(1)
	elseif #RoundPlayers >= 2 then
		print("Two.")
		task.wait(1)
	end
end

There’s no point in having a while loop nested inside of another while loop, a single conditional statement will suffice, to add onto the previous statement, a single conditional chain which uses elseif would be better (as you’d skip checking the 2nd statement if the 1st resolves as true), replace wait() with task.wait() for accuracy and finally perhaps consider reducing the delay from 5 seconds to 1 second.