Roblox Backpack Script Help

Help and Feedback Scripting Support
1

I made a backpack script check it and find the error pls.The script is

while wait() do
if game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name) then
for i, v in pairs(game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name)) do
if v.Name == script.Parent.Parent.Name then
script.Parent.Text = i
end
end
end
end

So what it does is if player has a number of the same tool in their backpack. it should show the number how many are there in his backpack.But it says error pls tell how to fix it.

2

First, can you format the code correctly?

2 Likes
4

What error is it showing?

Reformatted Code (In a code block) 
while wait() do
      if game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name) then
            for i, v in pairs(game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name)) do
            if v.Name == script.Parent.Parent.Name then
                  script.Parent.Text = i
            end
           end
       end
end
2 Likes
5

Yes thats what I typed but if I paste the script the format is changing like that.

7

Capture

It shows this @xZylter

8

The error is in

 for i, v in pairs(game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name)) do

You have to get all the children of the backpack, so you have to use :GetChildren() like this

 for i, v in pairs(game.Players.LocalPlayer.Backpack:GetChildren()) do
2 Likes
9 
while wait() do
      if game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name) then
            script.Parent.Text = game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name)
       end
end

This should work.

1 Like
10

The error is that GetFirstChild will only get the first instance with the script.parent.parents name. so you should use GetChildren which would get ALL of instances under the BackPack then see if its the same as the script.Parent.Parent.Name. the script below should work.

while wait() do
      if game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name) then
            for i, v in pairs(game.Players.LocalPlayer.Backpack:GetChildren) do
            if v.Name == script.Parent.Parent.Name then
                  script.Parent.Text = i
            end
           end
       end
end
2 Likes
11

Sorry, its not working for me. I think it needs a complicated script.

1 Like
12

For the script of GetChildren() , what its doing is its checking if there is the tool and if yes its showing backpack all tools count, instead of counting the tool with same name.

1 Like
13

Are there any errors in the console?

14

Using the for loop, you are looping through the objects inside of a folder or storage, but by saying BackPack:FindFirstChild(), you are making an error because you are looping through one object, which won’t work.

1 Like
15

Hold on is this script is supposed to count how many of this item is in the backpack right? if so you could do

function check(name)
	local number = 0
	local children = game.Players.LocalPlayer.Backpack:GetChildren()
	for i,v in pairs(children) do
		if v.Name == name then
			number = number + 1
		end
	end
	return number
end
while wait() do
	if game.Players.LocalPlayer.Backpack:FindFirstChild(script.Parent.Parent.Name) then
		local name = script.Parent.Parent.Name
		local amount = check(name)
		script.Parent.Text = amount
	end
end
2 Likes
16

Yes this is what I did before then it did not work and it says table got instance.I did a thing like

local number = 0
local name = script.Parent.Parent.Name
if v.Name = name then
number = number + 1
else
a call back function.

Thank you for it but it did not work saying got instance error

17

As the script you gave is a looped function what its doing is its checking once and adding a number.But since its a looped function for i,v and while wait do functions.Its checking and correcting of number looped and the number is increasing from 1 to inf in a few seconds.Causing lag for the player.

18

I think it needs a pcall function to.If you think yes tell me what to do as a pcall pls.

19

Ok it works now. I changed the numbers to default as 1 in player inventory and to avoid the mistaken i fixed it.Thank you everyone who helped :slight_smile: