Почему туррель убивает самого ближайщеговрага а потом не стреляет других пока они не станут ближе последнего?

Question

[Ad1yar]

Почему туррель убивает самого ближайщеговрага а потом не стреляет других пока они не станут ближе последнего?

void LockTarget()
    {
        if (target != null) return;

        GameObject[] zombies = GameObject.FindGameObjectsWithTag("Zombie");
        foreach (GameObject zombie in zombies)
        {
            octree.Add(zombie, zombie.GetComponent<Collider>().bounds);
        }

        closestDistanceSqr = maxDistance * maxDistance;
        GameObject closestTarget = null;

        List<GameObject> nearbyZombies = new List<GameObject>();

        octree.GetColliding(nearbyZombies, new Bounds(turret.position, new Vector3(maxDistance, maxDistance, maxDistance)*2));

        nearbyZombies.RemoveAll(x => x == null);

        foreach (GameObject zombie in nearbyZombies)
        {
            if (zombie == null)
            {
                nearbyZombies.RemoveAll(x => x == null);
                octree.Remove(zombie);
                closestDistanceSqr = maxDistance * maxDistance;
                continue;
            }

            float distanceSqr = (zombie.transform.position - turret.position).sqrMagnitude;

            if (distanceSqr < closestDistanceSqr)
            {
                closestDistanceSqr = distanceSqr;
                closestTarget = zombie;
            }
        }

        if (closestTarget != null)
        {
            target = closestTarget;
        }
    }

это стрельба (здесь тоже есть кусок)

void Shoot()
    {
        RaycastHit hit;
        if (Physics.Raycast(gun.position, -gun.forward, out hit, maxDistance, layerMask2))
        {
            if (hit.transform.gameObject == target)
            {
                hit.transform.TryGetComponent(out Zombie zombie);

                if ((timer += Time.deltaTime) > delay)
                {
                    if (!reload)
                    {
                        rockets[num].launch = true;
                        rockets[num].transform.parent = null;
                        num++;
                        if (num >= rockets.Length) reload = true;
                        Instantiate(shootSound, gun);
                        timer = 0;
                    }
                }

                if (!zombie.isAlive)
                {
                    closestDistanceSqr = maxDistance * maxDistance;
                    target = null;
                }
            }

            else
            {
                target = null;
            }
        }

        else
        {
            target = null;
        }
    }

Comments

[DrRen7]

LockTarget() когда запускается?
кинь в апдейт if(target == null)LockTarget();

[Ad1yar]

DrRen7, я таки сделал

[DrRen7]

Ad1yar, я чучуть упростил должно работать лучше

void LockTarget()
   {
       Collider[] zombies = Physics.OverlapSphere(turret.transform.position, maxDistance, _targetMask);
       Transform target = null; //должна быть глобальная или передавать через return
       if (!(zombies.Length > 0)) return;
       float mindist = maxDistance;
       foreach (Collider c in zombies)
       {
           float currentDist = Vector3.Distance(turret.transform.position, c.gameObject.transform.position);
           if ((currentDist<mindist))
           {
               mindist = currentDist;
               target = c.gameObject.transform;
           }
       }
   }

Плюс в Shot не надо будет таргет null и тд, а так кинуть LockTarget() в апдейт или запускать его через шут через Transform LockTarget() .... return;

[Ad1yar]

DrRen7, спасибо тебе, но а какже octree? https://github.com/Nition/UnityOctree

[DrRen7]

Ad1yar,
Думаю это надо использовать как то по другому.
как минимум

GameObject[] zombies = GameObject.FindGameObjectsWithTag("Zombie");
        foreach (GameObject zombie in zombies)
        {
            octree.Add(zombie, zombie.GetComponent<Collider>().bounds);
        }

ты если заходишь в метод сразу добавляешь, там могут быть повторные?

List<GameObject> nearbyZombies = new List<GameObject>();

листы не надо постоянно создавать 1 раз создай в глобальных и просто очищай его.
Далее

if (!zombie.isAlive)
                {
                    closestDistanceSqr = maxDistance * maxDistance;
                    target = null;
                }

как я понял ты объект не уничтожаешь, тогда вопрос есть вариант что он станет опять таргетом? вероятно в цикле выбора цели стоит сделать проверку на isAlive