SlideShare a Scribd company logo
1 of 20
Code Review
«Пишите код так, как будто
сопровождать его будет
склонный к насилию психопат,
который знает, где вы живете»
Resolve issues
●

Виявлення помилок

●

Виявлення недоліків

●

●

●

Відповідність стилю написання (Style Guide,
Best Practice)
Відповідність кода і поставленного
завдання
Обмін знаннями між участниками
Reviews
●

Pre-reviews

●

Post-fact

●

Random

●

Unplanning (WTF?)
Process
Основні аспекти
●

Основні аспекти

●

Хто має проводити інспекцію

●

Що будемо інспектувати

●

Як будемо проводити
Code quality, what is «bad»
Code quality, what is «Good»
Fight!
Instruments
●

Gerrit (GIT)

●

Git-flow

●

Rietveld (SVN)

●

Redmine plugins
Gerrit
GIT-flow
GIT-Flow Subteams
Що отримуємо
●

Посилення відповідності

●

Гарантії якості

●

Підвищення ясності коду

●

Навчання рецензентів

●

Вирівнювання пріорітетів

●

Навчання программістів
Profit ???
Average bug founding rating
●

30% with Unit Testing

●

35% with Unit+Integration Testing

●

35% with limited Beta testing

●

55% Project inspection

●

65% Code Inspection
1 hour of Inspection reduce 100 hour of
manual testing and defect repair © IBM
Гербалайф
●

●

●

●

●

●

●

Hewlett-Packard — экономія 21.5 млн. в рік
Затрати на виправлення дефектів з 40% від
вартості проекту до 20%.
1 година інспекції заміняє ~33 години
підтримки ПО
Программісти в 95% виконують зміни
правильно (до введення інспекції в 20%)
В 20 раз эфективніше black-box testing
На 30% швидше навчання Junior і мотивація
Senior для передачі знань
Soft & Books
●

●

●

GitFlow http://habrahabr.ru/post/106912/,
http://habrahabr.ru/post/147260/
Rewiev Board http://habrahabr.ru/post/187058/
Gerrit (GIT)
http://habrahabr.ru/company/badoo/blog/147508/

●

Rietveld (for SVN) (Google)

●

Stive McConnell — Code complete

●

Extreme Programming, XP (Kent Beck)
Де знайти книги? Google, ссилки у wiki, у
Макса. :)
Дякую за увагу

More Related Content

Viewers also liked

OMELETTE by Souhaila Lehhit
OMELETTE by Souhaila LehhitOMELETTE by Souhaila Lehhit
OMELETTE by Souhaila Lehhitesthermarc
 
StarUML NS Guide - Design
StarUML NS Guide - DesignStarUML NS Guide - Design
StarUML NS Guide - Design태욱 양
 
StarUML NS Guide - Introduction
StarUML NS Guide -  IntroductionStarUML NS Guide -  Introduction
StarUML NS Guide - Introduction태욱 양
 

Viewers also liked (7)

PolipropilenoProject
PolipropilenoProjectPolipropilenoProject
PolipropilenoProject
 
Food & drinks in Rome 2014
Food & drinks in Rome 2014Food & drinks in Rome 2014
Food & drinks in Rome 2014
 
OMELETTE by Souhaila Lehhit
OMELETTE by Souhaila LehhitOMELETTE by Souhaila Lehhit
OMELETTE by Souhaila Lehhit
 
StarUML NS Guide - Design
StarUML NS Guide - DesignStarUML NS Guide - Design
StarUML NS Guide - Design
 
Adaptaciones
AdaptacionesAdaptaciones
Adaptaciones
 
StarUML NS Guide - Introduction
StarUML NS Guide -  IntroductionStarUML NS Guide -  Introduction
StarUML NS Guide - Introduction
 
Pelajaran 7 8 mandarin
Pelajaran 7 8 mandarinPelajaran 7 8 mandarin
Pelajaran 7 8 mandarin
 

Similar to Code Review

РОМАН МАРІНСЬКИЙ «Організація та покращення QA Center of Excellence» QADay
РОМАН МАРІНСЬКИЙ «Організація та покращення QA Center of Excellence» QADayРОМАН МАРІНСЬКИЙ «Організація та покращення QA Center of Excellence» QADay
РОМАН МАРІНСЬКИЙ «Організація та покращення QA Center of Excellence» QADayGoQA
 
