Код-ревью и требования к коду джуна, мидла и сеньора

แชร์
ฝัง
  • เผยแพร่เมื่อ 24 ธ.ค. 2024

ความคิดเห็น • 109

  • @АнтонСтепанов-д3о
    @АнтонСтепанов-д3о ปีที่แล้ว +106

    Лайк сразу, только за то, что Вы все еще Сергей Немчинский!

    • @АнтонСтепанов-д3о
      @АнтонСтепанов-д3о ปีที่แล้ว +1

      @cafehumanhours Та харош. Уж этого я точно не переживу. Он капитальный кравчик!

    • @SergMirny_yt
      @SergMirny_yt ปีที่แล้ว

      😂👍

  • @tee-hee
    @tee-hee ปีที่แล้ว +6

    Разобраться, что там намычал заказчик в бизнес требованиях - задача ПМа, а не разработчика. Или даже бизнес аналитика, если он есть. Иначе, а зачем тогда они нужны вообще? Другое дело если в компании их нет.

  • @meduska-tyan
    @meduska-tyan ปีที่แล้ว +1

    Работаю 1С программистом. Что-то похожее на код ревью было только, когда начал работать. Иногда требую код ревью, когда нужно срочно изменить большое количество полей в базе и нет времени протестировать его.

  • @IvanSkliarenko-i8t
    @IvanSkliarenko-i8t ปีที่แล้ว +7

    Борода и очки + 30% к експертности)

  • @leetaipe
    @leetaipe ปีที่แล้ว +13

    Для проверки стиля кода есть линтеры, которые обычный ревьюер никогда не заменит по эффективности. На мой взгляд, если и имеет смысл делать код ревью то в первую очередь на уровне solution design и логики, и это отличная возможность для команды выработать какие-то общие принципы в написании кода, особенно для тех, кто новенький в команде/джун.

    • @SergMirny_yt
      @SergMirny_yt ปีที่แล้ว

      Так в этом же и заключается ревью (solilution design). А что б правильный отступ был или стилистика скобочек - это уже давно даже IDEшки делать умеют+ всякие flake8

    • @KostsovKonstantin
      @KostsovKonstantin ปีที่แล้ว

      На мой взгляд руководитель должен определять архитектуру, структуру и стиль. И все кто придерживается свободных взглядов или пытается умничать - остаётся без зарплаты.

    • @SergMirny_yt
      @SergMirny_yt ปีที่แล้ว

      @@KostsovKonstantin такиоб этом же и комментарий, не?)

    • @KostsovKonstantin
      @KostsovKonstantin ปีที่แล้ว +1

      @@SergMirny_yt , почти. Контекст немного другой.

  • @archie436
    @archie436 ปีที่แล้ว +4

    Дуже вдячний за відео

  • @ВладЛосев-н8т
    @ВладЛосев-н8т ปีที่แล้ว +1

    Классная у тебя рубаха :)

  • @secretadmirateur5979
    @secretadmirateur5979 ปีที่แล้ว +1

    Спасибо Сергей, еще не досмотрел видео, но уже благодарен!

  • @m777rmc3
    @m777rmc3 ปีที่แล้ว +1

    Дзякуй за Вашу працу

  • @RomanSumyua
    @RomanSumyua ปีที่แล้ว +11

    Не согласен с определением Джун/миддл /сеньйор. Вы путаете разработчиков с бизнеса аналитиками. Разбираться в бизнес логике, особенно большой компании, не задача разработчика. Понимание работы компании приходит со временем, но это никак не связано с уровнем сеньйорности разработчика. Особенно это касается больших компаний в которых большая текучка кадров.

    • @kawaikaino5277
      @kawaikaino5277 ปีที่แล้ว +1

      @@777camry777 Бред! Аналитик, не обязан понимать код) Бизнес процессы и логику самого приложения - да.

    • @kawaikaino5277
      @kawaikaino5277 ปีที่แล้ว +1

      @@777camry777 Отлично знаю, 4 года опыта в крупных компаниях. Если вас, заставили лезть в зону ответственности разработчиков, это не значит, что везде так

    • @winfle
      @winfle ปีที่แล้ว

      @@kawaikaino5277 Domain Driven Design

    • @drovoseg
      @drovoseg ปีที่แล้ว

      @@777camry777 Зачем аналитику читать код? Достаточно уметь читать и писать всякие open api, uml и схемы баз данных?

  • @markkovalyov9492
    @markkovalyov9492 ปีที่แล้ว +2

    Предел Бороды Ричарда Столлмана преодолен!
    По поводу код-ревью. Когда все эти гулг-глазы появились у меня возникла идея - фиксировать взгляд профессионала на код. По принципу температуры. Чем дольше core-reviewer смотрит на фрагмент кода - тем выше его температура. Далее можно сверить с другим. Самые температурные участки кода - подозрительны. Это еще не WTF но уже сомнительное местечко.

    • @PavelSidorovich
      @PavelSidorovich ปีที่แล้ว

      красиво! так же будет работать и в обратную сторону - подсвечивать некомпетентных или недостаточно ответственных ревьюверов

  • @KostsovKonstantin
    @KostsovKonstantin ปีที่แล้ว

    Интересная тема видео. Зашёл покритиковать. Но ты чертовски прав.

  • @Vlad_a450
    @Vlad_a450 ปีที่แล้ว

    Вау! Я - сеньор! А самооценка вечно на уровне чайника! )))))))))))))))
    Наверно сеньор этот тот, кто в принципе владеет технологией на 80+ %. Иначе трудно определить как считать. ИМХО Сеньор это: опыт работы в разных проектах и с разными технологиями с разными предприятиями в течение 3-5 лет с решением задачи уровня определения архитектуры проекта.

  • @deniskochetkov6827
    @deniskochetkov6827 ปีที่แล้ว +1

    Благодарю!

  • @igorehin
    @igorehin ปีที่แล้ว +3

    Спасибо Сергей! 100% так и есть как ты говоришь! Расскажи лучше как сравлятся с ситуацией когда тебе буквально навязывают писать код путём,который тебе не по душе и твой код фактически делает одну и ту же задачу и не как не противоречит цели! Я мидл девелопер в автомации и много чему научился у сеньёров,в том числе фулстакс,за что им безгранично благодарен! Взгляд и подход к задаче может быть разный у всех иногда разобратся в подходе другого девелопера ,это просто срыв башки. Зачем всё усложнять,если я написал код более практичным и читабельным? (Я не говорю о всех случаях когда в самом деле можно применить другой подход) Спасибо!

    • @lero4ka_valero4ka_77
      @lero4ka_valero4ka_77 ปีที่แล้ว

      Замер производительности сделай и у кого меньше очей набъет у того и хрен больше, а на словах не хуже этот как вид слева или снизу

    • @МихаилСкляренко-б4с
      @МихаилСкляренко-б4с ปีที่แล้ว

      @@lero4ka_valero4ka_77, у меня было подобное, я проводил замеры, но ревьюер как баран упирался, что здесь принят такой стайл и писать замороченный (на мой взгляд, но он считает иначе) код тоже принято в команде (к слову, в команде на то время были почти все новенькие) хотя предыдущая команда, по странным причинам, почему-то ушла... не долго я там проработал...

  • @artyai2272
    @artyai2272 ปีที่แล้ว

    я пока не смотрел что у тебя есть на канале, но хочу задать вопрос - попал на проект, где был очень большой технический долг, а клиент не хотел чтоб мы его решали, у всей команды было прям дичайшее уныние и аппатия и лень и наша команда не успевала по этой причине, взаимоисключающие баги и тд, обычно до этого я уходил заранее с таких проектов, а тут изза желания поддержать компанию решил остаться до победного. В итоге меня уволили, и мне теперь тошно смотреть на код даже собственных проектов... что делать, как вернуть жажду создавать?(( у меня изза этого аппатия, на работу не хочу устраиватся тоже по этой причине, сам - сеньор с почти 10 летним стажем... хочу это вспомнить - как я часами сидел в коде и жизнь была интерсна...

  • @alexb.2616
    @alexb.2616 ปีที่แล้ว

    Волосы - огонь!!! :)

  • @TwilightSun32
    @TwilightSun32 ปีที่แล้ว +1

    да, при централизованом код ревью ещё и код писать проблематично. а где-то на десятке разработчиков нужен уже второй архитектор/техлид. но в целом работает. от команды зависит.

  • @syracuse4612
    @syracuse4612 ปีที่แล้ว +4

    ооо даже посмотрю это

    • @brokenjessssssssus
      @brokenjessssssssus ปีที่แล้ว

      Серьёзно?! Сам syracuse соизволил посмотреть данное видео?

  • @artyai2272
    @artyai2272 ปีที่แล้ว

    как сделать так чтобы выводы сделанные на ретроспективе исполнялись потом командой?

  • @makar0nin
    @makar0nin ปีที่แล้ว +1

    По части "всё, что можно исправить" - не могу согласиться. Важный момент - прозрачность архитектуры. Я лично главную пользу код ревью вижу в том, что на это посмотрел другой человек и понял что оно делает. Хороший код должен быть написан так, чтоб не было потребности его объяснять. И не всегда возможно код собственного авторства непредвзято оценить по этому критерию.

  • @zentox8480
    @zentox8480 ปีที่แล้ว +2

    Борода всё ещё топ.

  • @winfle
    @winfle ปีที่แล้ว +2

    Дякую, Сергію!

  • @TheTerrapod
    @TheTerrapod ปีที่แล้ว +1

    2:35 в нескольких собесах такая задача ставилась, сделать код ревью, что задача вообще не решена и код джун не запускал, этого не говорится. На первом собесе я провалил его, потом понял в чём соль и на следующее уже оффер дали

    • @kawaikaino5277
      @kawaikaino5277 ปีที่แล้ว

      В чем соль)?

    • @TheTerrapod
      @TheTerrapod ปีที่แล้ว

      @@kawaikaino5277 я написал уже, перечитай

    • @nicholasspezza9449
      @nicholasspezza9449 ปีที่แล้ว

      в том, что он долпоёп, прошел собес там, где попалась задача, которую он уже 10 раз провалил, на 11 раз угадал.

    • @TheTerrapod
      @TheTerrapod ปีที่แล้ว

      @@nicholasspezza9449 2 против 11, фига ты загнул. И почему ты обзываешься? День плохой был?

  • @yuraer
    @yuraer ปีที่แล้ว +2

    Сергей, спасибо за годный контент. Какие требования к коду, к стилистике кода вы бы предъявляли?

  • @johnspireng1960
    @johnspireng1960 ปีที่แล้ว +1

    Спасибо, крутой выпуск - мне очень зашло!

    • @lero4ka_valero4ka_77
      @lero4ka_valero4ka_77 ปีที่แล้ว

      Чем зашло. Это простые истины. Или тут были рассказаны конкретные методики? Бабки на лавке также трепятся, а программеры обсуждают предметно. Все что вода - это не опыт и не знания, а вода как это

    • @johnspireng1960
      @johnspireng1960 ปีที่แล้ว

      @@lero4ka_valero4ka_77 если ты такой прошаренный и недовольный, что ты забыл на этом канале?

  • @vitaliikuzminov2888
    @vitaliikuzminov2888 ปีที่แล้ว +1

    и не страшен нам Chat GPT, пока Вы еще Сергей Немчинский)

  • @PsionicQ
    @PsionicQ ปีที่แล้ว +2

    Блин всё ещё Сергей Нимчинский, буду дальше ждать:(

  • @dmitrygidlevsky9787
    @dmitrygidlevsky9787 ปีที่แล้ว

    Уважаемая борода

  • @seoonlyRU
    @seoonlyRU ปีที่แล้ว

    лукасик от СЕООНЛИ!

  • @acab3146
    @acab3146 ปีที่แล้ว

    Борода топ!

  • @mightybobka
    @mightybobka ปีที่แล้ว +1

    Code review это то что делает из набор разработчиков команду.
    1. Это единственный способ обмена опытом внутри проекта.
    2. Это формирует общую ответственность за кодовую базу, а не то что John Doe у нас отвечает исключительно за платежи, а Jane Doe -- за работу с БД.
    3. В случае отсутствия тестов -- это единственный способ обеспечить качество продукта.
    4. Позволяет определить что задача была действительно сделана, иногда разработчик может сделать только часть задачи и просто забыть о каких-то мелочах, о которых напомнят коллеги.
    5. Даже мелкая проблема замеченная на этапе code review гораздо дешевле в исправлении, чем в релизе.
    и т.д.

  • @nanvlad
    @nanvlad ปีที่แล้ว

    А как можно быть «все ещё» основателем компании?)) этот факт уже никак не изменить (по крайней мере до возможности путешествий во времени)

    • @SergeyNemchinskiy
      @SergeyNemchinskiy  ปีที่แล้ว

      это просто устоявшаяся шутка

  • @ХориЗачем
    @ХориЗачем ปีที่แล้ว +1

    Живу в России жаль, что не могу купить ваш курс ибо сайт заблочен да и что бы вас и страну поддержать

  • @figazzchebupelkin8205
    @figazzchebupelkin8205 ปีที่แล้ว +1

    о, вы еще Сергей Немчинский)

  • @bricksbee-wp
    @bricksbee-wp ปีที่แล้ว

    Когда дождусь приветствия «Все еще бородатый Сергей Немчинский»?

  • @ent3rgm434
    @ent3rgm434 ปีที่แล้ว +1

    Не кажется ли Вам, Сергей, что Тимлиду необязательно писать новый код непосредственно на проекте(ах), ведь, как правило, программисты имеют пет-проекты, либо хотя бы они уделяют определенную часть свободного времени на изучение нового в ИТ и связанными с ним (или текущими/будущими проектами) отраслях?

    • @SergeyNemchinskiy
      @SergeyNemchinskiy  ปีที่แล้ว +1

      не обязательно. но хочется!

    • @Marty-u7o
      @Marty-u7o ปีที่แล้ว

      Вы что)) Как это не писать, писать))

    • @KostsovKonstantin
      @KostsovKonstantin ปีที่แล้ว

      как это потом поддерживать ?

  • @max__am
    @max__am ปีที่แล้ว +1

    зорос!

  • @ПавелК-ю9э
    @ПавелК-ю9э ปีที่แล้ว +1

    Каким образом понимание задачи заказчика делает тебя синьёром?
    Для примера, я работал в сфере заказчика около 20 лет + много друзей с универа кто и сейчас работает в банках и компаниях. Я отлично понимаю, какие задачи ставятся в банках перед командами и какую бизнес логику хочет видеть заказчик. Более того, иногда знаю, что он хочет реализовать в будущем, так как у друга в банке это уже давно пройденный этап.
    Поэтому понимание желаний заказчика на любом уровне не делает из меня синьёр программиста ни разу.

  • @agimgal
    @agimgal ปีที่แล้ว

    Если разработчик пишет один и тот же код... Ну вы поняли.

  • @oeaoo
    @oeaoo ปีที่แล้ว

    Код ревью как и любую практику можно превратить в хлам, своим отношением к проекту и коллегам. Увы.

  • @palkan2590
    @palkan2590 ปีที่แล้ว

    ни одно из твоих предсказаний не исполнилось, вот ты жук

  • @ТимурМ-ц7н
    @ТимурМ-ц7н ปีที่แล้ว +1

    Лайк сразу, в поддержку победы Украины! С уважением с Казахстана.

    • @alexandrapersukova
      @alexandrapersukova ปีที่แล้ว

      Спасибо вам за поддержку))

  • @alexmura3014
    @alexmura3014 ปีที่แล้ว

    Лайк за укр лого

  • @mtrhf8529
    @mtrhf8529 ปีที่แล้ว

    Здравствуйте. У меня к вам программистам "ютуберам" такой вопрос. Если у вас серьёзный уровень в программировании, то почему не идёте работать в крупные айти компании, например, Microsoft (зп 15-30млн руб в год)? Или почему в свое время не создали программу типа whatsapp, TH-cam, всякие андроид игры итд? С вашими то знаниями...

    • @nicholasspezza9449
      @nicholasspezza9449 ปีที่แล้ว

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

    • @Vladimir_Java_dev
      @Vladimir_Java_dev ปีที่แล้ว +1

      Последнее не валидно. Умение писать код не равно умению придумать пользующийся спросом продукт.
      Первое про позиции с мега зп - тут +\- , зависит от приоритетов человека. С другой стороны, помимо зарплат - те позиции предусматривают решение сложных задач, таких, которые не каждый осилит. Значит там возможен рост или поддержание тонуса - это еще один + к привлекательности работы на такой позиции.

  • @blackCitySkyline
    @blackCitySkyline ปีที่แล้ว

    а почему на русском?

  • @MrRomanvideo
    @MrRomanvideo ปีที่แล้ว

    А БА не подразумевается в команде? А груминг тасок тоже нет? Каждый разраб сам додумывает себе аксептанс критерии и дефинишн оф дан? Ну бред же.
    Да и вообще, бред , про то ,что синьорнее тот, кто в бизнес логике разбирается лучше. Видел кучу людей которые у себя на проекте тимлиды, на рынке не может пройти собес на синьора. Это про это! Или наоборот, пришел архитектор новый и любой мидл с годом опыта на проекте по бизнес логике порвет архитектора как тузик грелку. Так что не первый раз вижу , что вы говорите бред.

  • @yutka9139
    @yutka9139 ปีที่แล้ว

    бородач...

  • @denisarbatov6943
    @denisarbatov6943 ปีที่แล้ว

    Тематическая музыкальная композиция th-cam.com/video/GxohQ2bizmQ/w-d-xo.html

  • @lero4ka_valero4ka_77
    @lero4ka_valero4ka_77 ปีที่แล้ว +1

    Мдааааааа...... Джун - он джун и Синьор ему не по зубам априори. Если ждун и найдет хоть что-то в проекте синьора, то это будет капля, которая не стоит и не влияет на производительность или функционал и близко, а найти что-то стоящее чтобы реально стоило переписать вследствие безопасности, производительности, короче говоря качество, то ждун и в жизни там ничего не увидит. В противном случаи он не джун, а мидл. А по вашему это оля голливудский стиль: пришел полупокер и разнес всех к чертям, а все потому что ему нравится быть полупокером, а программирование он занимается для души

    • @drovoseg
      @drovoseg ปีที่แล้ว

      Джун это не стажер, это специалист с некоторым опытом. Хотя баги даже стажер может найти, их ведь все делают, сеньор это не святой.

  • @ПавелК-ю9э
    @ПавелК-ю9э ปีที่แล้ว

    А на счет "удаленной работы на заграницу" - вы вообще считали выгоду от данного мероприятия?
    5-6К евро.....выглядит на первый взгляд привлекательно. А если копнуть?
    1) Начнем с того, что в стране работодателя с Вас возьмут налоги и очень приличную часть от Вашей зарплаты и на руки будет уже совсем другая сумма.
    2) Предположим что-то случилось и Вы оказались в больнице на пару тройку месяцев. Что по оплате? Что с местом Вашим будет?
    3) Никаких пенсионных стажей не идет.
    4) Ещё и своё государство имеет право обложить Ваш доход подоходным налогом (как текущий, так и ранее полученный).
    5) За удаленку платят в 95% случаях меньше, чем за аналогичную работу в офисе.
    Сколько реально от 5-6К получите на руки?
    И вопрос. В чем изюм работы в "европе", если в часе езды от дома можно получать на руки те же самые по факту деньги и быть гораздо более защищенным?

    • @OlexLebed
      @OlexLebed ปีที่แล้ว +2

      зовнішньо економічна діяльність через в Україні через ФОП,
      1) в залежності від країни, є така штука як захист від подвійного оподаткування. Я надсилаю європейській компанії довідку від своєї податокової, що є податковим резидентом України, і за мене не будуть сплачувати податки в іншій країні.
      3) - ФОПи платять ЄСВ в пенсійний фонд, стаж рахується.
      4) - з ФОП системою все просто і прозоро - 5% від прибутку.

    • @UkraineBabuin
      @UkraineBabuin ปีที่แล้ว

      @@OlexLebed так об этом и речь )
      Один только налог в 5% на ФОП полностью нивелирует все потенциальные выгоды "работы на заграницу", не считая остальных плюшек от работы дома.

    • @ПавелК-ю9э
      @ПавелК-ю9э ปีที่แล้ว

      @@OlexLebed
      Можешь написать конкретные суммы? Вот смотри. Учу пока микросервисы. Часть ребят уже закончили и вышли на работу. Стандартная ставка у них от 3К+ долларов. Это ""грязными" + всякие ништяки, типа доп страховки у некоторых, у кого-то чай/кофе и т.д.
      После налогов 2600 примерно будет чистыми.
      + заболел - норм денег получаешь, пока в больнице загораешь.
      Напиши любую европейскую страну и напиши, сколько на руки по факту получишь и что будет, когда в больничку приляжешгь на пару недель.
      Что по правам? Как легко могут пинок под зад дать?
      Просто эти 5-6К подозреваю по факту, меньше чем тут 3К будут.

    • @OlexLebed
      @OlexLebed ปีที่แล้ว +2

      @@ПавелК-ю9э ці всі умови пишуться в контракті. Як домовишся. Це ж не трудовий договір, де тобі всі і все винні. Це вже підприємницька діяльність.

    • @OlexLebed
      @OlexLebed ปีที่แล้ว +1

      @@UkraineBabuin а вам треба, щоб держава доплачувала 5% ?? і ще банки всі при переказах накидували по 1% :-D

  • @СберегательныйБанк
    @СберегательныйБанк ปีที่แล้ว

    а знаете какое требование заказчик предъявляет к коду ? чтобы работал, а с этим у вас проблемы

  • @sashabroslavskiy7904
    @sashabroslavskiy7904 ปีที่แล้ว

    Зачем флаг Украины если спикер говорит на русском?)😂

  • @DrLepros
    @DrLepros ปีที่แล้ว

    Думал я недоджун который мимо VBA проходил, а я похорду неопытный синьер. Ну а чо? Реализовал в WMSке настолько эффективную систему репленишмента насколько возожно с учетом того что Австрийские SAP пацаны не готовы сделать вообще ничего? Дал людям не только понимание как правильно работать, но и инструменты, чтобы все делалось по нажатию одной кнопки, а дальше "магия оно все само"? Занимаюсь постоянным "рефакторингом", чтобы ежедневный континиоюс девелопмент не превращался в 100 состояний селект кейса, а просто передавал данные в класс и отрисовывал динамическую форму, при необходимости? Достаточно ли мне запроса "Мы короче хз. As is - вот так. Хуйня или нормально?" ?
    Во всех случаях ответ "Да". Безнес задача решена сильно лучше чем база международной компании с миллиардными прибылями в отдельно взятой стране. Задачу я воспринимаю и в технических терминах, и в терминах линейного персонала склада, и в терминах имплементации решений на уровне глобальной команды.🤔🤔🤔