Lights not turning on?
The following script is supposed to fade the lights off if they are on, and fade them on if they are off.
01 | local Me = script.Parent |
02 |
03 | local H1 = game.Workspace.House_Lights.HouseLight_1.ReadInside.Light.Light.SpotLight |
04 | local H2 = game.Workspace.House_Lights.HouseLight_2.ReadInside.Light.Light.SpotLight |
05 | local H3 = game.Workspace.House_Lights.HouseLight_3.ReadInside.Light.Light.SpotLight |
06 | local H4 = game.Workspace.House_Lights.HouseLight_4.ReadInside.Light.Light.SpotLight |
07 | local H5 = game.Workspace.House_Lights.HouseLight_5.ReadInside.Light.Light.SpotLight |
08 | local H6 = game.Workspace.House_Lights.HouseLight_6.ReadInside.Light.Light.SpotLight |
09 | local H7 = game.Workspace.House_Lights.HouseLight_7.ReadInside.Light.Light.SpotLight |
10 | local H8 = game.Workspace.House_Lights.HouseLight_8.ReadInside.Light.Light.SpotLight |
11 | local H9 = game.Workspace.House_Lights.HouseLight_9.ReadInside.Light.Light.SpotLight |
12 |
13 | local AllHouse = H1, H2, H3, H4, H5, H6, H7, H8, H9 --WHERE THE H's ARE LISTED |
14 |
15 | local on_or_off = "off" |
The problem I'm having is it only fades on ONE of the lights (that's H1). None of the others respond. I'm not going into the program and individually telling each and every single one to turn on because that would be ridiculous. If you need any more information about something, I'll be happy to comply.
1 answer
It's a good instinct that you don't want to repeat yourself. Repeating code for each one is not the right way to approach this :)
This is not how tuples work:
1 | local AllHouse = H1, H2, H3, H4, H5, H6, H7, H8, H9 |
When you have an assignment with commas, each corresponding thing on the left gets set to the corresponding thing on the right. In this statement, all of the ones after H1 are ignored.
There is no way to make one value be a 'merged' version of a bunch of values (like you wanted AllHouse to be), where setting something on it sets something on all of the lights. Instead, you make a collection of lights and do something to each element of the collection:
1 | -- make a list: |
2 | local lights = {H1, H2, H3, H4, H5, H6, H7, H8, H9} |
Note the curly braces.
lights is a list -- it does not have an .Brightness property. Instead, you use a for loop to do something for each light in lights:
1 | for i=60,0, -0.5 do |
2 | for _, light in pairs(lights) do |
3 | light.Brightness = i |
4 | end |
5 | wait(0.25) |
6 | end |
Less code to define your lights
Instead of typing out the full amount to define each H1 to H9, you could just organize your model so that only lights are in workspace.House_Lights. Then you can more easily generate the lights list:
1 | local lights = {} -- start with an empty list |
2 | for _, light in pairs(workspace.House_Lights:GetChildren()) do |
3 | local spotLight = light.ReadInside.Light.Light.SpotLight |
4 | table.insert(lights, spotLight) -- add the spot light to the `lights` list |
5 | end |
0