Code review. 4

แชร์
ฝัง
  • เผยแพร่เมื่อ 8 ก.พ. 2025

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

  • @alexandr6055
    @alexandr6055 9 หลายเดือนก่อน +8

    Первый раз смотрю такой формат. По моему крутая идея!

    • @dmdev
      @dmdev  9 หลายเดือนก่อน +1

      Это радует меня! Спасибо

  • @ВиталийСавилов-я4б
    @ВиталийСавилов-я4б 9 หลายเดือนก่อน +6

    Крутой формат, очень познавательно. Спасибо, будем ждать еще подобный контент.

    • @dmdev
      @dmdev  9 หลายเดือนก่อน +1

      Супер! Значит будет)

  • @ЕвгенийАлексеев-о9э
    @ЕвгенийАлексеев-о9э 9 หลายเดือนก่อน +4

    Очень годные замечания. Открыл для себя много нового

    • @dmdev
      @dmdev  8 หลายเดือนก่อน

      Очень рад!

    • @bbrother92
      @bbrother92 8 หลายเดือนก่อน

      @@dmdev Спасибо за ваши видео. Подписался, Могу спросить офтоп? - Когда у меня сложная настройка перед тестом (база данных, клиент Kafka и т.д.), где следует хранить все эти переменные состояния и информацию о конфигурации?
      И где это все лучше сетапить? Просто в полях класса и в @Before сетапить?

  • @АндрейРождественский-с9щ
    @АндрейРождественский-с9щ 9 หลายเดือนก่อน +3

    Очень круто. Подумай над идеей лайв кодинга какого нибудь не маленького проекта, не упираясь в обучение, а больше в бест практисы, рассуждение, чистый код и т.д.

    • @dmdev
      @dmdev  9 หลายเดือนก่อน

      Спасибо, обдумаю)

  • @a.asurov3933
    @a.asurov3933 9 หลายเดือนก่อน +1

    Спасибо большущеее)) Крутой формат и огроменное спасибо от команды Anvil Code за это ревью!

    • @dmdev
      @dmdev  9 หลายเดือนก่อน

      Очень рад, что смог помочь!

  • @ЕленаРадченко-ч8к
    @ЕленаРадченко-ч8к 7 หลายเดือนก่อน +1

    Большое спасибо за видео, а отдельное спасибо за файл!

    • @dmdev
      @dmdev  7 หลายเดือนก่อน

      Всегда пожалуйста!

  • @iswmq420
    @iswmq420 9 หลายเดือนก่อน +1

    Спасибо за такую работу!

    • @dmdev
      @dmdev  9 หลายเดือนก่อน

      Всегда пожалуйста

  • @Баходир-м5к
    @Баходир-м5к 4 หลายเดือนก่อน

    Было бы хорошо если было курс создание полноценного проекта с нуля с использованием всех технологии в roadmap😊😊

  • @java_one_love
    @java_one_love 9 หลายเดือนก่อน +4

    Мощь, спасибо🔥

    • @dmdev
      @dmdev  9 หลายเดือนก่อน

  • @nikolayveselov7490
    @nikolayveselov7490 9 หลายเดือนก่อน +1

    Спасибо, огромная работа!

    • @dmdev
      @dmdev  9 หลายเดือนก่อน

      Всегда пожалуйста

  • @brainattack11
    @brainattack11 9 หลายเดือนก่อน +1

    Спасибо большое!

    • @dmdev
      @dmdev  9 หลายเดือนก่อน

      Всегда пожалуйста!

  • @RishatKhussainov
    @RishatKhussainov 8 หลายเดือนก่อน

    Замечательное ревью, спасибо! Подскажите пожалуйста, на 44:10 и на 21:52 упоминается объединение нескольких endpoint'ов в controller'e в один общий endpoint по причине разных параметров фильтрации, однако PreAuthorize условия у endpoint'ов отличаются.
    Значит логику придется писать вручную, чтоб делегировать в другие сервисные методы с нужными PreAuthorize в зависимости от параметров фильтрации. Вышеописанную логику придется покрывать тестами, да и больше кода - больше ошибок. Если оставить все как есть в controller'ах, то писать вручную ничего не придется, ведь вызов нужного endpoint'a spring берет на себя в зависимости от path, что я бы и предпочел. Какие еще есть альтернативы?

    • @dmdev
      @dmdev  8 หลายเดือนก่อน +1

      В том-то и дело, что не может такого быть - это ошибка разного механизма проверки прав доступа. Доступ идет на РЕСУРС. И не зависит от параметров поиска!
      Например, идет получение ресурса по user_id и проверяется, что аутентифицированному пользователю доступен этот ресурс этого user_id.
      Но если тот же самый пользователь делает поиск того же самого ресурса по номеру телефона phone - то уже такой проверки нет. Что странно и легко допустить ошибку.

  • @bbrother92
    @bbrother92 8 หลายเดือนก่อน +1

    @dmdev Спасибо за ваши видео. Подписался, Могу спросить офтоп? - Когда у меня сложная настройка перед тестом (база данных, клиент Kafka и т.д.), где следует хранить все эти переменные состояния и информацию о конфигурации?
    И где это все лучше сетапить? Просто в полях класса и в @Before сетапить? Еще слышал это плохая практика юзать DI в юнит тестах

    • @dmdev
      @dmdev  8 หลายเดือนก่อน

      Это все я рассказываю в курсе Spring.

    • @bbrother92
      @bbrother92 8 หลายเดือนก่อน

      @@dmdev я тестер) . пжлст подскажите что-то

  • @Баходир-м5к
    @Баходир-м5к 8 หลายเดือนก่อน +1

    Здравствуйте, посоветуйте англоязычный курс по вашему Roadmap. Смотрю ваши курсы они очень класные❤❤.

    • @dmdev
      @dmdev  8 หลายเดือนก่อน

      К сожалению, не знаю таких англоязычных курсов :(

  • @vasiliybryukhanov3530
    @vasiliybryukhanov3530 9 หลายเดือนก่อน +1

    🔥

    • @dmdev
      @dmdev  9 หลายเดือนก่อน

  • @ooshkapooshkin8048
    @ooshkapooshkin8048 8 หลายเดือนก่อน +1

    Когда выйдет курс 𝐑𝐞𝐚𝐜𝐭𝐢𝐯𝐞 𝐒𝐩𝐫𝐢𝐧𝐠 / 𝐊𝐚𝐟𝐤𝐚?)

    • @dmdev
      @dmdev  8 หลายเดือนก่อน

      когда-нибудь точно выйдет. Но точных сроков нет - это ведь мой досуг в свободное от работы время)

    • @ooshkapooshkin8048
      @ooshkapooshkin8048 8 หลายเดือนก่อน

      @@dmdev дай те бог здоровья, я пересмотрел очень много курсов по спрингу, только начал изучать его, ты самый серьезный специалист в этом, мое уважение и почтение :)
      может быть как идея, чтобы руку набить на лучших практиках, порешай алгоритмы на джаве :)

  • @Баходир-м5к
    @Баходир-м5к 8 หลายเดือนก่อน

    Здравствуйте, у вас есть курсы на англ или посоветуйте англ курс похоже в формате вашых пж

    • @dmdev
      @dmdev  8 หลายเดือนก่อน

      К сожалению, не знаю таких англоязычных курсов. А мои не переведены :(

  • @AlekseySergeev278
    @AlekseySergeev278 9 หลายเดือนก่อน

    А это была первая или вторая ступень менторства?

    • @dmdev
      @dmdev  8 หลายเดือนก่อน

      Это не менторство вообще. Просто проект он рандомных ребят с dmdev talks

  • @alexandrmotorin5842
    @alexandrmotorin5842 9 หลายเดือนก่อน +1

    Очень круто!

    • @dmdev
      @dmdev  9 หลายเดือนก่อน

      Спасибо!