Unneeded NullReferenceException

Question

As far as I know, it runs perfectly:

 #pragma strict
 function Update()
 {
     if (Input.GetMouseButtonDown(0)){
         Debug.Log("click");
           var ray = Camera.main.ScreenPointToRay (Input.mousePosition);
           var hit : RaycastHit;
         if (Physics.Raycast (transform.position, transform.forward)){
             Debug.Log(ray.direction);
             hit.rigidbody.AddForce(ray.direction * 2000); //Nullrefrenceexception here
             //hit.rigidbody.AddForceAtPosition(ray.direction*2000,hit.point); (Same error with this code
             Debug.DrawLine (ray.origin, hit.point);
         }  
     }
 }

Answer 1

Answer by jgodfrey · Feb 20, 2016 at 09:47 PM

Errrr.... You're never assigning anything to the "hit" variable, so a NullRef exception makes sense here. I assume you want to use a Physics.Raycast overload that accepts a "out RaycastHit" argument.

Add comment · Show 5 · Share

Slashscreen · Feb 21, 2016 at 02:56 AM 0

Share

But it says var hit : RaycastHit;

jgodfrey · Feb 21, 2016 at 03:33 AM 0

Share

 var hit: RaycastHit;

Just creates a variable of type "RaycastHit". It doesn't actually place any value in the variable...

Slashscreen jgodfrey · Feb 21, 2016 at 05:12 AM 0

Share

That's what I just said.

  #pragma strict
  function Update()
  {
      if (Input.Get$$anonymous$$ouseButtonDown(0)){
          Debug.Log("click");
            var ray = Camera.main.ScreenPointToRay (Input.mousePosition);
            **var hit : RaycastHit;**
          if (Physics.Raycast (transform.position, transform.forward)){
              Debug.Log(ray.direction);
              hit.rigidbody.AddForce(ray.direction * 2000); //Nullrefrenceexception here
              //hit.rigidbody.AddForceAtPosition(ray.direction*2000,hit.point); (Same error with this code
              Debug.DrawLine (ray.origin, hit.point);
          }  
      }
  }

jgodfrey · Feb 21, 2016 at 02:54 PM 0

Share

Again...

You are not storing any value in your "hit" variable. As I mentioned the first time, you want to use a different overload of Physics.Raycast to get the RaycastHit information.

Specifically, you probably want:

 Physics.Raycast (transform.position, transform.forward, out hit)

Slashscreen jgodfrey · Feb 21, 2016 at 06:07 PM 0

Share

It returns that it expected a ), not hit

Unneeded NullReferenceException

1 Reply

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions