How to get a component from a prefab.

When you say the "other enemy seems to be invincible" which object are you talking about? This script appears to find one object named ToonSoldier_WW2_German_Infantry and decrement it's child objects health and then destroy the gameobject the script is attached to.

GameObject.Find looks in the entire scene for the first instance of an object with that name. If you have multiple enemies with that same name, there's no way to guarantee which one is being returned. It's very likely that the enemy that you're reducing the health of is not the same one that you're shooting at.

If the Collider is attached to the enemy, then you may be able to get the correct enemy health script by doing enemyTarget.gameObject.GetComponentInChildren<EnemyHealthScript>()

Alternatively, you may want to look at the documentation for Transform.Find. You would write enemyTarget.transform.Find("ToonSoldier_WW2_German_Infantry") to get the child gameobject of the enemy that you want.

1. Do not use "tags" they are slow because they have to be converted to C#, ins$$anonymous$$d use gameobject.name since I can guarantee all your enemies are named "Enemy"

2. Don't use GameObject.Find either, it looks through every gameobject in the scene and is INCREDIBLY slow. And you're also using it wrong, since once your Enemy collides with your trigger you're looking for it again AND doing it on every single trigger of this function.

3. Take a look at using a class system for your Enemy and Infantry, you're using some things that can easily be contained in their own objects, you don't want to rely this much on Unity scene functions.

People are right, and here's how I'd do it.

Should be gameObject, if that's the one from the scene, and don't forget to add (Clone) to the name since instances are called that way if you don't change the name manually.

"Find" is a great function, you just need to use it right: when instantiating, use smth like:

 var allEnemies = GameObject.FindGameObjectsWithTag("Enemy");
 var totalEnemiesSpawned: int;
 
 function Spawn()
 {
 var newEnemy = Instantiate(bla bla bla);
 totalEnemiesSpawned++;
 newEnemy.name = "ToonSoldier_WW2_German_Infantry" + totalEnemiesSpawned;
 }
 
 function OnTriggerExit()
 {
 for(var thisEnemy in allEnemies)
 {
 //your code including thisEnemy variable...
 }
 }

That will give unique names for all instances of the enemy. And THAT's when you use find by name among those with tags.

I am new to unity and the general field of game design and code. I could tell that the flaw was that it was searching for all of the enemy with the same name and that the code was over complicated (which I had earlier simplified but did not update). I could tell the flaws but did not have the experience to figure out so thanks for helping and I will probably be asking a lot more questions in the future. By the way @jmgek what do you mean by hardest names.

Hard set values* sorry about that using my phone so I have autocorrect

@jmgek I see what you mean, this is definitely something that I will change in other scripts.