Why does this gun script keep lagging my game?

[koziahss (Agent_Invalid)]

I see a couple of things that I would change:

• InvokeClient is highly unrecommended in this case for 2 reasons: Security and Performance

1. Security because InvokeClient will mean that the client send anything back and also could yield the server
2. Performance because it is possible to use a RemoteEvent instead of a RemoteFunction. RemoteFunctions sends data both ways whereas the RemoteEvent sends data 1 way.

• You would probably want to destroy the bullet regardless of what it hit. In the case that is hits your own character use Raycasting instead as it offers much more precision.

@chilly_ant

local bullet = Instance.new("Part",workspace)

View the docs: Bindable Events and Functions | Roblox Creator Documentation for more info.

• Recommended to not use the parent argument with Instance.new()

PSA: Don't use Instance.new() with parent argument

I’ve discovered a pretty bad performance issue in one of top games that has to do with Instance.new and wanted to write about this, since this is not obvious unless you know the system inside out. Tthere are several ways to create a ROBLOX object in Lua: local obj = Instance.new(‘type’); fill obj fields local obj = Instance.new(‘type’, parent); fill obj fields local obj = util.Create(‘type’, { field1 = value1, … }) If you care at all about performance, please only use the first option - I wi…

• Animation Tracks don’t unload

@chilly_ant

if idle then
		idle:Stop()
		idle = nil
	end

In other words if you Load an animation everytime the player equips a gun the Animation will stay loaded to the character untill the Humanoid dies.