My Inventory System adds multiple items and not just one, why is that?

Asked by

10 years ago

Basically, I have got the Inventory System to work pretty darn well. When the player presses E on their keyboard the item is then removed from the world and put into their inventory. The problem is that when the item is picked up, multiple quantitys of that time is added and not just one. It should add 1 item and not like 6.

01local player = game:GetService("Players").LocalPlayer
02local char = player.Character
03local mouse = player:GetMouse()
04local inv_gui = script.Parent.Inventory
05local inv = player:WaitForChild("Inventory")
06 
07while wait(0.1) do
08    for _,item in pairs(workspace["WinterSurvival"].DroppedItems:GetChildren()) do
09        if (item:IsA("Model")) then
10            local itemID
11            function findID(par)
12                for _,kid in ipairs(par:GetChildren()) do
13                    if (kid:IsA("Model")) then
14                        itemID = kid.ItemID.Value
15                    else
16                        findID(kid)
17                    end
18                end
19            end
20            findID(game:GetService("ReplicatedStorage").Items)
21            local label = script.Parent.PressE
22            if (char:FindFirstChild("Torso").Position - item.Handle.Position).magnitude < 7 then
23                label.Text = "Press [e] to pick up '"..item.Name.."'"
24                label.Visible = true
25                local debounce = true
26                mouse.KeyDown:connect(function(key)
27                    if (key:lower() == "e" and debounce) then
28                        debounce = false
29                        for _,slot in pairs(inv:GetChildren()) do
30                            if (slot.ItemID.Value == itemID) then
31                                slot.Qty.Value = slot.Qty.Value + 1
32                                label.Visible = false
33                                item:Destroy()
34                                break
35                            else
36                                if (slot.ItemID.Value == 0) then
37                                    slot.Qty.Value = 1
38                                    inv_gui[slot.Name].Qty.Visible = true
39                                    slot.ItemID.Value = itemID
40                                    slot.Value = item.Img.Value
41                                    label.Visible = false
42                                    item:Destroy()
43                                    break
44                                end
45                            end
46                        end
47                    end
48                    debounce = true
49                end)
50            else
51                label.Visible = false
52            end
53        else
54            return
55        end
56    end
57end
 View all 57 lines...

Thanks

Did you ever get this fixed? I'm looking through your post history and noticed this had no answers. adark 5487 — 10y

My Inventory System adds multiple items and not just one, why is that?

1 answer

Answer this question