- Home /
OnTriggerExit does not seem to be functioning, does not stop my coroutine
Ok so I have made a method where, when the player touches a collider (flowers), they get pollen every second. I have made an OnTriggerEnter for entering the collider and triggering the coroutine method, and an OnTriggerExit for stopping the pollen every second.
However, once the player stops touching the flower collider, the pollen count continues to increase! I cannot figure out how to get the pollen count to stop working once the player isn't touching the flowers. I have also noted that when moving to another flower, the coroutine fires again, making the pollen go up faster and that shouldn't be happening either. I have made sure all the layers were named correctly, and all the colliders needing to be triggers were checked.
Any suggestions would be super helpful!
using System;
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
public class AddPollenScript : MonoBehaviour
{
[SerializeField] int givePollen = 1;
int pollen = 0;
Collider2D flowerCollider;
[SerializeField] Text pollenText;
// Start is called before the first frame update
void Start()
{
flowerCollider = GetComponent<Collider2D>();
UpdatePollenText();
}
// Update is called once per frame
void Update()
{
//AddPollen();
//StopAddingPollen();
}
private void OnTriggerEnter2D(Collider2D other)
{
if (flowerCollider.IsTouchingLayers(LayerMask.GetMask("Good Bees", "Player")))
{
StartCoroutine(GivePollenPlayer());
Debug.Log("Touching Good Bee.");
}
if (flowerCollider.IsTouchingLayers(LayerMask.GetMask("Evil Bees")))
{
StartCoroutine(GivePollenEnemy());
Debug.Log("Touching Evil Bee");
}
}
private void OnTriggerExit2D(Collider2D other)
{
if (!flowerCollider.IsTouchingLayers(LayerMask.GetMask("Good Bees", "Player")))
{
StopCoroutine(GivePollenPlayer());
Debug.Log("Touching Good Bee.");
}
if (!flowerCollider.IsTouchingLayers(LayerMask.GetMask("Evil Bees")))
{
StopCoroutine(GivePollenEnemy());
Debug.Log("Touching Evil Bee");
}
}
private IEnumerator GivePollenPlayer()
{
while (true)
{
pollen += givePollen;
UpdatePollenText();
yield return new WaitForSeconds(1);
}
}
private IEnumerator GivePollenEnemy()
{
throw new NotImplementedException();
}
private void UpdatePollenText()
{
pollenText.text = pollen.ToString();
}
}
Answer by Wolfgang_Brabaender · Dec 16, 2019 at 10:58 AM
For controlled Start and Stop of Coroutines string references are not optimal. (I would say string references are always a mess... ;) ) Try sth. like
Coroutine coroutine;
void OnTriggerEnter2D(Collider2D other){
if(coroutine != null) return; //check if croutine is already running)
coroutine = GivePollenPlayer();
StartCoroutine(coroutine);
}
void OnTriggerExit2D(Collider2D other){
if(coroutine == null) return; //classic nullref escape
StopCoroutine(coroutine);
coroutine = null;
}
untested code ;)
Your answer
![](https://koobas.hobune.stream/wayback/20220612220139im_/https://answers.unity.com/themes/thub/images/avi.jpg)
Follow this Question
Related Questions
OnTriggerEnter2D works without being actually triggered. 0 Answers
Colliding two GameObjects 1 Answer
How to do collision of external objects on script? 1 Answer
Detect and react to collisions 2 Answers
Check Object Is Not Colliding 2 Answers