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

May the Code Review be with you [Russian]

May the Code Review be with you [Russian]

Code review is often mistaken for a silver bullet that solves all the developer problems. In reality it's not exactly like that, and this process has its requirements and constraints, just like everything else. We'll talk about what teams can benefit from code review and how to approach this process. We'll get into the details of process, measure the time it takes and learn to assess its usefulness.

Also the main rules of conducting a code review will be shown: both technical and ethical ones that can have positive impact not only on product quality but also on the team making it.

Egor Tolstoy

April 21, 2017
Tweet

More Decks by Egor Tolstoy

Other Decks in Technology

Transcript

  1. Егор Толстой
    telegram.me/
    telegram.me/
    etolstoy
    iosgr

    View Slide

  2. 2

    View Slide

  3. 3

    View Slide

  4. 4

    View Slide

  5. 5

    View Slide

  6. 6
    просмотр
    101
    1

    View Slide

  7. 7
    Анимации
    UX
    Motion Design
    Next Big Thing
    Красивая кнопочка

    View Slide

  8. 8
    May the Code Review
    be with you

    View Slide

  9. 9
    Цели
    Практики
    Софт
    Этика
    Code
    Review

    View Slide

  10. 10
    Цели
    Практики
    Софт
    Этика
    Code
    Review

    View Slide

  11. 11
    #1: Коллективное владение
    кодом

    View Slide

  12. 12
    12

    View Slide

  13. 13

    View Slide

  14. 14
    #2: Переиспользование кода

    View Slide

  15. 15

    View Slide

  16. 16

    View Slide

  17. 17

    View Slide

  18. 18
    #3: Обмен знаниями

    View Slide

  19. 19

    View Slide

  20. 20

    View Slide

  21. 21

    View Slide

  22. 22

    View Slide

  23. 23
    #4: Обнаружение ошибок

    View Slide

  24. 24
    Retain циклы
    Многопоточность
    Квалификаторы доступа
    Ошибки в логике

    View Slide

  25. 25

    View Slide

  26. 26

    View Slide

  27. 27
    #5: Единообразие проекта

    View Slide

  28. 28

    View Slide

  29. 29
    1 просмотр

    View Slide

  30. 30

    View Slide

  31. 31

    View Slide

  32. 32
    #1: Коллективное владение кодом
    #2: Переиспользование кода
    #3: Обмен знаниями
    #4: Обнаружение ошибок
    #5: Единообразие проекта

    View Slide

  33. 33
    Цели
    Практики
    Софт
    Этика
    Code
    Review

    View Slide

  34. 34
    Кто делает review?
    Когда делать review?
    Как делать review?
    Эффективно ли review?

    View Slide

  35. 35
    Кто делает review?
    Когда делать review?
    Как делать review?
    Эффективно ли review?

    View Slide

  36. 36
    Маленькая команда
    Много маленьких команд
    Большая команда
    Одиночка

    View Slide

  37. 37

    View Slide

  38. 38
    Автор
    Ревьюер 1
    Ревьюер 2
    Ревьюер 3
    Ревью

    View Slide

  39. 39
    Автор
    Ревьюер 1
    Ревьюер 2
    Ревьюер 3
    Ревью

    View Slide

  40. 40

    View Slide

  41. 41
    Тимлид
    Команда 1 Команда 2

    View Slide

  42. 42
    Тимлид
    Команда 1 Команда 2

    View Slide

  43. 43
    Тимлид
    Команда 1 Команда 2

    View Slide

  44. 44
    Тимлид
    Команда 1 Команда 2

    View Slide

  45. 45

    View Slide

  46. 46
    Автор

    View Slide

  47. 47
    Автор

    View Slide

  48. 48
    Автор

    View Slide

  49. 49
    A. Garbarev: 5 коммитов
    J. Blues: 4 коммита
    Кандидаты

    View Slide

  50. 50

    View Slide

  51. 51
    DataProvider1
    height = 26.0
    case 14:
    (strong) *delegate

    View Slide

  52. 52
    Чаты в Slack и Telegram
    Сходки PeerLab
    Другие коллеги

    View Slide

  53. 53
    Чаты в Slack и Telegram
    Сходки PeerLab
    Другие коллеги

    View Slide

  54. 54
    cocoadevelopers.club
    telegram.me/iosgt

    View Slide

  55. 55

    View Slide

  56. 56
    Чаты в Slack и Telegram
    Сходки PeerLab
    Другие коллеги

    View Slide

  57. 57
    • Обмен опытом
    • Code Review
    • Испытания библиотек
    • Просмотр видео
    peerlab.community

    View Slide

  58. 58
    Чаты в Slack и Telegram
    Сходки PeerLab
    Другие коллеги

    View Slide

  59. 59
    Не все хотят
    смотреть твой код

    View Slide

  60. 60
    Кто делает review?
    Когда делать review?
    Как делать review?
    Эффективно ли review?

    View Slide

  61. 61

    View Slide

  62. 62

    View Slide

  63. 63
    Кто делает review?
    Когда делать review?
    Как делать review?
    Эффективно ли review?

    View Slide

  64. 64

    View Slide

  65. 65
    Проводите
    архитектурные
    review

    View Slide

  66. 66
    0
    50
    100
    150
    200
    0 500 1000 1500 2000

    View Slide

  67. строки
    кода
    67
    0
    50
    100
    150
    200
    0 500 1000 1500 2000
    комментарии
    чем больше
    изменений,
    тем меньше
    комментариев

    View Slide

  68. 68
    Отправляйте
    дневную норму
    кода

    View Slide

  69. 69

    View Slide

  70. 70
    Проводите self
    assessment

    View Slide

  71. 71

    View Slide

  72. 72
    Подробно
    описывайте, что
    сделали

    View Slide

  73. 73

    View Slide

  74. 74
    Автоматизируйте
    чек-лист

    View Slide

  75. 75
    SwiftLint
    OCLint
    Infer
    Clang Analyzer
    CPD

    View Slide

  76. 76

    View Slide

  77. 77
    Автоматизация чек-листа для
    Code Review

    View Slide

  78. 78
    Стиль коммитов
    Наполнение коммитов
    Правописание в коммитах
    Линтеры
    Артефакты

    View Slide

  79. 79

    View Slide

  80. 80

    View Slide

  81. 81
    Кто делает review?
    Когда делать review?
    Как делать review?
    Эффективно ли review?

    View Slide

  82. 82
    Количество review
    Просмотренные строки кода
    Найденные ошибки

    View Slide

  83. 83
    Количество review
    Просмотренные строки кода
    Найденные ошибки

    View Slide

  84. 84
    Количество review
    Просмотренные строки кода
    Найденные ошибки

    View Slide

  85. 85
    Количество review
    Просмотренные строки кода
    Найденные ошибки

    View Slide

  86. 86
    Количество review
    Просмотренные строки кода
    Найденные ошибки

    View Slide

  87. 87
    Количество review
    Просмотренные строки кода
    Найденные ошибки

    View Slide

  88. 88
    #1: Коллективное владение кодом
    #2: Переиспользование кода
    #3: Обмен знаниями
    #4: Обнаружение ошибок
    #5: Единообразие проекта

    View Slide

  89. 89
    Goal-Question-Metric
    1. Определите цель
    2. Сформулируйте вопросы
    3. Подберите метрики

    View Slide

  90. 90
    Цель
    «Повысить уровень
    переиспользования кода»

    View Slide

  91. 91
    Вопросы
    Используются ли общие библиотеки?
    Вынесены ли визуальные стили?
    Какой уровень дублирования кода?

    View Slide

  92. 92
    Метрики
    Количество общих библиотек
    Количество захардкоженных стилей
    Уровень дублирования кода

    View Slide

  93. 93
    Выполнено 60% плана

    View Slide

  94. 94
    Inspection rate = LoC / hour
    Defect rate = problems / hour
    Defect density = problems / LoC

    View Slide

  95. 95

    View Slide

  96. 96
    210 LoC

    View Slide

  97. 97

    View Slide

  98. 98

    View Slide

  99. 99
    Inspection rate = 2.1 / 0.56 = 3.75
    Defect rate = 4 / 0.56 = 7.1
    Defect density = 4 / 2.1 = 1.9

    View Slide

  100. 100
    Inspection rate = 2.1 / 0.08 = 26.25
    Defect rate = 1 / 0.08 = 12.5
    Defect density = 1 / 2.1 = 0.48

    View Slide

  101. 101
    Inspection rate 3.75 vs 26.25
    Defect rate 7.1 vs 12.5
    Defect density 1.9 vs 0.48

    View Slide

  102. 102
    Состав участников
    Расписание участников
    Чек-лист
    Метрики эффективности

    View Slide

  103. 103
    Цели
    Практики
    Софт
    Этика
    Code
    Review

    View Slide

  104. 104
    Upsource / JetBrains
    Crucible / Atlassian
    GitHub

    View Slide

  105. 105

    View Slide

  106. 106

    View Slide

  107. 107
    Работа в IDE
    Интеграции с кучей сервисов
    Мощная статистика
    Code Ownership
    Гибкие уведомления

    View Slide

  108. 108

    View Slide

  109. 109

    View Slide

  110. 110
    Pre-commit review
    Review сниппетов
    Считает время на review
    Тесная интеграция с Atlassian
    Гибкие теги

    View Slide

  111. 111

    View Slide

  112. 112

    View Slide

  113. 113
    Можно ставить смайлики
    Крутой Code Ownership
    Шаблоны для PR
    Интеграции с инструментами
    Доступен из коробки

    View Slide

  114. 114

    View Slide

  115. 115
    Цели
    Практики
    Софт
    Этика
    Code
    Review

    View Slide

  116. 116

    View Slide

  117. 117
    Терминология
    действительно важна

    View Slide

  118. Профессионализм
    118
    Внимательность
    Адекватность
    Доброжелательность
    Отзывчивость
    Открытость
    Устойчивость к критике
    Коммуникабельность
    Ответственность

    View Slide

  119. 119

    View Slide

  120. 120
    Цели
    Практики
    Софт
    Этика
    Code
    Review

    View Slide

  121. 121

    View Slide

  122. 122

    View Slide

  123. Code Review нужен
    вам и вашей команде
    telegram.me/iosgr

    View Slide