Unity3D：当敌人（克隆）被击中时，原来的预制敌人受到伤害而不是克隆？

Question

The clones spawn with an EnemyHealth script where their health parameters are set. Within in this script is a function called TakeDamage(). My only guess is that I'm not explicitly defining which enemy needs to have it's health drained within TakeDamage() but I'm having some trouble wrapping my head around that because each clone has its own health and TakeDamage() is called from my PlayerAttack script when the weapon collides with the enemy, so I was assuming it would only happen to the colliding enemy. But I guess I need a defined way for TakeDamage() to only affect the enemy that's involved with the collision.

I'm self taught so I apologize if this is simple or a poor question, I've searched all over the place for several days now so I hope someone can help!

EnemyHealth Script:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class PlayerAttack : MonoBehaviour
{
    public EnemyHealth enemyHealth;
    public Animator animator;
    public LayerMask enemyLayers;
    public bool allowDamage = false;

    // Start is called before the first frame update
    void Start()
    {

    }

    // Update is called once per frame
    void Update()
    {
        if (Input.GetMouseButtonDown(0))
        {
            clickAttack();
        }
    }

    private void AnimationCheck()
    {
        if (animator.GetCurrentAnimatorStateInfo(0).IsName("Stab"))
        {
            allowDamage = true;
        }
        else
        {
            allowDamage = false;
        }
    }
    private void clickAttack()
    {
        animator.SetTrigger("Attacking");
    }

    void OnCollisionEnter(Collision collision)
    {
        if (collision.gameObject.CompareTag("Enemy"))
        {
            AnimationCheck();
            if (allowDamage == true)
            {
                enemyHealth.TakeDamage();
            }

        }
    }
}

PlayerAttack Script:

 using System.Collections; using System.Collections.Generic; using UnityEngine; public class PlayerAttack: MonoBehaviour { public EnemyHealth enemyHealth; public Animator animator; public LayerMask enemyLayers; public bool allowDamage = false; // Start is called before the first frame update void Start() { } // Update is called once per frame void Update() { if (Input.GetMouseButtonDown(0)) { clickAttack(); } } private void AnimationCheck() { if (animator.GetCurrentAnimatorStateInfo(0).IsName("Stab")) { allowDamage = true; } else { allowDamage = false; } } private void clickAttack() { animator.SetTrigger("Attacking"); } void OnCollisionEnter(Collision collision) { if (collision.gameObject.CompareTag("Enemy")) { AnimationCheck(); if (allowDamage == true) { enemyHealth.TakeDamage(); } } } }

Screenshot of Unity Scene:

Clone marked in blue is the one being hit, enemy marked in yellow is the original that's taking damage when other enemies are being hit.

Answer 1

The reason only one enemy is taking damage is because you are referencing one enemies health component. Whatever enemy you assign in the inspector to your enemyHealth variable is the one that will take damage.

Remove the enemyHealth variable from your PlayerAttack class.

Inside of the collision function, you will get the enemyHealth from the object that you collided with.

void OnCollisionEnter(Collision collision)
{
    if (collision.gameObject.CompareTag("Enemy"))
    {
        AnimationCheck();

        if (allowDamage)
        {
            var enemyHealth = collision.gameObject.GetComponent<EnemyHealth>();
            if (enemyHealth)
            {
                enemyHealth.TakeDamage();
            }
        }
    }
}

You can potentially get rid of the CompareTag line if only enemies will have the EnemyHealth script (which it sounds like is the case). Then you only need to check if the gameObject you collided with has that script to know it is an enemy.