Почему может замедляться скорость стрельбы, когда остаётся 1 враг?

Question

[yraiv]

Когда врагов много, то скорость стрельбы не меняется, но если враг остаётся один, то она резко снижается?

void Fire()
    {

        curTimeout += Time.deltaTime;
        if (curTimeout > fireRate)
        {
            Debug.Log("Fire");
            curTimeout = 0;
            _audioSoruce.PlayOneShot(FireSound);
            Rigidbody2D clone = Instantiate(bullet, gunPoint.position, Quaternion.identity) as Rigidbody2D;
            clone.GetComponent<BulletEnter>().atack = thisAtackTower;
            clone.transform.up = gunPoint.up;
            clone.velocity = clone.transform.up * speed;

        }
    }

Comments

[yraiv]

и curTimeout += Time.fixedDeltaTime;
не помог

[K0TlK]

yraiv, Выложите полностью код компонента, по этому обрезку не понять общей картины. Вот это вот страшилище

Rigidbody2D clone = Instantiate(bullet, gunPoint.position, Quaternion.identity) as Rigidbody2D;
clone.GetComponent<BulletEnter>().atack = thisAtackTower;
clone.transform.up = gunPoint.up;
clone.velocity = clone.transform.up * speed;

заменяется на это:

var bullet = Instantiate<Bullet>(bullet, gunPoint.position, Quaternion.identity);
bullet.Shot(direction);

Соответственно компонент Bullet должен иметь метод Shot(Vector2 direction) который принимает направление и придает себе движение в том направлении. И, если уж это какая-то пуля, то атаковать она должна сама, ей не должен кто-то говорить, что это ее таргет и вот его нужно атаковать.

[yraiv]

K0TlK, ох) вот полный код...

private Transform target;
    public int thisAtackTower;
    public float speed = 10; 
    public Rigidbody2D bullet; 
    public Transform gunPoint; 
    public float fireRate = 1;
    private float curTimeout;
    public int ZombieArrayGivePosition;
    public List<GameObject> zombieArray = new List<GameObject>();
    public List<GameObject> ZombieMaxArrayList = new List<GameObject>();
    public List<GameObject> zombieArrayNew = new List<GameObject>();
    public GameObject NewTarget;
    public AudioSource _audioSoruce;
    public AudioClip FireSound;
   





  
    void OnTriggerExit2D(Collider2D collider)
    {
        if (collider.gameObject.tag == "Zombie")
        {
            target = null;
            zombieArray.Remove(collider.gameObject);
            ZombieMaxArrayList.Remove(collider.gameObject);
        }
       

    }





    void OnTriggerEnter2D(Collider2D collider)
    {
        if (collider.gameObject.tag == "Zombie")
        {
            zombieArray.Add(collider.gameObject);
        }
    }

    void OnTriggerStay2D(Collider2D collider)
    {

        if (collider.gameObject.tag == "Zombie")
        {
            ChekDistante();
            if (target != null)
            {
                transform.rotation = Quaternion.Euler(transform.rotation.eulerAngles.x, transform.rotation.eulerAngles.y, Mathf.Atan2(target.transform.position.y - transform.position.y, target.transform.position.x - transform.position.x) * Mathf.Rad2Deg + 90);
                Fire();
            }
           
        }
    }


    private void ChekDistante()
    {
        ZombieArrayGivePosition = 0; 


        for (int i = 0; i < zombieArray.Count; i++)
        {

            if (zombieArray[i].GetComponent<ZombieStandart>().NewPosition > ZombieArrayGivePosition)
            {// проверяю, если максимальная точка меньше точки из массива, то беру её
                ZombieMaxArrayList.Clear();
                ZombieMaxArrayList.Add(zombieArray[i]);
                ZombieArrayGivePosition = zombieArray[i].GetComponent<ZombieStandart>().NewPosition;
            }
            else if (ZombieArrayGivePosition == zombieArray[i].GetComponent<ZombieStandart>().NewPosition)
            {

                ZombieMaxArrayList.Add(zombieArray[i]);

            }
         
        }
  

        if (zombieArray.Count > 0)
        {
            NewTarget = ZombieMaxArrayList[0];
            target = NewTarget.GetComponent<Transform>();
            ZombieMaxArrayList.Clear();
        }
           
    }
       


    void Fire()
    {
        curTimeout += Time.deltaTime;
        if (curTimeout > fireRate)
        {
            Debug.Log(Time.deltaTime);
            curTimeout = 0;
            _audioSoruce.PlayOneShot(FireSound);
            Rigidbody2D clone = Instantiate(bullet, gunPoint.position, Quaternion.identity) as Rigidbody2D;
            clone.GetComponent<BulletEnter>().atack = thisAtackTower;
            clone.transform.up = gunPoint.up;
            clone.velocity = clone.transform.up * speed;

        }
    }



  


}

[K0TlK]

yraiv,
Ожидайте.

[K0TlK]

yraiv,

public List<GameObject> zombieArray = new List<GameObject>();
public List<GameObject> ZombieMaxArrayList = new List<GameObject>();
public List<GameObject> zombieArrayNew = new List<GameObject>();

Почему их три? Зачем? Почему от GameObject, когда у вас уже есть класс ZombieStandart, который вы впоследствии получаете через GetComponent.

public Rigidbody2D bullet;
Почему Rigidbody, когда можно Bullet или что у вас там отвечает за пулю?
public int ZombieArrayGivePosition;
Что это такое, почему инт? Что оно означает? Зомби массив давать позиция?
public int thisAtackTower;
Это что такое, почему инт? Что оно означает?

void OnTriggerEnter2D(Collider2D collider)
    {
        if (collider.gameObject.tag == "Zombie")
        {
            zombieArray.Add(collider.gameObject);
        }
    }

