Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Загадочная история одного рефакторинга

Загадочная история одного рефакторинга

История рефакторинга hh.ru длинною в 2.5 года

Alexander Blinov

October 12, 2020
Tweet

More Decks by Alexander Blinov

Other Decks in Programming

Transcript

  1. Проблемы • Новая команда, ни***а не понятно что там внутри

    и какая логика • Два приложения процентов на 80 разные живут в одном модуле в разных flavors • Монолит «вертикальный», все написано на Activity + AsincTask-ах • Куча синлтонов • Сеть на своем доретрофитном велосипеде • Логика проверки на null и разного поведения прямо на UI • Местами вместо recycler списки собираются вьюшками • Баги • Будет очень гибкая навигация и диплинки повсюду • Релизный цикл — раз в 1-3 месяца • Билд сервер Антон • Крашрейт 99.95 • Умение анализировать процессы 03/2018
  2. Photo by Nikita Herasimenka on Unsplash Пилить фичи Довыносить модули

    Что делать дальше Чинить локальные проблемы
  3. • Smart Router-ы растолстели и мешают выносу остатка модулей •

    Модули выносили хаотично, критичный путь сборки длины 23 • DI кривоватый и в связи модулей много boilerplate • Скоро выйдет Jetpack Compose, и он про MVVM • На iOS дизайн система дала прирост к скорости 03/2020 Проблемы
  4. Photo by Hunters Race on Unsplash Выводы AS IS, TO

    BE, Actions, Measurement На метрики надо смотреть, но знать, что измеряешь Продавайте рефакторинг и облюдайте баланс во всем
  5. +1

  6. О чем мы говорили Выводы AS IS, TO BE, Actions,

    Measurement На метрики надо смотреть, но знать, что измеряешь Продавайте рефакторинг и облюдайте баланс во всем