Добавить в корзинуПозвонить
Найти в Дзене
Исходный Код | IT

400+ коммитов, 4 слоя архитектуры, 700 звезд - и краш на обычном CRUD-сценарии

🗣️ Так начинается хороший вопрос к любому проекту: Что именно мы проверяем, когда говорим «код нормальный»? ℹ️ На YouTube-канале коллеги вышел разбор демо-проекта от Senior Android-разработчика из Европы, который уже два года не может найти работу. ℹ️ Проект оформлен по всем канонам: правильная многомодульная архитектура (разделение на data/domain/presentation/UI), паттерны проектирования и полностью «зеленые» юнит-тесты. На схеме все выглядело убедительно. Но при ручном прогоне простого сценария приложение упало: ➡️ сохранить геопозицию; ➡️ посмотреть историю; ➡️ очистить историю; ➡️ вернуться назад. И здесь главное не в одном баге. Главное в том, что проект проверяли как набор изолированных частей, а сломался он как продукт. 💬 Мы вынесли из этого 3 сигнала для ревью и собеседований в комментариях! 🖥 Исходный код — перейти на сайт!

400+ коммитов, 4 слоя архитектуры, 700 звезд - и краш на обычном CRUD-сценарии.

🗣️ Так начинается хороший вопрос к любому проекту:

Что именно мы проверяем, когда говорим «код нормальный»?

ℹ️ На YouTube-канале коллеги вышел разбор демо-проекта от Senior Android-разработчика из Европы, который уже два года не может найти работу.

ℹ️ Проект оформлен по всем канонам: правильная многомодульная архитектура (разделение на data/domain/presentation/UI), паттерны проектирования и полностью «зеленые» юнит-тесты.

На схеме все выглядело убедительно. Но при ручном прогоне простого сценария приложение упало:

➡️ сохранить геопозицию;

➡️ посмотреть историю;

➡️ очистить историю;

➡️ вернуться назад.

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

💬 Мы вынесли из этого 3 сигнала для ревью и собеседований в комментариях!

🖥 Исходный код — перейти на сайт!