SlideShare ist ein Scribd-Unternehmen logo
1 von 17
Downloaden Sie, um offline zu lesen
Код-ревью,[object Object],На страже ваших багов.,[object Object],© Александр Швец,[object Object]
Что это?,[object Object],«Проверка кода с целью обнаружения и исправления ошибок, которые остались незамеченными в начальной фазе разработки»,[object Object]
Зачем?,[object Object],Повышаем качество кода,[object Object],Учимся писать код у других,[object Object],Узнаем новое о коде проекта,[object Object]
Почему не делаем?,[object Object]
Да как-то… ссыкотно,[object Object],Все увидят баги в моем коде / Мой код идеален!,[object Object],Нету времени.,[object Object],Клиент будет недоволен, что мы тратим бюджет на проверку качества кода, который и так должен быть идеален.,[object Object]
Где брать время?,[object Object]
Внедрить в процедуру проверки каждой задачи.,[object Object],QA,[object Object],Код-ревью,[object Object],Код,[object Object],QA,[object Object],Код-ревью,[object Object],Релиз,[object Object],QA,[object Object],QA,[object Object]
Давать эстимейты, учитывая 10% временина ревью.,[object Object],Эстимейт,[object Object]
В результате, время окупается за счет качества.,[object Object]
За чьи деньги?,[object Object],Повышаем качество кода.,[object Object],Повышаем вовлеченность разработчиков.,[object Object],Повышаем эффективность разработки и рейт.,[object Object],Profit!!!,[object Object]
Кто должен делать?,[object Object],Тех. лид/ менеджер,[object Object],       Обычно, опытнее и авторитетнее всех.,[object Object],2-3 других разработчиков,[object Object],       Две неопытные головы лучше одной неопытной.,[object Object]
При чем тут Drupal?,[object Object]
Приоритеты ревью,[object Object],Устранение уязвимостей,[object Object],Нахождение багов,[object Object],Исправление форматирования,[object Object]
Что просматриваем?,[object Object],1. Все PHP-файлы (.module, .inc, .install, .test),[object Object],Уязвимости.Баги. Форматирование.,[object Object],2. Все JS-файлы,[object Object],Баги.Не использование Drupal JS и JQuery. Форматирование.,[object Object],3. Все CSS-файлы,[object Object],Ошибки форматирования.,[object Object]
Рецепты код-ревью,[object Object],shvetsgroup.com/ru/blog/code,[object Object]
Спасибо за внимание!,[object Object]
Александр Швец,[object Object],ShvetsGroup,Директор,[object Object],     Почта:  1@shvetsgroup.com,[object Object],Телефон:+38 (097) 6833155,[object Object],Skype:neo.chief,[object Object]

Weitere ähnliche Inhalte

Was ist angesagt?

Management of projects
Management of projectsManagement of projects
Management of projectsMageCloud
 
KPI тестировщиков - особенности и проблемы
KPI тестировщиков - особенности и проблемыKPI тестировщиков - особенности и проблемы
KPI тестировщиков - особенности и проблемыRina Uzhevko
 
Code review как средство обеспечения качества программного обеспечения
Code review как средство обеспечения качества программного обеспеченияCode review как средство обеспечения качества программного обеспечения
Code review как средство обеспечения качества программного обеспеченияSQALab
 
"Dealing with legacy code"
"Dealing with legacy code""Dealing with legacy code"
"Dealing with legacy code"HYS Enterprise
 
рекомендации для начала карьерного роста
рекомендации для начала карьерного ростарекомендации для начала карьерного роста
рекомендации для начала карьерного ростаЕвгений Коробков
 
Гибкие методики разработки. Кто их придумал и почему
Гибкие методики разработки. Кто их придумал и почемуГибкие методики разработки. Кто их придумал и почему
Гибкие методики разработки. Кто их придумал и почемуDmitry Savchenko
 
Особенности внедрения KPI или как доказать, что Ваш «зеленый» проект реально ...
Особенности внедрения KPI или как доказать, что Ваш «зеленый» проект реально ...Особенности внедрения KPI или как доказать, что Ваш «зеленый» проект реально ...
Особенности внедрения KPI или как доказать, что Ваш «зеленый» проект реально ...CEE-SEC(R)
 
Crucible или почему для Code Review нужна не только голова, но и инструмент
Crucible или почему для Code Review нужна не только голова, но и инструментCrucible или почему для Code Review нужна не только голова, но и инструмент
Crucible или почему для Code Review нужна не только голова, но и инструментMaxim Kuzmich
 