Проверка по тэгу == плохо, нельзя так делать, можно легко ошибиться и написать в строке не то что надо и искать потом где ошибка. Это делается через получение компонента:

private void OnTriggerEnter2D(Collider2D collider)
{
    if (collider.TryGetComponent<ZombieStandart>(out ZombieStandart zombie)
    {
        zombieArray.Add(zombie);
    }
}

Соответственно zombieArray должен быть от типа ZombieStandart, а не от GameObject.
В OnTriggerExit и OnTriggerStay то же самое.
Все еще не понятно зачем ZombieMaxArrayList нужен, выглядит как какой-то рудимент.
Зачем эти приписки в конце (Array, List)? Чтобы не забыть, что это список? _zombies или подобное недостаточно говорит о том, что это какой-то массив данных?

transform.rotation = Quaternion.Euler(transform.rotation.eulerAngles.x, transform.rotation.eulerAngles.y, Mathf.Atan2(target.transform.position.y - transform.position.y, target.transform.position.x - transform.position.x) * Mathf.Rad2Deg + 90);

Это вы так метод transform.LookAt реализовали?
private void ChekDistante()
CheckDistance - если я правильно понял. Используйте гугл транслейт чтобы переводить слова, которые не знаете как правильно написать. Я не понимаю что делает этот метод. Какую он дистанцию проверяет, до чего? Если нужна дистанция до какого-то объекта, то есть метод Vector3.Distance.

Скорее всего, проблема в методе ChekDistante потому что там творится какая-то ересь, которую поймет только ее создатель. Да и метод Fire тоже не совсем то делает, что нужно. Он не должен сам ожидать, ожидание должна выполнять корутина, Fire должен просто проверить, есть ли возможность сейчас стрелять и стрельнуть. Далее уже запускается корутина и делает отсчет времени.

Ваша проблема в том, что вы выбрали себе слишком сложную задачу. Поделайте что-то простое типа FlappyBird, DoodleJump и других подобных гиперказуалок. Научитесь делать хотя бы эти игры, а потом переходите на жанры сложнее. Tower Defence или что вы пытаетесь сделать, слишком сложно для вас. Для начала изучите как разделять код на компоненты правильно и как с этими компонентами работать. Изучите основы.

Где изучать юнити - https://learn.unity.com/
Где изучать программирование - https://ulearn.me/

[yraiv]

K0TlK, ладно, спасибо)

[yraiv]

K0TlK, насчёт метода ChekDistante, так на будущее, мне интересно как реализовывается это. Дорога же в играх жанра td извилистая всяко-разно, кривая. Я не знал как правильно делать и накидал точки на местах поворота ( поворот на 90 градусов везде). Когда у меня враг появляется, то он двигается сразу к первой точки, а после того, как дошёл до неё, то автоматически находит следующую и идёт уже к ней public int ZombieArrayGivePosition; за это и отвечает, я считывал точки к которым стремится враг и брал тех у кого они наибольшие, получилось что он стреляет в тех, кто ближе к финишу идёт, я хотел ещё раз прогнать, чтоб отсортировать тех, кто по vector2.Distance смотреть кто ближе к финишу и выбирать в цель именно его, но посчитал, что это будет совсем ресурсно затратно и не правильно и решил не делать так. Как нормальные люди подобное реализуют? или это что-то из разряда, что в один ответ короткий не поместится?

[K0TlK]

yraiv, У башни есть область видимости. Самое простое - обычный триггер. При входе в этот триггер, если у башни нет таргета, то этот зомби им становится. Если же таргет есть, то этот зомби добавляется в список зомби, которые вошли в триггер. Соответственно, при смерти зомби, который был таргетом, целью становится первый зомби в списке (Этот зомби будет ближе всех к финишу, т.к. он первее вошел в триггер). При выходе из триггера, зомби удаляется из списка/таргета.
Но это работает, если у всех зомби одинаковая скорость, если она разная, то зависит от того, как строить путь для зомби. Проходиться по списку и выбирать ближайшего к финишу через дистанцию, хранить в зомби процент пройденного пути и по нему фильтровать, хранить в зомби последнюю пройденную контрольную точку и по ней фильтровать, как делали вы. Вариантов решения можно придумать сколько угодно.
Вам лучше научиться правильно именовать поля, свойства, методы, классы, потому что с вашими названиями, которые не отражают сути, вообще не понятно что для чего служит и приходится сидеть и вчитываться в код, искать где и как это работает. Вот здесь есть Naming convention, в котором расписано как что именовать.
И об оптимизации нужно думать, когда начинаются проблемы с ней.
Как я и писал раньше, Tower Defence пока что сложновато вам будет реализовать, а трудность реализации может быть для вас демотиватором, начните с простых вещей.

[yraiv]

K0TlK, кстати, спасибо за подсказку, я сделал так как сказали и оно стало работать нормально

void Fire()
    {
     

        if (CanFire == true)
        {
            _audioSoruce.PlayOneShot(FireSound);
            Rigidbody2D clone = Instantiate(bullet, gunPoint.position, Quaternion.identity) as Rigidbody2D; // не ругаться)
            clone.GetComponent<BulletEnter>().atack = thisAtackTower; // тут беру  атаку из башни, ибо с увеличением уровня урон меняется у неё и там разные комбинации всякие, они в связках работают 
            clone.transform.up = gunPoint.up;
            clone.velocity = clone.transform.up * speed;
            CanFire = false;
            StartCoroutine(WaitFire());

        }
    }



    IEnumerator WaitFire()
    {      
            yield return new WaitForSeconds(fireRate);
            CanFire = true;            
    }