Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Anyone know of a more efficient way for coding gui's being clicked?
Alright so say I've got this code:
01 | local OBJGui = player.PlayerGui.CatagoryGui.Frame |
02 |
03 | OBJGui.ObjectOne.MouseButton1Click:connect(function() |
04 | object = OBJGui.ObjectOne.Text |
05 | end) |
06 |
07 | OBJGui.ObjectTwo.MouseButton1Click:connect(function() |
08 | object = OBJGui.ObjectTwo.Text |
09 | end) |
10 |
11 | OBJGui.ObjectThree.MouseButton1Click:connect(function() |
12 | object = OBJGui.ObjectThree.Text |
13 | end) |
14 |
15 | OBJGui.ObjectFour.MouseButton1Click:connect(function() |
16 | object = OBJGui.ObjectFour.Text |
17 | end |
Is there any way to do this more efficiently instead of making countless amounts of _______.MouseButton1Click:connect(function()
This just seems horribly inefficient to me considering things like shop guis. I've tried to google it, but found nothing . Thanks for the help.
1 answer
Log in to vote
6
Yes, there is a way!
First, you want to change the numbers in the names of objects to actual numbers (not words). E.G,
Object1, Object2, Object3
Next, you want to loop through all of the objects using a number, then work from there, like so:
01 | local OBJGui = player.PlayerGui.CatagoryGui.Frame |
02 | local object -- i defined this earlier so it's more efficient |
03 |
04 | for i = 1,#OBJGui:GetChildren() do -- how many objects are inside the object holder |
05 | local obj = OBJGui:FindFirstChild("Object"..i) -- Object1, Object2, Object3, etc. |
06 | if obj then -- if it exists |
07 | obj.MouseButton1Down:connect(function() |
08 | object = obj.Text |
09 | end) |
10 | end |
11 | end |
Hope I helped!
~TDP