Scripting Helpers is winding down operations and is now read-only. More info→

How Do I Properly Set Up An Ammo System in Roblox?

Asked by

7 years ago

Alright. I learned how to make a RayCasting gun using the Roblox Wiki. Next thing, I made a little ammo system. Basically when I equip the gun, the script sets the gun's max ammo to 5. When I click/shoot the gun, the "ammo" value is reduced by one. After that, I made it so if the ammo value is equal or more than 1, the gun will shoot. If it is not, it does nothing. I then connected it to a GUI which displays the ammo value. It works! Only one thing... every time I equip the gun, the ammo is set back to 5.//I would like to know how to make it so the ammo value will save every time I un-equip the gun. I would also like it to save in the event the gun was dropped unto the ground, and another player was to pick it up. I would also like to know how to make a "stored ammo" where ammo is added in the event the player does something, say, clicks a part. Thank you so much! --PS-I am very sorry my code is very unorganized, I am sorry if you cannot understand it.

01local tool = script.Parent
02local player = game:GetService("Players").LocalPlayer
03local gui = game.Players.LocalPlayer.PlayerGui.AmmoGui.AmmoBar
04 
05tool.Equipped:connect(function(mouse)
06    local ammo = 5 -- MAX AMMO IN CLIP
07    gui.AmmoInClip.Text = ammo
08 
09 
10    mouse.Button1Down:connect(function()
11 
12 
13 
14        if ammo >= 1 then -- if ammo is over or equal to 1, shoot
15 
16 
17 
18        print("Mouse pressed!")
19        local ray = Ray.new(tool.Handle.CFrame.p, (mouse.Hit.p - tool.Handle.CFrame.p).unit * 300)
20        local part, position = workspace:FindPartOnRay(ray, player.Character, false, true)
21 
22        local beam = Instance.new("Part", workspace)
23        beam.BrickColor = BrickColor.new("Yellow")
24        beam.FormFactor = "Custom"
25        beam.Material = "Neon"
26        beam.Transparency = 0.25
27        beam.Anchored = true
28        beam.Locked = true
29        beam.CanCollide = false
30 
31        local distance = (tool.Handle.CFrame.p - position).magnitude
32        beam.Size = Vector3.new(0.3, 0.3, distance)
33        beam.CFrame = CFrame.new(tool.Handle.CFrame.p, position) * CFrame.new(0, 0, -distance / 2)
34 
35        game:GetService("Debris"):AddItem(beam, 0.01)
36 
37        if part then
38            local humanoid = part.Parent:FindFirstChild("Humanoid")
39 
40            if not humanoid then
41                humanoid = part.Parent.Parent:FindFirstChild("Humanoid")
42            end
43 
44            if humanoid then
45                humanoid:TakeDamage(30)
46            end
47 
48            ammo = (ammo-1) -- Subtract - shot from ammo
49            gui.AmmoInClip.Text = ammo
50 
51 
52            end
53        end
54    end)
55end)
 View all 55 lines...

1 answer

Answered by

Radstar1 270

7 years ago

Edited 7 years ago

Before I get into this, make sure you learn how to tab your script properly. Makes reading it and fixing problems much easier. Also in order to make your script cleaner it's better not to connect two functions inside of each other like you did with the Tool.Equipped then the Mouse.ButtonClick

Instead Make a variable named Equipped and change it to true once and once you click if that variable is true the rest will fire. I've put it inside of the code for you.

Actual Problem So basically you put local ammo inside of tool.Equipped which makes the ammo 5 everytime you equip the weapon. So take the variable local ammo outside of the tool.Equipped function. and make it like so:

01local tool = script.Parent
02local player = game:GetService("Players").LocalPlayer
03local gui = game.Players.LocalPlayer.PlayerGui.AmmoGui.AmmoBar
04local Equipped = false
05local ammo = 5 -- MAX AMMO IN CLIP Outside of the Tool.Equipped
06 
07tool.Equipped:Connect(function(mouse)
08    Equipped = true
09    gui.AmmoInClip.Text = ammo
10end)
11tool.Unequipped:Connect(function()
12    Equipped = false
13end)
14 
15mouse.Button1Down:connect(function()
16        if ammo >= 1 and Equipped then -- if ammo is over or equal to 1, shoot, and if weapon is equipped
17        print("Mouse pressed!")
18        local ray = Ray.new(tool.Handle.CFrame.p, (mouse.Hit.p - tool.Handle.CFrame.p).unit * 300)
19        local part, position = workspace:FindPartOnRay(ray, player.Character, false, true)
20        local beam = Instance.new("Part", workspace)
21        beam.BrickColor = BrickColor.new("Yellow")
22        beam.FormFactor = "Custom"
23        beam.Material = "Neon"
24        beam.Transparency = 0.25
25        beam.Anchored = true
26        beam.Locked = true
27        beam.CanCollide = false
28        local distance = (tool.Handle.CFrame.p - position).magnitude
29        beam.Size = Vector3.new(0.3, 0.3, distance)
30        beam.CFrame = CFrame.new(tool.Handle.CFrame.p, position) * CFrame.new(0, 0, -distance / 2)
31 
32        game:GetService("Debris"):AddItem(beam, 0.01)
33        if part then
34            local humanoid = part.Parent:FindFirstChild("Humanoid")
35            if not humanoid then
36                humanoid = part.Parent.Parent:FindFirstChild("Humanoid")
37            end
38            if humanoid then
39                humanoid:TakeDamage(30)
40            end
41            ammo = (ammo-1) -- Subtract - shot from ammo
42            gui.AmmoInClip.Text = ammo
43            end
44        end
45end)
 View all 45 lines...

If you want to learn how to save the actual ammo inside of it. Then all you would have to do is put NumberValue inside of the tool named: "Ammo" and use that instead of the variable named 'Ammo' that you have inside of the script. Decreasing this number value will keep it constant.

If you want to make something reload when you click it. You could add a click detector to like a brick(ammo pack) on the ground. Then inside of the brick make a script that says

1script.Parent.ClickDetector.MouseClick:Connect(function(Player)
2    if Player.Backpack:FindFirstChild(Tool.Name) then -- or wherever you put the tool
3        Tool.Ammo = 5
4    end
5end)

Thank you sooo much! Lol I never thought of using a number value instead of values inside the script!!! Thank you so much again! TheFierceWaffle 64 — 7y

Keep scripting, you will get much better. Radstar1 270 — 7y

Hello, MyosotisArae 0 — 3y

Perhaps this worked 3 years ago, but I tried this, and the value I modified in the local script (by the shoot) was unchanged when I read it from the server script (the part clicked). Is it a but I should report to Roblox ? MyosotisArae 0 — 3y

View all comments (2 more)

a bug, I mean MyosotisArae 0 — 3y

When I display it from the CommandBar, the value is right, though. So, this command does not give the same result in the script (Part:MouseClicked) than in the command bar : print(game.Players.MyosotisArae.Backpack.gun.nbAmmo.value) MyosotisArae 0 — 3y

How Do I Properly Set Up An Ammo System in Roblox?

1 answer

Answer this question