Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
1
How do I make that every button has a name of a tool?
Hello, I have a problem. There are 4 text buttons and 3 tools. I want that those text button's names would be the names of the tools. It prints "It worked" 12 times and all of the button names are a one tool name.
Script:
01 | function refresh() |
02 | print("refresh play") |
03 | for i,v in pairs(script.Parent.Frame:GetChildren()) do |
04 | if v:isA("TextButton") then |
05 | table.insert(buttons, v) |
06 | print("button added into table") |
07 | end |
08 | end |
09 |
10 | for i,v in pairs(buttons) do |
11 | for u,o in pairs(player.items:GetChildren()) do |
12 | buttons[i].Text = o.Name |
13 | print("it worked") |
14 | end |
15 | end |
16 | end |
17 | wait(10) |
18 | refresh() |
Thank you.
1 answer
Log in to vote
0
The issue lies in the loop
1 | for i,v in pairs(buttons) do -- button loop |
2 | for u,o in pairs(player.items:GetChildren()) do -- item loop |
3 | buttons[i].Text = o.Name |
4 | print("it worked") |
5 | end |
6 | end |
7 | end |
For every button you then get all items and set the item name to the same button meaning it will be set to the last item name in the loop.
You want to store the items then index them per button
1 | local itms = player.items:GetChildren() |
2 |
3 | for i,button in pairs(buttons) do |
4 | if itms[i] then |
5 | button.Text = itms[i].Name -- index the table of items |
6 | else |
7 | -- no tool for button |
8 | end |
9 | end |
Look at using events over refresh wait loops. You only need to refresh if something changed.
I hope this helps. Please comment if you have any other question about this code.
0
Thank you, that fixed the problem! JoneXI 51 — 5y