Cloning multiple items?

Asked by
11 years ago
So I wrote my script as an inventory pick up script. When I pick up ammo it adds the correct amount to the ammo value but when I pick up an object, it clones multiple items into my inventory.
01char = script.Parent
02torso = char:findFirstChild("Torso")
03player = game.Players.LocalPlayer
04control = player.Control
05text = player.PlayerGui.Cam.BlackOut.Interact
06resources = game.Lighting.Resources
07mouse = player:GetMouse()
08stats = player:findFirstChild("Player Stats")
09inventory = stats.Inventory
10maxdistance=10
11grabbable = nil
12select = player.PlayerGui.Cam.LookAt
13 
14function Distance(target)
15    local p = target.CFrame.p
16    local t = torso.CFrame.p
17    local distance = (p-t).magnitude
18    if distance <= maxdistance then
19        return true
20    else
21        return false
22    end
23end
24 
25function UnLook()
26    select.Adornee = nil
27    text.Visible = false
28end
29 
30function Grab(item,id,key)
31    if key == "e" then
32        local class = id.Value
33        if class == "Ammo" then
34            local value = inventory.Ammo:findFirstChild(item.Name)
35            value.Value = value.Value + item.Amount.Value
36            item:remove()
37            grabbable:disconnect()
38        end
39        if class == "Pistol" then
40            local pistol = resources.Weapons.Pistol:findFirstChild(item.Name):clone()
41            pistol.Parent = inventory.Weapons
42            item:remove()
43            grabbable:disconnect()
44        end
45    end
46end
47 
48function Look()
49    if control.Value == true then
50        local target = mouse.Target
51        if target ~= nil then
52            local id = target.Parent.Parent:FindFirstChild("Id")
53            if id ~= nil then
54                local item = id.Parent
55                local grab = Distance(target)
56                if grab == true then
57                    text.Text = item.Name
58                    text.Visible = true
59                    select.Adornee = item
60                    grabable = mouse.KeyDown:connect(function(key) Grab(item,id,key) end)   
61                else
62                    UnLook()
63                end
64            else
65                UnLook()
66            end
67        else
68            UnLook()
69        end
70    end
71end
72 
73mouse.Move:connect(Look)
 View all 73 lines...
Cloning multiple items?

1 answer

Answer this question
