Why will only one gun reload?

LOCAL https://pastebin.com/607mQc6J
SERVER https://pastebin.com/4L4Ue6Dg

Hello,

I have been editing these guns for a while now, after picking them up off a free model and making them work to my liking, and I have an issue where I can only reload the first gun that loads into the game, when I attempt to reload another gun it just does nothing.

Any help will be appreciated! :slight_smile:

Is anything being thrown into the output? errors? warns? What about adding prints? That will help you see what is running and what isn’t running. This should be where you start.

One thing that stands out to me is you are setting Ammo.Value to string and then after that you are applying numbers to it and doing math. If its a Value object its either a Number or a string (or the other value objects in other cases.). Take a look at that as that will error on you.

Keep in mind, the Scripting Support category is not meant for us to review broken code you haven’t even tried to fix. Look at the rules here:

1 Like
local function reload()
	print("hello this works")
	reloadRemote:FireServer()
	wait()
	if canReload and relDeb == false and script.Parent.Parent.Configs.Reload.Value == true then
		relDeb = true
		
		reloadTrack:Play()
		wait(.01)
		reloadRemote:FireServer()
		
		wait(2)
		relDeb = false
	end
end

Prints as soon as I press test, not equipped or R pressed. This is in the Local Script.

Should I ask someone to move it?

No, now you’ve given us some more context, so you’re okay now.

1 Like

Are you saying its not running the “equipped” and “reload” functions? Trying to understand the script is a little difficult, what are the “Canshoot” and “Canreload” variables pointing to? They are in a folder called remotes, are they Remote event objects? Value Objects? Make sure the entire script is loading by putting a print at the bottom. If that prints then you can know that there was no error when the script first initializes. Then go ahead and add a print to each function/event you want to test. See which ones run and which ones do not.

From the looks of it things appear to be setup sort of odd, for example setting the equipped variable to true but not actually checking in the functions if it is equipped or not. When someone presses “R” in that script the first thing you should be doing is checking to make sure the gun is equipped before firing the remote. What if you have two guns? Will the remote fire twice when you press R since both events are still being listened for? That being said, start by adding prints in key points in the script like i suggested. You only sent a small snippet so i don’t know where you added them in your last reply.

CanShoot + CanReload = Remote Functions

It’s only the Reloading that doesn’t work ON EVERY GUN BUT ONE (probably the first that loads in).
The print I placed in the snippet above printed as soon as I entered the game, instead of when I pressed R, or even equipped the gun. The scripts seems to be running fine, and the BackendScript makes sure that the gun is allowed to reload.

Okay, thanks for clarifying. If those are remote functions and you are looking to check something such as shooting and reloading you would want to use RemoteFunction:InvokeServer(), However, i would not recommend doing this as it will cause extreme latency when using the weapon so those checks are almost redundant amongst other things that could cause it to act strange. Do you have a model that i can look at to test in my own studio?

Sure, just make sure you add more than one, I will DM you the model.

Unbind the actions in the Unequipped event function, but also have the actions binded in the Equipped event function.

local function unequip()
    equipped = false
    local character = player.Character or player.CharacterAdded:Wait()
    local humanoid = character:WaitForChild("Humanoid")
   
    sprinting = false
    humanoid.WalkSpeed = 16
   
    mouse.Icon = oldIcon
 
    if holdTrack then
        holdTrack:Stop()
    end
    if sprintTrack then
        sprintTrack:Stop()
    end
    actionService:UnbindAction("Reload")
    actionService:UnbindAction("Sprint")
end

When scripting things such as guns and using ContextActionService, always unbind the actions on unequip and bind them on equip.

Send your output here, or try find any errors in the output and try fix them.