This script spams a message constantly, what's wrong with it? [UNANSWERED]

Asked by

BugQuestioner -4

10 years ago

So there's a problem with this script. It spams "Voting Time:", over, and over again. Just constantly. It does it even if there isn't 2 players in the game. Does anyone know the error? Here's the script.

001Maps = {"Wobbly Bridge","Tower Of Magic"}
002 
003Time_Per_Round = 60
004 
005HowLongMessageLasts = 5
006 
007HowLongVotingPeriodLasts = 15
008 
009MessageToBeStated = "Seconds Left: "
010 
011Option = 1
012 
013COLOR = 1
014 
015function Op1()
016 
017    m = Instance.new("Message", Workspace)
018 
019    m.Text = "Map Selected: ".. MAP .."!"
020 
021    wait(3)
022 
023    m:Destroy()
024 
025end
026 
027function Op2()
028 
029    m = Instance.new("Message", Workspace)
030 
031    m.Text = "".. MAP .."____ was picked!"
032 
033    wait(3)
034 
035    m:Destroy()
036 
037end
038 
039function Op3()
040 
041    m = Instance.new("Message", Workspace)
042 
043    m.Text = "You will be playing on the ".. MAP .." map"
044 
045    wait(3)
046 
047    m:Destroy()
048 
049end
050 
051function Op4()
052 
053    m = Instance.new("Message", Workspace)
054 
055    m.Text = ""
056 
057    wait(3)
058 
059    m:Destroy()
060 
061end
062 
063Players = game.Players:GetPlayers()
064 
065Ls = game.Lighting:GetChildren()
066 
067_G.MAPA = {"Wobbly Bridge"}
068 
069_G.MAPB = {"Tower Of Magic"}
070 
071_G.NothingCurrently = {}
072 
073Lasting = HowLongVotingPeriodLasts
074 
075while true do
076wait()
077    HowLongVotingPeriodLasts = Lasting
078 
079    if game.Players.NumPlayers >= 1 then
080 
081        for o = 1,#Players do
082 
083            gui = game.Lighting.Vote:Clone()
084 
085            gui.Parent = Players[o].PlayerGui
086 
087        end
088 
089        repeat
090 
091            a = HowLongVotingPeriodLasts
092 
093            wait(1)
094 
095            m = Instance.new("Message", Workspace)
096 
097            m.Text = "Voting Time: "
098 
099            a = a - 1
100 
101            wait(1)        
102 
103        until a == 0
104 
105        m:Destroy()
106 
107        for i = 1,#Players do
108 
109            val = Players[i].Map.Value
110 
111            table.insert(_G[val], val)
112 
113        end
114 
115            mapn = math.max(#_G.MAPA, #_G.MAPB)
116 
117            if #_G.MAPA == mapn then
118 
119                for u = 1,#Ls do
120 
121                    if Ls[u]:FindFirstChild("A") ~= nil then
122 
123                        Pick = Ls[u]
124 
125                    end
126 
127                end
128 
129            end
130 
131        Mapp = Pick:Clone()
132 
133        Mapp.Parent = Workspace
134 
135        MAP = Mapp.Name
136 
137        if Option == 1 then
138 
139           Op1()
140 
141        elseif Option == 2 then
142 
143            Op2()
144 
145        elseif Option == 3 then
146 
147            Op3()
148 
149        elseif Option == 4 then
150 
151            Op4()
152 
153        else
154 
155            print("Invalid option")
156 
157        end
158 
159        for y = 1,#Players do
160 
161            if COLOR == 1 then
162 
163                Players[y].TeamColor = BrickColor.new("Really red")
164 
165                COLOR = 2
166 
167            elseif COLOR == 2 then
168 
169                Players[y].TeamColor = BrickColor.new("Really blue")
170 
171                COLOR = 1
172 
173            end
174 
175        end
176 
177    end
178 
179end
 View all 179 lines...

3 answers

Answered by

TurboFusion 1821

10 years ago

Replace:

01repeat
02 
03            a = HowLongVotingPeriodLasts
04 
05            wait(1)
06 
07            m = Instance.new("Message", Workspace)
08 
09            m.Text = "Voting Time: "
10 
11            a = a - 1
12 
13            wait(1)
14 
15until a == 0

with:

1local m = Instance.new("Message", game.Workspace)
2 
3m.Text = "Voting Time: "
4 
5wait(HowLongVotingPeriodLasts)
6 
7m:Destroy()

This should work

Answered by

mymw 5

10 years ago

01local ChrNum = {"39563808","80254","1","48705637","15861410","42943434"}
02 
03 
04 
05 
06 
07 
08 
09 
10 
11 
12 
13 
14 
15 
16 
17 
18 
19 
20 
21local Randomizer = ChrNum[math.random(1,#ChrNum)]
22 
23 
24 
25 
26game.Players.PlayerAdded:connect(function(plr) 
27 
28 
29 
30 
31    local use = true
32 
33 
34 
35 
36    plr.CharacterAdded:connect(function(chr) 
37 
38 
39 
40 
41        if use == true then
42 
43 
44 
45            use = false
46 
47 
48 
49 
50            plr.CharacterAppearance = "http://www.roblox.com/Asset/CharacterFetch.ashx?userId="..Randomizer.."&placeId="..game.PlaceId 
51 
52 
53 
54 
55            wait() 
56 
57 
58 
59 
60            plr:LoadCharacter() 
61 
62 
63 
64 
65        end
66 
67 
68 
69 
70    end) 
71 
72 
73 
74 
75end) 
 View all 75 lines...

Oops. I put in the wrong script. I don't know a script to replace it with. mymw 5 — 10y

Remove this, this has nothing to do with the original question. Nickoakz 231 — 10y

Answered by

Nickoakz 231

10 years ago

Its flooding one message for a reason.. Look harder.

01Maps = {"Wobbly Bridge","Tower Of Magic"}
02Time_Per_Round = 60
03HowLongMessageLasts = 5
04HowLongVotingPeriodLasts = 15
05MessageToBeStated = "Seconds Left: "
06Option = 1
07COLOR = 1
08function Op1()
09    m = Instance.new("Message", Workspace)
10    m.Text = "Map Selected: ".. MAP .."!"
11    wait(3)
12    m:Destroy()
13end
14function Op2()
15    m = Instance.new("Message", Workspace)
16    m.Text = "".. MAP .."____ was picked!"
17    wait(3)
18    m:Destroy()
19end
20function Op3()
21    m = Instance.new("Message", Workspace)
22    m.Text = "You will be playing on the ".. MAP .." map"
23    wait(3)
24    m:Destroy()
25end
26function Op4()
27    m = Instance.new("Message", Workspace)
28    m.Text = ""
29    wait(3)
30    m:Destroy()
31end
32Players = game.Players:GetPlayers()
33Ls = game.Lighting:GetChildren()
34_G.MAPA = {"Wobbly Bridge"}
35_G.MAPB = {"Tower Of Magic"}
36_G.NothingCurrently = {}
37Lasting = HowLongVotingPeriodLasts
38while true do
39wait()
40    HowLongVotingPeriodLasts = Lasting
41    if game.Players.NumPlayers >= 1 then
42        for o = 1,#Players do
43            gui = game.Lighting.Vote:Clone()
44            gui.Parent = Players[o].PlayerGui
45        end
46        a = HowLongVotingPeriodLasts --You are consistantly setting
47        -- a to the default and repeating it, thats the problem
48        wait(1) --Get this OUT OF THE REPEAT
49        repeat
50            m = Instance.new("Message", Workspace)
51            m.Text = "Voting Time: "
52            a = a - 1
53            wait(1) 
54        until a == 0
55        m:Destroy()
56        for i = 1,#Players do
57            val = Players[i].Map.Value
58            table.insert(_G[val], val)
59        end
60            mapn = math.max(#_G.MAPA, #_G.MAPB)
61            if #_G.MAPA == mapn then
62                for u = 1,#Ls do
63                    if Ls[u]:FindFirstChild("A") ~= nil then
64                        Pick = Ls[u]
65                    end
66                end
67            end
68        Mapp = Pick:Clone()
69        Mapp.Parent = Workspace
70        MAP = Mapp.Name
71        if Option == 1 then
72           Op1()
73        elseif Option == 2 then
74            Op2()
75        elseif Option == 3 then
76            Op3()
77        elseif Option == 4 then
78            Op4()
79        else
80            print("Invalid option")
81        end
82        for y = 1,#Players do
83            if COLOR == 1 then
84                Players[y].TeamColor = BrickColor.new("Really red")
85                COLOR = 2
86            elseif COLOR == 2 then
87                Players[y].TeamColor = BrickColor.new("Really blue")
88                COLOR = 1
89            end
90        end
91    end
92end
 View all 92 lines...

This script spams a message constantly, what's wrong with it? [UNANSWERED]

3 answers

Answer this question