Code review. 4

Рет қаралды 2,979

dmdev

Күн бұрын

Пікірлер: 42

@alexandr6055 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@ЕвгенийАлексеев-о9э 4 ай бұрын

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

@dmdev 4 ай бұрын

Очень рад!

@bbrother92 4 ай бұрын

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

@ЕленаРадченко-ч8к 3 ай бұрын

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

@dmdev 3 ай бұрын

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

@ВиталийСавилов-я4б 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@АндрейРождественский-с9щ 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@АндрейАнтонов-у9в 4 ай бұрын

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

@dmdev 4 ай бұрын

@Баходир-м5к 5 күн бұрын

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

@a.asurov3933 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@nikolayveselov7490 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@iswmq420 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@brainattack11 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@Баходир-м5к 4 ай бұрын

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

@dmdev 3 ай бұрын

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

@vasiliybryukhanov3530 4 ай бұрын

🔥

@dmdev 4 ай бұрын

@bbrother92 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@bbrother92 4 ай бұрын

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

@ooshkapooshkin8048 4 ай бұрын

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

@dmdev 4 ай бұрын

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

@ooshkapooshkin8048 4 ай бұрын

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

@RishatKhussainov 3 ай бұрын

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

@dmdev 3 ай бұрын

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

@Баходир-м5к 4 ай бұрын

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

@dmdev 3 ай бұрын

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