When equipping more than once and firing it will do it more than once?

im currently working on a gun and im having this weird glitch and getting literally and headache because of it.

So if you equip the tool more than once like 5 times and click it will print "ok" 5 times, idk why i am getting this...

here is the script:

tool = script.Parent

local plr = game.Players.LocalPlayer
local mouse = plr:GetMouse()

Fire = false
equip = false

tool.Equipped:connect(function()
	equip = true
	
	mouse.Button1Down:connect(function(mouse)
		Fire = true
		if equip and Fire == true then
			print("ok")
		end
	end)
end)

tool.Unequipped:connect(function(mouse)
	equip = false
	Fire = false
end)

help?

This is because for each time you equip the tool, you're creating a new event listener, this is also why if you're equipping / unequipping 5 times, it will execute 5 times.

Remember that code should (almost) never have event listeners within event listeners, unless the first event listener is fired only once, as this will cause what you are experiencing right now.

The solution is to put the other event listener (mouse.Button1Down) outside of the tool.Equipped scope, this should fix your issue.

Basically, this is considered "bad code" for your current case:

tool.Equipped:Connect( function ()
	-- No, don't do this with this event
	mouse.Button1Down:Connect( function ()
		
	end)
end)

You should instead have it like the following code:

local hasEquipped = false;

tool.Equipped:Connect( function ()
	-- Ok, change some booleans and other variables if necessary
	hasEquipped = true;
end)

tool.Unequipped:Connect( function ()
	-- Ok, change the variable back
	hasEquipped = false;
end)

mouse.Button1Down:Connect( function ()
	-- Ok, make sure the user has equipped the tool
	if hasEquipped == true then
		print("Boom boom!");
	end
end)