Thread by @mobileunderhood

И тут я открываю ещё один поток, связанный с хорошим кодом, и дебагом. Хороший код должно быть легко дебажить - ибо мы точно в наших условиях написания кода допустим баги. И не всегда есть доступ к устройству.
Да тред про логи, но начнём с assert 😅

assert - функция которая нужна для программистов и отсекается в дебаге. Может иметь ещё названия: precondition, postcondition. Но суть у них одна.
С ней стоит быть аккуратней - нельзя в ней писать код, который что-то может изменить.
А то в дебаге работает, в релизе нет :)

Её мы используем, когда мы хотим сообщить себе/другим программистам, что что-то пошло не так - чаще всего какие-то данные отличаются от тех, что планировались.
Но код должен нормально продолжать работать при этом.

Написание ассертов позволяет избежать будущих проблем, когда ошибка долга остаётся не замеченной пока не выстрелит. А выстреливают обычно сильно.

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

А теперь самое интересное - я считаю, что ассерт должен попадать в логи релиза. Я считаю, что это важная информация, если у клиента произошел ассерт.

Ну а дальше про логи.
1. Логирование это привычка - если ей не обладает вся команда, то логи вас не сильно спасут

Ну просто из-за того, что половинчатые логи, могут дать ложную информацию - по логам проблема была в одном, а в реальности она была в другом, просто там не было логов.

2. Логи это не панацея. Если вы не понимаете зачем они вам, и их ценности, то лучше не надо. Это инструмент. Как и дебагер, или профайл - это один из инструментов доступный всем.

3. Но инструмент при правильном использовании решает две проблемы:
а. Дебаг в многопоточной среде - бывает, что остановка программы влияет на её исполнение.
б. Дебаг в случае если устройство далеко, а баг только на нём. В ковид особенно актуально 😅

Я бы хотел написать потоки "зачем вам логи?" и "как пользоваться логами?", но у меня не хватит времени.

Поэтому расскажу просто случай, недавний. Пока ещё помню:

Перед этим важная информация. В проекте я использую крашлитикс и конечно же логи настроена на неё.
Более того логи уровня assert и error прилетаю в крашлитикс как nonFatal, и говорят мне - где-то есть проблемы, не приводящие к крашу.

И так срабатывает ассерт: пользователь пробует войти в систему с незаполненными полями логин или пароль.
Почему проблема? Ну кнопка должна блокироваться по идее, если нет данных.

Но не критичная - сервер то проверит.

Тестирование пропустило - не заметило. Да и если запустить, то всё хорошо - ввожу кнопка активно, удаляю не активна.

Идём в логи: в логах видно, что это всегда второй вход. А первый вход отсекается из-за неверных данных.

Проверяем: вводим не верные данные, получаем ошибку, и... ничего.
Тут смотрим логи, там запись ещё - логин пароль почистился.
А у нас есть опциональная функция - не сохранять данные если ты залогинился. То есть все данные чиститятся

Ага, ок как воспроизвести удалось. Да это всё заняло 5 минут.
Но проблема то по факту оказывается не в том, что кнопка не блочиться (это тоже поправил), а в том - а чего данные то чистятся?

Получаем в итоге что это уже не ошибка UI, а ошибка бизнес логики.
А это уже критичнее.
Вот так за 20 минут было поправлено две разных ошибки - отсутствие обновление UI + косяк в бизнес логике.

Последняя возможно в будущем бы стрельнула, а может нет.
А UI просто не приятная.