Интересная задача с собеседования на Code Review

Рет қаралды 3,179

Күн бұрын

Пікірлер: 47

@jstFreak.. 2 ай бұрын

*Спасибо за видео за контент вообщем.* Я хотел узнать какие паттерны проектирования ты используешь, может ты расскажешь про методы solid или про архитектуру в следующем видео?

@ababavababa5914 4 ай бұрын

Уже ставлю автоматом лайк, респект и фарту масти

@pvt_joker6846 4 ай бұрын

Вячик красава, всё по фактам разложил!

@rianoneq 4 ай бұрын

Братан, хорош, давай, давай, вперёд! Контент в кайф, можно ещё? Вообще красавчик! Можно вот этого вот почаще? (напиши такой коммент все что в скобках сотри 50 рублей накину)

@face_control14 4 ай бұрын

Вери гуд, давно роликов не было. А теперь домашнее задание...

@michaelovechkin9865 4 ай бұрын

-сынок, закругляйся -но это новое видео Славы -топовый контент, разрешаю

@hwamtilan3738 4 ай бұрын

Бро, спасибо за такой суперский контент

@alexeyushakov6342 4 ай бұрын

Давай вот этого побольше.

@astero3133 Ай бұрын

Надеюсь автор продолжит выпускать видео

@endlinkkk3951 4 ай бұрын

Топ контент, лайк) Мои вопросы: во втором методе надо is None и тогда не понятно как мы передаем в ошибку имя и пароль если у нас кроме токена юзера ничего нет (юзера мы же не нашли). И разве норм в ошибку передавать пароль? прост я где-то слышал что это плохая практика

@programming_etc 4 ай бұрын

Да, брат ты полностью прав здесь. При том я сказал то правильно, а исправить забыл, короче очепятка вышла. То что пароль в ошибку передаётся тоже сомнительно, сыглы.

@zalimonpshigo 4 ай бұрын

Лучший!

@karenuuh 4 ай бұрын

легенда спасибо

@АндрейАлексеев-х3д Ай бұрын

Этот лайвкодинг точно на мидла? как будто бы на джуна

@wtfisakilometerrrrrr 4 ай бұрын

добро пошло, большо лайф кодинга

@denisnikanorov225 4 ай бұрын

Интересно про какую компанию речь в начале)

@programming_etc 4 ай бұрын

А кстати у вас тестовое ж вроде было ревью

@bat0ri- 4 ай бұрын

user is None и эксепшн кидаем забыл убрать not))

@aprilrush2172 2 ай бұрын

Это так задумано, байт на комменты😂

@edis671games 4 ай бұрын

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

@WityazKajit 4 ай бұрын

В функции authenticate лучше реквест тоже вынести и принимать только токен, разве не?

@programming_etc 4 ай бұрын

Абсолютно верно

@Chel1k7 4 ай бұрын

Обосрался на 13 минуте и не убрал лишний not в проверке на наличие юзера

@programming_etc 4 ай бұрын

Должно быть use is None, тут ды

@НиколайРюмин-й6и 4 ай бұрын

А второй реквест чего не убрал то?

@programming_etc 4 ай бұрын

А я чё, за вас всё должен делать? Ну не, ребятки, дальше сами

@NPoff4957 4 ай бұрын

Мафаня ушел в программирование?

@Timur_Kan 4 ай бұрын

Пароль нужно было посолить. Если база утечет - соль не даст подобрать пароли по таблицам. Сервису это бы добавило безопасности, а собеседуемому плюсов

@programming_etc 4 ай бұрын

@@Timur_Kan сто проц

@RexerNotes 3 ай бұрын

Кидать UserNotFoundError с паролем открытым внутри очень опасно, так делать не надо

@Chanel_for_CODE 4 ай бұрын

Смотря на чужой код, не забывай писать свой)

@dmitriynaumov1990 4 ай бұрын

Дело в том, что иногда чужой код оказывается говнокодом.

@antonidassss 4 ай бұрын

ты не убрал импорт md5, так что переделывай видос (сделаем вид, что я сам заметил, а не ты сказал про это в конце)

@tinylith1603 4 ай бұрын

А зачем мы вообще позволяем пользователям иметь одинаковые имена?

@tinylith1603 4 ай бұрын

Ваще ничо не понял чо в коде происходит короче, если про джанго говорить, я не шарю за него, но погуглив нашёл, что там есть функция authenticate, которая вроде как должна на SQL инъекции проверять, а тут в коде User.objects какой-то, откуда оно берётся-то? o_O

@programming_etc 4 ай бұрын

@@tinylith1603 там не одинаковые имена будут у пользоватеплей, а токены генерились всегда одинаковые, и по этому их легче было бы ломануть

@mierce 4 ай бұрын

а я что...а я на jvm языках сижу

@programming_etc 4 ай бұрын

сигма

@RyanovskY 4 ай бұрын

кринж, чел делает условие, если пользователь существует, то отправить ошибку UserNotFoundError. Если если исправить ошибку в коде, то зачем писать "if user is not None" если достаточно "if user" 🤨

@programming_etc 4 ай бұрын

Проверки разные могут быть, например та же самая пустая строка и None. это разные вещи, по этому я привык проверять явно.

@programming_etc 4 ай бұрын

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

@RyanovskY 4 ай бұрын

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

@programming_etc 4 ай бұрын

@@RyanovskY показательно могу тебе закон Ома напомнить, атсасын

@RyanovskY 4 ай бұрын

@@programming_etc хахахах