Код-ревью

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

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

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

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

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

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

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

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

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

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

При чем тут Drupal?

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

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

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

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