Is there a easier way to get all and determine the text buttons?

[ObnoxiousMate]

I would like to know if there’s a easier way to do the same function but in a more nicer or organized way in a GUI localscript rather than having to repeat this process.

Sure there’s a easier and faster way to do this right?

[Katrist]

Loop through the numbers and retrieve the buttons by their names.

for i = 0, 9 do
	local button = numpadFrame:WaitForChild(`Numkey{i}`)
	button.Activated:Connect(playBeep)
end

[ObnoxiousMate]

Thanks for the help, but I don’t think I put the code in right. I got the following error:
NumKey0 is not a valid member of Frame “Players.ObnoxiousMate.PlayerGui.KeypadGUI.BackgroundFrame.NumpadFrame”

[Katrist]

Use Numkey instead of NumKey. I updated my code.

[ObnoxiousMate]

Should’ve known, thank you so much for the help!

[Forummer]

Could just use GetChildren and check if the child is a TextButton instance.

[Katrist]

I did that initially, but it’s cleaner to loop over the numbers and avoid an unnecessary check

[Forummer]

The client should still be doing the following to account for unforeseen replication delays.

local button = frame:FindFirstChild("Button"..index)
if button then

[Katrist]

That’s true, I’ll add a :WaitForChild instead so the button is accounted for when it loads in.

[Forummer]

I personally dislike WaitForChild so I’d do the following, but to each their own.

local function onChildAdded(button)
	button.MouseButton1Click:Connect(func)
end

frame.ChildAdded:Connect(onChildAdded) --listen to frame's child added event (for not yet replicated buttons)

--loop through buttons that have already replicated

[7eoeb (beoe7)]

Ahh yes UIGridLayout.MouseButton1Click

You would have to do

if button:IsA("TextButton") then
    button.MouseButton1Click:Connect(func)
end

[Forummer]

The variable names don’t even match the original script, of course you’d edit the template I provided to account for this.