ТОП рекомендаций по код-ревьюСидишь ты, значит, пилишь фичу для нашего интернет-магазина. Добавляешь хитрую логику для расчета скидок или обновления статусов заказов. Локально все работает, тесты зеленые. Собираешь это дело, отправляешь в ветку и создаешь Pull Request – типа, "готовенько, народ, забирайте в прод!". И ждешь, что вот-вот твоя гениальность окажется у пользователя.Но вместо этого тебе прилетают комментарии. Стрелочки указывают на твой код, рядом какие-то вопросы, предложения, иногда даже восклицательные знаки. Ух ты, елки-палки! Сразу мысли: "Почему они придираются? Это же работает!". Это и есть код-ревью, и многим новичкам (да и не только) оно кажется пыткой или экзаменом.Давай разберемся, как делать ревью, чтобы коллеги сами хотели учитывать твои комментарии и ждали твоей проверки как праздника.Зачем вообще это надо?Ты написал кусок кода, а другие ребята из команды смотрят на него свежим взглядом. И это важно, потому что когда ты погружен в задачу, глаз замыливается. Ты можешь не заметить банальную опечатку, неочевидный баг, который вылезет только при определенных условиях, или просто не очень оптимальное решение. Ревьюеры помогают найти и устранить такие штуки.А еще ревью – это обмен знаниями. Ты видишь, как пишут другие, учишься новым приемам, узнаешь о фишках языка или фреймворка, о которых даже не подозревал. А когда ревьюишь ты, то глубже погружаешься в чужой код, видишь разные подходы к решению задач. Это одна из самых эффективных инвестиций в собственное развитие и качество кода.Что искать в коде?Сначала просто прочитай код, как книгу. В хорошем коде тебе должна быть понятна логика, и ты должен сразу видеть, что делает каждый метод. Если приходится ломать голову, значит, код можно сделать более читаемым, например, переименовать переменные и разбить длинный метод на несколько маленьких.Дальше включай режим детектива и ищи потенциальные проблемы. Попробуй обнаружить места, где может вылететь null; проверь циклы: нет ли там лишних операций, которые замедлят работу? Проверь, не тянет ли кто-то все данные из базы, когда нужна только пара строчек?И смотри на общую структуру. Соответствует ли код тем принципам, которые приняты в вашей команде? Не добавили ли в проект избыточные зависимости?Комментируй как босс!Главное правило: будь конкретным и вежливым. Вместо "Тут плохая логика" напиши: "Смотри, вот в этой строке мы считаем скидку до проверки на минимальную сумму заказа. А что, если сумма меньше? Получится, что скидка посчитается, но потом не применится. Может, сначала проверять сумму, а потом уже считать скидку?". Видишь разницу? Ты указываешь на проблему, объясняешь почему это проблема (на примере их же кода!), и предлагаешь вариант решения, но не настаиваешь.Используй вопросы: "А что будет, если...?", "Не думал над вариантом сделать...?", "Почему сделано именно так?". Это не звучит как обвинение, а скорее как приглашение к дискуссии. И всегда начинай ревью с чего-то позитивного, если есть за что зацепиться. "Круто, что добавил тесты на этот кейс!" или "Хорошо структурировано!".Тебе пишут?Если непонятно – спрашивай! "Не совсем понял, почему этот подход лучше в данном случае? Можешь привести пример?". Не бойся показаться глупым. Вопросы показывают, что ты хочешь разобраться. Иногда в процессе обсуждения оказывается, что и ревьюер что-то упустил, или твой вариант тоже хорош, и твой коллега только рад будет, если ты отстоишь свою правоту. Это нормальный рабочий процесс.Не спорь ради спора. Твоя цель – сделать код лучше, а не доказать, что ты знаешь все лучше всех. Принимай конструктивную критику, исправляй, учись. А если с чем-то категорически не согласен, аргументируй спокойно.Когда вся команда активно участвует в ревью, баги вылавливаются быстрее, знания распространяются, а новички растут. Ревью – это один из самых мощных инструментов для обучения и развития в команде. Используй его как возможность стать сильнее как разработчик. Это реально работает!Рассказывай в комментах, какой самый полезный совет ты получил на ревью!
Оставить комментарий/отзыв