Calculating World position of screen's corners?

Is this the correct way to do this?
To add some context, these positions will be used to place box collider around the screen.

Code (csharp):

1. // Screens coordinate corner location
2.     var upperLeftScreen = Vector3(0, Screen.height, ship.transform.position.y );
3.     var upperRightScreen = Vector3(Screen.width, Screen.height, ship.transform.position.y);
4.     var lowerLeftScreen = Vector3(0, 0, ship.transform.position.y);
5.     var lowerRightScreen = Vector3(Screen.width, 0, ship.transform.position.y);
6.    
7.     //Corner locations in world coordinates
8.     var upperLeft = camera.ScreenToWorldPoint(upperLeftScreen);
9.     var upperRight = camera.ScreenToWorldPoint(upperRightScreen);
10.     var lowerLeft = camera.ScreenToWorldPoint(lowerLeftScreen);
11.     var lowerRight = camera.ScreenToWorldPoint(lowerRightScreen);

 

Looks right to me!

 

Hmm, it doesn't work though...

 

In what way is it not working? The z component of your Vector3 variables should be measured in world units from the camera you are using, i.e. it is relative.

See the docs: http://unity3d.com/support/documentation/ScriptReference/Camera.ScreenToWorldPoint.html

If you use camera.nearClipPlane like in the example, you will get the exact world position of the screen corners.

 

Fixed it!

Here's what I did:

Code (csharp):

1. // Screens coordinate corner location
2.     var upperLeftScreen = Vector3(0, Screen.height, depth );
3.     var upperRightScreen = Vector3(Screen.width, Screen.height, depth);
4.     var lowerLeftScreen = Vector3(0, 0, depth);
5.     var lowerRightScreen = Vector3(Screen.width, 0, depth);
6.    
7.     //Corner locations in world coordinates
8.     var upperLeft = camera.ScreenToWorldPoint(upperLeftScreen);
9.     var upperRight = camera.ScreenToWorldPoint(upperRightScreen);
10.     var lowerLeft = camera.ScreenToWorldPoint(lowerLeftScreen);
11.     var lowerRight = camera.ScreenToWorldPoint(lowerRightScreen);
12.     upperLeft.y = upperRight.y = lowerLeft.y = lowerRight.y = ship.transform.position.y

where

Code (csharp):

1. depth = (ship.transform.position.y-camera.transform.position.y);

 

Thanks for sharing your solution.
I came up with an alternative way of computing the same thing, I'm not sure which one is lighter on the CPU though.
Also, my solution only works if you're using an orthographic camera in 2D.

Code (CSharp):

1.         var upperLeft = camera.transform.position + new Vector3(-camera.aspect * camera.orthographicSize, camera.orthographicSize);
2.         var upperRight = camera.transform.position + new Vector3(camera.aspect * camera.orthographicSize, camera.orthographicSize);
3.         var lowerLeft = camera.transform.position + new Vector3(-camera.aspect * camera.orthographicSize, -camera.orthographicSize);
4.         var lowerRight = camera.transform.position + new Vector3(camera.aspect * camera.orthographicSize, -camera.orthographicSize);

If your camera aspect doesn't change during the game, you can even precompute these Vector3 once in the class and save a bit more on computation.