Clean up after player left server

Help and Feedback Scripting Support
1
  1. I want to ask if i have
local PlayerAnimationsCache = {}
local PlayerCurrentAnimators = {}
local PlayerDebounces = {} -- Store debounces for m1 for each player

for all weapon script (server script) i have. each weapon will have a script
and at the end of script, i have this to prevent memory leak

function onPlayerRemoving(player)

	PlayerAnimationsCache[player.UserId] = nil
	PlayerCurrentAnimators[player.UserId] = nil
	PlayerDebounces[player.UserId] = nil
end

game.Players.PlayerRemoving:Connect(onPlayerRemoving)

  1. would this cause a problem as more weapons are added to a game? The thing im worrying about is this would run for a player who does not even own that weapon and 25 scripts (weapons ) will run when player leaving as well.

  2. Feel free to tell me a better way to do this as well. I’d appreciate a lot!

2

this is ChatGPT said so far “” Having the PlayerRemoving connection for each weapon in your game, even if you have around 100 weapons, should not cause significant lag. The PlayerRemoving event is a standard way to clean up player-specific data when a player leaves the game, and it’s a common practice in Roblox game development." But i have no way to verify this~~

3

So you have this script, in 100 different weapons? Wouldn’t it just be better to use a loop. Regardless, even if you didn’t, I don’t think 100 different events removing these items from the table wont cause any major issues, but it is certainly not the most efficient way

4

Why don’t you store all these variables in a module script since you want to use it for multiple scripts?

And now you can simply call the function with out having to activate all other scripts.

Edit : sorry wrong person

1 Like
5

I never thought about this. Ill try it out

6

yup, each weapon’s script (around 25 )will have those variables and a clean-up function. how’d you loop it?