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

Why is my Round System never working?

Asked by

9 years ago

I looked at so many tutorials but they only work for 'Printing' or 'Hint's'. Also, I tried to change the hint's to text, but it won't work.

I've tried many things to fix this problem but none of them are doing the trick, so I was wondering if someone could help. Thanks!

Errors: Text won't change. Map's won't load. It will not teleport you to the map.

Code:

001local playerNotifications = game.StarterGui.ScreenGui.Frame.TextLabel
002 
003function waitForPlayers()
004    if game.Players.NumPlayers < 1 then
005        playerNotifications.Text = "Waiting for players!"
006    end
007end
008 
009function intermissionTimer()
010    if game.Players.NumPlayers > 1 then
011        for countDown = 10, 1, -1 do
012        playerNotifications.Text = "Intermission: " ..countDown
013    end
014end
015 
016function displayGamemode()
017    - Couldn't find code for this.
018end
019 
020function mapSelection()
021    local mapList = game.ReplicatedStorage.Maps:GetChildren()
022    local randomMap = math.random(1, #mapList)
023    playerNotifications.Text = "Choosing the map!"
024    wait(4)
025    local mapChosen = mapList[randomMap]
026    playerNotifications.Text = "The map is: " ..mapChosen.Name
027    local mapClone = mapChosen:Clone()
028    mapClone.Parent = game.Workspace
029end
030 
031function teleportAllPlayers()
032    local target = CFrame.new(mapClone.Spawns.Position)
033    for i, player in ipairs(game.Players:GetChildren()) do
034        player.Character.Torso.CFrame = target + Vector3.new(0, i * 5, 0)
035        player.Playing.Value = 1
036    end
037end
038 
039function startingTimer()
040    for countDown = 3, 1, -1 do
041        playerNotifications.Text = "Game begins in: " ..countDown
042    end
043end
044 
045function roundTimer()
046    for countDown = 20, 1, -1 do
047        playerNotifications.Text = "Time left: " ..countDown
048    end
049end
050 
051function roundEnded()
052    - Couldn't find code for this.
053end
054 
055function teleportAlivePlayers()
056    local target = CFrame.new(workspace.TeleportTarget.Position)
057    for i, player in ipairs(game.Players:GetChildren()) do
058        player.Character.Torso.CFrame = target + Vector3.new(0, i * 5, 0)
059        player.Playing.Value = 0
060    end
061end
062 
063function mapDestroy()
064    - Couldn't find code for this.
065end
066 
067function showResults()
068    - Couldn't find code for this.
069end
070 
071game:GetService('Players').PlayerAdded:connect(function(player)
072    player.CharacterAdded:connect(function(character)
073        character:WaitForChild("Humanoid").Died:connect(function()
074            player.Playing.Value = 0
075        end)
076    end)
077 
078    local playing = Instance.new("IntValue", player)
079    playing.Value = 0
080    playing.Name = "Playing"
081end)
082 
083while(true) do
084      waitForPlayers()
085      wait(0)
086      intermissionTimer()
087      wait(0)
088      displayGamemode()
089      wait(5)
090      mapSelection()
091      wait(5)
092      teleportAllPlayers()
093      wait(0)
094      startingTimer()
095      wait(0)
096      roundTimer()
097      wait(0)
098      roundEnded()
099      wait(0)
100      teleportAlivePlayers()
101      wait(0)
102      mapDestroy()
103      wait(0)
104      showResults() 
105   end
106end
 View all 106 lines...

You have too many ends at the end. Move the end from 105 to line 12, since the script is thinking you're trying to end the function, not the loop all the way at line 11 M39a9am3R 3210 — 9y

You need 2 hyphens or it will error all the code below. https://gyazo.com/45afd362dc675514eccee844b46855b9 Azmidium 388 — 9y

1 answer

Answered by

TheDeadlyPanther 2460

9 years ago

Problem #1: Notifications

StarterGui is basically where the guis get cloned from, it doesn't change everyone's guis. So, instead, you make a function, right at the start that you call whenever you want to change text:

01function notification(text)
02    for _,v in pairs (game.Players:GetChildren())
03        local find = v.PlayerGui:FindFirstChild("ScreenGui")
04        if find then
05            find.Frame.TextLabel.Text = text
06        end
07    end
08end
09 
10-- to call it: notification("your text here")

Problem #2: Map

It should work, I see no problems. It is most likely something before the map function has errored.

Problem #3: Teleporting

The problem is that MapChosen is a local inside the MapSelection function, which means it doesn't exist inside TeleportPlayers().

To fix this, I would recommend you just create a local right at the start of the script, on Line 2 or something, called "CurrentMap". When it chooses the map, it sets CurrentMap to MapChosen, so inside the MapSelection function:

`1`	`CurrentMap` `=` `MapChosen`

This will mean you can get the current map in the entire script.

Also, here are some things to fix:

1. Make WaitForPlayers() a while loop, so:

1function WaitForPlayers()
2    while game.Players.NumPlayers < 2 do
3        notification("You need 1 more player to start the game.")
4    end
5end

2.IntermissionTime() needs an end under the for loop.

Hope I helped :)

Why is my Round System never working?

1 answer

Answer this question