0

Добрый день!

Особо не знаю куда обращаться, так что залетел на первый сайт, который мне выдал поисковик по запросу "Как можно посоветоваться с "программистами")

Тут больше не вопрос, так как не знаю что спрашивать)

Но суть такова

С другом пытаемся осваивать азы Unity и соответственно C#. Сами в этом новички полные и с кодом раньше работали классе в 3-4 в PascalABC. Хотелось бы просто услышать мнение, советы, правки, указания на ошибки и т.д. по коду. Точнее несколько скриптов. Скорее всего до конца это не работает как надо, но мы стараемся, честно) Я вот только сегодня наконец понял, что такое параметры в методе и как их можно применять.

Смотрите, основной код ИИ противника (EnemyAIWithFOV.cs) был сделан с огромной помощью ChatGPT, но остальная часть была написана процентов на 90 своими силами, поэтому могут быть огромные ошибки в логике и подобной теме. На комментарии не обращайте внимание. Когда я дорабатывал скрипт, то делал пометки для друга, что бы он понял, что я менял.

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

public class EnemyAIwithFOV : MonoBehaviour
{


    [Header("Переменные для настройки баланса")]
    public attackEvent attackEvent;

    [Space]

    [Header("Абсолютное обноружение")]
    [Tooltip("Минимальная дистанция при которой юнит заметит другого юнита (так же влияет на абсолютную дистанцию срабатывания вокруг)")]
    [Range(1, 3)] public int minDistance = 1; // через нее же и настраивать маленькую зону обзора
    [Header("Настройка дистанции атаки")]
    [Tooltip("Дистанция атаки (сама атака в разработке)))")]
    [Range(0, 20)] public float attackRadius = 10;
    [Header("Настройка дистанции обзора")]
    [Tooltip("Дистанция обзора")]
    [Range(1, 40)] public float sightRadius = 16;
    [Header("Настройка угла обзора")]
    [Tooltip("Угол обзора")]
    [Range(0, 360)] public int fieldOfViewAngle = 60; // угол обзора
    private int absoluteSign = 2; // это тип будет дистанция при которой он полюбому увидит даже если не в радиусе( на всякий скрыл, тк калибровка у нее по максимуму простая)

    [Space]
    [Space]
    [Space]

    [Header("Цели для выбора приоритета")]
    [Tooltip("Сюда добавить главную цель преследования для юнита")]
    public Transform primaryTarget;
    [Tooltip("Сюда добавить второстепенную цель преследования для юнита")]
    public Transform secondaryTarget;


    [Header("Атрибуты")]
    [Tooltip("Сюда перетащить компонент с NavMeshAgent")]
    public NavMeshAgent nav;

    [Space]
    [Space]
    [Space]

    [Header("Цели")]

    [Space]

    [Header("Текущая цель")]
    [SerializeField] private Transform currentTarget;
    [Header("Лист персонажей находящихся в радиусе")]
    [SerializeField] private List<Transform> targetsInRange = new List<Transform>();
    [Header("Лист персонажей находящихся в видимости")]
    [SerializeField] private List<Transform> targetsInSight = new List<Transform>();

    public Animator EnemyAnim;
    public bool attack;
    public bool attackPlayer;
    public bool attackStella;



    public void SearchTargetPosition()
    {
        Collider[] hitColliders = Physics.OverlapSphere(transform.position, attackRadius);
        foreach (Collider hitCollider in hitColliders)
        {
            if (hitCollider.CompareTag("Enemy") || hitCollider.CompareTag("Player"))
            {
                targetsInRange.Add(hitCollider.transform);
            }
        }
    }

    private void Start()
    {
        EnemyAnim = GetComponent<Animator>();
        nav = GetComponent<NavMeshAgent>();
        SearchTargetPosition(); //  тоже просто вынес отдельно (там внизу все методы)
        attack = false;
        attackPlayer = false;
        attackStella = false;

    }

    

    public void Attack()
    {
        StartCoroutine("AttackWithTime");
        
        EnemyAnim.SetTrigger("Attack");
    }

