Публичное Code Review Go-проекта

Поділитися
Вставка
  • Опубліковано 1 січ 2025

КОМЕНТАРІ • 44

  • @nikolay_tuzov
    @nikolay_tuzov  Рік тому +3

    👾Если хотите узнавать о наших активностях первыми, подписывайтесь на мой Телеграм-канал: t.me/ntuzov

  • @nskforward
    @nskforward Рік тому +23

    Формат очень зашел! Пожалуйста, побольше таких выпусков. Глебу отдельный респект, очень приятно его слушать. Давиду совет не обижаться, потому что только так он может сделать резкий рывок в своём развитии. На удивление, «напористого» Евгения мне тоже интересно было выслушать, он говорил аргументированные вещи. Раз такой резонанс с этим Евгением в комментариях, прошу следующее онлайн ревью кода сделать ему. Николай, можно отыскать этого Евгения? Народ, поддержите лайком!

    • @nikolay_tuzov
      @nikolay_tuzov  Рік тому +1

      А чего его искать, он в нашем чате регулярно общается =)

  • @АртемФомин-э6и
    @АртемФомин-э6и Рік тому +10

    Формат - огонь! Надеюсь, будут еще подобные созвоны

  • @ziliboba73
    @ziliboba73 Рік тому +14

    Спасибо, услышал несколко полезных вещей для себя. На будущее конечно хотелось бы чтобы разрешение картинки с кодом было получше.

  • @georgiy_kulagin
    @georgiy_kulagin Рік тому +7

    Ребят, а закиньте плез ссылки на статьи, о которых говорил Глеб на 18:45, связанные аппаратной работой процессора и особенностях работы языка с ней.

  • @roudder6606
    @roudder6606 Рік тому +17

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

    • @TheDavBag
      @TheDavBag Рік тому +3

      у всех разная подача =) возможно Женя больше всех переживал за либу

    • @Levelord92
      @Levelord92 Рік тому +3

      он вообще какой-то зарвавшийся, я бы не хотел к такому индивиду в команду попасть

    • @calculaction
      @calculaction Рік тому

      И Глеб и Евгений дали одинаково полезные советы, как мне видится. У Евгения просто подача более эмоциональная и конкретная, а Глеб более сдержанный и спокойный - вот и вся разница (имхо)

  • @alexsibrin715
    @alexsibrin715 Рік тому

    Мистер Глеб, большое спасибо за Ваше грамотное объяснение, очень круто, понятно и интересно! Давид, спасибо за смелость! Не наелся дайте ещё! А то бунт!

  • @invisibleinvisible83
    @invisibleinvisible83 Рік тому

    Спасибо, ребят формат супер. Всем участникам респектище!!!! Николай, отдельное спасибо. Процветания каналу 🙏🏻🌹

  • @Levelord92
    @Levelord92 Рік тому +10

    Николай в начале упомянул, что критика была вежливая, и так и было, пока не пошла секция "комментарий от Евгения (1)"
    женёк, унылый ты душнила, поумерь свой пыл, и поучись такту у своих коллег по цеху

  • @cycymara
    @cycymara 4 місяці тому

    Даааа, поддушивает, но зато качественный код, это круто! Давайте больше хардкода :D

  • @decay9984
    @decay9984 Рік тому

    Спасибо, ценная информация

  • @artemistomin5035
    @artemistomin5035 Рік тому +3

    Евгений явно не прошел собес по софт скилам)

  • @Shindos-Kopernik
    @Shindos-Kopernik Рік тому

    Отличное видео!
    Познавательный контент, разбор проекта под видением нескольких ревьюверов!
    Каждый со своей позиции смотрел и это замечательно! Давиду респект и удачи!

  • @eugeniyyar860
    @eugeniyyar860 Рік тому

    Golang не знаю но приятно слушать.

  • @robertm.4443
    @robertm.4443 3 місяці тому

    Евгений это 0% пользы, 100% токсичности

  • @tumenit
    @tumenit Рік тому

    Спасибо!
    Насчет безусловного поднятия логов вверх, спорно, я бы от ситуации отталкивался, иногда и по месту логировать можно.
    Насчет отказа от прослушивания ctx.Done() в пользу обработки ctx.Err() , это будет работать только если мы не блокруемся в цикле на чтении из канала с данными, а если блокируемся, то при закрытии контекста мы не дойдем до проверки ctx.Err() или я все неправильно понял )

  • @РустамНафиков-ж5з

    Спасибо за видео! А какого уровня код в этом пакете? Просто я джуниор разработчик, работающий. И я вот думаю, это нормально, что мне очень тяжело дается понимание сути работы пакета и того, для чего он вообще нужен? Может я просто не сталкивался с задачами такого рода

    • @АртемВирский
      @АртемВирский Рік тому

      Это нормально. Проект, как я понял, для шустрого обмена сообщениями между сервисами в приложении с микросервисной архитектурой.
      Для ждуна, даже если смысл этой штуки понятен, на практике не скоро придется столкнуться.

  • @Smerrrtnik
    @Smerrrtnik Рік тому +1

    Глеб, очень круто, чы находите время на помощь другим.
    Было бы отлично, если бы вы нашли время совместно с команда разработки WT пофиксить баги.
    А то в чужом глазу соломинку увидишь...

  • @МаркЧолак
    @МаркЧолак 11 місяців тому

    58:13 - с такой аргументацией и контекст нельзя тащить в большинство функций :)
    Ну это так, набросить.

  • @yodude2493
    @yodude2493 Рік тому +13

    какой же душный этот Евгений, подключился просто повыпендриваться, испортил эфир

    • @AVIL13
      @AVIL13 Рік тому +4

      Самое забавное, что ничего полезного не сказал.
      Попытался набросить, предложил: "усложни какими нибудь алгоритмами, хз какими, он не в курсе" сказал что пиши код лучше он бы не заапрувил, и смылся.
      Очень жалко людей кто работает вместе с такими персонажами.
      Обратите внимание на тон, как с высока общается с Давидом и как заискивающе с Глебом.

    • @СергейИванов-ы7ч5ы
      @СергейИванов-ы7ч5ы Рік тому +3

      @@AVIL13 Привыкай. Если нужно сделать десятки ревью за раз, то на политесы времени нет

    • @AVIL13
      @AVIL13 Рік тому +7

      @@СергейИванов-ы7ч5ы на самом деле это не так.
      Просто есть определенные правила, о которых надо договориться.
      К примеру нет ни какой пользы от ревью вроде: "мне не нравится, переделывай".
      Во первых нужно описать что именно не устраивает и как ты это видишь. Иначе все твои комментарии будут в этом духе. Поэтому они сразу все бессмысленны.
      И нужно стараться в ревью давить не на эмоции или авторитет, а на смысл и бизнес-процессы. Тогда 10 ревью в день, это норма, а не что то героическое.

    • @СергейИванов-ы7ч5ы
      @СергейИванов-ы7ч5ы Рік тому

      @@AVIL13 Я не писал, что не нужно объяснять, а только про то, что не нужны лишние обидки, тк многие программисты - это аутисты с трудностями в коммуникации, увы

    • @robertm.4443
      @robertm.4443 3 місяці тому

      @@СергейИванов-ы7ч5ы Глебу это не мешает, а ревью он провел явно побольше

  • @rumartru
    @rumartru Рік тому

    Спасибо! Отличный формат, но разрешение видео...

  • @maxdzh212
    @maxdzh212 Рік тому

    Закрывать канал должен тот кто в него пишет, это самая надежная схема. И это позволит читателю после закрытия контекста прочесть все оставшееся из канала простым range

  • @СергейИванов-ы7ч5ы

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

    • @СергейИванов-ы7ч5ы
      @СергейИванов-ы7ч5ы Рік тому

      ЗЫ Мысли на лестнице. С одной стороны, го урезанный до минимума, имеет минимальный порог вхождения, но я бы не рекомендовал его для начинающих, тк язык это только лопата, а нужно прежде всего знать, как копать. А для него нет нормальных фреймворков, литературы и примеров правильных подходов

    • @s1ovac308
      @s1ovac308 Рік тому +2

      Ой, чувствую пылающий зад джависта :)))
      Где вам не хватает литературы по нашему языку? Спросите, могу предоставить

    • @СергейИванов-ы7ч5ы
      @СергейИванов-ы7ч5ы Рік тому

      @@s1ovac308
      Ты лучше сразу напиши, что он из себя представляет и какие могут быть реализации, посмеемся над твоим задом

    • @СергейИванов-ы7ч5ы
      @СергейИванов-ы7ч5ы Рік тому

      @@s1ovac308 Стесняюсь спросить, какие претензии у тебя к унылой java, кроме того, что вы не понимаете, что под капотом и слишком много разных вариантов решений одного и того же, которые без правильного понимания приводят к кривым решениям?

    • @Levelord92
      @Levelord92 Рік тому

      @@s1ovac308 можешь мне предоставить пару тайтлов литературы) Мне не докопаться, а искренне нужно проверенную книгу почитать