Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
How to Make already have item object?
Hello, I would like to know how do I remove clones from the tool in the backpack, for example, once I already have the item, when I click to buy an object it is cloning in the back pack. Could someone help me?
1 | function buy() |
2 | if script.Parent.Parent.Parent.Parent.Parent.Parent.leaderstats.Level.Value >= 45000 then |
3 | script.Parent.Parent.Parent.Parent.Parent.Parent.leaderstats.Level.Value = script.Parent.Parent.Parent.Parent.Parent.Parent.leaderstats.Level.Value-0 |
4 | local clone1 = game.ServerStorage.Sword:Clone() |
5 | clone1.Parent = script.Parent.Parent.Parent.Parent.Parent.Parent.Backpack |
6 | end |
7 | end |
8 |
9 | script.Parent.MouseButton1Down:connect(buy) |
1
you should use variables, instead of saying "script.Parent.Parent.Parent.Parent.Parent.Parent.Parent" and so on niroqeo 123 — 4y
3 answers
Log in to vote
0
Check the iten existence with a for loop:
01 | local AlreadyHave = false |
02 | for i, v in pairs (Player.Backpack:GetChildren()) do |
03 | if v.Name == Item.Name then |
04 | AlreadyHave = true |
05 | end |
06 | end |
07 |
08 | if not AlreadyHave then |
09 | -- do stuff |
10 | end |
0
Works!! Thanks. Games_DBZ -15 — 4y
Log in to vote
0
It would really help if you used variables as mentioned above, making your code readable.
What you want to do to prevent multiple clones, is to use a debounce and a for loop to iterate through the player's backpack to see if the "Sword" already exists.
01 | local player = game.Players.LocalPlayer |
02 | local Backpack = player:FindFirstChild("Backpack") |
03 | local Item = game:GetService("ServerStorage").Sword |
04 | local debounce = false -- Local debounce variable |
05 |
06 | script.Parent.TextButton.MouseButton1Click:Connect(function() |
07 | if debounce == false then -- Only runs if debounce is false |
08 | Item:Clone().Parent = Backpack |
09 | end |
10 |
11 | for i,v in pairs(Backpack:GetChildren()) do -- For loop through children of backpack |
12 | if v.Name == "Sword" then -- Check to see if there's an item called Sword |
13 | debounce = true -- Debounce set to true, so the above code wouldn't run |
14 | end |
15 | end |
16 | end) |
Log in to vote
0
Well, I am going to provide you a solution with what almost everyone said. It is like looping through every item in your backpack.
01 | function buy() |
02 |
03 | local var = script.Parent.Parent.Parent.Parent.Parent.Parent -- Creating a variable for simplification |
04 |
05 | local PlayerBackpack = var.Backpack:GetChildren() -- Creates a table with contents of the backpack |
06 |
07 | if var.leaderstats.Level.Value >= 45000 then |
08 |
09 | for i, tool in pairs(PlayerBackpack) do -- Runs a loopin the table created above |
10 | if tool:IsA("Tool") then -- Checks if the item is a Tool then |
11 | if tool.Name == "Sword" then -- Checks if the tool's name is Sword (eg) |
12 | var.leaderstats.Level.Value = var.leaderstats.Level.Value - 0 |
13 | local clone1 = game.ServerStorage.Sword:Clone() |
14 | clone1.Parent = var.Backpack |
15 | end |
View all 23 lines...
Lemme know if it helps!