I’m trying to make a system where the game doesn’t start unless there’s 3 or more players.
It doesn’t seem to execute the scripting when the players get to 3.
I’ve tried a for loop to count the players, I’ve tried using a variable to get the amount, Example:
local OnlinePlayers = game.Players:GetPlayers()
if #OnlinePlayers >= 3 then
-- blah blah blah
end
local Players = game:GetService("Players")
local OnlinePlayers = Players:GetPlayers()
if #OnlinePlayers >= 3 then
for Number, Player in pairs(OnlinePlayers) do
if Player.TeamColor == BrickColor.new("Institutional white") then
Player.Character:FindFirstChild("HumanoidRootPart").Position = Vector3.new(0, 1, 0)
Player.TeamColor = BrickColor.new("Dark stone grey")
end
end
end
Have you placed this part on a loop, or a Players.PlayerAdded connection?
if #OnlinePlayers >= 3 then
for Number, Player in pairs(OnlinePlayers) do
if Player.TeamColor == BrickColor.new("Institutional white") then
Player.Character:FindFirstChild("HumanoidRootPart").Position = Vector3.new(0, 1, 0)
Player.TeamColor = BrickColor.new("Dark stone grey")
end
end
end
If not then it’ll only check it once, so you’ll need to either put it in a loop or a Players.PlayerAdded connection
I would say that you listen when a player joins the game and then hear an Int by 1. Once you have 3 players in the game you can loop through all players as you have done
local Players = game:GetService("Players")
local CurrentPlayer = 0
Players.PlayerAdded:Connect(function(Plr)
CurrentPlayer += 1
if CurrentPlayer >= 3 then
for _, Player in pairs(Players:GetChildren()) do
if Player.TeamColor == BrickColor.new("Institutional white") then
Player.Character:FindFirstChild("HumanoidRootPart").Position = Vector3.new(0, 1, 0)
Player.TeamColor = BrickColor.new("Dark stone grey")
end
end
end
end)
It is important that you also reduce the player when he leaves the game.
This will only run once, when the server starts, and when the server starts there likely won’t be any players whatsoever, you have to keep track of the players that join and leave
local Players = game:GetService('Players')
local playerCount = 0
local playerAddedConnection
playerAddedConnection = Players.PlayerAdded:Connect(function()
playerCount += 1 -- Add 1 to the player count when a player joins
if playerCount == 3 then
playerAddedConnection:Disconnect()
-- Start the game
end
end)
Players.PlayerRemoving:Connect(function()
playerCount -= 1 -- Remove 1 from the player count when a player exits
end)
No need to do a player count, you can keep checking how many players there are in the server each time you’re attempting to start the game.
local PlayerService = game:GetService("Players")
local function StartGame()
if #PlayerService:GetPlayers() >= 3 then
--Attribute teams ect...
end
end
PlayerService.PlayerAdded:Connect(StartGame)
Here’s another way to do it, in case the game can start only once and new players can join while the game has already started.
local PlayerService = game:GetService("Players")
local GameStarted = false
local function StartGame()
GameStarted = true
--Spawn and attribute team to all players
end
local function JoinGame(NewPlayer)
--Spawn and attribute team to NewPlayer
end
PlayerService.PlayerAdded:Connect(function(NewPlayer)
if #PlayerService:GetPlayers() >= 3 and GameStarted == false then
StartGame()
elseif GameStarted == true then
JoinGame(NewPlayer)
end
end)
You realize that the player count is intentional, right? It’s much faster and better to simply add and check a number than having to call :GetPlayers() and getting the length of a table when a player joins the server
Not really, you got a useless active variable and an additional connection (player removing) only for it while #PlayerService:GetPlayers() is the only thing you could do to get the exact same result… it take less memory usage and save up 5 lines of code, but you’re still free to do it however you want if you prefer your method over mine.
