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