Длинный путь к DevOps?
Длинный путь к DevOps?Длинный путь к DevOps?
Длинный путь к DevOps?CEE-SEC(R)
 
Лайфхаки при аутсорсе - Эдуард Христусь, Func
Лайфхаки при аутсорсе - Эдуард Христусь, FuncЛайфхаки при аутсорсе - Эдуард Христусь, Func
Лайфхаки при аутсорсе - Эдуард Христусь, FunciDealMachine
 
Exploratory testing
Exploratory testingExploratory testing
Exploratory testingGleb Rybalko
 
Олег Бунин (Онтико) | Менеджмент и бизнес-процессы в разработке highload-прое...
Олег Бунин (Онтико) | Менеджмент и бизнес-процессы в разработке highload-прое...Олег Бунин (Онтико) | Менеджмент и бизнес-процессы в разработке highload-прое...
Олег Бунин (Онтико) | Менеджмент и бизнес-процессы в разработке highload-прое...RIF-Technology
 
евгения фирсова нерелизное тестирование
евгения фирсова   нерелизное тестированиеевгения фирсова   нерелизное тестирование
евгения фирсова нерелизное тестированиеAlexei Lupan
 
Гибкое тестирование
Гибкое тестированиеГибкое тестирование
Гибкое тестированиеRina Uzhevko
 
QA Fes 2016. Анастасия Асеева. Роль тестирования в Devops
QA Fes 2016. Анастасия Асеева. Роль тестирования в DevopsQA Fes 2016. Анастасия Асеева. Роль тестирования в Devops
QA Fes 2016. Анастасия Асеева. Роль тестирования в DevopsQAFest
 
Оценка проектов тестирования
Оценка проектов тестированияОценка проектов тестирования
Оценка проектов тестированияRina Uzhevko
 
Сергей Смирнов (Altair Engineering Inc.) | Организация работы распределенной ...
Сергей Смирнов (Altair Engineering Inc.) | Организация работы распределенной ...Сергей Смирнов (Altair Engineering Inc.) | Организация работы распределенной ...
Сергей Смирнов (Altair Engineering Inc.) | Организация работы распределенной ...RIF-Technology
 
JS Lab2017_Алексей Зеленюк_Сбалансированное окружение для вашей продуктивности
JS Lab2017_Алексей Зеленюк_Сбалансированное окружение для вашей продуктивностиJS Lab2017_Алексей Зеленюк_Сбалансированное окружение для вашей продуктивности
JS Lab2017_Алексей Зеленюк_Сбалансированное окружение для вашей продуктивностиGeeksLab Odessa
 

Was ist angesagt? (20)

Agile
AgileAgile
Agile
 
Management of projects
Management of projectsManagement of projects
Management of projects
 
KPI тестировщиков - особенности и проблемы
KPI тестировщиков - особенности и проблемыKPI тестировщиков - особенности и проблемы
KPI тестировщиков - особенности и проблемы
 
Code review как средство обеспечения качества программного обеспечения
Code review как средство обеспечения качества программного обеспеченияCode review как средство обеспечения качества программного обеспечения
Code review как средство обеспечения качества программного обеспечения
 
"Dealing with legacy code"
"Dealing with legacy code""Dealing with legacy code"
"Dealing with legacy code"
 
рекомендации для начала карьерного роста
рекомендации для начала карьерного ростарекомендации для начала карьерного роста
рекомендации для начала карьерного роста
 
Гибкие методики разработки. Кто их придумал и почему
Гибкие методики разработки. Кто их придумал и почемуГибкие методики разработки. Кто их придумал и почему
Гибкие методики разработки. Кто их придумал и почему
 
Особенности внедрения KPI или как доказать, что Ваш «зеленый» проект реально ...
Особенности внедрения KPI или как доказать, что Ваш «зеленый» проект реально ...Особенности внедрения KPI или как доказать, что Ваш «зеленый» проект реально ...
Особенности внедрения KPI или как доказать, что Ваш «зеленый» проект реально ...
 
Crucible или почему для Code Review нужна не только голова, но и инструмент
Crucible или почему для Code Review нужна не только голова, но и инструментCrucible или почему для Code Review нужна не только голова, но и инструмент
Crucible или почему для Code Review нужна не только голова, но и инструмент
 
Длинный путь к DevOps?
Длинный путь к DevOps?Длинный путь к DevOps?
Длинный путь к DevOps?
 
