Need Help Using C# To Active Multiple Objects with one Switch

Unity Engine
1

What I am trying to do is use one Switch to turn multiple Light2d Using GameObject[ ]. Well It works but I get a Null Reference Object reference not set to an instance of object. The lights still get turned on even if they are not in the public array and that is not what I want. What I need is that one I jump on a switch it activates a bunch of lights (Light2D) that are in the public array with out the errors. Here is the code that I am using to try to accomplish this. I have tried so much and nothing works.

public class LightsOn : MonoBehaviour
{
    public GameObject lSwitch;
    public GameObject[] lights;
    void Start()
    {
        lights = GameObject.FindGameObjectsWithTag("LightToActivate");

      
    }

    void Update()
    {
        LightIsOn();
    }

    public void LightIsOn()
    {
        if (lSwitch.GetComponent<SwitchActivate>().isOn == true)
        {
            foreach (GameObject lights in lights)
           {

                lights.GetComponent<UnityEngine.Experimental.Rendering.LWRP.Light2D>().intensity = 1;
            }

        }
    }
}
2
3

Two things come to mind:

foreach (GameObject lights in lights)

You name both the GameObject an the Array ‘lights’. Does that even compile? Make sure to name your variables carefully.

In Editor, check if there is an empty ‘slot’ in your Array, i.e. the number of objects show for length the is greater than the number of objects you filled in. This can easily happen if you deleted an object that was previously in the array.

4

It does work all of the lights light up when I step on the switch but I get a NullReferenceException: Object reference not set to an instance of an object. I have never used an array before like this.

5

All slots are full I do not get it.

6

GetComponent might return null if object is not active