Ревью кода

Что это значит

Ревью кода - проверка изменений в исходном коде другим разработчиком перед их слиянием с основной версией программы. Проверяющий ищет ошибки, риски безопасности, нечитаемые места и отклонения от правил команды.

Чаще всего оно проходит в системе управления кодом на основе Git через pull request или merge request: автор предлагает изменение, проверяющий его комментирует и одобряет, только когда всё в порядке.

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

Как это работает на практике

Ревью кода дополняет автоматизированные тесты и CI/CD, проверяющие работоспособность, тогда как человек оценивает логику, архитектуру и читаемость. Вместе они образуют основу качества программ.

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

В системах, рассчитывающих налоги, зарплаты или цены, изменения в расчётах стоит проверять и человеку, понимающему предметную область, а не только программисту.

На что обратить внимание

Слишком большие изменения трудно проверить, и проверяющий часто одобряет их формально. Лучше небольшие частые изменения, в которых ошибки действительно находят.

Ревью кода не заменяет аудит безопасности или тест на проникновение. Уязвимости, возникающие при сочетании нескольких частей системы, при проверке одного изменения часто не видны.