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

How can I reduce client lag when firing a remote event multiple times?

Asked by

Adamiscool2004 4

4 years ago

I have developed a BETA version of a game, think of papers please, it's a loop of a random frame showing, detecting to keypresses, sending it off to the server, and moving a part, that loop repeats with a while true do loop.

Whenever the frame shows, the user can either press [F] or [G], and dependant on the information, it's either right or wrong. The problem is that I need to wait for the user to make the input so I used a remote event to tell the server when that input occurred, but it causes more lag each time it is fired, by around 3 loops in a lag spike is noticeable.

I thought of making the part movement client-side (because it is a single-player game) but I have no idea how to do that, and if it ever becomes a multiple-player game, that would not really be practical.

How can I repeat the loop indefinitely without lag spikes occurring?

This is the loop:

01while true do
02    tween_car_in()
03 
04    spawn_docs()
05 
06    user_input()
07 
08    choose_passport()
09 
10    check_option()
11 
12    destroy_docs()
13 
14    tween_car_out()
15 
16    reset_cycle()
17end

This is the relevant function:

1repeat
2    rs.OptionPicked.OnServerEvent:Connect(function()
3        chosen = true
4        rs.OptionPicked:FireClient(player)
5    end)
6    wait(0.25)
7until chosen == true

And this is the keypress detection:

01user_input.InputBegan:Connect(function(key_input)
02    if key_input.KeyCode == Enum.KeyCode.F and picked == false and script.Parent.Visible == true then
03        --Irrelevant Code
04        wait(2)
05        game.ReplicatedStorage.OptionPicked:FireServer(true)
06        --Irrelevant Code
07    elseif key_input.KeyCode == Enum.KeyCode.G and picked == false and script.Parent.Visible == true then
08        --Irrelevant Code
09        wait(2)
10        game.ReplicatedStorage.OptionPicked:FireServer(true)
11        --Irrelevant Code
12    end
13end)

Note that in the relevant function I also need to pass the same event from client to client so I need to pass it through the server and then pass it onto the other client Adamiscool2004 4 — 4y

Try putting the loop in the script handling the remote event. Gooncreeper 98 — 4y

That is the script handling the loop, it's only purpose is to run the loop. All of the other actions (such as making the frame appear) are done through local scripts. All this script does is tween a few parts, play some animations, pick a random number (so it can display the relevant frame), and just pass this information through remote events to the local scripts so they can do their part. Adamiscool2004 4 — 4y

1 answer

Answered by

prooheckcp 340

4 years ago

First of all, never use While Trues, they are not healthy for any kind of behavior, instead of it use While wait() do. Either way there are two ways you can do this, one if having the while on the server side meaning u send all the variables that can only be obtained on the client side to the server side and do the action there, the second option is trying to make this event based instead of using a while, per example if u only want something to happen after the user presses a button you would use a key pressed event and so on

That's objectively untrue. `while true do` is the preferable of writing the loop, whereas `while wait() do` is not. Also, your response would be better placed as a comment, not an answer, since you didn't completely answer his question. Gey4Jesus69 2705 — 4y

The `while true do` is already on the server side, and making it event based would not be possible because there is not (or at least I don't think there is) a equivalent of `goto [line]` as in Javascript which is the only way I think I could make it not a loop, it would be great if you would put in some code samples showing me how to not make it a loop, thanks. Adamiscool2004 4 — 4y

How can I reduce client lag when firing a remote event multiple times?

1 answer

Answer this question