Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.

Код-ревью в Drupal

1,563 views

Published on

Published in: Technology
  • Be the first to comment

  • Be the first to like this

Код-ревью в Drupal

  1. 1. Код-ревью<br />На страже ваших багов.<br />© Александр Швец<br />
  2. 2. Что это?<br />«Проверка кода с целью обнаружения и исправления ошибок, которые остались незамеченными в начальной фазе разработки»<br />
  3. 3. Зачем?<br />Повышаем качество кода<br />Учимся писать код у других<br />Узнаем новое о коде проекта<br />
  4. 4. Почему не делаем?<br />
  5. 5. Да как-то… ссыкотно<br />Все увидят баги в моем коде / Мой код идеален!<br />Нету времени.<br />Клиент будет недоволен, что мы тратим бюджет на проверку качества кода, который и так должен быть идеален.<br />
  6. 6. Где брать время?<br />
  7. 7. Внедрить в процедуру проверки каждой задачи.<br />QA<br />Код-ревью<br />Код<br />QA<br />Код-ревью<br />Релиз<br />QA<br />QA<br />
  8. 8. Давать эстимейты, учитывая 10% временина ревью.<br />Эстимейт<br />
  9. 9. В результате, время окупается за счет качества.<br />
  10. 10. За чьи деньги?<br />Повышаем качество кода.<br />Повышаем вовлеченность разработчиков.<br />Повышаем эффективность разработки и рейт.<br />Profit!!!<br />
  11. 11. Кто должен делать?<br />Тех. лид/ менеджер<br /> Обычно, опытнее и авторитетнее всех.<br />2-3 других разработчиков<br /> Две неопытные головы лучше одной неопытной.<br />
  12. 12. При чем тут Drupal?<br />
  13. 13. Приоритеты ревью<br />Устранение уязвимостей<br />Нахождение багов<br />Исправление форматирования<br />
  14. 14. Что просматриваем?<br />1. Все PHP-файлы (.module, .inc, .install, .test)<br />Уязвимости.Баги. Форматирование.<br />2. Все JS-файлы<br />Баги.Не использование Drupal JS и JQuery. Форматирование.<br />3. Все CSS-файлы<br />Ошибки форматирования.<br />
  15. 15. Рецепты код-ревью<br />shvetsgroup.com/ru/blog/code<br />
  16. 16. Спасибо за внимание!<br />
  17. 17. Александр Швец<br />ShvetsGroup,Директор<br /> Почта: 1@shvetsgroup.com<br />Телефон:+38 (097) 6833155<br />Skype:neo.chief<br />

×