Код ревью участника Laravel комьюнити CutCode

  Рет қаралды 4,688

Просто о Laravel. CutCode

Просто о Laravel. CutCode

Күн бұрын

Экспериментальная рубрика CodeReview на CutCode, посвященная разбору репозиториев участников нашего комьюнити в telegram - goo.su/dsGP7PI .
#codereview#laravel#cutcode
---------------------------------------------------------------------------------
🚀📹👨‍🏫 Как насчет прокачки своих навыков с помощью наших обучающих видеокурсов по web-разработке? Переходи на мой сайт 👇
learn.cutcode.dev/?
❗️❗️❗️Присоединяйся к нашему комьюнити в телеграм - там и советом помогут и много интересного - goo.su/dsGP7PI
---------------------------------------------------------------------------------
⏰ Таймкоды:
00:00 Введение
02:02 Composer.json и регистрация
04:14 LinkController
06:55 TypeHint и AdminController
08:12 CodeStyle
10:44 LiveWire и blade компоненты
11:14 Для чего дают задания на собеседования
13:44 Подведение итогов
Всех поклонников laravel я приветствую на канале CutCode! Друзья мы начинаем новую рубрику, пока что она экспериментальная, жду фидбэк от вас в комментариях - нужно ли ее продолжать. Рубрика будет посвящена случайному Code Review. Как вы знаете у нас есть комьюнити в телеграме (t.me/laravel_chat), где есть топик код ревью. Ребята показывают репозитории и те кому интересно и есть время делают ревью. Недавно к нам зашел разработчик который отправил проект на review и сегодня мы его с вами рассмотрим. В нем есть что обсудить.
Для начала предыстория - вместе с репозиторием разработчик написал что проект это тестовое задание для собеседования на должность мидл разработчика с окладом от 2 до 3,5 тысяч долларов. На мой взгляд немало. И автора отправили домой даже не допустив до собеседования, сказав что код нечитаемый. Собственно и вопрос был к нам - ребята подскажите что именно нечитаемое, ведь вроде все гуд. Но изначально я отнесся к процессу иначе, я в целом не понимаю смысл таких тестовых заданий всякий todo листы и так далее. Они скорее нужны чтобы просто отсеять совсем слабых ребят. Но если вы выполняете задание, у вас цель чтобы вас взяли на работу, должность скажем laravel разработчик как в данном случае. Еще и Middle, то значит в задании нужно показать что Laravel вы знаете хорошо и раз middle, значит вы не просто пользователь пакетов, мастер установки зависимостей и дергания методов, а разбираетесь в принципах разработки, используете преимущества ООП, у вас чистые понятный код
---------------------------------------------------------------------------------
📹 делитесь этим видео с друзьями:
• Как установить Laravel...
🔔 подпишитесь на KZfaq-канал: kzfaq.info?s...
📼 Курс по Laravel с нуля:
• Курс по Laravel 8 обуч...
Код ревью участника Laravel комьюнити CutCode
---------------------------------------------------------------------------------
🔗 наш сайт: cutcode.dev/?
📱 Наш telegram-канал: t.me/laravel_cutcode

