Code review
#разработка

Одним из процессов, который повышает качество кода в команде является code review.

В этом процессе принимают участие две стороны: автор кода и рецензент. Собственно, рецензент и принимает участие о добавлении кода в общую базу проекта.

На практике я сталкивался с двумя типами ревью:
• ревью проводит один человек (чаще всего тимлид)
• ревью проводят все члены команды, а решение в спорных ситуациях принимает тимлид.

Мне больше нравится второй вариант. Основным плюсом является то, что в процессе чтения кода принимают участие все члены команды, поэтому все понимают большую часть кодовой базы. Ну и тимлид может заняться другими делами, а не только ревьювить код.

Однако есть и недостатки ревью. Например психологическая сторона: по сути, ты указываешь на косяки в коде другому человеку. Мало кто из людей любит, когда им указывают на их ошибки, поэтому общение при ревью требует определенной деликатности. Каждой стороне необходимо отноститься к ревью как к способу обмена знаниями и способ улучшения кода. Ну и формулировки никогда не должны быть грубыми, какая бы глупая ошибка не была совершена.

А вы поводите ревью кода в своей команде?