Create basic butterfly AI. - #152
Conversation
| @@ -0,0 +1,6 @@ | |||
| public enum BehaviourStateButterfly | |||
| { | |||
| None = -1, | |||
There was a problem hiding this comment.
CodeConvention 28: Любое перечисление должно содержать None. Если это int перечисление, то None должен соответствовать 0.
There was a problem hiding this comment.
Я по этому поводу спросил, как быть? Потому что VS2022 настоятельно рекомендовала None делать равным -1 но так никто и не ответил.
|
|
||
| public void OnEnable() | ||
| { | ||
| _physicsService = Services.SharedInstance.PhysicsService; |
There was a problem hiding this comment.
тут прилетает null, потому что инициализация микросервисов происходит позже чем активируется текущий SO (в текущей ветке, инициализация выполнена в Start() )
There was a problem hiding this comment.
Да, но он при каждом апдейте обнуляется... Как это почему?!
Если мы уже сохранили ссылку на _physicsService, почему он при каждом апдейте слетает то?
|
|
||
| public void Act(ButterflyModel butterflyModel) | ||
| { | ||
| //TODO - почему тут каждый кадр слетает _physicsService?!?!?!?!?!? |
There was a problem hiding this comment.
Разве в каждом? В первом кадре присваивается значение, а дальше уже в условия не попадает. Проверь у себя еще раз.
There was a problem hiding this comment.
Вот именно что в каждом апдейте слетает....
Я цже все перепроверил, почему так и затянул со сдачей этого... убрал все лишнее... Оставил только бабочку... Слетает при каждом апдейте. Приходится заново инициализировать в методе Act() и что-то мне подсказывает это так и осталось в главной ветке master
There was a problem hiding this comment.
Возможно, слетает в этой ветеке. Я только что проверила в рабочей, ничего не слетает.
|
|
||
| private void Idle(ButterflyModel butterflyModel) | ||
| { | ||
| Debug.Log($"Бабочка сидит на месте"); |
There was a problem hiding this comment.
На будущее: в самом проекте использовать CustomDebug
There was a problem hiding this comment.
а у нас есть такой? )) мне то он вообще не нужен, это я так показать смену состояния
There was a problem hiding this comment.
Я не уверена, есть ли в этой ветке, но просто стоит это учитывать, если понадобится в основной ветке :)
| public const string PLAYER = "Player"; | ||
|
|
||
| public const string RABBIT = "Rabbit"; | ||
| public const string BUTTERFLY = "Butterfly"; |
There was a problem hiding this comment.
Зачем создавать тег, если он нигде не используется?
There was a problem hiding this comment.
используется! При создании, при коллизии у вас тут используется система тегов )) что меня жутко взбесило кажется, но уже не помню точно
There was a problem hiding this comment.
Где-то в проекте может еще и используются, а так нужно использовать Type вместо тегов
|
|
||
| public void Act(ButterflyModel butterflyModel) | ||
| { | ||
| //TODO - почему тут каждый кадр слетает _physicsService?!?!?!?!?!? |
There was a problem hiding this comment.
Возможно, слетает в этой ветеке. Я только что проверила в рабочей, ничего не слетает.
|
|
||
| private void Idle(ButterflyModel butterflyModel) | ||
| { | ||
| Debug.Log($"Бабочка сидит на месте"); |
There was a problem hiding this comment.
Я не уверена, есть ли в этой ветке, но просто стоит это учитывать, если понадобится в основной ветке :)
3 participants
Вот такой вариант бабочки пойдет?
Мне не понятно почему там каждый кадр слетает _physicsService?