ревью кода
-
Хабрахабр
QIWI Грейдер — как оценивать сотрудников быстро, удобно, прозрачно
Оценка персонала помогает решить сразу несколько задач. Во-первых, что важно для самого сотрудника — он будет знать, насколько хорошо справляется с работой, какие навыки стоит подкачать, а какие уже в порядке. Во-вторых, обратная связь внутри команды или подразделения — это всегда важно, с какой стороны ни посмотри. В-третьих, от оценки и зависят такие приятные штуки, как премии, да и карьерный…
Читать далее » -
Хабрахабр
Как наладить код-ревью, если ты совсем один
Код-ревью – это хорошо. Оно помогает улучшить качество кода, наладить сотрудничество в команде, познакомить каждого разработчика с самыми отдаленными уголками кодовой базы, до которых он бы не скоро добрался, делая только назначенные ему задачи. Из всего того, чем обычно занимаются программисты, именно в код-ревью лучше всего видна справедливость поговорки «Одна голова – хорошо, а две – лучше». А как быть…
Читать далее » -
Хабрахабр
Процесс ревью кода структурно порочен. Вот, как его исправить
Классический процесс ревью кода страшен до безобразия и даже некоторого восторга. Разберём, почему это так, и что с этим делать. Взгляд на ревью кода из DALL·E 2 В программировании мы любим задумываться над тем, как поизящнее написать программу, как выбрать методологию поэлегантнее, как бы удачнее применить паттерн. На этом наши возможности не останавливаются, и мы учим наши программы рефлексировать. То…
Читать далее » -
Хабрахабр
Пример полезного комментария
В хороших книгах по программированию пишут, что код должен быть самодокументирующимся. А комментарии нужны там, где делается что-то нетривиальное. Наша команда разделяет это мнение, и недавно нам попался фрагмент кода, который отлично это демонстрирует. Код, который мы рассмотрим далее, был выписан в процессе работы над статьёй "Обработка дат притягивает ошибки или 77 дефектов в Qt 6". Анализатор PVS-Studio обратил внимание…
Читать далее » -
Хабрахабр
Пример полезного комментария
В хороших книгах по программированию пишут, что код должен быть самодокументирующимся. А комментарии нужны там, где делается что-то нетривиальное. Наша команда разделяет это мнение, и недавно нам попался фрагмент кода, который отлично это демонстрирует. Код, который мы рассмотрим далее, был выписан в процессе работы над статьёй "Обработка дат притягивает ошибки или 77 дефектов в Qt 6". Анализатор PVS-Studio обратил внимание…
Читать далее » -
Хабрахабр
Код ревью: как быть хорошим автором
Привет! Меня зовут Сергей Загурский, я работаю в Joom в команде инфраструктуры. В своей практике ревьюера кода я регулярно сталкиваюсь с тем, что автор не понимает, что ревьюер не является волшебным чёрным ящиком, в который можно закинуть любые изменения и получить по ним обратную связь. Ревьюер, как и автор, будучи человеком, обладает рядом слабостей. И автор должен (если, конечно, он…
Читать далее » -
Хабрахабр
Code review Терминатор. Ревью, за которое вам скажут спасибо
Рыжик помогает мне ревьюить код. А когда ему что-то не нравится — тоже настоящий Терминатор «Code review Терминатор», — однажды назвал меня коллега после особо продуктивного ревью. С одной стороны, это тешило ЧСВ и было приятно. С другой — коллега действительно научился чему-то новому, и это позволило писать ему более качественный код. Так что win-win. После смены работы сменились и…
Читать далее » -
Хабрахабр
[Перевод] Чему я научился у ведущего программиста
Год назад я начал работать на полную ставку в Bloomberg. И тогда же задумал написать эту статью. Я думал, что буду полон идей, которые смогу выплеснуть на бумагу, когда придёт время. Но уже через месяц понял, что всё будет не так просто: я уже начал забывать то, чему научился. Либо знания настолько хорошо усвоились, что мой разум заставил меня поверить,…
Читать далее » -
Хабрахабр
Процесс ревью кода в hh.ru
Мне на глаза попался документ с правилами и рекомендациями по процессу ревью кода внутри компании. Я решил, что такой полезной информацией надо поделиться с внешним миром. С благословения автора я публикую работу. Большинство указанных правил носят рекомендательный характер и надо руководствоваться в первую очередь здравым смыслом, а не слепым соблюдением. Общие положения В hh.ru ревью проходит в формате пулл-реквестов на…
Читать далее » -
Хабрахабр
Рецепт полезного код-ревью от разработчика из Яндекса
Меня зовут Сергей, последние пять лет я работаю в Яндексе. Привет. Писал код на JavaScript, Python и C++. За это время участвовал в разработке одиннадцати проектов. Но в каждой команде, на всех проектах, вне зависимости от языка программирования я использовал код-ревью. Некоторые проекты делал в одиночку, другие разрабатывал в группе из восьми человек. Иногда, глядя на чужой код, хочется воскликнуть:…
Читать далее »