    public IEnumerator AttackWithTime()
    {
        yield return new WaitForSeconds(0.8f);
        int attackTarget = attackEvent.targetToDamage;
        attackEvent.AttackEnemy(attackTarget);
        yield return new WaitForSeconds(1.6f);
    }

    private void Update()
    {

        absoluteSign = minDistance;

        Collider[] hitColliders = Physics.OverlapSphere(transform.position, attackRadius);
        foreach (Collider hitCollider in hitColliders)
        {
            if (hitCollider.CompareTag("Enemy") || hitCollider.CompareTag("Player"))
            {

                Transform target = hitCollider.transform;
                Vector3 targetDirection = (target.position - transform.position).normalized;

                if (currentTarget != null && Vector3.Distance(nav.transform.position, currentTarget.position) <= attackRadius)
                {
                    nav.isStopped = true;
                    EnemyAnim.SetBool("nav.isStoped", nav.isStopped);

                }
                else
                {
                    nav.isStopped = false;
                    EnemyAnim.SetBool("nav.isStoped", nav.isStopped);
                }
                foreach (Collider enemy in hitColliders)
                {
                    if (enemy.CompareTag("Player") && currentTarget == CompareTag("Player"))
                    {
                        attack = true;
                        attackPlayer = true;
                        Attack();
                    }
                    else
                    {
                        attack = false;
                        attackPlayer = false;
                        attackStella = false;
                        StartCoroutine("AttackWithTime");
                    }
                    if (enemy.CompareTag("Enemy") && currentTarget == CompareTag("Enemy"))
                    {
                        attack = true;
                        attackStella = true;
                        Attack();
                    }
                    else
                    {
                        attack = false;
                        attackPlayer = false;
                        attackStella = false;
                        StartCoroutine("AttackWithTime");
                    }

                    
                    transform.LookAt(new Vector3(currentTarget.position.x, transform.position.y, currentTarget.position.z));

                    
                }
            }
        }

        if (currentTarget == null || !targetsInRange.Contains(currentTarget))
        {
            currentTarget = GetNewTarget();
        }

        if (currentTarget != null && Vector3.Distance(transform.position, currentTarget.position) >= minDistance)
        {
            TransformNavMesh();//тут я просто навмеш в отдельный метод вынес, что бы проще его потом было изменять под нужды
        }
        targetsInSight.Clear();
        Collider[] targetsInViewRadius = Physics.OverlapSphere(transform.position, sightRadius); // сфера тут осталась, там потом просто ограничение будет стоять, что перед объектом в -угол обзора/2 и +угол обзора/2 будет чекать только
        Collider[] targetInAbsolutRadius = Physics.OverlapSphere(transform.position, absoluteSign);
        foreach (Collider targetCollider in targetInAbsolutRadius) // тут как раз таки триггер для срабатывания маленькой зоны на игроке, что бы прям близко нельзя было подобраться
        {
            if (targetCollider.CompareTag("Player"))
            {
                targetsInSight.Add(targetCollider.transform);
            }
        }
        foreach (Collider targetCollider in targetsInViewRadius)
        {
            if (targetCollider.CompareTag("Enemy") || targetCollider.CompareTag("Player"))
            {
                Transform target = targetCollider.transform;
                Vector3 targetDirection = (target.position - transform.position).normalized;
                float targetDistance = Vector3.Distance(transform.position, target.position);

                if (targetDistance <= sightRadius && Vector3.Angle(transform.forward, targetDirection) < fieldOfViewAngle / 2 || targetDistance <= absoluteSign) // ну вот как раз ограничение это
                {

                    RaycastHit hit; // а это просто проверка на препятствие
                    Debug.DrawLine(transform.position, target.position);
                    if (Physics.Raycast(transform.position, targetDirection, out hit, targetDistance))
                    {
                        if (hit.transform == target)
                        {
                            targetsInSight.Add(target);
                        }
                    }
                }
            }
        }
        if (secondaryTarget != null && targetsInSight.Contains(secondaryTarget) && !targetsInSight.Contains(primaryTarget))
        {
            currentTarget = secondaryTarget; // тут шаришь, ничего не менялось, просто выбор целей
        }
        else if (currentTarget == secondaryTarget && !targetsInSight.Contains(secondaryTarget))
        {
            currentTarget = primaryTarget;
        }
        if (currentTarget != null && Vector3.Distance(transform.position, currentTarget.position) > sightRadius)
        {
            currentTarget = null;
        }
    }
    private Transform GetNewTarget()
    {
        Transform newTarget = null;

        if (primaryTarget != null)
        {
            newTarget = primaryTarget;
        }
        if (secondaryTarget != null && targetsInSight.Contains(secondaryTarget))
        {
            newTarget = secondaryTarget;
        }
        if (newTarget == null && targetsInRange.Count > 0)
        {
            newTarget = targetsInRange[Random.Range(0, targetsInRange.Count)];
        }
        return newTarget;
    }