Лайфхаки при аутсорсе - Эдуард Христусь, Func
Лайфхаки при аутсорсе - Эдуард Христусь, FuncЛайфхаки при аутсорсе - Эдуард Христусь, Func
Лайфхаки при аутсорсе - Эдуард Христусь, Func
 
Exploratory testing
Exploratory testingExploratory testing
Exploratory testing
 
Олег Бунин (Онтико) | Менеджмент и бизнес-процессы в разработке highload-прое...
Олег Бунин (Онтико) | Менеджмент и бизнес-процессы в разработке highload-прое...Олег Бунин (Онтико) | Менеджмент и бизнес-процессы в разработке highload-прое...
Олег Бунин (Онтико) | Менеджмент и бизнес-процессы в разработке highload-прое...
 
евгения фирсова нерелизное тестирование
евгения фирсова   нерелизное тестированиеевгения фирсова   нерелизное тестирование
евгения фирсова нерелизное тестирование
 
Гибкое тестирование
Гибкое тестированиеГибкое тестирование
Гибкое тестирование
 
QA Fes 2016. Анастасия Асеева. Роль тестирования в Devops
QA Fes 2016. Анастасия Асеева. Роль тестирования в DevopsQA Fes 2016. Анастасия Асеева. Роль тестирования в Devops
QA Fes 2016. Анастасия Асеева. Роль тестирования в Devops
 
What Tests Are For?
What Tests Are For?What Tests Are For?
What Tests Are For?
 
Оценка проектов тестирования
Оценка проектов тестированияОценка проектов тестирования
Оценка проектов тестирования
 
Сергей Смирнов (Altair Engineering Inc.) | Организация работы распределенной ...
Сергей Смирнов (Altair Engineering Inc.) | Организация работы распределенной ...Сергей Смирнов (Altair Engineering Inc.) | Организация работы распределенной ...
Сергей Смирнов (Altair Engineering Inc.) | Организация работы распределенной ...
 
JS Lab2017_Алексей Зеленюк_Сбалансированное окружение для вашей продуктивности
JS Lab2017_Алексей Зеленюк_Сбалансированное окружение для вашей продуктивностиJS Lab2017_Алексей Зеленюк_Сбалансированное окружение для вашей продуктивности
JS Lab2017_Алексей Зеленюк_Сбалансированное окружение для вашей продуктивности
 

Andere mochten auch

Тренды Drupal
Тренды DrupalТренды Drupal
Тренды DrupalShvetsGroup
 
Embedded Librarians: Diverse Initiatives, Common Challenges.
Embedded Librarians: Diverse Initiatives, Common Challenges. Embedded Librarians: Diverse Initiatives, Common Challenges.
Embedded Librarians: Diverse Initiatives, Common Challenges. davidshumaker
 
Collaborate! (Is There Any Other Way?)
Collaborate! (Is There Any Other Way?)Collaborate! (Is There Any Other Way?)
Collaborate! (Is There Any Other Way?)davidshumaker
 
Drupal дистрибутивы
Drupal дистрибутивыDrupal дистрибутивы
Drupal дистрибутивыShvetsGroup
 
The New Net-Centric Librarian
The New Net-Centric LibrarianThe New Net-Centric Librarian
The New Net-Centric Librariandavidshumaker
 

Andere mochten auch (9)

Estonia
EstoniaEstonia
Estonia
 
makalah
makalahmakalah
makalah
 
Тренды Drupal
Тренды DrupalТренды Drupal
Тренды Drupal
 
Embedded Librarians: Diverse Initiatives, Common Challenges.
Embedded Librarians: Diverse Initiatives, Common Challenges. Embedded Librarians: Diverse Initiatives, Common Challenges.
Embedded Librarians: Diverse Initiatives, Common Challenges.
 
Collaborate! (Is There Any Other Way?)
Collaborate! (Is There Any Other Way?)Collaborate! (Is There Any Other Way?)
Collaborate! (Is There Any Other Way?)
 
Drupal дистрибутивы
Drupal дистрибутивыDrupal дистрибутивы
Drupal дистрибутивы
 
The New Net-Centric Librarian
The New Net-Centric LibrarianThe New Net-Centric Librarian
The New Net-Centric Librarian
 
makalah
makalahmakalah
makalah
 
makalah
makalahmakalah
makalah
 

Ähnlich wie Код-ревью в Drupal

