Про code review

#soer #itubeteam

[leandrmiklashevich]

Золотой человек! Как всё чётко и по делу высказано. Так трымаць)))

[Прикольноепроектноепрограммиро]

Code Review - нереально полезная вещь, причем полезная даже при написании "Hello, world", когда твой код смотрит одноклассник или друг. Помню, очень гордился программой, где было что-то наподобие
if (x == 12) return 'A';
if (x == 13) return 'B';
if (x == 14) return 'C';
...
В общем, около 15 слов if и return на 15 строках. После code review от одногруппника спеси у меня поубавилось))))

[albrehtdurer]

Позволю не согласиться...Если код релевантен - предметной области задачи, он действительно читается легко как книга ...но при одном условии- читающий должен владеть предметной областью этой задачи

[ВячеславЮшков-пщ]

Пойдет в копилку полезных рекомендаций к осмыслению. Есть такой момент: более опытные разработчики тоже могут ошибаться - им приходит в голову гениальная алгоритмическая схема, которая решает все проблемы на этом участке кода, но не всегда здраво оценивают сроки реализации для исправления проблемного говнокода, а потом начинается ранговая борьба с остальными, что может демотивировать часть команды. Как бороться с писькомерством внутри команды, которое вредит бизнесу?

[ПавелВолков-ъъ]

Не согласен с мнением о том, что кодревьюер должен быть обязательно на голову выше остальных членов команды. У нас, например, часто ставят на должность кодревьюера среднего (а иногда даже и младшего разработчика) с целью быстро и качественно его обучить + если человек новый, то ему это очень быстро помогает привыкнуть к стилю написания кода в нашей команде. Кодим в основном на плюсах.

[boycovclub]

Извините может не в тему, а что вы думаете насчет фреймворка React Native?

[stranger]

Именно по этому простую программу год будите делать. Когда фулстек за день все напишет

[NoldoWalker]

Спасибо за обзор темы. Возможно стоит рассказать про code style и code convention.

[Das.Kleine.Krokodil]

а обязан ли разработчик делать изменения по результатам код ревью?

[eugenemazurov]

Не, если появилась мысль добавить комментарий, то уже есть проблема. За комментарии надо давать молоко... Автор мало работал на проектах, тема раскрыта плохо.

[fivaproldge]

Привет, ответь на вопрос про белую рубашку и в целом приличный вид, если не сложно. Ты же свои видосы из дома снимаешь? И работаешь явно по удаленке... Белая рубашка и все такое - зачем? При удаленке оно помогает войти в рабочий ритм, находясь дома?

[musicmadness]

Посмотрел до 1:42. Интересная тема, прошу сделать видео где в первых 20% полностью раскрывается тема ревью.

[albrehtdurer]

Я так и не понял...Зачем делать кодревью, если нет претензий к производительности кода? Возможно вы говорите не про ооп стиль программирования? Как плохой пример -java, Хороший - smalltalk.

[iNemoden]

Не согласен про «убивает демократию», кол ревью это и есть демократия, если требуется минимум 2 код ревьюера, таким образом у нас уже 3 участника - программист, написавший код и его мнение, выраженное в коде и ещё 2 мнения код ревьюеров, демократия таким образом достигается обсуждением кода минимум 3мя сторонами, а не 2мя где, судя по видео, тот кто делает код ревью заведомо более опытный разработчик. Насчёт того, что код ревью должен делать более опытный разраб тоже не согласен, - код ревью должен делать любой член команды, будь то джун на стажировке или архитектор с 20 летним стажем работы и речь даже не про передачу знаний, т.к. передавать знания можно более эффективными способами, это одна из но не основная задача ревью кода. Если мой код проверит старший товарищ, он может указать на недостатки с точки зрения своего опыта, если же мне код ревью сделает джун, это может быть полезно и джуну и мне, первый может быть получит знания, я получу фидбак понятен ли мой код менее опытным разрабам (это ведь тоже важно)