door opens on event

Hello,
i tried to wrtie script which will open door when the player will be in trigger. But when i try to open door function OnTriggerStay is calling itself 4 times. How can i fix this issue?

Code (CSharp):

1.  
2.     public bool trigered=false;
3.     public bool Opened=true;
4.  
5.     private Animation anim;
6.     private GameObject player;
7.  
8.  
9.     void Start () {
10.         anim = GetComponent<Animation> ();
11.         player = GameObject.FindGameObjectWithTag ("Player");
12.     }
13.     void OnTriggerStay(Collider other) {
14.         if (other == player)
15.         {
16.             trigered = !trigered;
17.             Opened=!Opened;
18.             if (Input.GetKeyDown (KeyCode.E))
19.             {
20.                     if (Opened==true)
21.                     anim.Play ("CloseDoor");
22.                     else
23.                     anim.Play ("OpenDoor");
24.                    
25.                 }
26.             }
27.         }
28.     }

 

OnTriggerStay is called every frame the colliders are touching. Is that what you want? Have you considered OnTriggerEnter which is only called the first time the colliders touch

 

But inOnTrigerEnter the moment when it is possible to open door is very short. Thats why i put there triggered as a bool to check how long i will be able to OpenDoor I want to be able to open door whenever i want. Thats why i gave there OnTrigerStay. Have u different way to do that?

 

The main issue you have is your boolean values. While the player is in the trigger, both trigered and Opened are switched in every frame which should definitely not be the case.
First, you can forget about trigered, because it is not used at all and it has a typo . If you want to use it, you should set it to true in there and have the trigger exit event as well which then turns it off.
Besides that, Opened should only be changed in when you start to play the animation. As you start the close animation, set it to false, when you start the open animation, set it to true. If you do that, don't forget the curly braces!

 

Yeah, you are correct! Now it works fine! Thank you so much!