Select random item from a folder

Help and Feedback Scripting Support
#1

Hello, i was working on my game and i was making a script that would select a random tool out of a Folder
Here is my code:

local folder = game.Lighting.Common
local tools = {
folder:GetChildren()
}
local choosen_tool = tools[math.random(1, #tools)]

I am sure i made a mistake somewhere, thanks for your help!

#2

Yes, you were very close. Just take out the brackets around folder:GetChildren(). GetChildren() automatically returns a table for all of the children, so you don’t need to put that table inside of another table.

local folder = game.Lighting.Common
local tools = folder:GetChildren()

local choosen_tool = tools[math.random(1, #tools)]
2 Likes
#3

It’s already in a table when you define the choosen_tool variable w h e e z e, just remove the table brackets

#4

:GetChildren() returns a table by itself, so encasing it with brackets is creating a table within a table. Just remove the brackets.

#5

I have removed the brackets and its not working

Π‘Π½ΠΈΠΌΠΎΠΊ экрана 2021-01-18 Π² 19.22.57
Π‘Π½ΠΈΠΌΠΎΠΊ экрана 2021-01-18 Π² 19.22.571920Γ—1080 328 KB

take a look at the error at the output

#6

You got an instance from the folder. Since you are using it for a string value, you need to get the name of that instance, so just add .Name to the end of it. On Line 19 do this:

gui.Frame.Gradient.val1.Value = chosen_tool.Name
#7

That is because you are trying to change a String value to an instance, I am unsure about your file structure and what value you are trying to get from the Instance.

#8

The problem isnt the Gui value, its the random selection

local choosen_tool = tools[math.random(1, #tools)]

Do i need to add the Name at the :Getchildren()?

#9

Did you try adding .Name to the end of the line? You cannot put an Instance into a string value, but you can put the Instance’s name inside.

4 Likes
#10

Thanks alot man and everyone who helped,
I missed the name and mixed up String and Instance
Thank you!

1 Like