Почему боты не хотят идти по точкам?

Question

[Ad1yar]

Бот не хочет идти по точкам, я уже не знаю что делать все перерыл везде такой же метод используют как и я
вот код:

public class NextBots2 : MonoBehaviour
{
    public NavMeshAgent navMeshAgent;
    public Transform player;

    public static int difficult = -1;

    public Transform door;

    public Transform[] points;

    [SerializeField] private int currentPoint;

    void Start()
    {
        currentPoint = -1;
        navMeshAgent = GetComponent<NavMeshAgent>();
    }

    void Update()
    {
        CheckPlayerOnRoom();

        if (currentPoint != -1)
        {
            navMeshAgent.SetDestination(points[currentPoint].position);
        }

        if (navMeshAgent.remainingDistance < 0.5f)
        {
            currentPoint++;
        }

        if (currentPoint >= points.Length)
        {
            currentPoint = 0;
        }

        print(currentPoint);

        difficult = Settings.diff;

        if (difficult == 0) navMeshAgent.speed = 5f;

        else if (difficult == 1) navMeshAgent.speed = 7f;

        else if (difficult == 2) navMeshAgent.speed = 9f;

        else if (difficult == 3) navMeshAgent.speed = 11f;
    }

    void CheckPlayerOnRoom()
    {
        if (Check.playerOnRoom && door.transform.rotation == Quaternion.Euler(0, -90, 0))
        {
            if (currentPoint == -1) currentPoint = 0;
        }

        else
        {
            currentPoint = -1;
            navMeshAgent.SetDestination(player.transform.position);
        }
    }
}

Comments

[freeExec]

Если что-то не работает, то удаляют весь лишний код к делу не относящийся и пытаются разобраться в нём. Т.е. в общем случае должна остаться одна строчка.

[twobomb]

Я бы начал наверно с изучения что такое отладка, как пользоваться инструментами отладки...

[Ad1yar]

twobomb, я пробовал но он вместо того чтобы пойти на 0 точку увеличивает ее на два тоесть с 0 до 2

[twobomb]

Ad1yar, Вот отлично, следующий шаг найти кто и почему увеличивает её с 0 на 2.
Нужно искать где происходит инкрементация, какое условие на это влияет, возможно что-то не так с ним... Почему оно постояннно возвращает true?

[Ad1yar]

twobomb, может когда он увел currentpoint?

[twobomb]

Ad1yar, Возможно изменение интовой currentPoint на 0 или другие значения не меняет текущую цель navmesha?
Как вы думаете, можно быть нужно совместно еще и SetDestination всетаки вызывать при смене currentPoint или не?

[Ad1yar]

twobomb, я посмотрел и увидел что при 0 он просто стоит на месте а 1 это уже другая точка к которй мне не надо в итоге я сделал что начальная теперь на 1 но теперь ОНИ тоесть 3 бота, 2 бота идут к 2 точке а последний к 3...

[twobomb]

Ad1yar, Зачем вам вообще переменная currentPoint? Если текущая точка хранится в свойстве destination navmesha?
Если вам нужно перейти на следующую, делаете indexOf по массиву points, получаете индекс и берете следующий если он есть.

[Ad1yar]

twobomb, indexOf? впервые слышу

[twobomb]

Ad1yar, Ну не суть, функцию которая получает индекс такого же элемента в массиве.
Тоесть у вас есть позиция она же цель она же свойствео destination navmesha, и есть массив точек из которых вы взяли эту самую точку и присвоили той самой целью , через метод SetDestination.
Так вот вам нужно узнать какой индекс в массиве целей, имеет текущая цель.

[Ad1yar]

twobomb, я ее сделал вот код(лишнее убрал):

public NavMeshAgent navMeshAgent;
    public Transform player;

    public Transform[] points;

    [SerializeField] private int currentPoint;

    void Start()
    {
        currentPoint = 0;
        navMeshAgent = GetComponent<NavMeshAgent>();
    }

    void Update()
    {
        CheckPlayerOnRoom();

        if (currentPoint >= points.Length) currentPoint = 0;

        if (navMeshAgent.remainingDistance < 1f)
        {
            currentPoint += 1;
            navMeshAgent.SetDestination(points[currentPoint].position);
        }
    }

    void CheckPlayerOnRoom()
    {
        if (Check.playerOnRoom && door.transform.rotation == Quaternion.Euler(0, -90, 0))
        {
            if(currentPoint == 0) navMeshAgent.SetDestination(points[1].position);
        }

        else
        {
            currentPoint = 0;
            navMeshAgent.SetDestination(player.transform.position);
        }
    }

[twobomb]

Ad1yar, Нет, если убрать лишнее то будет что-то типа такого

public NavMeshAgent navMeshAgent;
        public Transform player;

        public Transform[] points;

        void Start(){
            navMeshAgent = GetComponent<NavMeshAgent>();
            navMeshAgent.SetDestination(points[0].position)
        }

        void Update(){
            CheckPlayerOnRoom();

            if (navMeshAgent.remainingDistance < 1f){
                var inx = Array.IndexOf(points, points.FirstOrDefault(e => e.position == navMeshAgent.destination));
                if(inx != -1)
                    navMeshAgent.SetDestination(points[(inx+1)% points.Length].position);
            }
        }

        void CheckPlayerOnRoom(){
            if (Check.playerOnRoom && door.transform.rotation == Quaternion.Euler(0, -90, 0) && Array.IndexOf(points, points.FirstOrDefault(e => e.position == navMeshAgent.destination) == 0)
                navMeshAgent.SetDestination(points[1].position);
            else
                navMeshAgent.SetDestination(player.transform.position);
            
        }

[twobomb]

Ad1yar, Ну и это как-бы физика и обновление позиций происходит всеравно только в физике, поэтому то что вызывается в Update лучше вызывать в FixedUpdate, зачем нужно лишние вызова

[Ad1yar]

twobomb, самая главная цитата программиста "Работает - не трогай"

Answer 1

[Ad1yar]

решено вот код(лишнее убрал):

public NavMeshAgent navMeshAgent;
    public Transform player;

    public Transform[] points;

    [SerializeField] private int currentPoint;

    void Start()
    {
        currentPoint = 0;
        navMeshAgent = GetComponent<NavMeshAgent>();
    }

    void Update()
    {
        CheckPlayerOnRoom();

        if (navMeshAgent.remainingDistance < 1f)
        {
            currentPoint += 1;
            if (currentPoint >= points.Length) currentPoint = 0;
            navMeshAgent.SetDestination(points[currentPoint].position);
        }
    }

    void CheckPlayerOnRoom()
    {
        if (Check.playerOnRoom && door.transform.rotation == Quaternion.Euler(0, -90, 0))
        {
            if(currentPoint == 0) navMeshAgent.SetDestination(points[1].position);
        }

        else
        {
            currentPoint = 0;
            navMeshAgent.SetDestination(player.transform.position);
        }
    }