Как на код-ревью помогать младшим коллегам расти?
- Как обучать через код-ревью?
Объяснять почему, а не только что: «здесь будет гонка, если два запроса придут одновременно — вот как это проявится», со ссылкой на материал. Задавать наводящие вопросы, чтобы человек нашёл проблему сам. Разделять обязательное и пожелания, не заваливать десятками мелочей. Отмечать хорошие решения. Повторяющиеся замечания — повод для гайда или правила линтера, а не для одного и того же комментария.
- Сложное обсуждение — голосом, а не в двадцати комментариях
- Не переписывать код за автора в своём коммите — так он не научится
- Ревью от младшего к старшему тоже полезно — оба учатся
А ещё Хорошее ревью — то, после которого следующий PR этого человека лучше
Подробно
Пример комментария:
«Тут запрос к базе в цикле — на 1000 заказов будет 1000 запросов (проблема N+1). Можно загрузить позиции одним запросом через prefetch_related — вот пример в orders/report.py. Как думаешь, что будет с временем ответа на большом клиенте?»
Системно: собрать частые замечания в чек-лист для авторов PR.
Что хотят услышать
- Объяснение причин и последствий.
- Вопросы вместо готовых ответов.
- Приоритеты замечаний и похвала.
- Повторяющееся — в гайды и автоматизацию.
Проверьте себя
Один верный вариант. Подумайте, потом откройте ответ.
Показать варианты и ответ
- ✗Переписать код за автора, чтобы он сразу увидел правильный вариант
- ✗Оставлять короткие «переделай» — разберётся сам и запомнит лучше
- ✓Объяснять, почему решение проблемно, задавать наводящие вопросы
- ✗Помечать как обязательные все замечания, включая стиль, чтобы приучить к порядку
Разбор темы
- Рост коллег — наставничество, онбординг, ревью и общие знания
Как помогать расти младшим коллегам, вводить новичка в работу, учить через код-ревью, поддерживать документацию и не быть единственным экспертом.
Связанные вопросы
Разбор собеседования с sotto
После собеседования sotto разбирает разговор: оценка, каждый вопрос — хорошо, частично или слабо, чего не хватило в ответе и над чем поработать к следующему разу. Первые 30 минут бесплатно.