Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.
Сode reviews as an important component of
the successful mobile product
Nick Sergeev, CTO @ Room 8 Studio
Input
Сжатые сроки (deadly deadlines)
Факторы свойственные [почти] любому проекту:
Input
Сжатые сроки (deadly deadlines)
Постоянные изменения требований (change requests every day)
Факторы свойственные [по...
Input
Сжатые сроки (deadly deadlines)
Постоянные изменения требований (change requests every day)
Разный уровень девелопер...
The issues we want to solve
Одинаковая осведомленность девелоперов об
изменениях/нововведениях в разных частях проекта
(kn...
The issues we want to solve
Одинаковая осведомленность девелоперов об
изменениях/нововведениях в разных частях проекта
(kn...
The issues we want to solve
Одинаковая осведомленность девелоперов об
изменениях/нововведениях в разных частях проекта
(kn...
Code review types
Team lead code review
Code review types
Team lead code review
XP elements
Code review types
Team lead code review
XP elements
Team code review
Code review types
Team lead code review
XP elements
Team code review
Team Code Review Format
Митинг длиной до 1 часа
Время проведения - середина дня - когда важные дела уже сделаны и необходи...
Additional points
Что делать если объем кода слишком большой?
Ревьюим только самый важный функционал, core features
Застав...
Additional points
Что делать если объем кода слишком большой?
Ревьюим только самый важный функционал, core features
Застав...
What do we got?
Все знают все
What do we got?
Все знают все
Ошибки разного рода выявляются на ранних этапах
What do we got?
Все знают все
Ошибки разного рода выявляются на ранних этапах
Ego effect
What do we got?
Все знают все
Ошибки разного рода выявляются на ранних этапах
Ego effect
It’s fun because it’s social
Last thoughts
Keep it lightweight! [Не перегружайте деталями участников, не заставляйте их
сразу писать громоздкие отчеты,...
Last thoughts
Keep it lightweight! [Не перегружайте деталями участников, не заставляйте их
сразу писать громоздкие отчеты,...
Last thoughts
Keep it lightweight! [Не перегружайте деталями участников, не заставляйте их
сразу писать громоздкие отчеты,...
Thanks! :)
@nick_sergeev
www.room8studio.com
Upcoming SlideShare
Loading in …5
×

Room8: Внедрение практик code review как важная составляющая успеха мобильного продукта

776 views

Published on

Успех любого программного продукта зависит от многих факторов и одним из основных является качество кода. В условиях часто меняющихся требований и параллельной разработки нескольких проектов следить за данным показателем невероятно сложно. Обсуждаемые в докладе практики призваны помочь руководителям отделов разработки и разработчикам решить эту проблему.

  • Be the first to comment

  • Be the first to like this

Room8: Внедрение практик code review как важная составляющая успеха мобильного продукта

  1. 1. Сode reviews as an important component of the successful mobile product Nick Sergeev, CTO @ Room 8 Studio
  2. 2. Input Сжатые сроки (deadly deadlines) Факторы свойственные [почти] любому проекту:
  3. 3. Input Сжатые сроки (deadly deadlines) Постоянные изменения требований (change requests every day) Факторы свойственные [почти] любому проекту:
  4. 4. Input Сжатые сроки (deadly deadlines) Постоянные изменения требований (change requests every day) Разный уровень девелоперов в команде (кадровая проблема) Факторы свойственные [почти] любому проекту:
  5. 5. The issues we want to solve Одинаковая осведомленность девелоперов об изменениях/нововведениях в разных частях проекта (knowledge/practices sharing)
  6. 6. The issues we want to solve Одинаковая осведомленность девелоперов об изменениях/нововведениях в разных частях проекта (knowledge/practices sharing) Быстрое вхождение в проектную команду девелоперов разного уровня
  7. 7. The issues we want to solve Одинаковая осведомленность девелоперов об изменениях/нововведениях в разных частях проекта (knowledge/practices sharing) Быстрое вхождение в проектную команду девелоперов разного уровня Bad practices sharing/rapid problem solving
  8. 8. Code review types Team lead code review
  9. 9. Code review types Team lead code review XP elements
  10. 10. Code review types Team lead code review XP elements Team code review
  11. 11. Code review types Team lead code review XP elements Team code review
  12. 12. Team Code Review Format Митинг длиной до 1 часа Время проведения - середина дня - когда важные дела уже сделаны и необходим перерыв Частота проведения - 1 раз в 2 спринта Участвует вся команда Приглашаем наиболее опытных людей из других команд Каждый девелопер представляет реализованные им фичи, команда обсуждает реализацию 1 человек ведет протокол, он содержит информацию о найденных проблемах и хороших решениях Протокол размещается во внутренней базе знаний (confluence) Каждый девелопер может вести собственный чек-лист На одном из последующих planning meetings при необходимости выделяется время на работы по исправлению
  13. 13. Additional points Что делать если объем кода слишком большой? Ревьюим только самый важный функционал, core features Заставляем себя комментировать код :)
  14. 14. Additional points Что делать если объем кода слишком большой? Ревьюим только самый важный функционал, core features Заставляем себя комментировать код :) Цель найти виноватого - второстепенна, гораздо более важно найти проблему
  15. 15. What do we got? Все знают все
  16. 16. What do we got? Все знают все Ошибки разного рода выявляются на ранних этапах
  17. 17. What do we got? Все знают все Ошибки разного рода выявляются на ранних этапах Ego effect
  18. 18. What do we got? Все знают все Ошибки разного рода выявляются на ранних этапах Ego effect It’s fun because it’s social
  19. 19. Last thoughts Keep it lightweight! [Не перегружайте деталями участников, не заставляйте их сразу писать громоздкие отчеты, сам процесс должен быть максимально простым] Внедряя любой процесс или практику надо помнить следующие вещи:
  20. 20. Last thoughts Keep it lightweight! [Не перегружайте деталями участников, не заставляйте их сразу писать громоздкие отчеты, сам процесс должен быть максимально простым] Внедряя любой процесс или практику надо помнить следующие вещи: Don’t micro manage! [Не заставляйте людей проверять каждую запятую, не заставляйте их следовать по шагам инструкции, которую вы составили или тому, что у вас в голове - позвольте им адаптироваться или адаптировать процесс]
  21. 21. Last thoughts Keep it lightweight! [Не перегружайте деталями участников, не заставляйте их сразу писать громоздкие отчеты, сам процесс должен быть максимально простым] Внедряя любой процесс или практику надо помнить следующие вещи: Don’t force it! [Никогда не ставьте жесткие сроки при внедрении новых практик, люди всегда сопротивляются новому - дайте им почувствовать преимущества самим] Don’t micro manage! [Не заставляйте людей проверять каждую запятую, не заставляйте их следовать по шагам инструкции, которую вы составили или тому, что у вас в голове - позвольте им адаптироваться или адаптировать процесс]
  22. 22. Thanks! :) @nick_sergeev www.room8studio.com

×