Code Review, как это фактически происходит?

Question

[Vetal Matitskiy]

Доброго дня, уважаемые гуру разработки.
Подскажите пожалуйста что обычно подразумевают под Code Review? Это некоторые собрания вживую когда написавший код рассказывает что он делал а другие задают вопросы или нечто другое? смотрел публикации на хабре и других ресурсах по этой теме, но там все очень обобщенно и вскользь. с кучей хвалебных отзывов но не раскрывают фактическую суть понятия. Знакомый с курсов говорит, что у них Code Review, это когда кто-то из менторов/преподователей через онлайн смотрит твой код и дает по ходу просмотра коментарии. Какая из трактовок наиболее правильная?

Answer 1

[Espleth]

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

Comments

[Vetal Matitskiy]

при таком подходе происходит прямое общение между кодописателем и ревьювером? или же это подразумевает что ревьювер самостоятельно смотрит код и может при необходимости делать какие-то пометки/замечания?

[Artem Filippov]

Обычно Code Review проводится прямо в GitHub или BitBucket, сразу со всеми поправками и замечаниями к пулл-реквесту, выходит очень удобно и продуктивно.

[Espleth]

Vetal Matitskiy: как сказал Artem Filippov, обычно код ревьюер смотрит код самостоятельно. Он делает замечания по коду, которые ему хотелось бы исправить, отправляет эти замечания человеку, написавшему этот код, и после того, как этот человек исправляет все эти замечания, код можно будет принимать.

Answer 2

[Денис Инешин]

Для проведения Code Review существуют специальные инструменты. Например на Гитхабе, можно ревьюить любой пулл реквест и комментировать любую строчку кода. Т.е. без проблем делается удаленно.

Comments

[Vetal Matitskiy]

правильно ли я понимаю что Code Review это чаще всего удаленный процесс?

Answer 3

[Максим Кузнецов]

Вот хороший ответ на Ваш вопрос: habrahabr.ru/post/142564
На мой взгляд, CR - единственное средство борьбы с трудно уловимыми ошибками, которые обычно не удаётся воспроизвести на тестах, и которые вследствие хитрого сплетения обстоятельств не дают спокойно работать нашим пользователям.
Причём у нас три уровня борьбы, на каждом из которых используется CR: при детальном конструировании архитектор или ведущий разработчик анализирует готовый код на предмет корректного вмешательства в его работу (принцип "не навреди") и на предмет повторного использования готовых компонентов, затем после реализации проводится обзор кода на предмет его оптимальности и детального понимания принципов работы реализованного компонента (последнее важно для разработчиков, которые будут использовать компонент при дальнейшей работе над кодом), и, конечно, при диагностике сбоев системы.