QA Fest 2016. Алексей Виноградов. Цель тестирования. А на самом деле?
QA Fest 2016. Алексей Виноградов. Цель тестирования. А на самом деле?QA Fest 2016. Алексей Виноградов. Цель тестирования. А на самом деле?
QA Fest 2016. Алексей Виноградов. Цель тестирования. А на самом деле?QAFest
 
Юлия Викторова; Александр Тарасов. DevOps без булшита.
Юлия Викторова; Александр Тарасов. DevOps без булшита.Юлия Викторова; Александр Тарасов. DevOps без булшита.
Юлия Викторова; Александр Тарасов. DevOps без булшита.ScrumTrek
 
"Этот код плохой, его нужно переписать". Слышали? Как обосновать
"Этот код плохой, его нужно переписать". Слышали? Как обосновать"Этот код плохой, его нужно переписать". Слышали? Как обосновать
"Этот код плохой, его нужно переписать". Слышали? Как обосноватьMaksym Bezuglyi
 
Олег Бунин, 20 рисков, которые необходимо учесть при расчёте сложного проекта
Олег Бунин, 20 рисков, которые необходимо учесть при расчёте сложного проектаОлег Бунин, 20 рисков, которые необходимо учесть при расчёте сложного проекта
Олег Бунин, 20 рисков, которые необходимо учесть при расчёте сложного проектаScrumTrek
 
Эволюция внутренних процессов в большой команде
Эволюция внутренних процессов в большой командеЭволюция внутренних процессов в большой команде
Эволюция внутренних процессов в большой командеAgilie Team
 
Виталий Шибаев - Креативный менеджмент глазами разработчика: как выжить в agi...
Виталий Шибаев - Креативный менеджмент глазами разработчика: как выжить в agi...Виталий Шибаев - Креативный менеджмент глазами разработчика: как выжить в agi...
Виталий Шибаев - Креативный менеджмент глазами разработчика: как выжить в agi...HappyDev
 
Повышаем и следим за качеством PHP кода
Повышаем и следим за качеством PHP кодаПовышаем и следим за качеством PHP кода
Повышаем и следим за качеством PHP кодаAleksandr Makhomet
 
Технологический цикл и соблюдение фаз производства.
Технологический цикл и соблюдение фаз производства.Технологический цикл и соблюдение фаз производства.
Технологический цикл и соблюдение фаз производства.Сергей Сторожев
 
Software craftsmanship 8
Software craftsmanship 8Software craftsmanship 8
Software craftsmanship 8Pavel Veinik
 
Алексей Лустин. Непрерывная проверка качества кода.
Алексей Лустин. Непрерывная проверка качества кода.Алексей Лустин. Непрерывная проверка качества кода.
Алексей Лустин. Непрерывная проверка качества кода.ScrumTrek
 
ДЗ №2
ДЗ №2ДЗ №2
ДЗ №2aimus
 
Повышение качества тестов и автоматическая валидация REST API документации
Повышение качества тестов и автоматическая валидация REST API документацииПовышение качества тестов и автоматическая валидация REST API документации
Повышение качества тестов и автоматическая валидация REST API документацииCEE-SEC(R)
 
Профессии в IT
Профессии в ITПрофессии в IT
Профессии в ITSam Faktorovich
 
Гибкие методологии разработки: максимальный результат для бизнеса с минимальн...
Гибкие методологии разработки: максимальный результат для бизнеса с минимальн...Гибкие методологии разработки: максимальный результат для бизнеса с минимальн...
Гибкие методологии разработки: максимальный результат для бизнеса с минимальн...Alexey Tigarev
 
Tech Talks @NSU: Рассказ о разных профессиях в IT-индустрии, или почему не вс...
Tech Talks @NSU: Рассказ о разных профессиях в IT-индустрии, или почему не вс...Tech Talks @NSU: Рассказ о разных профессиях в IT-индустрии, или почему не вс...
Tech Talks @NSU: Рассказ о разных профессиях в IT-индустрии, или почему не вс...Tech Talks @NSU
 
В.Денисенков - Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
В.Денисенков - Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...В.Денисенков - Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
В.Денисенков - Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...borovoystudio
 
Developmentmanage3.0
Developmentmanage3.0Developmentmanage3.0
Developmentmanage3.0WRider
 
В.Денисенков Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
В.Денисенков   Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...В.Денисенков   Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
В.Денисенков Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...borovoystudio
 

Ähnlich wie Код-ревью в Drupal (20)

