Допустим, вы разработчик. Вы отвечаете за свою часть работы – например, отправку данных на сервер. В команде уже есть проект, вы вместе его поддерживаете и улучшаете.
Когда вы пишете новую функцию, она не попадает сразу в проект. Сначала её проверяют – это и есть код-ревью.
Что смотрят на проверке
Кто-то из старших коллег берёт ваш код и изучает:
- ошибки и баги;
- стиль – соответствует ли код правилам компании;
- оформление – есть ли отступы, соблюдена ли читаемость;
- комментарии – если их нужно оставлять;
- логика решения – вы выбрали красивое решение или сломали полпроекта?
- влияние на других – не заставите ли вы коллег переписывать свою работу;
- безопасность – может ли злоумышленник использовать код во вред.
Если есть проблемы – код отправляют на доработку. Если всё ок – он идёт дальше, обычно на тестирование.
Кто проверяет
Обычно код-ревью делают более опытные разработчики. Джуниоров проверяют мидлы, мидлов – сеньоры, сеньоров – другие сеньоры или тимлиды. В больших компаниях могут быть отдельные люди, которые смотрят код у всех и следят за стилем.
В маленькой команде этим занимается ведущий программист.
Как проходит процесс
- Вы написали код и отправили на проверку.
- Проверяющий смотрит код, исправляет ошибки или пишет замечания. Иногда может подойти и объяснить лично.
- Если замечаний много – вы переделываете. Если мало или нет – проверяющий правит сам и передаёт дальше.
Зачем это надо
Если вы работаете один или с другом – код-ревью не нужно. Вы и так обсудите всё на месте.
Когда в команде много людей – каждый пишет по-своему. Даже если код работает, поддерживать его потом – боль. Править чужой код, если он написан плохо, – долго и дорого. Поэтому на код-ревью разработчики приводят код к общему знаменателю, чтобы потом не мучиться.
Комментарии
Пока нет отзывов