Тестування ПЗ
Тестування ПЗТестування ПЗ
Тестування ПЗKyrylo Bezpalyi
 
How to Leverage your Skill Set for Product by Matic PM
How to Leverage your Skill Set for Product by Matic PMHow to Leverage your Skill Set for Product by Matic PM
How to Leverage your Skill Set for Product by Matic PMProduct School
 
Як прокачати трьох студентів за п’ять тижнів
Як прокачати трьох студентів за п’ять тижнівЯк прокачати трьох студентів за п’ять тижнів
Як прокачати трьох студентів за п’ять тижнівStfalcon Meetups
 
Як прокачати трьох студентів за п’ять тижнів
Як прокачати трьох студентів за п’ять тижнівЯк прокачати трьох студентів за п’ять тижнів
Як прокачати трьох студентів за п’ять тижнівArtem Henvald
 
Іван Веркалець "Як аутсорсинговій компанії ефективно працювати із стартапами"...
Іван Веркалець "Як аутсорсинговій компанії ефективно працювати із стартапами"...Іван Веркалець "Як аутсорсинговій компанії ефективно працювати із стартапами"...
Іван Веркалець "Як аутсорсинговій компанії ефективно працювати із стартапами"...Lviv Startup Club
 
Lviv Freelance Forum Іван Веркалець: “Як аутсорсинговій компанії ефективно пр...
Lviv Freelance Forum Іван Веркалець: “Як аутсорсинговій компанії ефективно пр...Lviv Freelance Forum Іван Веркалець: “Як аутсорсинговій компанії ефективно пр...
Lviv Freelance Forum Іван Веркалець: “Як аутсорсинговій компанії ефективно пр...Lviv Startup Club
 

Similar to Code Review (9)

РОМАН МАРІНСЬКИЙ «Організація та покращення QA Center of Excellence» QADay
РОМАН МАРІНСЬКИЙ «Організація та покращення QA Center of Excellence» QADayРОМАН МАРІНСЬКИЙ «Організація та покращення QA Center of Excellence» QADay
РОМАН МАРІНСЬКИЙ «Організація та покращення QA Center of Excellence» QADay
 
Sergey Dmytryshyn, Crowdin
Sergey Dmytryshyn, CrowdinSergey Dmytryshyn, Crowdin
Sergey Dmytryshyn, Crowdin
 
Тестування ПЗ
Тестування ПЗТестування ПЗ
Тестування ПЗ
 
Crowd in
Crowd inCrowd in
Crowd in
 
How to Leverage your Skill Set for Product by Matic PM
How to Leverage your Skill Set for Product by Matic PMHow to Leverage your Skill Set for Product by Matic PM
How to Leverage your Skill Set for Product by Matic PM
 
Як прокачати трьох студентів за п’ять тижнів
Як прокачати трьох студентів за п’ять тижнівЯк прокачати трьох студентів за п’ять тижнів
Як прокачати трьох студентів за п’ять тижнів
 
Як прокачати трьох студентів за п’ять тижнів
Як прокачати трьох студентів за п’ять тижнівЯк прокачати трьох студентів за п’ять тижнів
Як прокачати трьох студентів за п’ять тижнів
 
Іван Веркалець "Як аутсорсинговій компанії ефективно працювати із стартапами"...
Іван Веркалець "Як аутсорсинговій компанії ефективно працювати із стартапами"...Іван Веркалець "Як аутсорсинговій компанії ефективно працювати із стартапами"...
Іван Веркалець "Як аутсорсинговій компанії ефективно працювати із стартапами"...
 
Lviv Freelance Forum Іван Веркалець: “Як аутсорсинговій компанії ефективно пр...
Lviv Freelance Forum Іван Веркалець: “Як аутсорсинговій компанії ефективно пр...Lviv Freelance Forum Іван Веркалець: “Як аутсорсинговій компанії ефективно пр...
Lviv Freelance Forum Іван Веркалець: “Як аутсорсинговій компанії ефективно пр...
 

Code Review