Пікірлер: 86
@tochivk
@tochivk Жыл бұрын
Хотелось бы, чтобы в следующий раз, вы сразу в видое показывали, как правильно должен выглядеть код. То есть, исправлять его прям в видео.
@daloopdaloop
@daloopdaloop Жыл бұрын
Эта рубрика должна быть и почаще!
@vi_ju
@vi_ju Жыл бұрын
Видео кайф, но мне кажется было бы прикольно, если показывал бы наглядно как нужно было бы написать (не во всех моментах конечно 😅, а то будет очень долго). Ставлю лайк
@user-we5ht9jy4q
@user-we5ht9jy4q Жыл бұрын
абсолютно поддерживаю! а если не лень, то репу в гит, с первым комитом, До и вторым После, прям большое спасибо сказать будет можно =)))
@plutuss
@plutuss Жыл бұрын
Интересное видео. Да, код нужно улучшить. Меня два года назад с таким возмоно не много лучше кодом на джуна брать не хотели. Сейчас уже на мидла люди пытаются. Если автор читает. .... Не сдавайся проходи больше собиседований и будешь видеть слабые моменты. И фикси их ..)))
@Aleksvin8888
@Aleksvin8888 11 ай бұрын
прикольно подивився і дещо нове дізнався прям захтілось одразу побігти і свій код поправити ))))
@YazmyratAtayew
@YazmyratAtayew 7 ай бұрын
полезная ! Обязательно продолжай. И почаще!
@dobrydyadka
@dobrydyadka Жыл бұрын
Бро, спасибо за разбор! Повеселил ))
@user-qb7tv7yi1i
@user-qb7tv7yi1i Жыл бұрын
Очень полезная рубрика! Обязательно продолжай. И почаще!
@turdakhov
@turdakhov Жыл бұрын
Больше таких видосов! Спасибо!
@user-ju9ut9dz3y
@user-ju9ut9dz3y Жыл бұрын
Рубрика огонь! Спасибо! )
@kind1y
@kind1y 11 ай бұрын
Рубрика - пушка! Хотелось бы ещё сразу видеть исправления.
@CutCodeRu
@CutCodeRu 11 ай бұрын
посмотрим на будущее
@fukurou48
@fukurou48 Жыл бұрын
Спасибо, полезно!
@CutCodeRu
@CutCodeRu Жыл бұрын
🔥
@rpy6ocTb
@rpy6ocTb Жыл бұрын
Рубрика отличная, давай на постоянку её =) и как уже писали, было бы хорошо показать, как лучше сделать, но не прям очевидные моменты
@CutCodeRu
@CutCodeRu Жыл бұрын
Снимем и такое
@friendvane7612
@friendvane7612 Жыл бұрын
Отличная рубрика, почаще говорите о лучших практиках)
@CutCodeRu
@CutCodeRu 11 ай бұрын
спасибо!
@pepper-fm
@pepper-fm Жыл бұрын
топ рубрика я бы ещё сослался на roadmap от Повиласа (laravel daily) о том, что нужно знать разрабу на какой уровень
@CutCodeRu
@CutCodeRu 11 ай бұрын
у нас свой Roadmap - cutcode.dev/roadmap . попробуйте!
@Pavelbrov
@Pavelbrov Жыл бұрын
Я за, особенно за то, чтоб показывалось до/после
@CutCodeRu
@CutCodeRu Жыл бұрын
Может после этого ролика, автор исправит ошибки и посмотрим тогда "после"
@Edvard-Aliev
@Edvard-Aliev Жыл бұрын
@@CutCodeRu Нужно от Вас мастер класс! Ждем от Вас! Очень хочется увитился!
@artem-web-developer
@artem-web-developer Жыл бұрын
Интересно)
@CutCodeRu
@CutCodeRu 11 ай бұрын
🔥
@ArabicLang.online
@ArabicLang.online Жыл бұрын
Рубрика супер, однозначно нужна и для начинающих, и для опытных.
@CutCodeRu
@CutCodeRu Жыл бұрын
💪
@iddqd2285
@iddqd2285 Жыл бұрын
Еще не смотрел видос, но это 100% отличная идея.
@CutCodeRu
@CutCodeRu 11 ай бұрын
👍
@asadaa3095
@asadaa3095 Жыл бұрын
Спасибо большое, хотелось бы смотреть и на сенёров тоже
@user-gp5yd3sw3w
@user-gp5yd3sw3w Жыл бұрын
Тема интересная, но ожидал в конце ролика исправленные куски кода, как было на старых видео этого канала 😅
@CutCodeRu
@CutCodeRu Жыл бұрын
Это другая рубрика)
@user-gp5yd3sw3w
@user-gp5yd3sw3w Жыл бұрын
@@CutCodeRu всё же, было бы интересно посмотреть на исправленный вариант кода в таких случаях)
@CutCodeRu
@CutCodeRu Жыл бұрын
@@user-gp5yd3sw3w думаю плохой, хороший код еще обязательно выйдет
@bulletproof1304
@bulletproof1304 Жыл бұрын
Выглядит как Джун решил пройти собеседование на мидла. Ну или тестовое сделано на отье***. Понатыкано пакетов, стандартных методов из коробки, многие из которых вообще не используются. Как сказал автор канала делал задание не разработчик, а пользователь пакетов ларавель. А подписчику желаю удачи в развитии, у всех были и будут ошибки.
@user-nf1td4hh7y
@user-nf1td4hh7y Жыл бұрын
Круто, спасибо большое! Продолжайте пожалуйста) понимаю, куда расти после такого видео 🙏
@molxs5339
@molxs5339 9 ай бұрын
Капец, я еще даже не джун, но про такие вещи как DI знаю и активно использую. А тут целый мидл жесть….
@soversus5374
@soversus5374 3 ай бұрын
Когда диалог строится: лучше было бы так... Не понятно, кому лучше, чем лучше... и почему бы и нет. )
@gamingplay5415
@gamingplay5415 Жыл бұрын
это нужно показыват) какие никакие подсказки можно подсказывать😊
@user-hy3bb4mn8o
@user-hy3bb4mn8o Жыл бұрын
Интересно! В группе задавали как раз задавали вопрос, где бы найти такую услугу как код ревью от сеньора
@CutCodeRu
@CutCodeRu 11 ай бұрын
думаете, будет востребовано?
@user-hy3bb4mn8o
@user-hy3bb4mn8o 11 ай бұрын
@@CutCodeRu не знаю насчет востребованности, но интерес то есть
@user-po3id7ee7n
@user-po3id7ee7n Жыл бұрын
Всё по делу, но очень сильно не согласен на 9.52. Я даже не представляю такую ситуацию при которой на рабочем проекте в продакшене меняют id uint64 на uuid string. Обычно uuid добавляют надстройкой сверху, например, для привязки к внешним системам, а внутри продолжают использовать id. Да и id() это int|string что плохо для проекта в части типизации. Благо если автотесты подскажут проблему заранее или стат анализатор.
@CutCodeRu
@CutCodeRu 11 ай бұрын
спасибо за комментарий
@Amedomaroful
@Amedomaroful Жыл бұрын
Я за, побольше таких видео
@CutCodeRu
@CutCodeRu Жыл бұрын
Спасибо за фидбек
@modestusmunh93
@modestusmunh93 Жыл бұрын
Рубрика нужна!
@CutCodeRu
@CutCodeRu 11 ай бұрын
💯
@syracuse4612
@syracuse4612 6 ай бұрын
Правильно понял это на мидла ТЗ?
@syracuse4612
@syracuse4612 6 ай бұрын
сейчас копайлот в миллион раз чище напишет, какой там джун
@areaweb
@areaweb Жыл бұрын
Согласен с Данилом, код прям плохой, нужно тренироваться ))
@pavelhol9652
@pavelhol9652 Жыл бұрын
Тренируются спортсмены, разрабы - учатся, потому как программирование, это область знаний
@CutCodeRu
@CutCodeRu 11 ай бұрын
💯
@snikitin32
@snikitin32 4 ай бұрын
Рассуждать про SOLID в контроллерах любого фреймворка как минимум глупо) Любой контроллер уже нарушает первый принцип solid)))
@turdakhov
@turdakhov Жыл бұрын
Обзор с рефакторингом не на словах был бы еще круче. Спасибо!
@modestusmunh93
@modestusmunh93 Жыл бұрын
Согласен! Весь проект рефакторить не нужно, но пару-тройку методов/контроллеров можно было бы коснуться.
@CutCodeRu
@CutCodeRu 11 ай бұрын
учту!
@fil-d1d
@fil-d1d Жыл бұрын
IDE спалил имя автора 😂
@CutCodeRu
@CutCodeRu 11 ай бұрын
🙂
@iddqd2285
@iddqd2285 Жыл бұрын
Слабый код, не понимаю на что человек надеялся, сразу видно что у него нет опыта, надо с годик на джуне посидеть на нормальном проекте и будет щастье. Очень хочется на ревью увидеть что-то более
@soversus5374
@soversus5374 3 ай бұрын
Нормально там все... вы придираетесь. ) Клиенту плевать, главное, что бы работало. Клиента интересует монетизация, а не где у вас там скобка и тайпхинт.
@vitaercx
@vitaercx Жыл бұрын
Не возьмусь защищать автора кода, но и рекомендации и комментарии автора обзора не поддерживаю. Критика очевидная, использовать solid принципы, не использовать пакеты, форматировать код. Заученные слова, которые мог дать любой. Особенно, где-то деструктивная даже идея - что главное, показать себя, и чуть ли не писать "велосипед" для этого, а не пользоваться пакетами. Неопытный ум послушав это может воспринять это буквально и понтоваться там, где можно написать просто и элегантно. При всей критике внешних пакетов, ваш канал пестрит заголовками "обзор пакета". Да и ваша админка это тоже пакет, который вы предлагаете к использованию. Т.е. расходится то, что вы даете и то за что в итоге критикуете. Пакеты это инструменты, и если бы автор кода показал грамотное ими пользование, показал то, что бизнес может в разы сократить время разработки и эффективно использовать время, это было бы аргументом и пусть использует этот инструмент, чем писать свой тратя лишнее время. Не могу также согласиться, что автор кода показал себя как хороший пользователь Laravel. Потому что то, как он использует инструменты фреймворка так их использовать не рекомендуется.
@CutCodeRu
@CutCodeRu Жыл бұрын
Если бы да кабы, но автор не показал себя не как пользователь пакетов, не как изобретатель велосипедов! Благодарю за комментарий
@user-pg7ri5jg6p
@user-pg7ri5jg6p Жыл бұрын
Я стажер/джун не уверен что лучше написал но не которые моменты даже я как стажер/ джун лучше написал бы
@CutCodeRu
@CutCodeRu 11 ай бұрын
👍
@modestusmunh93
@modestusmunh93 Жыл бұрын
Итальянская кухня хахаах
@TsA1ex
@TsA1ex Жыл бұрын
Увидел валидацию в контроллере (вместо форм реквеста) и работу с моделями (вместо сервисов) и подумал что тут максимум Junior+ с натяжкой P.S. Нестрогое сравнение дальше. Погорячился с оценкой. Рекомендую начать с курсов основ (желательно не Лаврика) и документации
@egorow4innikow565
@egorow4innikow565 Жыл бұрын
Не функционал, а функциональность. Хех kzfaq.info/get/bejne/osqffrGem9eshqs.html
@GreatOsmanSultan
@GreatOsmanSultan Жыл бұрын
Почаще такую критику с обзором🎉🎉🎉🎉
@CutCodeRu
@CutCodeRu 11 ай бұрын
по возможности буду выпускать!
@gian_tiaga
@gian_tiaga Жыл бұрын
Это код даже не джуна.)) Я бы даже в стажеры не взял. Без обид автору кода
@CutCodeRu
@CutCodeRu 11 ай бұрын
у него все впереди
@soversus5374
@soversus5374 3 ай бұрын
Это пока вы не пошли на собеседование... )
@Diqeeeeeeeeeei
@Diqeeeeeeeeeei Жыл бұрын
Нам нужно больше ревью
@CutCodeRu
@CutCodeRu 11 ай бұрын
буду выпускать
@itdev4680
@itdev4680 Жыл бұрын
3500 не мало но и не много, но тут момент что это код близко не мидла. Джун, даже не аппер
@enmaboya
@enmaboya Жыл бұрын
да даже не джун, тут слишком много косяков
@user-ct4tl3yp1z
@user-ct4tl3yp1z Жыл бұрын
@@enmaboya а что вы хотите от джунов тогда? Чтоб сразу все знал и писал чисто? так это не джун, а сеньер.
@enmaboya
@enmaboya Жыл бұрын
@@user-ct4tl3yp1z чтобы он умел читать документацию хотя бы. то что в видео - уровень человека который просто копирует решения из интернета, при этом не особо вникая что именно он скопировал и зачем, но с этого в принципе все начинали, так что ничего страшного не вижу, просто непонятно зачем было собеситься на мидла
@CutCodeRu
@CutCodeRu 11 ай бұрын
👨‍🎓
@soversus5374
@soversus5374 3 ай бұрын
@@enmaboya , копировать решение из интернета, тоже уметь надо и далеко не всем дано. ) Даже велосипед изобрести не каждый способен.
@itdev4680
@itdev4680 Жыл бұрын
Не стоит светить имя его
@CutCodeRu
@CutCodeRu Жыл бұрын
Автор дал добро на запись видео и я не вижу ничего плохого, человек в процессе обучения
@zCubEz
@zCubEz Жыл бұрын
Эта рубрика должна быть и почаще!
@CutCodeRu
@CutCodeRu 11 ай бұрын
👌
Как бесплатно замутить iphone 15 pro max
00:59
ЖЕЛЕЗНЫЙ КОРОЛЬ
Рет қаралды 8 МЛН
How Many Balloons Does It Take To Fly?
00:18
MrBeast
Рет қаралды 207 МЛН
Викторина от МАМЫ 🆘 | WICSUR #shorts
00:58
Бискас
Рет қаралды 5 МЛН
Один день из жизни разработчика
5:42
Просто о Laravel. CutCode
Рет қаралды 1,7 М.
Чего ожидать от HTTP/3 + Go
51:07
Нина Пакшина
Рет қаралды 9 М.
Laravel solved race conditions
14:13
Aaron Francis
Рет қаралды 15 М.
Laravel Events / Observers
15:23
Просто о Laravel. CutCode
Рет қаралды 15 М.
JIT в PHP 8: Просто о сложном
19:26
Andrey Lozunov - Fullstack Web Developer
Рет қаралды 14 М.
Laravel Policies. Как организованно проверять права пользователей
30:33
Lectoria. Обучение веб-разработке.
Рет қаралды 9 М.
Как бесплатно замутить iphone 15 pro max
00:59
ЖЕЛЕЗНЫЙ КОРОЛЬ
Рет қаралды 8 МЛН