    public void TransformNavMesh()
    {
        nav.SetDestination(currentTarget.transform.position); // тут в целом даже одна строчка вышла))

    }

    private void OnDrawGizmosSelected()
    {
        Gizmos.color = Color.red;
        Gizmos.DrawWireSphere(transform.position, attackRadius);

        Gizmos.color = Color.cyan;
        Gizmos.DrawWireSphere(transform.position, minDistance);

        Gizmos.color = Color.yellow;
        Gizmos.DrawWireSphere(transform.position, sightRadius);

        //тута просто для визуализации вынес две линии угла обзора
        Gizmos.color = Color.green;
        Vector3 rightDir = Quaternion.AngleAxis(fieldOfViewAngle / 2, Vector3.up) * transform.forward;
        Vector3 leftDir = Quaternion.AngleAxis(-fieldOfViewAngle / 2, Vector3.up) * transform.forward;
        Gizmos.DrawLine(transform.position, transform.position + rightDir * sightRadius);
        Gizmos.DrawLine(transform.position, transform.position + leftDir * sightRadius);

    }
}

Это скрипт ИИ противника. Тут получается намешано дофига всего). Основная его задача бежать в сторону некого Обелиска, остановиться и атаковать (он всегда знает где он), но если в поле зрения попадается игрок, то он бежит к нему и атакует. Сама атака до конца не реализована и ей я занимался вчера и сегодня. Первый раз пробовал ссылаться и на переменные и методы из других скриптов и работал с параметрами).

Тут само действие внесения урона

public void Attack()
        {
            StartCoroutine("AttackWithTime");
            
            EnemyAnim.SetTrigger("Attack");
        }
    
        public IEnumerator AttackWithTime()
        {
            yield return new WaitForSeconds(0.8f);
            int attackTarget = attackEvent.targetToDamage;
            attackEvent.AttackEnemy(attackTarget);
            yield return new WaitForSeconds(1.6f);
        }

Атака реализована для меня максимально запутана, Получается он получает сведенья, что в радиусе атаки присутствуют колайдеры с нужными тегами, создает "состояние" что он атакует и кого, а после передает данные вот в этот скрипт.

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



public class attackEvent : MonoBehaviour
{
    public healthBar healthBar;
    public healthStella healthStella;
    public float minDamage;
    public float maxDamage;
    public float curretDamage;
    public EnemyAIwithFOV enemyScript;
    public bool attackPlayer;
    public bool attackStella;
    public int targetToDamage;

    public void Start()
    {
        attackPlayer = false;
        attackStella = false;
    }

    public void Update()
    {
        CheckTargetForDamage();     
    }

    public void CheckTargetForDamage()
    {
        if (enemyScript.attack == true && enemyScript.attackPlayer == true)
        {
            attackPlayer = true;
            
        }
        else attackPlayer = false;

        if (enemyScript.attack == true && enemyScript.attackStella == true)
        {
            attackStella = true;
        }
        else attackStella = false;

        if (attackPlayer)
        {
            targetToDamage = 1;
            Debug.Log("Враг бьет игрока");
        }
        if (attackStella)
        {
            targetToDamage = 2;
            Debug.Log("Враг бьет обелиск");
        }
        if (!attackPlayer && !attackStella)
        {
            targetToDamage = 0;
            Debug.Log("Враг никого не бьет");
        }
    }

