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

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

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

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

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

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

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

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

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

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

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

При чем тут Drupal?

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

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

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

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

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