Use a loop, this is gonna be the buggiest code I ever wrote T_T

I am hoping that this works... It is a small loop, I didn't test it because I don't have much time to test this. I don't know if it will work or not, this is yours buddy, good luck!

If it works, click the answer button, thanks :)

I set it up in a loop, made it wait the timelength of a random PACER type for disabled and thats basically all.

How to shorten a script

The easiest way that is apparent to me is to have one function that both enables and disables the sounds. I will write an example of what you could do and explain how every element works:

I assume that you can take it from here. I just wanted to provide you with an example of how to efficiently go about this issue of yours. I hope this helps and have a great day scripting!

You can use a for loop to achieve this. It would also be more efficient to use one function instead of two. Basically, we'll loop through all of the sounds and toggle them accordingly.

This answer's to be used as a reference

Just to change it up a little bit. :>

To accomplish something like this, you'd use a generic for to loop through the objects. Although it might be a good idea to use GetChildren to loop through the objects (as others have pointed out), I think GetDescendants would favor better, as we wont have to use two separate for loops to get the sounds and play/stop them. Here's an example:

VS

Pretty short and sweet answer, right? :>

Now, onto...

Stuff touched on, but never really explained

1. Generic For - To quote the page, "The generic for loop allows you to traverse all values returned by an iterator function." In other words, it allows you to iterate over a table; it'll return two values: The index, and the value for said index.

2. GetChildren - Creates a table with all the children within a object.

3. GetDescendants - Similar to GetChildren, but it also accounts for descendants; for example, if there's a model with parts in the Workspace, Workspace:GetDescendants() will get the model and the parts within the model.

4. IsA -- To put simply, it checks an object's ClassName, and if the ClassName is what you gave IsA, it'll return true, otherwise it'll return false.

If you have any questions, please let me know. Thanks for reading, and have a good day. :)

Use for loops to edit each child individually. Make a variable for script.Parent.Parent instead of repeating script.Parent.Parent for multiple lines.

The bracket operator

The dot operator (.) is similar to the bracket operator in that we can use it to get children of an object and get properties like so:

The bonus of this is that we need to use a string to retrieve a child/property with that name. The good thing about using a string instead of just the object name is that we can use spaces, it can start with a number and we can concatenate strings.

Concatenation

When you concatenate something, you are combining it together. We concatenate strings by putting 2 periods in between the strings. Here is an example of concatenating 2 strings together:

Blue and Pink are my favorite colors

It combines both strings into 1.

Simplifying your function

We can actually turn your function into a single script by combining your two functions into 1. We can use your variable enabled as a way to see if we should enable or disable your objects.

For loops

For loop will loop over and over again for a set amount of times. We can use this to our advantage to make it loop for every item inside of a table. We can use pairs to return the objects inside of a table.

The first array loops 10 times because we set it to do that. The second one will loop 3 times because we said we wanted it to loop for each value inside of the array.

Final Product

Hope it helps!

you can use a generic for loop which literates through a table, in which :GetChildren() returns.

pretty simple script that will help 100% if it doesn't work then tell me because i wouldn't know why and if it did something you didn't want it to do then try asking your questions more specifically.