What is wrong with this script?

Question

i keep getting t$$anonymous$$s error with t$$anonymous$$s script. "NullReferenceException: Object reference not set to an instance of an object" T$$anonymous$$s is supposed to be in javascript... in case i did somet$$anonymous$$ng wrong, language-wise.

its somet$$anonymous$$ng with the "if(lightObject.transform.Rotation.x > 31){" but i dont know what im doing wrong.

Please help!

Heres the code:

 var lightObject : GameObject;
 var breat$$anonymous$$ngObj : GameObject;
 var hasSprinted : boolean = false;
 
 function Update () {
 
 if(breat$$anonymous$$ngObj.GetComponent(SprintScript).isSprinting == true){
 
  lightObject.transform.Rotate(Time.deltaTime * 50, 0, 0);
  
  hasSprinted = true;
 
  if(lightObject.transform.Rotation.x > 31){
  
  lightObject.transform.Rotation.x = 31;
  
  hasSprinted = true;
  
  }
  }
  
 else if(breat$$anonymous$$ngObj.GetComponent(SprintScript).isSprinting == false && hasSprinted == true){
 
  lightObject.transform.Rotate(Time.deltaTime * -50, 0, 0);
  
  if(lightObject.transform.Rotation.x > 0){
  
  lightObject.transform.Rotation.x = 0;
  }
  }
 }

Answer 1

Best Answer

Answer by Eric5h5 · Aug 05, 2012 at 12:56 AM

There's no such t$$anonymous$$ng as transform.Rotation. Capitalization is very important. Also, that won't work even when corrected, since transform.rotation is a quaternion, w$$anonymous$$ch is a 4-dimensional representation of an object's rotation, so rotation.x doesn't correspond to the x axis and is not in degrees anyway. Since reading one component from eulerAngles tends to be unreliable (since there's more than one valid way to represent an object's rotation), you're better off tracking the rotation yourself in a separate variable.

Add comment · Show 12 · Share

DNP · Aug 05, 2012 at 12:59 AM 0

Share

What would the variable represent?? would it be lightObject.localRotation?

Eric5h5 · Aug 05, 2012 at 01:03 AM 1

Share

No, localRotation is also a quaternion. The variable would represent the rotation in degrees; see the MouseLook script in the character controller standard package for an example of a script that uses its own variables to track rotation of specific axes instead of trying to read it from the object (which is fairly problematic).

DNP · Aug 05, 2012 at 01:15 AM 0

Share

Alright. I see what you mean. But could you post an example? or if your bored perhaps a fix? becuase it will help me understand what you mean. Also, in the mouse look script, im guessing you mean these variables.

public enum RotationAxes { MouseXAndY = 0, MouseX = 1, MouseY = 2 } public RotationAxes axes = RotationAxes.MouseXAndY; public float sensitivityX = 15F; public float sensitivityY = 15F;

public float minimumX = -360F; public float maximumX = 360F;

public float minimumY = -60F; public float maximumY = 60F;

float rotationY = 0F;

Eric5h5 · Aug 05, 2012 at 01:27 AM 0

Share

I'm referring to the rotationY variable.

DNP · Aug 05, 2012 at 03:12 AM 0

Share

Why does it have an "F" next to the value???

Show more comments

Answer 2

Answer by slayer29179 · Aug 05, 2012 at 12:41 AM

Sure :) Try These;

 if(lightObject.transform.Rotation.x > 31){
 
 if(lightObject.transform.rotation.x > 31){
 
 if(lightObject.transform.Rotate.x > 31){
 
 if(lightObject.transform.rotate.x > 31){
 
 if(lightObject.transform.Vector3.right > 31){

and see if they work :)

Add comment · Show 4 · Share

DNP · Aug 05, 2012 at 12:48 AM 0

Share

The second one works! thank you!

slayer29179 · Aug 05, 2012 at 12:52 AM 0

Share

Woohoo ^^ my first accepted answer :D thank you ! :D

DNP · Aug 05, 2012 at 12:53 AM 0

Share

no problem :3

Eric5h5 · Aug 05, 2012 at 12:57 AM 0

Share

Actually none of those is correct, sorry...see my answer for an explanation.

What is wrong with this script?

2 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions