Ответы пользователя по тегу C#
  • Зачем нужно прописывать float rotationY = transform.localEulerAngles.y; (unity)?

    K0TlK
    @K0TlK
    Буллю людей.
    Эффект будет тот же

    Нет не будет.
    В случае с rotationY, если у тебя было ненулевое вращение по оси Y, то оно сохранится, т.е. Если localEulerAngles.y было 90, то оно и останется 90. Если же просто 0 написать, то localEulerAngles.y тоже станет 0.
    Ответ написан
    Комментировать
  • Как можно улучшить код этот прыжка?

    K0TlK
    @K0TlK
    Буллю людей.
    Убери проверку по тэгу.
    transform.position замени на Rigidbody.MovePosition.
    Таймер из апдейта в корутину перемести и в Jump ее запускай. Чтобы остановить прыжок, закэшируй корутину и останавливай ее, когда нужно.
    40 вынеси в отдельное поле.
    Это у тебя не PhysicsJump, физики здесь никакой нет, это анимированный прыжок, переименуй. IsOnGrounded - "является на приземленный". grounded достаточно. У тебя IsOnGrounded всегда противоположно IsJumping. Зачем здесь два флага?
    Это лишний монобех, перемести код отсюда в Movement свой. Если движение у тебя вдоль поверхности, то нормаль, которую ты получаешь при движении, можно использовать в качестве замены флагов grounded и jumping (если нормаль не нулевая, то можно прыгать, если нулевая, то прыжок происходит). Соответственно, теперь тебе не нужны тэги, т.к. землю ты проверяешь через нормаль, флаги, по той же причине, не нужен апдейт, т.к. анимация происходит в корутине, не нужен лишний монобех, чтобы не делать из объекта кучу монобехов. Остается только паблик метод Jump и корутина.
    Ответ написан
    5 комментариев
  • Соответсвует ли код принципам солид?

    K0TlK
    @K0TlK
    Буллю людей.
    Разделять код на миллион монобехов != ооп. Единственный способ достичь правильного ооп в юнити - отделять всю логику от движка, а монобехи использовать как вьюшку, чтобы отображать всякое. Но реализуемый для тебя способ - использовать интерфейсы, под каждый паблик метод отдельный интерфейс, взаимодействовать через эти интерфейсы, учиться нормально прокидывать зависимости: всегда должна быть какая-то точка входа, где будут инициализироваться компоненты и передаваться все зависимости.
    Virtual методы = плохо, класс в идеале должен быть либо abstract либо sealed.
    Protected поля = public поля = нарушение инкапсуляции/иммутабельности.
    PlayerHealth, PlayerEnergy, Health - это все одно и то же с разными реализациями, есть интерфейс IHealth все под него и просто этот интерфейс реализуешь.
    AudioPlayer вообще какой-то ужас. Во-первых, почему GameObject takeDamageSourseObject и т.д Почему GameObject, если ты потом получаешь у него компонент AudioSource, ты можешь конкретные компоненты в инспекторе передавать, т.е. не GameObject, a AudioSource и GetComponent потом делать не нужно будет. Во-вторых, из всего этого можно было сделать один компонент, в котором будет один метод Play, который будет принимать AudioSource и проигрывать его. Либо напрямую пропихивать AudioSource и воспроизводить звук.
    Нет какого-то единого кодстайла, где-то есть нижнее подчеркивание, где-то нет, где-то есть _cs где-то нет, для чего эта _cs я так и не понял. Где-то сериализуемое приватное поле, где-то тупо паблик.
    Про солид вообще смысла говорить нет. Интерфейсов пара штук на весь проект. Зависимости нормально не прокидываются, всё через поля.
    Это так, навскидку. Думаю, если зайти в папку bot, то можно будет диссертацию написать по содержимому этой папки.
    Ответ написан
    4 комментария
  • Почему бот идет к цели не всегда?

    K0TlK
    @K0TlK
    Буллю людей.
    Любой искусственный интеллект, возможности которого выходят за рамки "принеси подай иди подальше не мешай", проще будет сделать через Behaviour Tree. Без него, чем сложнее у тебя будет бот, тем больше кода придется писать и тем больше будет каких-то непонятных условий, что у тебя, собственно, и проявляется в апдейте, полностью состоящем из if else if else if else. С Behaviour Tree все куда проще, есть много разных классов, отвечающих за что-то одно, все они совмещаются в разные деревья.
    Пример я покажу с использованием ассета Behaviour Designer, т.к., если я буду делать свою реализацию BT, то, боюсь, места для текста в ответе не хватит. Стоит Designer 90$, к счастью, пиратить софт очень плохо и так делают только плохие люди.

    Для того, чтобы бот ходил, нужно направление движения, соответственно, это направление нужно откуда-то брать:
    using UnityEngine;
    
    namespace BehaviourTree
    {
        public interface IMovementInput
        {
            Vector3 Direction { get; }
        }
    }


    Ну и, собственно, сам бот:
    using UnityEngine;
    
    namespace BehaviourTree
    {
        [RequireComponent(typeof(Rigidbody))]
        public class Bot : MonoBehaviour
        {
            [SerializeField] private MonoBehaviour _botInput = null;
            [SerializeField] private float _speed = 10f;
    
            private Rigidbody _rb;
    
            private IMovementInput BotInput => (IMovementInput) _botInput;
    
            private void OnValidate()
            {
                if (_botInput is IMovementInput) return;
    
                Debug.LogError($"{nameof(_botInput)} should implement {nameof(IMovementInput)}");
                _botInput = null;
            }
    
            private void Awake()
            {
                _rb = GetComponent<Rigidbody>();
            }
    
            private void FixedUpdate()
            {
                var velocity = new Vector3(BotInput.Direction.x, 0, BotInput.Direction.z) * _speed;
                _rb.velocity = velocity;
            }
    
            public void LookAt(Vector3 direction)
            {
                var rotation = Quaternion.LookRotation(direction);
    
                transform.rotation = rotation;
            }
        }
    }


    Инжектим интерфейс ввода через инспектор и в FixedUpdate задаем скорость. Чтобы посмотреть в каком-то направлении, нужно вызвать метод LookAt. Чтобы прокинуть интерфейс через инспектор нужен какой-то монобех, который будет реализовывать интерфейс. Вот он:
    using UnityEngine;
    
    namespace BehaviourTree
    {
        public class BotInput : MonoBehaviour, IMovementInput
        {
            public Vector3 Direction { get; set; }
        }
    }

    В дальнейшем через этот BotInput я буду перемещать бота, просто задавая нужное направление. С ботом все, теперь можно накинуть Bot на какой-нибудь капсюль на сцене, на него же повесить BotInput и в поле BotInput компонента Bot поместить тот самый BotInput.

    62c017ffaf2a1016283427.png

    Теперь нужна какая-то цель, чтобы бот за ней шел. Интерфейс:
    using UnityEngine;
    
    namespace BehaviourTree
    {
        public interface ITarget
        {
            Vector3 Position { get; }
        }
    }

    Реализация:
    using UnityEngine;
    
    namespace BehaviourTree
    {
        public class SomeTarget : MonoBehaviour, ITarget
        {
            public Vector3 Position => transform.position;
        }
    }

    Все то же самое, что и с ботом. Поместить на какой-нибудь капсюль, отличный от капсюля бота.

    Ну а теперь BT. Добавляешь своему боту компонент Behaviour Tree: в инспекторе бота -> Add Component -> Behaviour Designer -> Behaviour Tree.
    Далее нужно как-то туда добавить инпут бота, для этого создаем скрипт:
    using BehaviorDesigner.Runtime;
    
    namespace BehaviourTree.Brain
    {
        public class SharedBotInput : SharedVariable<BotInput>
        {
            public static implicit operator SharedBotInput(BotInput input) => new SharedBotInput {Value = input};
        }
    }

    Теперь BotInput можно прокидывать в переменные Behaviour Designer. Кстати о них. Открываем Tools -> Behaviour Designer -> Editor, далее тыкаем на бота, к которому прикреплен компонент Behaviour Tree, в дизайнере открываем Variables в поле Name вводим botInput, в поле Type выбираем BotInput, нажимает Add. Далее в инспекторе бота перетаскиваем компонент BotInput в поле botInput компонента Behaviour Tree.

    62c018177e9a0543052458.png

    Теперь в Behavoiur Designer можно устанавливать значение Direction компонента BotInput.
    Сначала я покажу весь код, что я написал для управления ботом, а потом то, как это все выглядит в дизайнере.
    Бот будет очень простой - он будет просто ходить в рандомном направлении, а если увидит свой таргет, то будет тупо бежать в него.

    Собственно, хождение в рандомном направлении:
    using BehaviorDesigner.Runtime;
    using BehaviorDesigner.Runtime.Tasks;
    using UnityEngine;
    
    namespace BehaviourTree.Brain
    {
        public class SetRandomDirection : Action
        {
            public SharedVector3 Direction;
    
            public override TaskStatus OnUpdate()
            {
                Direction.Value = Vector3.Scale(Random.insideUnitSphere.normalized, new Vector3(1, 0, 1));
                return TaskStatus.Success;
            }
        }
    }

    Наследуясь от Action, SetRandomDirection появляется в нодах(Tasks) дизайнера в выпадающем меню Actions
    Теперь нужно как-то установить направление движения бота.
    using BehaviorDesigner.Runtime;
    using BehaviorDesigner.Runtime.Tasks;
    
    namespace BehaviourTree.Brain
    {
        public class SetInput : Action
        {
            public SharedBotInput SharedInput;
            public SharedVector3 Input;
    
            public override TaskStatus OnUpdate()
            {
                SharedInput.Value.Direction = Input.Value;
                return TaskStatus.Success;
            }
        }
    }

    Собственно, здесь все понятно. Берем значение из Input и присваиваем его в SharedInput.
    Поворот в определенном направлении:
    using BehaviorDesigner.Runtime.Tasks;
    
    namespace BehaviourTree.Brain
    {
        public class LookAtDirection : Action
        {
            public SharedBotInput Direction;
            public Bot Bot;
    
            public override TaskStatus OnUpdate()
            {
                Bot.LookAt(Direction.Value.Direction);
                return TaskStatus.Success;
            }
        }
    }

    Далее идет нахождение цели и следование за ней, соответственно цель эту надо тоже добавить в дизайнер.
    using BehaviorDesigner.Runtime;
    
    namespace BehaviourTree.Brain
    {
        public class SharedTarget : SharedVariable<SomeTarget>
        {
            public static implicit operator SharedTarget(SomeTarget target) => new SharedTarget {Value = target};
        }
    }

    Установка направления к цели:
    using BehaviorDesigner.Runtime;
    using BehaviorDesigner.Runtime.Tasks;
    using UnityEngine;
    
    namespace BehaviourTree.Brain
    {
        public class SetDirectionToTarget : Action
        {
            public SomeTarget Target;
            public Bot Origin;
            public SharedVector3 Direction;
    
            public override TaskStatus OnUpdate()
            {
                var direction = (Target.Position - Origin.transform.position).normalized;
                Direction.Value = new Vector3(direction.x, 0, direction.z);
                return TaskStatus.Success;
            }
        }
    }


    Теперь нужно как-то дать возможность боту увидеть цель. Интерфейс:
    namespace BehaviourTree.Brain
    {
        public interface IEye
        {
            bool InSight(ITarget target);
        }
    }

    Реализация:
    using UnityEngine;
    
    namespace BehaviourTree.Brain
    {
        public class Eye : IEye
        {
            private readonly float _fov;
            private readonly Transform _origin;
    
            public Eye(float fov, Transform origin)
            {
                _fov = fov;
                _origin = origin;
            }
    
            public bool InSight(ITarget target)
            {
                var direction = target.Position - _origin.position;
    
                var angle = Mathf.Acos(Vector3.Dot(_origin.forward.normalized, direction.normalized)) * Mathf.Rad2Deg;
    
                return angle <= _fov;
            }
        }
    }

    Здесь все просто. Глаз, у которого есть Field of View в методе InSight проверяем, входит ли позиция таргета в этот фов. Проверяется все очень просто. Находим направление от бота к таргету, вычисляем угол между этим вектором направления и направлением взгляда бота, сравниваем. Чтобы вычислить угол нужно знать всего лишь формулу скалярного произведения векторов из нее выводим то, что у меня в коде умножение на Mathf.Rad2Deg потому что Mathf.Acos возвращает угол в радианах, а я его перевожу в градусы.
    Ну и последнее - нахождение цели:

    Все таки нужно было писать свою реализацию BT, потому что места все равно не хватило, продолжение в комментах.
    Ответ написан
    5 комментариев
  • Что я не так делаю с интерфейсами?

    K0TlK
    @K0TlK
    Буллю людей.
    Что я не так делаю с интерфейсами?

    Всё.
    У тебя у одного интерфейса слишком много ответственностей у него и Warside какой-то есть и дамаг может принимать и умереть может.

    Разделяй этот интерфейс на несколько.

    namespace Health
    {
        public interface IHealth
        {
            void Lose(int amount);
            void Restore(int amount);
        }
        
        public interface IMutable<out T>
        {
            T Current { get; }
        }
        
        public interface IFinal
        {
            event Action Over;
        }
    }


    Получается такой Health:

    using System;
    using UnityEngine;
    
    namespace Health
    {
        public class Health : IHealth, IFinal, IMutable<int>
        {
            public event Action Over;
            private readonly int _max;
            private const int Min = 0;
    
            public Health(int max)
            {
                _max = max;
                Current = _max;
            }
            
            public int Current { get; private set; }
            
            public void Lose(int amount)
            {
                SetCurrent(Current - amount);
            }
    
            public void Restore(int amount)
            {
                SetCurrent(Current + amount);
            }
    
            private void SetCurrent(int amount)
            {
                Current = Mathf.Clamp(amount, Min, _max);
                
                if (Current == Min) Over?.Invoke();
            }
            
        }
    }


    Health не должен быть отдельным компонентом, который будет висеть на условном рыцаре. Рыцарь будет содержать в себе этот Health, но напрямую хп ему изменять никто не будет, поэтому нужен еще один интерфейс IDamageable:
    namespace Health
    {
        public interface IDamageable
        {
            void ApplyDamage(int amount);
        }
    }


    И сам рыцарь:
    using UnityEngine;
    
    namespace Health
    {
        public class Knight : MonoBehaviour, IDamageable
        {
            [SerializeField] private int _maxHealth = 100;
            private Health _health;
    
            private void Awake()
            {
                _health = new Health(_maxHealth);
            }
    
            private void OnEnable()
            {
                _health.Over += Die;
            }
    
            private void OnDisable()
            {
                _health.Over -= Die;
            }
    
            public void ApplyDamage(int amount)
            {
                _health.Lose(amount);
                Debug.Log($"Damaged, hp left - {_health.Current}");
            }
    
            private void Die()
            {
                Debug.Log("Died");
                Destroy(gameObject);
            }
        }
    }


    Теперь, чтобы нанести урон рыцарю, нужно получить компонент IDamageable и вызвать его метод ApplyDamage:
    using UnityEngine;
    
    namespace Health
    {
        public class Enemy : MonoBehaviour
        {
            [SerializeField] private int _damage = 50;
            
            private void OnTriggerEnter2D(Collider2D other)
            {
                if (other.TryGetComponent(out IDamageable damageable))
                {
                    damageable.ApplyDamage(_damage);
                }
            }
        }
    }


    Всё. Используй TryGetComponent и тогда не нужно будет делать миллион проверок является ли что-то null.
    Warside твой должен висеть на рыцаре, а не на хп, поэтому делай отдельный интерфейс под этот Warside.
    Ответ написан
    2 комментария
  • Плавное движение и вращение персонажа с помощью acceleration.x?

    K0TlK
    @K0TlK
    Буллю людей.
    Код твой изменять не буду, сам все перепишешь на моем примере.

    Начнем с низов. Есть ввод от игрока. Для этого ввода вводим интерфейс:
    namespace SmoothMovement
    {
        public interface IPlayerInput
        {
            float Acceleration { get; }
        }
    }


    Далее нам нужно сглаживать этот Acceleration, значит вводим еще один интерфейс:
    namespace SmoothMovement
    {
        public interface ISmoothAcceleration
        {
            float Smooth(float acceleration, float input);
        }
    }


    Далее реализуем IPlayerInput:
    using UnityEngine;
    
    namespace SmoothMovement
    {
        public class MobileInput : MonoBehaviour, IPlayerInput
        {
            public float Acceleration { get; private set; }
    
            [SerializeField] private float _minAcceleration = -1f;
            [SerializeField] private float _maxAcceleration = 1f;
            [SerializeField] private float _smoothMultiplier = 5f;
            [Range(0, 1)] [SerializeField] private float _fadingSpeed = 0.01f;
    
            private ISmoothAcceleration _smoothing;
    
            private void Awake()
            {
                _smoothing = new SmoothedAcceleration(_minAcceleration, _maxAcceleration, _smoothMultiplier, _fadingSpeed);
            }
    
            private void Update()
            {
                Acceleration = _smoothing.Smooth(Acceleration, Input.acceleration.x);
            }
        }
    }


    В апдейте присваиваем свойству Acceleration сглаженное значение инпута. Далее само сглаживание:
    using UnityEngine;
    
    namespace SmoothMovement
    {
        public class SmoothedAcceleration : ISmoothAcceleration
        {
            private readonly float _multiplier;
            private readonly float _minValue;
            private readonly float _maxValue;
            private readonly float _fadingSpeed;
    
            
            public SmoothedAcceleration(float minValue, float maxValue, float multiplier, float fadeSpeed)
            {
                _minValue = minValue;
                _maxValue = maxValue;
                _multiplier = multiplier;
                _fadingSpeed = fadeSpeed;
            }
            
            public float Smooth(float acceleration, float input)
            {
                if (input == 0)
                {
                    acceleration = Mathf.Lerp(acceleration, 0, _fadingSpeed);
                    return acceleration;
                }
                
                acceleration += input * _multiplier * Time.deltaTime;
                acceleration = Mathf.Clamp(acceleration, _minValue, _maxValue);
    
                return acceleration;
            }
        }
    }


    Есть минимальные и максимальное значение ускорения, множитель - чем он больше, тем быстрее разгоняться будет и скорость затухания ускорения - чем больше тем быстрее ускорение будет стремиться к нулю. И тест:
    using UnityEngine;
    
    namespace SmoothMovement
    {
        public class TestMovement : MonoBehaviour
        {
            [SerializeField] private MonoBehaviour _input = null;
            [SerializeField] private float _speed = 10f;
            
            
            private IPlayerInput Input => (IPlayerInput)_input;
    
    
            private void OnValidate()
            {
                if (_input is IPlayerInput) return;
                
                Debug.LogError($"{nameof(_input)} should implement {nameof(IPlayerInput)}");
                _input = null;
            }
    
            private void FixedUpdate()
            {
                Move(Input.Acceleration);
            }
    
            private void Move(float direction)
            {
                if (direction == 0) return;
                
                var position = transform.position;
                position.x += direction * _speed * Time.deltaTime;
                transform.position = position;
            }
        }
    }

    Инжектим IPlayerInput через инспектор, двигаем геймобжект. С вращением делай сам что-нибудь, я не знаю как у тебя там что должно вращаться
    Ответ написан
    Комментировать
  • Почему OnTriggerStay2D работает не сразу?

    K0TlK
    @K0TlK
    Буллю людей.
    Потому что OnTriggerStay вызывается не каждый кадр. Отдели ввод игрока. WakeUp - костыль, которого не должно быть. Вот примерно то, что тебе надо:
    public class KeyboardInput : MonoBehaviour, IPlayerInput
    {
        public event Action ActionButtonPressed;
    
        private void Update()
        {
            if (Input.GetKeyDown(KeyCode.F))
            {
                ActionButtonPressed?.Invoke();
            }
        }
    }
    
    public interface IPlayerInput
    {
        public event Action ActionButtonPressed;
    }


    Отдельно инпут.

    [RequireComponent(typeof(Collider2D))]
    public class DialogueTrigger : MonoBehaviour
    {
        [SerializeField] private KeyboardInput _input;
    
        private void OnTriggerEnter2D(Collider2D collision)
        {
            if (collision.TryGetComponent(out Player player))
            {
                _input.ActionButtonPressed += StartDialogue;
            }
        }
    
        private void OnTriggerExit2D(Collider2D collision)
        {
            if (collision.TryGetComponent(out Player player))
            {
                _input.ActionButtonPressed -= StartDialogue;
                EndDialogue();
            }
        }
    
        private void StartDialogue()
        {
            Debug.Log("DialogueStarted");
        }
    
        private void EndDialogue()
        {
            Debug.Log("DialogueEnded");
        }
    
    }


    Отдельно все остальное. На входе подписываемся, на выходе отписываемся. Все. Не нужен никакой OnTriggerStay и WakeUp.
    Ответ написан
    8 комментариев
  • Как отследить, что объект находится около точки в 2d на unity?

    K0TlK
    @K0TlK
    Буллю людей.
    Проверяй дистанцию между платформой и точкой
    var distance = (point.position - transform.position).magnitude;

    Vector3.magnitude Возвращает длину вектора. Вот пример:
    public class Example : MonoBehaviour
    {
        [SerializeField] private Transform _pointA, _pointB = null;
        [SerializeField] private float _speed = 10f;
        [SerializeField] private float _closeDistance = 0.2f;
    
        private Transform _currentPoint;
    
        private void Start()
        {
            _currentPoint = _pointA;
        }
    
        private void FixedUpdate()
        {
            MoveTo(_currentPoint.position);
        }
    
        private void MoveTo(Vector3 position)
        {
            var nextPosition = transform.position;
            var delta = (position - transform.position).normalized;
    
            delta *= _speed * Time.deltaTime;
    
            nextPosition += delta;
    
            transform.position = nextPosition;
    
            UpdatePoint();
        }
    
        private void UpdatePoint()
        {
            var distance = (_currentPoint.position - transform.position).magnitude;
            if (distance <= _closeDistance)
            {
                SwitchPoint();
            }
        }
    
        private void SwitchPoint()
        {
            _currentPoint = _currentPoint == _pointA ? _pointB : _pointA;
        }
    
    }


    В UpdatePoint я рассчитываю расстояние между точкой и позицией платформы и, если оно меньше максимальной дистанции приближения, то меняю точку, к которой стремится платформа.

    С тем же успехом можно сравнивать позиции платформы и точки, к которой она стремится и, если они равны, менять точку.
    Ответ написан
    Комментировать
  • Почему деньги не вычитаются при покупке из переменной?

    K0TlK
    @K0TlK
    Буллю людей.
    Какой раз ты уже задаешь этот вопрос? Третий? Ты ничему не научишься, если будешь просто копипастить код, который кто-то написал за тебя. Ты где-то нашел код, скопировал его и пытаешься изменить, не понимая того, как все работает.
    Ты и не отнимаешь эти деньги из первого скрипта, ты загружаешь данные из json в BuyCharacter, изменяешь их и сохраняешь.
    Тебе сюда и сюда
    Ответ написан
  • Как сделать так, чтобы функция вызывалась при изменении переменной в Unity?

    K0TlK
    @K0TlK
    Буллю людей.
    Если ползунок - юнитивский слайдер, то у него есть событие onValueChanged. Подписываешь на это событие метод, который изменяет громкость и все.
    Что-то типа этого:
    [SerializeField] private Slider _slider;
    
    private void OnEnable()
    {
        _slider.onValueChanged.AddListener(ChangeVolume);
    }
    
    private void OnDisable()
    {
        _slider.onValueChanged.RemoveListener(ChangeVolume);
    }
    
    private void ChangeVolume(float amount)
    {
        //изменить громкость на значение amount
    }
    Ответ написан
    1 комментарий
  • Как вызвать класс из другого скрипта (прикреплены к одному объекту)?

    K0TlK
    @K0TlK
    Буллю людей.
    Stop это не класс, а метод. Если они прикреплены к одному объекту, то можно получить ком>понент через GetComponent, как вы это делаете с rigidbody. В вашем случае это -

    GetComponent<combatPlaye>(), а далее уже у закэшированного компонента вызывать метод, который вам нужен.
    Ответ написан
    7 комментариев