Теперь Кью работает в режиме чтения

Мы сохранили весь контент, но добавить что-то новое уже нельзя

На что смотреть при проведении код-ревью?

ПрограммированиеCode review
Антон Ахатов
  · 175
Лучший
Project Manager at Yandex  · 16 окт 2021
Чтобы на что-то смотреть, нужно определить, зачем вы проводите код-ревью.
На код-ревью можно валидировать:
  • архитектуру приложения, чтобы написанный код не выбивался из общей концепции
  • бизнес-логику, чтобы новый код не ломал какие-то существующие модели приложения, о которых разработчик может не знать
  • правильность реализации алгоритмов, если это применимо
  • сложные логические ошибки, например, управление многопоточностью
  • читаемость кода, так как код будут читать и другие разработчики
  • стиль кода, чтобы сохранялись общие правила к наименованиям функций, переменных и так далее
  • ошибки, конечно же
В общем, нужно определить задачу, которую код-ревью должно решать в вашем случае. А затем уже выбирать, на что смотреть.
1 эксперт согласен
Разработчик Rails/React  · 11 февр 2022
Я бы начал с оценки объёма коммита, чтобы можно было вообще рассчитать свои силы. Дальше вспомнить от кого этот код, что вообще от человека ожидать, может он делает какие-то типичные ошибки и их стоит проверить в первую очередь. В целом главное, наверное, оценка адекватности кода решаемой задаче.