Online Mind Mapping and Brainstorming

Create your own awesome maps

Online Mind Mapping and Brainstorming

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 - 3 reviews range from 0 to 5

Ревью кода

Цели

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

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

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

Устаревшее

Причины ЗА

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Рецензия (passaround)

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

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

Интерфейсы

Тесты

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

Реализация

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

Изменения

Всё решение

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

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

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

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

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

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

Время будет!

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

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

Ищешь сам

В парах

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

Спец-отряд

Танцуют все

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

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

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

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

Static code analyser до ревью

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

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

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

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

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

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

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

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

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

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

Замечания

Кто ведет?

Кейсы

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

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

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

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

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

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

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

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

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

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