    public void Damage(float minDamage, float maxDamage, out float curretDamage)
    {
        curretDamage = Random.Range(minDamage, maxDamage);
    }

    public void SetDamage()
    {
        Damage(minDamage, maxDamage, out curretDamage);

    }

    public void AttackEnemy(int whoToBeat)
    {
            SetDamage();
        if (whoToBeat == 1)
        {
            healthBar.GetDamage();
        }
        if (whoToBeat == 2)
        {
            healthStella.GetDamage();
        }
            
            
        
    }


}

В нем он проверяет кого бьет противник и так же присваивает для цели "типо" индекс (возможно это можно было реализовать через массивы, но я пока плохо в них разбираюсь)

public void CheckTargetForDamage()
        {
            if (enemyScript.attack == true && enemyScript.attackPlayer == true)
            {
                attackPlayer = true;
                
            }
            else attackPlayer = false;
    
            if (enemyScript.attack == true && enemyScript.attackStella == true)
            {
                attackStella = true;
            }
            else attackStella = false;
    
            if (attackPlayer)
            {
                targetToDamage = 1;
                Debug.Log("Враг бьет игрока");
            }
            if (attackStella)
            {
                targetToDamage = 2;
                Debug.Log("Враг бьет обелиск");
            }
            if (!attackPlayer && !attackStella)
            {
                targetToDamage = 0;
                Debug.Log("Враг никого не бьет");
            }
        }
    

Так же там генерируется случайный урон

И после из этого скрипта все отправляется в показатели здоровья целей.

Это ХП для игрока

    using System.Collections;
    using System.Collections.Generic;
    using UnityEngine;
    using UnityEngine.UI;
    
    
    public class healthBar : MonoBehaviour
    {
        
        [Header("Максимальное ХП")]
        public int MaxHealth = 100;
        public float healthValue;
        private int MinHealth = 0;
        public Slider HealthBar;
        public attackEvent attackEvent;
    
        public void Start()
        {
            attackEvent = GetComponent<attackEvent>();
            HealthBar.maxValue = MaxHealth;
            HealthBar.minValue = MinHealth;
            healthValue = Mathf.Clamp(healthValue, MinHealth, MaxHealth);
            
    
        }
    
        public void Update()
     {
       HealthBar.value = healthValue;
     }
    
        public void ChangeHealth(out float curretHP, float damage)
        {
            curretHP = healthValue;
            curretHP -= damage;
            healthValue = curretHP;
            
        }
    
    
        public void GetDamage()
        {
    
            float SetDamage = attackEvent.curretDamage;
            ChangeHealth(out healthValue,SetDamage);
    
        }
    }

Это ХП Обелиска

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


public class healthStella : MonoBehaviour
{

    [Header("Максимальное ХП")]
    public float healthValueStella = 100;
    public attackEvent attackEvent;
    public string curretStellaHealth;

    public void Start()
    {
        attackEvent = GetComponent<attackEvent>();
    }

        public void Update()
    {
        HealthStella();
    }

    public void HealthStella()
    {
        healthValueStella.ToString();
        curretStellaHealth = healthValueStella + " Stella Health";
    }

    public void ChangeHealth(out float curretHP, float damage)
    {
        curretHP = healthValueStella;
        curretHP -= damage;
        healthValueStella = curretHP;

    }


    public void GetDamage()
    {

        float SetDamage = attackEvent.curretDamage;
        ChangeHealth(out healthValueStella, SetDamage);

    }

}

Эт просто для выведение на экране ХП Обелиска

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

public class screemHealthStella : MonoBehaviour
{
    public healthStella healthStella;
    public Text healthText;
    void Update()
    {
        
        healthText.text = healthStella.curretStellaHealth;
    }
}

