I am not sure If I got this gun script right can someone check it for me?

This is quite a long one but If anyone can check it for me that would be great for my upcoming game I am developing.

originalAmmo = script.Parent.Ammo.Value
m = Instance.new("Message")

function computeDirection(vec)
	local lenSquared = vec.magnitude^2
	local invSqrt = 1 / math.sqrt(lenSquared)
	return Vector3.new(vec.x * invSqrt, vec.y * invSqrt, vec.z * invSqrt)
end



function updateAmmo()
	m.Text = " "
	--for i = 1,script.Parent.Ammo.Value do
	--	m.Text = m.Text .. "|"
	--end
	--for i = 1, (originalAmmo - script.Parent.Ammo.Value) do
	--	m.Text = m.Text .. " "
	--end
	m.Text = m.Text .. " " .. script.Parent.Ammo.Value.. "/12"
end


function fire(v)
	for i = 1,1 do   
                script.Parent.Handle.Fire:play()
	        script.Parent.Ammo.Value = script.Parent.Ammo.Value - 1
                updateAmmo()
                local dir = v - script.Parent["Handle"].Position
		dir = computeDirection(dir
		local pos = script.Parent["Handle"].Position + (dir * 8)
		local p = Instance.new("Part")
		p.Name = "Projectile"
		p.CFrame = CFrame.new(pos, pos + dir)
		p.BrickColor = BrickColor.new(24)
		p.Reflectance = 0.1
		p.Velocity = (script.Parent.Parent["Head"].Position - v).unit * -150
		p.Size = Vector3.new(1, 0.4, 1)
		p.formFactor = 2
		local mesh = script.Parent.Mesh:clone()
		mesh.Parent = p
		local upforce = Instance.new("BodyForce")
		upforce.force = Vector3.new(0, p:GetMass() * 196, 0)
		upforce.Parent = p
		local s = script.Parent["ProjectileScript"]:Clone()
		s.Disabled = false
		s.Parent = p
		p.Parent = game.Workspace
		wait(0)
	end
end


function onActivated()
	if script.Parent.Enabled == true then
		--script.Parent.Enabled = false
		if script.Parent.Ammo.Value > 0 then
			fire(script.Parent.Parent["Humanoid"].TargetPoint)
		else
			if script.Parent.Clips.Value >= -30000000000000 then
				script.Parent.Enabled = false
				script.Parent.Handle.Reload:play()
				m.Text = "reloading"
				for i =1,5 do
					wait(5) --Reload Time
					m.Text = m.Text .. "."
				end
				script.Parent.Clips.Value = script.Parent.Clips.Value - 1
				script.Parent.Ammo.Value = originalAmmo
				updateAmmo()
				script.Parent.Enabled = true
			else
				m.Text = "No more clips!"
			end
		end
		wait(0.11)
		--script.Parent.Enabled = true
	end
end


function onEquipped()
	local p = game.Players:GetChildren()
	for i = 1,#p do
		if p[i].Character == script.Parent.Parent then
			m.Parent = p[i]
		end
	end
	updateAmmo()
end

function onUnequipped()
	m.Parent = nil
end


script.Parent.Activated:connect(onActivated)
script.Parent.Equipped:connect(onEquipped)
script.Parent.Unequipped:connect(onUnequipped)