An ugly fire-arc-checking function

I have a rather ugly C# function that determines if a turret with a limited fire arc is allowed to rotate far enough to shoot at a target. I'm posting this for two reasons: Firstly, I'd like to see if someone has a better solution, since this feels clumsy to me. Secondly, I haven't found accessible code for this purpose, which seems like it should be a fairly common one, so someone might stumble across this and find it useful.

There is a Transform called turretBase, and its forward is the line against which its rotation limits are defined, in the up, down, left, and right directions. It checks to see if the local angle to the direction to the world-position argument is within the bounds of the rotation limits. I can't just use a single Vector3.Angle check because I am defining each direction's limit separately.

Code (csharp):

1.  
2.     public bool TargetIsInArc(Vector3 pos)
3.     {
4.         Vector3 dir = turretBase.InverseTransformDirection(pos - turretBase.position);
5.         float   turnLimit  = dir.x < 0 ? leftRotationLimit : rightRotationLimit;
6.         float   pitchLimit = dir.y < 0 ? downRotationLimit : upRotationLimit;
7.         bool    canSee = (Vector3.Angle(Vector3.forward, new Vector3(dir.x, 0, dir.z)) <= turnLimit
8.                           &&
9.                           Vector3.Angle(Vector3.forward, new Vector3(0, dir.y, dir.z)) <= pitchLimit);
10.  
11.         Debug.DrawLine(turretBase.position,
12.                        pos,
13.                        canSee ? Color.green : Color.red,
14.                        Time.deltaTime);
15.  
16.         return canSee;
17.     }
18.  

So, any suggestions?

 

Looks fine to me. I`ve done clumsier things
Maybe look at Mathf.atan2 ? http://docs.unity3d.com/ScriptReference/Mathf.Atan2.html