Код-ревью в цифрах

Уровень: ★★★☆☆ (опытные инженеры)

Что делает хороший процесс код-ревью хорошим? Давайте спросим об этом самих разработчиков.

Хорошее исследование по поводу практик код-ревью — https://www.codacy.com/ebooks/guide-to-code-reviews.

Ребята опросили 682 разработчика. Сначала просили оценить качество кода в проекте (по шкале от 1 до 5) и распределение времени (сколько уходит на новые фичи, сколько на баги, сколько на технический долг). Дальше задавали вопросы о процессах разработки и код-ревью. В конце делали выводы о корреляции ответов, оценки качества кода и распределении времени.

В итоге сформировали 4 правила хорошего код-ревью:

1. Не пропускайте фичи без код-ревью и ревьюйте код до деплоймента
2. Все разработчики должны время от времени ревьюить код, а не только тим лиды или самые опытные инженеры
3. Разработчик должен тратить на ревью от 0,5 до 1 дня на ревью в неделю. Не больше, не меньше
4. Надо делать глубокие код-ревью. В итоге это выигрышная стратегия — и % времени на новые фичи будет расти

По ходу дела ребята приводят интересные инсайты:

1. В среднем разработчики тратят 45% времени на фикс багов(28%) и техдолг(17%)
2. Интересная корреляция: увеличение субъективной оценки качества кода на 1 балл (шкала от 1 до 5) приводит к уменьшением времени на фикс багов на 5% и техдолга на 1%
3. Больше всего на оценку качество кода по мнению разработчиков влияет код-ревью