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

How to automatically close gui after 4 seconds?!

Asked by

3 years ago

Im making a murder mystery type game and i have guis to display the role of the player when the round starts. The game is locked in first person so i cant add a button and i want it to automatically close after 4 seconds. however, everything i try wont work. here is my code:

01local killerWeapon = game.ServerStorage.Knife
02local plrs = game.Players
03local inRound = game.ReplicatedStorage.inRound
04 
05local survivors = {}
06 
07 
08inRound.Changed:Connect(function()
09    if inRound.Value == true then
10        local chosen = plrs:GetChildren()[math.random(1, #plrs:GetChildren())] 
11 
12        chosen.PlayerGui.Picker.Background.RoleGiven.Text = "Murderer"
13        chosen.PlayerGui.Picker.Background.RoleGiven.TextColor3 = Color3.fromRGB(255, 0, 0)
14 
15        chosen.PlayerGui.Picker.Background.Visible = true
16 
17        killerWeapon:Clone().Parent = chosen.Backpack
18        print("killer")
19 
20 
21 
22 
23 
24        for i, plr in pairs(plrs:GetChildren()) do
25 
26            if plr ~= chosen then
27 
28                table.insert(survivors, plr)
29 
30                plr.PlayerGui.Picker.Background.RoleGiven.Text = "Survivor"
31                plr.PlayerGui.Picker.Background.RoleGiven.TextColor3 = Color3.fromRGB(0, 255, 0)
32 
33                plr.PlayerGui.Picker.Background.Visible = true
34                print("survivor visible")
35 
36            end
37        end
38    end
39end)
 View all 39 lines...

2 answers

Answered by

TheLuminent 1204

3 years ago

When looping through players, consider adding a coroutine if you're waiting for a specific period of time.

A coroutine creates a new thread & allows the environment to continue running without yielding; this is what we call Multithreading.

01local PlayersService = game:GetService("Players")
02local ReplicatedStorage = game:GetService("ReplicatedStorage")
03local ServerStorage = game:GetService("ServerStorage")
04 
05local BoolValue = ReplicatedStorage:WaitForChild("inRound")
06local Knife = ServerStorage:WaitForChild("Knife")
07local Survivors = {}
08 
09ReplicatedStorage:WaitForChild("inRound").Changed:Connect(function(property)
10    if tostring(property) == "Value" then
11        local inRound = BoolValue.Value
12        if inRound == true then
13            local Chosen = PlayersService:GetPlayers()[math.random(1, #PlayersService:GetPlayers())]
14            local PlayerGui = Chosen:FindFirstChildWhichIsA("PlayerGui") or Chosen:WaitForChild("PlayerGui")
15 
16            local Picker = PlayerGui:WaitForChild("Picker")
17            local Background = Picker:WaitForChild("Background")
18            local RoleGiven = Background:WaitForChild("RoleGiven")
19 
20            RoleGiven.Text = "Murderer"
21            RoleGiven.TextColor3 = Color3.fromRGB(255, 64, 64)
22 
23            coroutine.wrap(function()
24                wait(4)
25                Background.Visible = false
26 
27                local BackpackItem = Knife:Clone()
28                BackpackItem.Name = "Knife"
29                BackpackItem.Parent = Chosen:FindFirstChildWhichIsA("Backpack") or Chosen:WaitForChild("Backpack")
30            end)()
31 
32            pcall(print, Chosen:GetFullName() .. " is now the suspected killer.")
33 
34            for _, Player in pairs(PlayersService:GetPlayers()) do
35                if Player ~= Chosen and Player.Character and typeof(Player.Character) == "Instance" and Player.Character:IsDescendantOf(workspace) then
36                    local Humanoid = Player.Character:FindFirstChildWhichIsA("Humanoid")
37                    if Humanoid:GetState() ~= Enum.HumanoidStateType.Dead and Humanoid.Health > 0 then
38                        table.insert(Survivors, Player)
39                        local PlayerGui = Player:FindFirstChildWhichIsA("PlayerGui") or Player:WaitForChild("PlayerGui")
40 
41                        local Picker = PlayerGui:WaitForChild("Picker")
42                        local Background = Picker:WaitForChild("Background")
43                        local RoleGiven = Background:WaitForChild("RoleGiven")
44 
45                        RoleGiven.Text = "Victim"
46                        RoleGiven.TextColor3 = Color3.fromRGB(64, 255, 64)
47 
48                        coroutine.wrap(function()
49                            wait(4)
50                            Background.Visible = false
51                        end)()
52 
53                        pcall(print, Player:GetFullName() .. " is a victim of the suspected killer.")
54                    end
55                end
56            end
57        end
58    end
59end)
 View all 59 lines...

I tried this code but the gui didnt appear. I also put some prints in but they didnt work either. YourAverageMyth 45 — 3y

because you suck at scripting ez ez ez ez pingsock 111 — 3y

Answered by

Omq_ItzJasmin 666

3 years ago

Just add a wait()...??

1-- between lines 37 and 38 maybe
2wait(4)
3for i, plr in pairs(plrs:GetChildren()) do
4    plr.PlayerGui.Picker.Background.Visible = false
5end

adding a wait causes 1 player to see the survivor role, then when it dissapears the next person sees the survivor role. YourAverageMyth 45 — 3y

Sorry for the late reply, but i thought if you add it outside the loops like i said maybe between lines 37 and 38, it would again go through every player and instead of adding the role and making it visible, it just makes the role not visible? Omq_ItzJasmin 666 — 3y

How to automatically close gui after 4 seconds?!

2 answers

Answer this question