Properly using debounce

[low bobcat]

I'm not sure if the problem is me debouncing or something but my UpdateLogs() function keeps running repeatidly and so is making my remote event fire way too many times can someone help me figure out a way to prevent this? Code provided in the screenshot.

[deep oxide]

1. You shouldn't listen to .OnServerEvent in a Player Function, that will memory leak if there is more than 1 player. Move it outside on its own

2. You should not loop UpdateLog(), you're connecting a connection every frame, that's gonna have serious lag issues. Just loop and connect the functions once

3. Your db check should be in the function, as in, if db then return end

4. Also thought I'd just let you know that task.wait() is better than wait(), I forgot the reason but might be performancd

[low bobcat]

DANG IMA HAVE TO CHANGE SO MUCH 😭😭😭 thanks doe ill work on it as soon as i get back from my walk anything else you add ill also look at when im back

[deep oxide]

Alright

[low bobcat]

What you suggest I use instead of .OnServerEvent doe?

[deep oxide]

No its correct but

You see how its inside the PlayerAdded function?

Just copy the OnServerEvent and move it outside of that function