DV
Понял, а в релиз идет обработка исключения или как?
Size: a a a
2020 June 08
🐸
Или в логах при тестах
🐸
В релизе их не должно быть
DV
В релизе их не должно быть
Исключений? То в таком случае, могу ли я в релизе оставить тот вариант который я изначально показал?
P#
Павел ответил, что пусть выбросит исключение, и мы будем знать, что где-то на объекте не висит в конкретном случае "враг".
Че опять повел?
🐸
Это уже зависит от контекста. Где-то можно, где-то не должно быть подобной заглушки
P#
Исключений? То в таком случае, могу ли я в релизе оставить тот вариант который я изначально показал?
Покаж ещё раз
DV
Переслано от Danila Vanilla
var tempEnemy = enemiesToDamage[i].GetComponent<enemy>();
if(tempEnemy)
{
enemy.TakeDamage(damage);
}
DV
Я челу ответил в основе. Павел поругался на проверку на null
🐸
Тут первое - это именование. Нужно IDamagable
P#
Переслано от Danila Vanilla
var tempEnemy = enemiesToDamage[i].GetComponent<enemy>();
if(tempEnemy)
{
enemy.TakeDamage(damage);
}
А если enemiestodamage.Count<=i
DV
Тут первое - это именование. Нужно IDamagable
Вопрос не ко мне
DV
Я парню указал на ошибку
AD
Гет компонент в цикле это шедевр
DV
И написал решение
🐸
Гет компонент в цикле это шедевр
Да ладно?
P#
Ну вообще проверка на нулл это не так уж и плохо
AD
Ну вообще проверка на нулл это не так уж и плохо
Это ужос
P#
Просто тут зависит от человека, если человек сознательно ожидает что в каком то формате ему может приходить нулл и так должно быть по логике то все норм
🐸
Ну вообще проверка на нулл это не так уж и плохо
Скорее всего, Павел решил, что это кусок метода, куда список энеми должен приходить - тогда всё верно. Если это просто проверка результата сферкаста, то можно и простым нуллчеком