Why do I lose more value than what the script says to lose?
I have a gun, and it has an ammo value. Everytime I shoot the gun, it works fine.. I have the max amount of ammo (15). It shoots it all, then I reload and shoot it again, still, everything works.
But when I unequipt my gun, then I take it back, suddenly it shoots 8 bullets instead. And its like the ammo value changed from 15 to 8. The script tells the gun to lose only 1 bullet PER SHOT. But for some reason it loses way more ammo.
I tried making a ScriptModule to store the ammo there, same issue. I tried making a Value inside the gun script to store the ammo there, same issue. I tried making a Value inside the gun it senf (NumberValue) to store the ammo there, same issue.
LOCAL SCRIPT (Mostly the input framework, to send a remote events to the server script)
01 | local shootremote = script.Parent:WaitForChild("ShootRemote") |
02 | local reloadremote = script.Parent:WaitForChild("ReloadRemote") |
03 |
04 | out = false |
05 |
06 | script.Parent.Equipped:Connect(function() |
07 | out = true |
08 | idle:Play() --this is an animation ignore it |
09 | mouse.Button1Down:connect(function() |
10 | shootremote:FireServer() |
11 | end) |
12 | end) |
13 | script.Parent.Unequipped:Connect(function() |
14 | out = false |
15 | idle:Stop() --this is an animation ignore it |
SERVER SCRIPT (The script that handles everything else, shooting, effects, reload.)
01 | local ammo = script.Parent.Ammo.Value -- These both are the value of a number value inside the gun, this is the last method I tried to use, but as I said, didnt work, like the other 2 methods. |
02 | local clip = script.Parent.Clip.Value |
03 |
04 | function shoot(player) |
05 | local char = player.Character |
06 | local hum = char.Humanoid |
07 | local came = game.Workspace.Camera |
08 |
09 | local recoilanim = hum:LoadAnimation(anims.recoil) |
10 |
11 | if ammo > 0 then |
12 | main.Fire:Play() |
13 | main.Flash.Enabled = true |
14 | main.PointLight.Enabled = true |
15 | recoilanim:Play() |
0
0
0
0
0
1 answer
I'm assuming the problem is, you're connecting the function that shoots the gun every time its equipped. It should be an easy fix, let me explain.
1 | script.Parent.Equipped:Connect(function() |
2 | out = true |
3 | idle:Play() |
4 | mouse.Button1Down:connect(function() -- connects the function to shoot. |
5 | shootremote:FireServer() |
6 | end) |
7 | end) |
The above is your equip function in your LocalScript.
You see, when you connect an event (in this case, mouse.Button1Down:connect) the function will fire every time the event is fired.
Once you've equipped the gun, it'll connect your click event to the shoot function, so from that point, every time your mouse is clicked, your shootremote:FireServer will run.
So if you equip the gun twice, it'll connect your shooting function twice, therefore clicking once will fire two shooting functions, allowing you to shoot twice and lose twice the ammo.
There are a few of ways to fix this, but one thing is the same: We want only one shooting function firing when you shoot.
Fix 1:
You could disconnect the shooting function when your gun is unequipped. So if you equip the gun, it'll connect to your shooting function, then when you unequip your gun, the shooting function will be disconnected and won't run anymore until you connect it again.
01 | script.Parent.Equipped:Connect(function() |
02 | out = true |
03 | idle:Play() --this is an animation ignore it |
04 | shootfunction = mouse.Button1Down:connect(function() -- connects the function to shoot. |
05 | shootremote:FireServer() |
06 | end) |
07 | end) |
08 |
09 | script.Parent.Unequipped:Connect(function() |
10 | out = false |
11 | idle:Stop() |
12 | shootfunction:Disconnect() -- disconnects the shooting function. |
13 | end) |
Fix 2:
You could have the shooting function always connected, but it will only run if a certain requirement is met by using if statements.
In this case, we want the requirement to be Gun is equipped so it will only shoot if the gun is equipped.
Fortunately, you already have a variable that defines whether or not the gun is equipped, which is out.
01 | mouse.Button1Down:Connect(function() -- connects the function to shoot. |
02 | if out == true then -- if the gun is equipped, |
03 | shootremote:FireServer() -- fires the shoot RemoteEvent. |
04 | end |
05 | end) |
06 |
07 | script.Parent.Equipped:Connect(function() |
08 | out = true |
09 | idle:Play() |
10 | end) |
11 |
12 | script.Parent.Unequipped:Connect(function() |
13 | out = false |
14 | idle:Stop() |
15 | end) |
Fix 3:
Use the Activated event.
The Instance Tool has an event called Activated. it is fired whenever the tool is activated.
In other words, it will fire when you click with the tool equipped.
So we use this, make the gun shoot when it's activated.
01 | script.Parent.Activated:Connect(function() -- connects the shooting function when the gun is activated. |
02 | shootremote:FireServer() -- fires the shoot RemoteEvent. |
03 | end) |
04 |
05 | script.Parent.Equipped:Connect(function() |
06 | out = true |
07 | idle:Play() |
08 | end) |
09 |
10 | script.Parent.Unequipped:Connect(function() |
11 | out = false |
12 | idle:Stop() |
13 | end) |
Edit: Try using :Connect() instead of :connect(), since :connect() is deprecated.
Sorry if you didn't understand my explanation, I hope it helped you in anyway.
If you have any questions, I'll try to answer them when I'm online.
0
0