Mouse Click is passing the GUI

Hi!
I wrote a simple script which is selecting the GameObject clicked by Mouse:

Code (CSharp):

1. public static GameObject SelectedGameObject()
2.     {
3.         ray = Camera.main.ScreenPointToRay(Input.mousePosition);
4.         if (Physics.Raycast(ray, out hit, 10f))
5.         {
6.             if (hit.transform.gameObject.tag != "Player" && hit.transform.gameObject.tag != "ItemDatabase" && hit.transform.gameObject.tag != "Coller")
7.             {
8.                 //return hit.transform.gameObject;
9.                 if (hit.transform.gameObject != null)
10.                 {
11.                     selected = hit.transform.gameObject;
12.                     QuickMenu.Initialized = false;
13.                     return selected;  
14.                 }
15.                 else
16.                 {
17.                     QuickMenu.Initialized = false;
18.                     return selected;    
19.                 }
20.              
21.             }
22.             if(hit.transform.gameObject == null)
23.             {
24.                 return null;
25.             }
26.              
27.         }
28.             return null;
29.     }

The problem is when I'm clicking on the GUIControls when the GameObject is behind my GUI.My script is selecting the GameObject behind the GUI but I don't want this result.

 

You can use UnityEngine.EventSystems together with EventSystem.current.IsPointerOverGameObject() and have a bool to decide wither you were on top of a UI element or not like this,

Code (CSharp):

1. if (EventSystem.current.IsPointerOverGameObject())
2.                     {
3.                         wasOverUI = true;
4.                     }

You can couple this with a Canvas Group to decide which UI element is interactable or not.

 

@Tamerqarrain
I'm using OnGUI