Well first of all (first if => false and GetKeyup => GetKeyUp):
function Update() {
if (Input.GetKeyDown("Fire1")) {
if (light.enabled == false)
light.enabled = true;
}
if (Input.GetKeyUp("Fire1")) {
if (light.enabled == true)
light.enabled = false;
}
}
Secondly, as Khenkel mentioned, light is already used as default, so try renaming it.
Quick tip, you can use F2 to refactor the name, so you only have to do it once and the system will apply for the rest of the script :)
Thirdly, you can enable a "Light" component, right now you have a GameObject component, you can either deactive only the light component, or the entire gameObject.
If you want to use the "enable" on a GameObject, you should use SetActive(true); (or false)
All of the Above leads to (Possibly):
public var newNameForLight: GameObject;
function Update() {
if (Input.GetKeyDown("Fire1")) {
if (newNameForLight.activeSelf == false)
newNameForLight.SetActive(true);
}
if (Input.GetKeyUp("Fire1")) {
if (newNameForLight.activeSelf == true)
newNameForLight.SetActive(false);
}
}
OR
public var newNameForLight: Light;
function Update() {
if (Input.GetKeyDown("Fire1")) {
if (newNameForLight.enabled == false)
newNameForLight.enabled = true;
}
if (Input.GetKeyUp("Fire1")) {
if (newNameForLight.enabled == true)
newNameForLight.enabled = false;
}
}
In both cases, don't forget to drag in the public variable :)
Hope that helps, if I misunderstood or made grammar mistakes, my apologies ;)
↧
