Create your own awesome maps

Even on the go

with our free apps for iPhone, iPad and Android

Get Started

Already have an account?
Log In

Ревью кода by Mind Map: Ревью кода
5.0 stars - 1 reviews range from 0 to 5

Ревью кода

Цели

"Социализация" кода

Распространение знаний о системе

Коллективное владение кодом

Достижение консенсуса

Обучение

Повышение качества

Проверка завершения

Устаревшее

Соответствие стилям и правилам

Устранение ошибок

Причины ЗА

Две пары глаз лучше

Эффект плюшего мишки

Проговаривание

Другой режим работы мозга

Выравнивание знаний о системе в команде

Код "для людей"

Причины против

Личностные конфликты

Звездная болезнь автора кода

Может стать скучной процедурой

Может повысить трудозатраты

Социальные (критика -> недоверие, обиды)

Ложное чувство безопасности

Степень формальности

Групповое ревью (inspection,team review)

Прохождение (walkthrogh)

Парное программирование (pair programming)

Рецензия (passaround)

Целевое ревью (ad hoc review)

С чего начать?

Интерфейсы

Тесты

Документация

Реализация

Что ревьюить?

Изменения

Всё решение

Ключевые точки (классы, методы)

Как внедрить?

Общее решение команды

Начать с малого и медленно

Адаптировать под себя

Интегрировать в процессе

Время будет!

Стратегия выбора ревьювера

Авторитетный человек

Ищешь сам

В парах

Контрольный центр, выбирающий ревьювера

Спец-отряд

Танцуют все

Связи на стороне

Полезные советы

Проводить до или перед чекином (перед лучше)

Отслеживать ревьюверов

Он знает код

Статистика перегрузки ревьюверов

Отвественность

Static code analyser до ревью

Инспектируем код, а не автора

НЕ пропускаем ревью

Ревьювер в курсе задачи

Должно быть удобным процессом

Правила ревью д.б. общедоступны

Документировать ревью

Ищем достоинства, а не только недостатки

Тесты тоже требуют ревью

Есть вопросы, непонятно как работает - спрашиваем

Стратегия ведения ревью

Замечания

фиксятся сразу

оформляются как задачи

Кто ведет?

Автор

Ревьювер

Кейсы

Изменений много

Исправляем

Комитим как есть и ставим задачи

Примитивные ошибки правим сразу

Не делаем наспех

Не затягиваем проведение

Если ревью не прошло 2 раза - опасность

Меняем автора

Добавляем ревьювера

Автор не понял смысл замечаний

Программируем в паре

Типичные ошибки

Ревьювер правит код

Ревьювер вносит свое представление решения

Можно сказать "все отлично"