Scripting Helpers is winding down operations and is now read-only. More info→
Ad
Log in to vote
0

Ammo gui isn't working on a gun, can somebody help me with what I've done wrong?

Asked by 7 years ago

I got an ammo gui ed with a LocalScript but it aparently doesn't work :/

local players = game:GetService("Players")
local player = players.LocalPlayer
local weapon = .Parent
local ammo = .Parent:WaitForChild('CurrentAmmo').Value
local maxammo = .Parent:WaitForChild('MaxAmmo').Value

weapon.Equipped:connect(function()
    local gui = Instance.new('ScreenGui', player.PlayerGui)
    gui.Name = 'WeaponGui'
    local currentammogui = Instance.new('TextBox', gui)
    currentammogui.Name = 'CurrentAmmo'
    currentammogui.Text = 'Current: '..ammo
    currentammogui.Size = UDim2.new(0.17, 0, 0.07, 0)
    currentammogui.Position = UDim2.new(0.81, 0, 0.9, 0)
    currentammogui.BackgroundTransparency = 1
    currentammogui.FontSize = 'Size28'
    currentammogui.Font = 'Code'
    local maxammogui = Instance.new('TextBox', gui)
    maxammogui.Name = 'MaxAmmo'
    maxammogui.Text = 'Max: '..maxammo
    maxammogui.Size = UDim2.new(0.17, 0, 0.07, 0)
    maxammogui.Position = UDim2.new(0.81, 0, 0.85, 0)
    maxammogui.BackgroundTransparency = 1
    maxammogui.FontSize = 'Size28'
    maxammogui.Font = 'Code'
end)

ammo.Changed:connect(function(newammo)
    local currentammogui = player.PlayerGui.WeaponGui:WaitForChild('CurrentAmmo')
    currentammogui.Text = 'Current: '..newammo
end)

weapon.Unequipped:connect(function()
    player.PlayerGui.WeaponGui:Remove()
end)

Photo of how my Studio looks like: http://prntscr.com/e7is61

1 answer

Log in to vote
0
Answered by
Goulstem 8144 Badge of Merit Moderation Voter Administrator Community Moderator
7 years ago
Edited 7 years ago

When setting the initial text of currentammogui and the maxammogui, you need to index the ammo Value's Value Property.

--line 12
currentammogui.Text = 'Current: '..ammo.Value

--line 20
maxammogui.Text = 'Max: '..maxammo.Value
0
It is, it just copied wrong somehow :/ VladimVladim 78 — 7y
0
I have a script to change the value, but it isn't this script :/ VladimVladim 78 — 7y
0
Alright, I think I found your error then. Edited my answer. Next time supply any errors that appear in the Output. Makes it super easy for us to fix your code. Goulstem 8144 — 7y
0
That's not the problem as look correctly at the script is says script.Parent:WaitForChild('CurrentAmmo').Value VladimVladim 78 — 7y
Ad

Answer this question