Link
Embed
Share
Beginning
This slide
Copy link URL
Copy link URL
Copy iframe embed code
Copy iframe embed code
Copy javascript embed code
Copy javascript embed code
Share
Tweet
Share
Tweet
Slide 1
Slide 1 text
Егор Толстой telegram.me/ telegram.me/ etolstoy iosgr
Slide 2
Slide 2 text
2
Slide 3
Slide 3 text
3
Slide 4
Slide 4 text
4
Slide 5
Slide 5 text
5
Slide 6
Slide 6 text
6 просмотр 101 1
Slide 7
Slide 7 text
7 Анимации UX Motion Design Next Big Thing Красивая кнопочка
Slide 8
Slide 8 text
8 May the Code Review be with you
Slide 9
Slide 9 text
9 Цели Практики Софт Этика Code Review
Slide 10
Slide 10 text
10 Цели Практики Софт Этика Code Review
Slide 11
Slide 11 text
11 #1: Коллективное владение кодом
Slide 12
Slide 12 text
12 12
Slide 13
Slide 13 text
13
Slide 14
Slide 14 text
14 #2: Переиспользование кода
Slide 15
Slide 15 text
15
Slide 16
Slide 16 text
16
Slide 17
Slide 17 text
17
Slide 18
Slide 18 text
18 #3: Обмен знаниями
Slide 19
Slide 19 text
19
Slide 20
Slide 20 text
20
Slide 21
Slide 21 text
21
Slide 22
Slide 22 text
22
Slide 23
Slide 23 text
23 #4: Обнаружение ошибок
Slide 24
Slide 24 text
24 Retain циклы Многопоточность Квалификаторы доступа Ошибки в логике
Slide 25
Slide 25 text
25
Slide 26
Slide 26 text
26
Slide 27
Slide 27 text
27 #5: Единообразие проекта
Slide 28
Slide 28 text
28
Slide 29
Slide 29 text
29 1 просмотр
Slide 30
Slide 30 text
30
Slide 31
Slide 31 text
31
Slide 32
Slide 32 text
32 #1: Коллективное владение кодом #2: Переиспользование кода #3: Обмен знаниями #4: Обнаружение ошибок #5: Единообразие проекта
Slide 33
Slide 33 text
33 Цели Практики Софт Этика Code Review
Slide 34
Slide 34 text
34 Кто делает review? Когда делать review? Как делать review? Эффективно ли review?
Slide 35
Slide 35 text
35 Кто делает review? Когда делать review? Как делать review? Эффективно ли review?
Slide 36
Slide 36 text
36 Маленькая команда Много маленьких команд Большая команда Одиночка
Slide 37
Slide 37 text
37
Slide 38
Slide 38 text
38 Автор Ревьюер 1 Ревьюер 2 Ревьюер 3 Ревью
Slide 39
Slide 39 text
39 Автор Ревьюер 1 Ревьюер 2 Ревьюер 3 Ревью
Slide 40
Slide 40 text
40
Slide 41
Slide 41 text
41 Тимлид Команда 1 Команда 2
Slide 42
Slide 42 text
42 Тимлид Команда 1 Команда 2
Slide 43
Slide 43 text
43 Тимлид Команда 1 Команда 2
Slide 44
Slide 44 text
44 Тимлид Команда 1 Команда 2
Slide 45
Slide 45 text
45
Slide 46
Slide 46 text
46 Автор
Slide 47
Slide 47 text
47 Автор
Slide 48
Slide 48 text
48 Автор
Slide 49
Slide 49 text
49 A. Garbarev: 5 коммитов J. Blues: 4 коммита Кандидаты
Slide 50
Slide 50 text
50
Slide 51
Slide 51 text
51 DataProvider1 height = 26.0 case 14: (strong) *delegate
Slide 52
Slide 52 text
52 Чаты в Slack и Telegram Сходки PeerLab Другие коллеги
Slide 53
Slide 53 text
53 Чаты в Slack и Telegram Сходки PeerLab Другие коллеги
Slide 54
Slide 54 text
54 cocoadevelopers.club telegram.me/iosgt
Slide 55
Slide 55 text
55
Slide 56
Slide 56 text
56 Чаты в Slack и Telegram Сходки PeerLab Другие коллеги
Slide 57
Slide 57 text
57 • Обмен опытом • Code Review • Испытания библиотек • Просмотр видео peerlab.community
Slide 58
Slide 58 text
58 Чаты в Slack и Telegram Сходки PeerLab Другие коллеги
Slide 59
Slide 59 text
59 Не все хотят смотреть твой код
Slide 60
Slide 60 text
60 Кто делает review? Когда делать review? Как делать review? Эффективно ли review?
Slide 61
Slide 61 text
61
Slide 62
Slide 62 text
62
Slide 63
Slide 63 text
63 Кто делает review? Когда делать review? Как делать review? Эффективно ли review?
Slide 64
Slide 64 text
64
Slide 65
Slide 65 text
65 Проводите архитектурные review
Slide 66
Slide 66 text
66 0 50 100 150 200 0 500 1000 1500 2000
Slide 67
Slide 67 text
строки кода 67 0 50 100 150 200 0 500 1000 1500 2000 комментарии чем больше изменений, тем меньше комментариев
Slide 68
Slide 68 text
68 Отправляйте дневную норму кода
Slide 69
Slide 69 text
69
Slide 70
Slide 70 text
70 Проводите self assessment
Slide 71
Slide 71 text
71
Slide 72
Slide 72 text
72 Подробно описывайте, что сделали
Slide 73
Slide 73 text
73
Slide 74
Slide 74 text
74 Автоматизируйте чек-лист
Slide 75
Slide 75 text
75 SwiftLint OCLint Infer Clang Analyzer CPD
Slide 76
Slide 76 text
76
Slide 77
Slide 77 text
77 Автоматизация чек-листа для Code Review
Slide 78
Slide 78 text
78 Стиль коммитов Наполнение коммитов Правописание в коммитах Линтеры Артефакты
Slide 79
Slide 79 text
79
Slide 80
Slide 80 text
80
Slide 81
Slide 81 text
81 Кто делает review? Когда делать review? Как делать review? Эффективно ли review?
Slide 82
Slide 82 text
82 Количество review Просмотренные строки кода Найденные ошибки
Slide 83
Slide 83 text
83 Количество review Просмотренные строки кода Найденные ошибки
Slide 84
Slide 84 text
84 Количество review Просмотренные строки кода Найденные ошибки
Slide 85
Slide 85 text
85 Количество review Просмотренные строки кода Найденные ошибки
Slide 86
Slide 86 text
86 Количество review Просмотренные строки кода Найденные ошибки
Slide 87
Slide 87 text
87 Количество review Просмотренные строки кода Найденные ошибки
Slide 88
Slide 88 text
88 #1: Коллективное владение кодом #2: Переиспользование кода #3: Обмен знаниями #4: Обнаружение ошибок #5: Единообразие проекта
Slide 89
Slide 89 text
89 Goal-Question-Metric 1. Определите цель 2. Сформулируйте вопросы 3. Подберите метрики
Slide 90
Slide 90 text
90 Цель «Повысить уровень переиспользования кода»
Slide 91
Slide 91 text
91 Вопросы Используются ли общие библиотеки? Вынесены ли визуальные стили? Какой уровень дублирования кода?
Slide 92
Slide 92 text
92 Метрики Количество общих библиотек Количество захардкоженных стилей Уровень дублирования кода
Slide 93
Slide 93 text
93 Выполнено 60% плана
Slide 94
Slide 94 text
94 Inspection rate = LoC / hour Defect rate = problems / hour Defect density = problems / LoC
Slide 95
Slide 95 text
95
Slide 96
Slide 96 text
96 210 LoC
Slide 97
Slide 97 text
97
Slide 98
Slide 98 text
98
Slide 99
Slide 99 text
99 Inspection rate = 2.1 / 0.56 = 3.75 Defect rate = 4 / 0.56 = 7.1 Defect density = 4 / 2.1 = 1.9
Slide 100
Slide 100 text
100 Inspection rate = 2.1 / 0.08 = 26.25 Defect rate = 1 / 0.08 = 12.5 Defect density = 1 / 2.1 = 0.48
Slide 101
Slide 101 text
101 Inspection rate 3.75 vs 26.25 Defect rate 7.1 vs 12.5 Defect density 1.9 vs 0.48
Slide 102
Slide 102 text
102 Состав участников Расписание участников Чек-лист Метрики эффективности
Slide 103
Slide 103 text
103 Цели Практики Софт Этика Code Review
Slide 104
Slide 104 text
104 Upsource / JetBrains Crucible / Atlassian GitHub
Slide 105
Slide 105 text
105
Slide 106
Slide 106 text
106
Slide 107
Slide 107 text
107 Работа в IDE Интеграции с кучей сервисов Мощная статистика Code Ownership Гибкие уведомления
Slide 108
Slide 108 text
108
Slide 109
Slide 109 text
109
Slide 110
Slide 110 text
110 Pre-commit review Review сниппетов Считает время на review Тесная интеграция с Atlassian Гибкие теги
Slide 111
Slide 111 text
111
Slide 112
Slide 112 text
112
Slide 113
Slide 113 text
113 Можно ставить смайлики Крутой Code Ownership Шаблоны для PR Интеграции с инструментами Доступен из коробки
Slide 114
Slide 114 text
114
Slide 115
Slide 115 text
115 Цели Практики Софт Этика Code Review
Slide 116
Slide 116 text
116
Slide 117
Slide 117 text
117 Терминология действительно важна
Slide 118
Slide 118 text
Профессионализм 118 Внимательность Адекватность Доброжелательность Отзывчивость Открытость Устойчивость к критике Коммуникабельность Ответственность
Slide 119
Slide 119 text
119
Slide 120
Slide 120 text
120 Цели Практики Софт Этика Code Review
Slide 121
Slide 121 text
121
Slide 122
Slide 122 text
122
Slide 123
Slide 123 text
Code Review нужен вам и вашей команде telegram.me/iosgr