Рефакторинг в условиях быстрорастущего стартапа. Егор Фесенко

Все хотят работать с хорошо написанной системой, в которой фикс багов будет занимать секунды, а добавление новых фич покажется легкой и незатейливой задачей. Но, к сожалению, изначально предусмотреть все шаги в развитии продукта мы не можем, и любой код через какое-то время перестанет быть идеальным для поставленных задач. Это нормально. И потому рефакторинг — процесс, который должен проводиться постоянно.
В докладе я расскажу:
- как найти время на рефакторинг;
- на что обращать внимание перед тем, как начать переписывать проект;
Покажу немного примеров из нашего опыта.

Transcript

1. Рефакторинг в условиях быстрорастущего стартапа In software development 'perfect' is a verb, not an adjective
2. 2 Егор Фесенко iOS Software Engineer

3. 3 Revolut Компания основана в 2015 г. 8+ млн пользователей 20+ офисов в мире 1700+ сотрудников Оценка компании $1,7 млрд
4. 4 Revolut locations
6. 6 Проблема со стороны бизнеса ❗ Обычно менеджер не видит прямую связь между рефакторингом и прибылью компании.
7. 7 ❗ Обычно менеджер не видит прямую связь между рефакторингом и прибылью компании. ‼ «Время на рефакторинг» — Грааль в мире разработки. Проблема со стороны бизнеса
8. 8 Проблема со стороны разработки ❗ Ребят, тут очень сложно всё, поэтому я оставил как было, но фича работает, всё нормально.
9. 9 Проблема со стороны разработки ❗ Ребят, тут очень сложно всё, поэтому я оставил как было, но фича работает, всё нормально. ‼ Всегда всё сложно. Сделать фичу — часть работы. Оставить все в лучшем состоянии, чем было до вас — тоже.
10. 10 Что делать • Решайте задачи бизнеса. Вы с менеджером в одной лодке, и его успех — ваш успех.
11. 11 Что делать • Решайте задачи бизнеса. Вы с менеджером в одной лодке, и его успех — ваш успех. • Найдите минимально возможное улучшение и запланируйте его в рамках следующего спринта.
12. 12 Что делать • Решайте задачи бизнеса. Вы с менеджером в одной лодке, и его успех — ваш успех. • Найдите минимально возможное улучшение и запланируйте его в рамках следующего спринта. • Регулярно показывайте заказчику ценность рефакторинга и его значение для продукта, пользователей.
13. 13 Примеры • Баг с доступом к вкладам
14. 14 Доменная модель
15. 15 Доменная модель
16. 16 Доменная модель
17. 17 Доменная модель
18. 18 Примеры • Баг с доступом к вкладам • Проблемы с производительностью на главном экране
20. 20 Слой работы с БД и слой работы с сетью
21. Так, ладно, давай подумаем
22. Как всё было 22
23. Что хотели получить 23
24. 24 рефакторинг
25. Слой работы с сетью
26. 26 Слой работы с сетью
27. 27 «Interface» MoneyBoxes ApiProtocol
28. 28 «Interface» StandingOrders ApiProtocol «Interface» AutoInvest ApiProtocol «Interface» MoneyBoxOperations ApiProtocol
30. Data Provider
31. •Ходил в сеть •Отправлял данные в локальное хранилище •Передавал информацию между экранами Что делал DataProvider 31
35. •Ходил в сеть (Repository) •Отправлял данные в локальное хранилище (Repository) •Передавал информацию между экранами Что делал DataProvider 35
36. Что хотели получить 36
37. •Ходил в сеть (Repository) •Отправлял данные в локальное хранилище (Repository) •Передавал информацию между экранами Что делал DataProvider 37
38. •Ходил в сеть (Repository) •Отправлял данные в локальное хранилище (Repository) •Передавал информацию между экранами Что делал DataProvider 38
39. Presentation Layer
40. 40 Presentation Layer
41. 41 Presentation Layer
42. 42 Presentation Layer
43. 43 Presentation Layer
44. 44 Presentation Layer
45. 45 Выводы • Помните, что вы зарабатываете деньги для компании, а не занимаетесь экспериментами над архитектурой.
46. 46 Выводы • Помните, что вы зарабатываете деньги для компании, а не занимаетесь экспериментами над архитектурой. • Попробуйте сделать наименьшее возможное улучшение и посмотрите, куда это приведет вас.
47. 47 Выводы • Помните, что вы зарабатываете деньги для компании, а не занимаетесь экспериментами над архитектурой. • Попробуйте сделать наименьшее возможное улучшение и посмотрите, куда это приведет вас. • Будьте на одной волне с бизнесом. Смотрите на себя как на часть команды, а не машинку для набирания кода.
48. 48 Выводы • Помните, что вы зарабатываете деньги для компании, а не занимаетесь экспериментами над архитектурой. • Попробуйте сделать наименьшее возможное улучшение и посмотрите, куда это приведет вас. • Будьте на одной волне с бизнесом. Смотрите на себя как на часть команды, а не машинку для набирания кода. • Идеала не существует, но это не повод не стремиться к нему.
49. Егор Фесенко iOS Software Engineer @ Revolut georgy.fesenko@revolut.com github.com/FesenkoG t.me/Fesenkoegor