Презентация на тему: Код-ревью

Код-ревью На страже ваших багов.

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

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

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

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

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

Внедрить в процедуру проверки каждой задачи.

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

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

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

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

При чем тут Drupal?

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

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

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

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

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