Improving Your Mobile Development (rus)

Совершенствование разработки мобильных приложений

Доклад о том, как мы улучшали процессы разработки в мобильной (и не только) команде, что использовали и почему, к чему стремились.

IT'Summer 2016
http://cioconf.ru/program/#2276

Ключевые слова: Continuous Integration, Continuous Delivery, code-review, статический анализ, QA

Transcript

1. Совершенствование разработки мобильных приложений Илья Третьяков
2. Когда-то давно… 2009 год https://developer.android.com/about/versions/android-1.6-highlights.html http://www.mrgadget.com.au/gadget/2009/top-grossing-apps-at-apple-appstore-australia-dec-2009/
3. • «Мы не можем релизиться - Иван сейчас в отпуске» • «Мы не можем выложить обновление - сборка сломана» • «Ммм, нет смысла ради этого багфикса обновлять приложение» • «Ты уверен, что эта сборка использует production- сервер?» • «Нужна новая сборка, в старой забыли увеличить версию»
4. Code review
5. Code review • раннее исправление ошибок
6. Code review • раннее исправление ошибок • обучение младших программистов
7. Code review • раннее исправление ошибок • обучение младших программистов • изучение кода коллег
8. Code review • раннее исправление ошибок • обучение младших программистов • изучение кода коллег • уменьшение дублирования функционала (не кода!)
9. Code review • ограничивайте количество кода
10. Code review https://smartbear.com/learn/code-review/best-practices-for-peer-code-review/
11. Code review • ограничивайте количество кода • ограничивайте время и скорость
12. Code review https://smartbear.com/learn/code-review/best-practices-for-peer-code-review/
13. Code review • ограничивайте количество кода • ограничивайте время • очень важна регулярность • 1 задача - 1 инспекция кода • помните о комфорте Best Kept Secrets of Peer Code Review / Jason Cohen
14. Code review • периодический повтор замечаний • очевидные замечания • важные вещи упускались
15. Статический анализ кода • не отнимает время у разработчиков • исключает очевидные замечания до ревью • позволяет оценить технический долг
16. Статический анализ кода
17. Continuous Integration push hook/poll build tests static analysis notification CI git save artifact
18. http://nvie.com/posts/a-successful-git-branching-model/
19. Распространение • usb-кабель/apk-file • Fabric Beta/Hockeyapp • свой сервер • Google Play
20. Распространение Версия • своя для каждой поставляемой ветки • используется номер сборки • versionCode = 76 • versionName = 1.4.76 • используется QA/пользователем при описании багов • используется программистами (тег, логи, отчеты об ошибках)
21. Распространение Список изменений (changelog) • что изменилось? • что появилось? • варианты составления: • руками • git log • ссылки на тикеты / документацию
22. Распространение Сбор данных о крешах • сохранение дебаг-символов для расшифровки стектрейсов (mapping.txt/dSYM.zip) • сохранение данных о креше • стектрейс • название устройства • версия Android • технические данные и тд • логи
23. Распространение Сбор данных о крешах
24. Обычно Должно быть Серверные и мобильные разработчики Кадр из фильма Captain America: Civil War http://marvel.com/movies/movie/219/captain_america_civil_war Photo by Alberto E. Rodriguez/Getty Images for Disney Мобильное API: Серверные разработчики VS Мобильные, Роман мочалов, http://www.slideshare.net/ImproveGroup/api-vs
25. Описание API • «А посмотри сам» • *.doc файл • thrift, protobuffers • swagger, apiary
26. Спецификация API • история изменений • протокол передачи данных (обычно https) • кодировка, типы полей, структуры данных • ошибки • локализация • аутентификация
27. isItSunnyToday обязательный boolean 1. 0 2. 1 3. true 4. false 5. null 6. "да" { "isItSunnyToday": … }
28. –Мобильный разработчик «ПриложениеAPI не работает!» «Приложение не работает!» –Заказчик
29. Причины • спецификацию не читают • вам не сказали, что спецификацию обновили • API не готово
30. https://www.flickr.com/photos/birgerking/6821074104
31. Статистика • не знают про гайды • знают, но не читали • знают, читали, не хотят следовать 10 % 90 %
32. Статистика • не знают про гайды • знают, но не читали • знают, читали, не хотят следовать 10 % 90 % «42,7 процента статистики берутся с потолка» Стивен Райт, «Симпсоны и их математические секреты» Саймон Сигнх
33. Дизайн реального проекта http://photoshopetiquette.com/
34. http://photoshopetiquette.com/ Дизайн реального проекта
35. http://photoshopetiquette.com/ Дизайн реального проекта
36. http://photoshopetiquette.com/ Дизайн реального проекта
37. Metrics & keylines
38. zeplin.io
39. zeplin.io
40. zeplin.io
41. Совершенствование разработки мобильных приложений Илья Третьяков twitter.com/ jvilya linkedin.com/in/ ilyavtretyakov