Clones [amount] random buttons [Completed]

Help and Feedback Scripting Support
,
#1

Clone 3 random buttons from a frame.

image
image831×245 46.8 KB

I tried to clone 3 random buttons but it still clone 1 button, no errors.

image

local Folder = script.Parent.ItemLists:GetChildren()

local Child = Folder[math.random(3, #Folder)]:Clone()

if Child:IsA("TextButton") then
	Child.Parent = script.Parent.Items
end

Hope you understand what I meant.

1 Like
#2

Use a for i,v in pairs / ipairs loop

#3

Wrong kind of loop, but close.
You would actually want a for i = 1, math.random(1, 3) do loop.

#4

There you go @Rexiler1

local rand = Random.new()
local Folder = script.Parent.ItemLists:GetChildren()

for count = 1, 3 do
     local Child = Folder[rand:NextInteger(3, #Folder)]:Clone()
     if Child:IsA("TextButton") then
	     Child.Parent = script.Parent.Items
     end
end
#5

Try this @Rexiler1:

local Folder = script.Parent.ItemLists:GetChildren()

for i = 1, 3 do
	local Child = Folder[math.random(3, #Folder)]:Clone()
	if Child:IsA("TextButton") then
		Child.Parent = script.Parent.Items
	end
end
#6

Thanks you @SnowyStxrm and @neeols
Your both scripts actually works, idk which one to mark solution lol.

#7

Use mine, using Random.new() and rand:NextInteger is unneeded. math.random() will suffice. (plus he literally just copied my code and edited it to use Random.new() and rand:NextInteger, editing his post after mine to make it look like I copied him…)

1 Like
#8

image

seems like it cloned the same button instead of different one :moyai:

#9

change the three in math.random to a 1 and see if that fixes it

#10

image

still happening though :moyai:

#11 

for count = 1,5 do


local Child = Folder[math.random(3, #Folder)]:Clone()
end
#12

Bro what? I posted mine 20 mins before yours! I came back and edited it later. Pretty sure NextInteger is needed here since its a loop.

#13 
local Child = Folder[math.random(3, #Folder)]:Clone()
local old = Child
for count = 1,5 do

if old == Child then
Child = Folder[math.random(3, #Folder)]:Clone()
Child:Destroy()
else
Child = Folder[math.random(3, #Folder)]:Clone()
old = child
end
end
#14

Your code was not that. Furthermore rand:NextInteger is not needed because math.random() is called every time.

#15

Do you want them to never duplicate?

#16

image

Nvm, I changed something inside the script, it works now

1 Like
#17

No worries. Don’t forget to mark my post as a solution!

#18

You are like in need of a solution

#19

Using Random.new() and rand:NextInteger is actually needed, the reason why it’s working because I use Random.new() and rand:NextInteger
Instead of cloning the same button, it clones a different buttons

1 Like
#20 

local random = folder[math.random(1, #folder)]:Clone()

for count = 1,5 do

local folder = game.ReplicatedStorage.Buttons:GetChildren()

local random = folder[math.random(1, #folder)]:Clone()

random.Parent = game.Players.mymomtookmyps5away.PlayerGui.ScreenGui.Frame

end