Буду очень рад, если подскажете, как можно сделать получше, что использовать, как сократить, как сделать более круче или более гибко и податливо для изменений.Скорее всего данные скрипты очень не оптимизированы.

Если прочли и просмотрели все, то отдельное спасибо)

2
  • Ой, заметил ошибку у себя же. HealthBar.value = healthValue скорее всего нужно вывести в метод Update 3 мая в 15:11
  • 1
    Не пишите дополнительную информацию в комментариях, вместо этого вопрос можно отредактировать - нажмите кнопку править
    – aepot
    3 мая в 15:53

1 ответ 1

0

ОЧЕНЬ много букв, самые главные намешано дофига всего. Ты с трудом нашел ошибку, другой человек вряд ли сориентируется, да и ты сам спустя время тоже. Комментариев перебор, даже мешают, нужен просто нормальный нейминг.

Поля класса поделены Space, верный признак намешенности, что бы в этом ориентироваться, нужно дробить класс на маленькие с одной единственной ответственностью. Например обнаружение:

[Serializable]
public class Perception : MonoBehaviour
{
    [SerializeField] private float _seeRange = 16;
    [SerializeField] private float _viewAngle = 90f;
    [Space]
    [SerializeField] private float _senseRange = 2;

    public bool IsDetected (Transform observer, Transform target)
    {
        Vector2 observerPosition = new Vector2(observer.position.x, observer.position.z);
        Vector2 targetPosition = new Vector2(target.position.x, target.position.z);
        Vector3 direction = observer.forward;
        Vector2 observerDirection = new Vector2(direction.x, direction.z);
        return IsDetected(observerPosition, observerDirection, targetPosition);
    }

    public bool IsDetected (Vector2 observerPosition, Vector2 observerDirection, Vector2 targetPosition)
    {
        Vector2 delta = targetPosition - observerPosition;
        float deltaSqrMagnitude = delta.sqrMagnitude;
        if (deltaSqrMagnitude > _seeRange * _seeRange) // сравнение длины без вычисление квадратного корня
            return false; // out of see range
        if (deltaSqrMagnitude < _senseRange * _senseRange )
            return true; // in sense range
        float dot = Vector2.Dot(delta.normalized, observerDirection.normalized); // Cos of delta-direction corner
        float deltaAngle = Mathf.Acos(dot) * Mathf.Rad2Deg; // degree of delta-direction corner
        return deltaAngle < _viewAngle * 0.5f; // onView
    }
}

И Perception это просто поле твоего класса вместо одной из кучек полей и методов.


И никаких комментариев к именам нет! от слова совсем! только функционалу! Весь нейминг в контексте, как Perception.SeeRange, а EnemyAIwithFOV.MinDistance - угадай с ста попыток.

И так далее...

3
  • Ага, спасибо большое! Теперь останется только научиться это все разделять правильно) Я пока что не знаю по какому принципу можно создавать в одном скрипте несколько классов или какие методы ('bool IsDetected' надеюсь это метод) помимо 'void' можно создавать. Система атаки и здоровья это моя первая попытка вывести части кода в другие скрипты и пытаться это все соединить. Спасибо большое! Постараюсь это все учесть и исправиться))) 4 мая в 4:07
  • Да, это метод класса. У класса есть только поля класса, конструкторы, методы класса и get set свойства (которые на самом деле тоже методы). Переменные бывают только в методах, как dot и onView, не путать их с полями классов). Что бы общаться с другими, терминалогию нужно знать!
    – Yaroslav
    4 мая в 10:53
  • Декомпазиция кода важна, что бы классы следовали принципу единственной ответственности single responsibility principle. Лучше учиться этому на ранних этапах, избежишь тонны говна, которго и так хватать будет. Есть выражение: "легко написать код понятный машине, а хороший код понятен другому программисту", где зачастую "другой" это ты сам, спустя время, когда баг ищешь)
    – Yaroslav
    4 мая в 10:53

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.