Progress script doesn't work if more than 1 person is in the game

Help and Feedback Scripting Support
1

Hello!

I’ve ran into a very frustrating issue. I have a progress system that works for one player only, however many players are going to be playing in one map, so this won’t work. I don’t know where to go from here. Basically, I have a bunch of parts, that when touched, they increase the players’ progress. I don’t know how to proceed or if I should restart by doing something else. Thanks in advance for any help.

Basically, if a player touches the part, they will be added to the table. Unless the table already contains their name, they will gain a very small amount of progress, which should end up having their progress bar fill up to ~100%. The table check is so they can’t gain more than one point per brick. Note that it’s a server script.

local players = {}
local deb = false
local progressToAdd = 1 / #script.Parent.Parent:GetChildren()

script.Parent.Touched:Connect(function(p)
	if p.Parent:FindFirstChild("Humanoid") then
		if deb == false then
			deb = not deb
			if table.find(players, p.Parent.Name) == nil then
				local val = game.Players[p.Parent.Name]:FindFirstChild("PlayerGui").ProgressTimer.ProgressBar.Config.Progress
				val.Value = val.Value + progressToAdd
				table.insert(players, #players + 1, p.Parent.Name)
			end
			deb = not deb
		end
	end
end)
2

Try adding a player variable in the function. I think you only made it so one person is added. I changed the table.insert line, because if you already have the players as a variable, they count as 1.

script.Parent.Touched:Connect(function(p)
	if p.Parent:FindFirstChild("Humanoid") then
          local Player = game.Players.LocalPlayer
		if deb == false then
			deb = not deb
			if table.find(players, p.Parent.Name) == nil then
				local val = game.Players[p.Parent.Name]:FindFirstChild("PlayerGui").ProgressTimer.ProgressBar.Config.Progress
				val.Value = val.Value + progressToAdd
				table.insert(players,Player) -- already inside the table lemme know if it doesn't work
			end
			deb = not deb
		end
	end
end)
1 Like
3

This is a server script so LocalPlayer wouldn’t work here.

4

My bad. Then I suggest using this article

This’ll run inside the server script for you and fix your problem.

Hope that helps, logging off for now hope your code works bro.

1 Like
5

Don’t think your modified table.insert would work either, wouldn’t it return a concatenation error? The second argument should be a number for the position in the table.

6

table.insert can also be called with just two arguments to insert the value at the end of the table.

3 Likes
7

Did not know that, thanks!
Doesn’t seem to be a solution though unfortunately.

1 Like