Отвечаю на комментарии к видео про ревью кода

  Рет қаралды 5,311

S0ER

S0ER

2 жыл бұрын

#soer #itubeteam
Основной канал для общения и публикации новых видео - Телегарм - t.me/softwareengineervlog
Спонсорство - donate.s0er.ru
Сайт платным контентом - soer.pro
Зеркало для видео Дзен Видео - zen.yandex.ru/id/5f578bdf22e2...
GitHub - github.com/soerdev
Чат для программистов - / discord
Группа ВК - codeartblog

Пікірлер: 28
@SeniorSoftwareVlogger
@SeniorSoftwareVlogger 2 жыл бұрын
1 + 1 = 11, это каждый фронтенщик знает вообще-то
@karelalex
@karelalex 2 жыл бұрын
Я не знаю, наверное, надо меньше в бакенд залезать. 😀
@S0ERDEVS
@S0ERDEVS 2 жыл бұрын
А 11 - это три )
@konstantinsolkin487
@konstantinsolkin487 2 жыл бұрын
@@S0ERDEVS Его идеи будут актуальны всегда
@programisli
@programisli 2 жыл бұрын
Из моего опыта мало кто делает реальное ревью кода, обычно это способ потратить время на кофе и просто нажать кнопку Approve.
@daniilthegunner843
@daniilthegunner843 2 жыл бұрын
Когда видос "отвечаю на комментарии к видео, где я отвечаю на комментарии к видео про код ревью"?)
@nikolaymatveychuk6145
@nikolaymatveychuk6145 2 жыл бұрын
Интересное видео получилось. Спасибо!
@user-tm7hs1un2m
@user-tm7hs1un2m 2 жыл бұрын
Уважаемый Soer, было бы интересно увидеть выпуск про легаси код: каким будет Ваше видение того, как справляться с большими массами устаревшего, трудноподдерживаемого кода, не покрытого тестами и документацией и т.д. Например, писать заново или работать с тем, что есть (и если всё же разгребать, то как наиболее эффективно привести всё в порядок). Спасибо)
@dev_insider
@dev_insider 2 жыл бұрын
14:28 «то как работают другие калеки» 😆
@saiko4972
@saiko4972 2 жыл бұрын
Очень нравится картинка. Скажите, пожалуйста, на какую камеру снимаете и как у вас получается это небольшое размытие, дымка(?)
@S0ERDEVS
@S0ERDEVS 2 жыл бұрын
Sony 7s 28mm/3.5 Размытие за счёт фокуса объектива. Дымка за счёт вытягивания тёмных мест в серый цвет.
@KolomiecSergeyK
@KolomiecSergeyK 2 жыл бұрын
Could you please talk a bit about git flow. How to make big feature in git better? Should we squash commits or not. When do we need to do this (squash) Maybe some example? Thanks!
@stanislavsh6582
@stanislavsh6582 2 жыл бұрын
Я вот только не могу понять, почему на футболке лого нутаку...
@user-df5pk9qp6k
@user-df5pk9qp6k 2 жыл бұрын
Можно узнать название темы для KDE, которую Вы используете?
@skynowa2626
@skynowa2626 2 жыл бұрын
Т.е. code-style придумали просто так
@a.danilenko
@a.danilenko 2 жыл бұрын
Пора, наверное, code review относить к разряду "темы, вызывающие бесконечное количество споров".
@Seacrest.
@Seacrest. 2 жыл бұрын
недисциплинарный код должен быть осужден
@kselnaag2482
@kselnaag2482 2 жыл бұрын
Абсолютно не согласен с автором канала. Здесь работает правило единственности ответственности: прогер пишет код, чтоб он работал и прошел ревью; ревьюер читает и апрувит код, берет на себя полную ответственность этот код. Поэтому все, что требует ревьюер, кодер обязан поправить, т.к. дальше ответственность ревьюера. Простые, понятные всем правила, не имеющие двоякого толкования. Я, например, ревьюер, какая у меня мотивация делать работу, если она ни на что не влияет (это же просто совет) ? На сколько часто именно ревьюер находит баг, особенно в ситуации со сниженной мотивацией ? Являюсь приверженцем как раз Бугаенко, у него вообще 2 ревью: первое - кросс-ревью, выполняется другим членом команды, на этом этапе отсеивается большинство багов, несоответствий архитектуры, проблемм с производительностью и.т.д.; второе - когда таска прошла первое ревью, код смотрит и апрувит архитектор на соответствие феншую и всему такому, чем о5ть берет на себя ответственность, что этот код будет смерджен в основную ветку. Еще раз подчеркиваю: основной принцип не быть добреньким, поддерживать атмосферу быть лояльным и т.д., это ни разу не гарантирует выполнение задачи качественно и в срок, а сохранять четкие и понятные правила для всех.
@S0ERDEVS
@S0ERDEVS 2 жыл бұрын
Ага, только а задаче "код работал и прошёл ревью" две отаетсвенности. SRP как раз работает в моем варианте - программист отвечает за работу кода, ревьюер проверяет понятность кода. )
@kselnaag2482
@kselnaag2482 2 жыл бұрын
@@S0ERDEVS Не стоит передергивать. Вы прекрасно понимаете, что кроется под формулировкой "принцип единой ответственности". Если вам угодно, то программист отвечает за пропихивание всеми правдами и неправдами своего коммита в мэйн-ветку, а уж через какие фильтры он вынужден пробиваться - это дело десятое. И ревью, наряду с юнит тестами, линтерами и прочими CI/CD пайплайнами имеет только одну цель - отфильтровать неподходящий код. Но тут вопрос, в основном, не в технических тонкостях и понятийных неточностях, а в одном простом вопросе: готов ли "менеджер" (например техлид) брать ответственность за то, что его команда творит. Способен ли человек, управляющий процессом разработки, сконструировать адекватную систему этой самой разработки. Или он, не зная что делать, покупает смузи и биллиардный стол в офис (цитата Бугаенко, близкая к оригинальной =D ) ? Каждый управленец отвечает на этот вопрос сам.
@S0ERDEVS
@S0ERDEVS 2 жыл бұрын
@@kselnaag2482 я то прекрасно понимаю srp, только не понимаю зачем вы его вспомнили. Он тут не к месту.
@S0ERDEVS
@S0ERDEVS 2 жыл бұрын
Что касается идей Егора, применительно к практике управления, то лучше чем он сам никто и не скажет - kzfaq.info/get/bejne/r5xkjceXmbWtaX0.html
@Max-nr1bv
@Max-nr1bv 2 жыл бұрын
Стараюсь не работать в таких командах. Мне больше всего нравится неформальная власть. Для команды очень опасна ситуация, когда техлид некомпетентен (я не говорю про вас, но такое бывает) и навязывает свою точку зрения, потому что он здесь власть и он ответственный и швец и жнец и на всех ЯП игрец. Субъективно мое мнение, что в либеральной атмосфере просто легче работать всем. Может сиюминутно продуктивность ниже, но на долгой дистанции проект выигрывает. Мой техлид на прошлом проекте не брал отпуск несколько лет просто потому что не уставал, работал размеренно. И бизнес сыт и программисты целы
@timurbabkin8113
@timurbabkin8113 2 жыл бұрын
Ты все равно меня не убедил: если у человека есть цель стать лучше, то нет разницы приказываешь ты ему или советуешь, он так или иначе будет воспринимать это как направление для развития. Так было у меня, когда я был джуном, так и у моих джунов, когда я стал тимлидом. Самое главное - аргументировать замечания, чтобы человек понимал почему нужно делать так, а не иначе, и к чему то или иное решение может привести. А так, с остальными тейками, в целом, согласен.
@S0ERDEVS
@S0ERDEVS 2 жыл бұрын
Имеешь полное право, мои видео это просто совет )
The day of the sea 🌊 🤣❤️ #demariki
00:22
Demariki
Рет қаралды 51 МЛН
Универ. 13 лет спустя - ВСЕ СЕРИИ ПОДРЯД
9:07:11
Комедии 2023
Рет қаралды 4,9 МЛН
Проектируем OpenSource приложение
17:31
SQM 18/24: Defects Density [software quality crash course] [eng sub]
1:24:01
The day of the sea 🌊 🤣❤️ #demariki
00:22
Demariki
Рет қаралды 51 МЛН