QA Fest 2016. Алексей Виноградов. Цель тестирования. А на самом деле?
QA Fest 2016. Алексей Виноградов. Цель тестирования. А на самом деле?QA Fest 2016. Алексей Виноградов. Цель тестирования. А на самом деле?
QA Fest 2016. Алексей Виноградов. Цель тестирования. А на самом деле?
 
Юлия Викторова; Александр Тарасов. DevOps без булшита.
Юлия Викторова; Александр Тарасов. DevOps без булшита.Юлия Викторова; Александр Тарасов. DevOps без булшита.
Юлия Викторова; Александр Тарасов. DevOps без булшита.
 
"Этот код плохой, его нужно переписать". Слышали? Как обосновать
"Этот код плохой, его нужно переписать". Слышали? Как обосновать"Этот код плохой, его нужно переписать". Слышали? Как обосновать
"Этот код плохой, его нужно переписать". Слышали? Как обосновать
 
Олег Бунин, 20 рисков, которые необходимо учесть при расчёте сложного проекта
Олег Бунин, 20 рисков, которые необходимо учесть при расчёте сложного проектаОлег Бунин, 20 рисков, которые необходимо учесть при расчёте сложного проекта
Олег Бунин, 20 рисков, которые необходимо учесть при расчёте сложного проекта
 
Эволюция внутренних процессов в большой команде
Эволюция внутренних процессов в большой командеЭволюция внутренних процессов в большой команде
Эволюция внутренних процессов в большой команде
 
Виталий Шибаев - Креативный менеджмент глазами разработчика: как выжить в agi...
Виталий Шибаев - Креативный менеджмент глазами разработчика: как выжить в agi...Виталий Шибаев - Креативный менеджмент глазами разработчика: как выжить в agi...
Виталий Шибаев - Креативный менеджмент глазами разработчика: как выжить в agi...
 
Повышаем и следим за качеством PHP кода
Повышаем и следим за качеством PHP кодаПовышаем и следим за качеством PHP кода
Повышаем и следим за качеством PHP кода
 
Quality Assurance
Quality AssuranceQuality Assurance
Quality Assurance
 
Технологический цикл и соблюдение фаз производства.
Технологический цикл и соблюдение фаз производства.Технологический цикл и соблюдение фаз производства.
Технологический цикл и соблюдение фаз производства.
 
Software craftsmanship 8
Software craftsmanship 8Software craftsmanship 8
Software craftsmanship 8
 
Code review psyhology
Code review psyhologyCode review psyhology
Code review psyhology
 
Алексей Лустин. Непрерывная проверка качества кода.
Алексей Лустин. Непрерывная проверка качества кода.Алексей Лустин. Непрерывная проверка качества кода.
Алексей Лустин. Непрерывная проверка качества кода.
 
ДЗ №2
ДЗ №2ДЗ №2
ДЗ №2
 
Повышение качества тестов и автоматическая валидация REST API документации
Повышение качества тестов и автоматическая валидация REST API документацииПовышение качества тестов и автоматическая валидация REST API документации
Повышение качества тестов и автоматическая валидация REST API документации
 
Профессии в IT
Профессии в ITПрофессии в IT
Профессии в IT
 
Гибкие методологии разработки: максимальный результат для бизнеса с минимальн...
Гибкие методологии разработки: максимальный результат для бизнеса с минимальн...Гибкие методологии разработки: максимальный результат для бизнеса с минимальн...
Гибкие методологии разработки: максимальный результат для бизнеса с минимальн...
 
Tech Talks @NSU: Рассказ о разных профессиях в IT-индустрии, или почему не вс...
Tech Talks @NSU: Рассказ о разных профессиях в IT-индустрии, или почему не вс...Tech Talks @NSU: Рассказ о разных профессиях в IT-индустрии, или почему не вс...
Tech Talks @NSU: Рассказ о разных профессиях в IT-индустрии, или почему не вс...
 
В.Денисенков - Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
В.Денисенков - Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...В.Денисенков - Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
В.Денисенков - Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
 
Developmentmanage3.0
Developmentmanage3.0Developmentmanage3.0
Developmentmanage3.0
 
В.Денисенков Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
В.Денисенков   Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...В.Денисенков   Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
В.Денисенков Семь раз отмерь. Все что надо знать о выборе подрядчиков, прог...
 

Код-ревью в Drupal

  • 1.
  • 2.
  • 3.
  • 4.
  • 5.
  • 6.
  • 7.
  • 8.
  • 9.
  • 10.
  • 11.
  • 12.
  • 13.
  • 14.
  • 15.
  • 16.
  • 17.