ScreenToWorldPoint and Mouse Position

Question

I'm trying to have an object move to a certain area on tap. I've done "research" (looked at 3 articles, 2 videos, then the unity scripting API) and it led me to this.

 public int TapDamage = 10;

 Vector3 StartPos;
 Vector3 p = new Vector3();
 Camera c = Camera.main;
 Event e = Event.current;
 Vector2 mousePos = new Vector2();


 
 void MovingClickBox()
 {


     if (Input.GetMouseButtonDown(0) == true)
     {
         mousePos = Input.mousePosition;

         mousePos.x = e.mousePosition.x;
         mousePos.y = c.pixelHeight - e.mousePosition.y;


         p = c.ScreenToWorldPoint(new Vector3(mousePos.x, mousePos.y, c.nearClipPlane));

         gameObject.transform.position = p;


     }
  
 }


 private void FixedUpdate()
 {
     MovingClickBox();
 }

Right now this isn't moving the object at all and I don't know how to fix it. Please help. Also, unity is saying that get_main isn't allowed in a monobehaviour constructor; don't know what that means but seems useful.

Answer 1

Best Answer

Answer by Bunny83 · Feb 14, 2018 at 02:36 AM

There are several things wrong here:

First of all understand the difference between ScreenSpace and GUI space. ScreenSpace has it's origin at the bottom left of the screen while GUI space has it's origin at the top left.
Input.mousePosition returns the mouse position in ScreenSpace. The Event class gives the mouse position in GUI space.
ScreenToWorldPoint expects the input point in ScreenSpace. So there's no need to invert the y axis.
The Event class should only be used in OnGUI. It's tightly coupled with OnGUI.
Do never check one-time events (like MouseDown, MouseUp) inside FixedUpdate. FixedUpdate is only ever relevant when you deal with continous forces in the physics system. Anything else should or have to be done in Update. FixedUpdate runs in most cases at a slower rate and therefore can "miss" frames. Since one time events are only active for one frame they can not be checked consistently inside FixedUpdate.
Keep in mind that the near clipping plane for a perspective camera is usually extremely small. So your object wouldn't move much in terms of world space movement. Also if you place an object on the clipping plane it usually wouldn't be visible since as the name tells you the scene is clipped at the clipping plane.

In a 3d environment you usually want to place an object either a certain distance from the camera or at the raycast hit point in the scene. In a 2d game when using an orthographic camera you may want to set the z position of the returned vector to 0. However from your description and from your code we can't determine if this is a 2d or 3d problem.

Add comment · Show 3 · Share

yummy81 · Feb 14, 2018 at 10:13 AM 0

Share

@Bunny83 You said that FixedUpdate is for the physics system. That's right, but why according to examples in the documentation, methods like Physics.SphereCast (and maybe other methods from Physics class too, for which there are no examples provided), are placed in Update() ?

meat5000 ♦ yummy81 · Feb 14, 2018 at 10:20 AM 1

Share

Such Physics methods do not require Updating frame to frame. This means they do not need the benefit of the Fixed Tick Rate and so can be fired from Update.

yummy81 meat5000 ♦ · Feb 14, 2018 at 10:47 AM 0

Share

Thank You!

Answer 2

Answer by Annick35 · Feb 14, 2018 at 12:07 AM

Add the Camera c = Camera.main; into the start method.

So keep Camera c; at line 5.

Then add

 void Start()
 {
     c = Camera.main;
 }

at around line 10.

So it should look like

 public int TapDamage = 10;
  Vector3 StartPos;
  Vector3 p = new Vector3();
  Camera c;
  Event e = Event.current;
  Vector2 mousePos = new Vector2();
 
 void Start()
 {
      c = Camera.main;
 }
  
  void MovingClickBox()
  {
      if (Input.GetMouseButtonDown(0) == true)
      {
          mousePos = Input.mousePosition;
          mousePos.x = e.mousePosition.x;
          mousePos.y = c.pixelHeight - e.mousePosition.y;
          p = c.ScreenToWorldPoint(new Vector3(mousePos.x, mousePos.y, c.nearClipPlane));
          gameObject.transform.position = p;
      }  
  }
  private void FixedUpdate()
  {
      MovingClickBox();
  }

Add comment · Show 5 · Share

Golden_Gamer · Feb 14, 2018 at 01:13 AM 0

Share

I tried this and it got rid of the old exception but added 2 more: Internal_GetGUIDepth is not allowed to be called from a monobehaviour constructor and this second one was here already just forgot to mention it; it says that object reference not set to an instance of an object. Also, I should have said this at the start of this comment; it still doesn't move.

Annick35 Golden_Gamer · Feb 14, 2018 at 01:57 AM 0

Share

To fix the Event Error, you would put take Event e = Event.Current; and change it to Event e;

then add this to Start()

 void Start()
 {
     c = Camera.main;
     e = Event.current;
 }

Annick35 Annick35 · Feb 14, 2018 at 01:59 AM 0

Share

Then to fix the Object reference on your void MovingClickBox() change it to

 void MovingClickBox()
   {
       if (Input.GetMouseButtonDown(0) == true)
       {
           mousePos = Input.mousePosition;
           mousePos.x = Input.mousePosition.x;
           mousePos.y = c.pixelHeight - Input.mousePosition.y;
           p = c.ScreenToWorldPoint(new Vector3(mousePos.x, mousePos.y, c.nearClipPlane));
           gameObject.transform.position = p;
       }  
   }

Show more comments

ScreenToWorldPoint and Mouse Position

2 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions