Код-ревью На страже ваших багов. © Александр Швец. - презентация

1 Код-ревью На страже ваших багов. © Александр Швец

2 Что это? «Проверка кода с целью обнаружения и исправления ошибок, которые остались незамеченными в начальной фазе разработки»

3 Зачем? Повышаем качество кода Учимся писать код у других Узнаем новое о коде проекта

4 Почему не делаем?

5 Да как-то… ссыкотно 1.Все увидят баги в моем коде / Мой код идеален! 2.Нету времени. 3.Клиент будет недоволен, что мы тратим бюджет на проверку качества кода, который и так должен быть идеален.

6 Где брать время?

7 Код QA Код- ревью Релиз QA Код- ревью Внедрить в процедуру проверки каждой задачи.

8 Давать эстимейты, учитывая 10% времени на ревью. Эстимейт

9 В результате, время окупается за счет качества.

10 За чьи деньги? 1.Повышаем качество кода. 2.Повышаем вовлеченность разработчиков. 3.Повышаем эффективность разработки и рейт. 4.Profit!!!

11 Кто должен делать? Тех. лид / менеджер Обычно, опытнее и авторитетнее всех. 2-3 других разработчиков Две неопытные головы лучше одной неопытной.

12 При чем тут Drupal?

13 Приоритеты ревью 1.Устранение уязвимостей 2.Нахождение багов 3.Исправление форматирования

14 Что просматриваем? 1. Все PHP-файлы (.module,.inc,.install,.test) Уязвимости. Баги. Форматирование. 2. Все JS-файлы Баги. Не использование Drupal JS и JQuery. Форматирование. 3. Все CSS-файлы Ошибки форматирования.

15 Рецепты код-ревью shvetsgroup.com/ru/blog/code

16 Спасибо за внимание!

17 Александр Швец ShvetsGroup, Директор Почта: Телефон: +38 (097) Skype: neo.chief