Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
2
Why doesn't this function work more than one time?
works perfectly only the first time it happens, what's the problem?
01 | local Players = game:GetService("Players") |
02 | local ReplicatedStorage = game:GetService("ReplicatedStorage") |
03 |
04 | local player = Players.LocalPlayer |
05 | local welcomePlayerEvent = ReplicatedStorage:WaitForChild("FishEvent") |
06 | local playerGui = player:WaitForChild("PlayerGui") |
07 |
08 |
09 | local function onWelcomePlayerFired() |
10 | do |
11 | local henlo = script.Parent.ActivateFish:Clone() |
12 | henlo.Parent = script.Parent |
13 | henlo.Name = "Clone" |
14 | henlo.Disabled = false |
15 | wait(4) |
16 | end |
17 | end |
18 |
19 | welcomePlayerEvent.OnClientEvent:Connect(onWelcomePlayerFired) |
0
Where is the script located? Also that 'do' is pretty useless. It just makes the code more complicated. TheDeadlyPanther 2460 — 7y
0
The 'wait(4)' is also useless, since it's at the end of a function/coroutine. The problem is likely that the first ActivateFish changes something important so that running it a second time doesn't work. If you printed out every time onWelcomePlayerFired, I expect you'd find that the function is running every time the event is fired. chess123mate 5873 — 7y
1 answer
Log in to vote
0
First let's get your variables down
1 | local Players = game:GetService("Players") |
2 | local ReplicatedStorage = game:GetService("ReplicatedStorage") |
3 |
4 | local player = Players.LocalPlayer |
5 | local welcomePlayerEvent = ReplicatedStorage:WaitForChild("FishEvent") |
6 | local playerGui = player:WaitForChild("PlayerGui") |
The do on line 10 is pretty much useless so let's get rid of it, also lets combine the function and calling it.
1 | welcomePlayerEvent.OnClientEvent:connect(function() |
2 |
3 | end) |
Next i'm going to add your code in with just a bit of rearranging(idk i like it in a order ;p)
1 | welcomePlayerEvent.OnClientEvent:connect(function() |
2 | local henlo = script.Parent.ActivateFish:Clone() |
3 | henlo.Name = "Clone" |
4 | henlo.Parent = script.Parent |
5 | henlo.Disabled = false |
6 | end) |
No point in adding the wait(4) because the event can be called over and over
Let's put it together
01 | local Players = game:GetService("Players") |
02 | local ReplicatedStorage = game:GetService("ReplicatedStorage") |
03 |
04 | local player = Players.LocalPlayer |
05 | local welcomePlayerEvent = ReplicatedStorage:WaitForChild("FishEvent") |
06 | local playerGui = player:WaitForChild("PlayerGui") |
07 |
08 | welcomePlayerEvent.OnClientEvent:connect(function() |
09 | local henlo = script.Parent.ActivateFish:Clone() |
10 | henlo.Name = "Clone" |
11 | henlo.Parent = script.Parent |
12 | henlo.Disabled = false |
13 | end) |
As long as no other script is deleting that ActivateFish instance you should be fine.
0
Thanks but this doesn't change the fact that it only works one time greybird700 63 — 7y
0
@greybird700 if the event is only being called one time then that means YOU are only calling it one time, this part of the script is correct, maybe your problem lies in the script you are firing the event from. DanzLua 2879 — 7y