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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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