Добрый день!
Особо не знаю куда обращаться, так что залетел на первый сайт, который мне выдал поисковик по запросу "Как можно посоветоваться с "программистами")
Тут больше не вопрос, так как не знаю что спрашивать)
Но суть такова
С другом пытаемся осваивать азы 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;
}
}
Буду очень рад, если подскажете, как можно сделать получше, что использовать, как сократить, как сделать более круче или более гибко и податливо для изменений.Скорее всего данные скрипты очень не оптимизированы.
Если прочли и просмотрели все, то отдельное спасибо)