Foreach/list loop problem

Question

I am having a problem with a foreach loop. When my AOE ability hits an enemy it creates a CircleCollider2D and puts all the enemies on a list, which then the loop goes through and adds damage to their scripts.

private void OnTriggerEnter2D(Collider2D other)
{
    if(other.tag == "Enemy")
    {
        targets.Add(other);

        foreach (Collider2D enemy in targets)
        {
            EnemyHealth hp = enemy.GetComponentInChildren<EnemyHealth>();

            hp.TakeDamage(abilityConfig.baseDamage);

            print(enemy.name + " takes " + abilityConfig.baseDamage + "damage!");
        }
    }        
}

But as you can see from the picture below the loop goes 3 times over one enemy, 2 times over the second one and once over the third.

As these enemies are the same type they all have the same EnemyHealth script. If anyone would point me in the right direction I would greatly appreciate it.

Answer 1

OnCollisionEnter executes for each entity that collided. So you don't need foreach loop at all.

private void OnTriggerEnter2D(Collider2D other)
{
    if (other.tag == "Enemy")
    {
        EnemyHealth hp = enemy.GetComponentInChildren<EnemyHealth>();
        hp.TakeDamage(abilityConfig.baseDamage);
        print(enemy.name + " takes " + abilityConfig.baseDamage + " damage!");
    }
}

Answer 2

In response to your comment, if you want to hit every enemy than i suggest you get all the reference of your enemies and then apply the damage like following

private void OnTriggerEnter2D(Collider2D other)
{
    if(other.tag == "Enemy")
    {
        // Change targets type to List<GameObjects>
        targets.Clear();
        targets.AddRange(GameObject.FindGameObjectsWithTag("Enemy"));

        foreach (GameObject enemy in targets)
        {
            EnemyHealth hp = enemy.GetComponentInChildren<EnemyHealth>();

            hp.TakeDamage(abilityConfig.baseDamage);

            print(enemy.name + " takes " + abilityConfig.baseDamage + " damage!");
        }
    }        
}  

We are here basically checking if we hit any enemy, if thats the case we are first clear the list then add all the enemies to the list, so no duplicates there,

Then we are looping through all pretty much the same