OnTriggerEnter being called multiple times in succession?

Question

Hey all,

Basically, I have a box that the player has to be able to collect. When the player "collects" it, the box will actually just instantly move to another spawnpoint, and I'll also execute scoring stuff. When the player object passes over a box, OnTriggerEnter(Collider hitObject) is called, and I do a check to make sure that the object I hit is actually a box that can be collected. If it is, I move the box to a different location to give the effect

The problem is that when my player object hits the box, OnTriggerEnter is called two or three times, which will undoubtedly cause issues for keeping track of the score. It's my understanding that OnTriggerEnter is called once upon the initial collision with the object, so I'm not sure why it's being called multiple times per collision.

Some of my code follows:

 //From the player object's script
 
 void OnTriggerEnter (Collider hitObject)
     {    
         if (hitObject.tag == "peep")
         {
             hitObject.transform.position = new Vector3(0,-100,0);
             Debug.Log("Collected");
             hitObject.transform.GetComponent<peepControl>().collect();
         }
     }
 
 //From the peepControl script
 
 public void collect()
     {
         int i = Random.Range(0,allSpawns.Length);
         while (allSpawns[i] == currentSpawn)
             i = Random.Range(0,allSpawns.Length);
         
         transform.position = allSpawns[i].transform.position;
         currentSpawn = allSpawns[i];
     }

Through Debug.Log I found that the OnTriggerEnter function was testing true on its if statement three times before the object actually moved at all. Suggestions?

Answer 1

Best Answer

Answer by fafase · Jul 02, 2014 at 12:36 AM

Is the collider on your player a Character Controller or a sphere collider or a capsule collider? Those being circular may have two (or more) contact points.

Just use a boolean to prevent multiple calls.

 void OnTriggerEnter (Collider hitObject)
 {
     if(isColliding) return;
     isColliding = true;
     // Rest of the code
 }
 
 void Update(){
     isColliding = false;
 }

Even though your collider is not one of those I mentioned, this should work. If the boolean is true, quit the method, else set it to true and do the rest.

Since Physics is done all at once and before update in the same frame, if you have a second collision detected, nothing will happen since the first already set the boolean.

Update is just putting it back to false.

EDIT: In order to remove the update, one can also do as such:

 void OnTriggerEnter (Collider hitObject)
 {
     if(isColliding) return;
     isColliding = true;
     // Rest of the code
     StartCoroutine(Reset());
 }

 IEnumerator Reset()
 {
      yield return new WaitForEndOfFrame();
      isColliding = false;
 }

Add comment · Show 11 · Share

TRON-dll · Jul 02, 2014 at 08:07 AM 0

Share

TruffelsAndOranges · Jul 21, 2015 at 04:35 PM 0

Share

fafase · Jul 24, 2015 at 06:05 PM 0

Share

Siraaaj-Ahmed · Dec 28, 2016 at 10:59 PM 0

Share

aloften · Aug 30, 2017 at 02:11 AM 0

Share

fafase aloften · Sep 01, 2017 at 05:28 AM 0

Share

Show more comments

Answer 2

For some reason, setting a boolean check for colliders can sometimes cause the boolean value to either flip, or to turn on/off the OnTriggerEnter/OnTriggerExit methods while remaining in the trigger area, like opening a door and closing it repeatedly while in the trigger area, or flipping the boolean value so it closes when you approach and opens when you walk away.

using a simple variable iterator prevents the actual method from being called repeatedly, guaranteeing only one execution per collision.

 //previous code in class
 //using a door opening and closing as an example
 [SerializeField] private GameObject device;
     private int i = 0;
 
 
     void OnTriggerEnter(Collider other){
         DoorScript door = device.GetComponent<DoorScript>();
         if (i == 0) {
             if (door != null) {
                 door.Operate ();
                 i++;
             }
         }
 
     }
     void OnTriggerExit(Collider other){
         DoorScript door = device.GetComponent<DoorScript>();
         if (i == 1) {
             if (door != null) {
                 door.Operate ();
                 i = 0;
             }
         }
 
     }
 /* All DoorScript contains is a function Operate() and a boolean for if the door is open, and an if/else statement in Operate() for moving door to open if bool is false (set to closed) and moving door to closed if bool is true (set to open)

Answer 3

Answer by sisse008 · Mar 25, 2018 at 10:52 AM

ontriggerenter should be on each "collectible" object. not on the player:

 class Target
 {
         private collected = false;
         void collect() {
           ....
           collected = true;
          }
        void OnTriggerEnter (Collider other)
        {   
            if (!collected && other.tag == "player")
            {
                transform.position = new Vector3(0,-100,0);
                Debug.Log(gameObject.name +" was Collected");
                collect();
          }
      }

Add comment · Show 1 · Share

SchafiSchafoesselchen · May 06, 2018 at 12:00 AM 0

Share

Answer 4

Answer by shawnsi · Mar 28 at 03:18 PM

I know this is an old topic, but I wanted to share another solution just in case. I solved this issue by setting character Rigidbody's IsKinematic to "true". My guess is that when IsKinematic is false, the colliders would push the character back enough to trigger OnTriggerExist. This caused a trigger loop and caused it to run OnTriggerEnter and OnTriggerExit multiple times.

Add comment · Share

OnTriggerEnter being called multiple times in succession?

4 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions