Код ревью участника Laravel комьюнити CutCode
HTML-код
- Опубликовано: 3 авг 2024
- Экспериментальная рубрика 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...
🔔 подпишитесь на RUclips-канал: ruclips.net/user/CutCodeRu?s...
📼 Курс по Laravel с нуля:
• Курс по Laravel 8 обуч...
Код ревью участника Laravel комьюнити CutCode
---------------------------------------------------------------------------------
🔗 наш сайт: cutcode.dev/?
📱 Наш telegram-канал: t.me/laravel_cutcode
Хотелось бы, чтобы в следующий раз, вы сразу в видое показывали, как правильно должен выглядеть код. То есть, исправлять его прям в видео.
Эта рубрика должна быть и почаще!
Видео кайф, но мне кажется было бы прикольно, если показывал бы наглядно как нужно было бы написать (не во всех моментах конечно 😅, а то будет очень долго). Ставлю лайк
абсолютно поддерживаю!
а если не лень, то репу в гит, с первым комитом, До и вторым После, прям большое спасибо сказать будет можно =)))
прикольно подивився і дещо нове дізнався прям захтілось одразу побігти і свій код поправити ))))
полезная ! Обязательно продолжай. И почаще!
Интересное видео. Да, код нужно улучшить. Меня два года назад с таким возмоно не много лучше кодом на джуна брать не хотели. Сейчас уже на мидла люди пытаются. Если автор читает. .... Не сдавайся проходи больше собиседований и будешь видеть слабые моменты. И фикси их ..)))
Очень полезная рубрика! Обязательно продолжай. И почаще!
топ рубрика
я бы ещё сослался на roadmap от Повиласа (laravel daily) о том, что нужно знать разрабу на какой уровень
у нас свой Roadmap - cutcode.dev/roadmap . попробуйте!
Рубрика отличная, давай на постоянку её =) и как уже писали, было бы хорошо показать, как лучше сделать, но не прям очевидные моменты
Снимем и такое
Бро, спасибо за разбор! Повеселил ))
Рубрика - пушка! Хотелось бы ещё сразу видеть исправления.
посмотрим на будущее
Интересно)
🔥
Спасибо, полезно!
🔥
Я за, особенно за то, чтоб показывалось до/после
Может после этого ролика, автор исправит ошибки и посмотрим тогда "после"
@@CutCodeRu Нужно от Вас мастер класс! Ждем от Вас! Очень хочется увитился!
Больше таких видосов! Спасибо!
Рубрика огонь! Спасибо! )
Тема интересная, но ожидал в конце ролика исправленные куски кода, как было на старых видео этого канала 😅
Это другая рубрика)
@@CutCodeRu всё же, было бы интересно посмотреть на исправленный вариант кода в таких случаях)
@@user-gp5yd3sw3w думаю плохой, хороший код еще обязательно выйдет
Выглядит как Джун решил пройти собеседование на мидла. Ну или тестовое сделано на отье***. Понатыкано пакетов, стандартных методов из коробки, многие из которых вообще не используются. Как сказал автор канала делал задание не разработчик, а пользователь пакетов ларавель. А подписчику желаю удачи в развитии, у всех были и будут ошибки.
Отличная рубрика, почаще говорите о лучших практиках)
спасибо!
Рубрика супер, однозначно нужна и для начинающих, и для опытных.
💪
Спасибо большое, хотелось бы смотреть и на сенёров тоже
Еще не смотрел видос, но это 100% отличная идея.
👍
Капец, я еще даже не джун, но про такие вещи как DI знаю и активно использую. А тут целый мидл жесть….
Когда диалог строится: лучше было бы так...
Не понятно, кому лучше, чем лучше... и почему бы и нет. )
Интересно! В группе задавали как раз задавали вопрос, где бы найти такую услугу как код ревью от сеньора
думаете, будет востребовано?
@@CutCodeRu не знаю насчет востребованности, но интерес то есть
это нужно показыват) какие никакие подсказки можно подсказывать😊
Всё по делу, но очень сильно не согласен на 9.52. Я даже не представляю такую ситуацию при которой на рабочем проекте в продакшене меняют id uint64 на uuid string. Обычно uuid добавляют надстройкой сверху, например, для привязки к внешним системам, а внутри продолжают использовать id. Да и id() это int|string что плохо для проекта в части типизации. Благо если автотесты подскажут проблему заранее или стат анализатор.
спасибо за комментарий
Круто, спасибо большое! Продолжайте пожалуйста) понимаю, куда расти после такого видео 🙏
Рубрика нужна!
💯
Нам нужно больше ревью
буду выпускать
Я за, побольше таких видео
Спасибо за фидбек
Согласен с Данилом, код прям плохой, нужно тренироваться ))
Тренируются спортсмены, разрабы - учатся, потому как программирование, это область знаний
💯
Обзор с рефакторингом не на словах был бы еще круче. Спасибо!
Согласен! Весь проект рефакторить не нужно, но пару-тройку методов/контроллеров можно было бы коснуться.
учту!
Правильно понял это на мидла ТЗ?
сейчас копайлот в миллион раз чище напишет, какой там джун
Увидел валидацию в контроллере (вместо форм реквеста) и работу с моделями (вместо сервисов) и подумал что тут максимум Junior+ с натяжкой
P.S. Нестрогое сравнение дальше. Погорячился с оценкой. Рекомендую начать с курсов основ (желательно не Лаврика) и документации
IDE спалил имя автора 😂
🙂
Не возьмусь защищать автора кода, но и рекомендации и комментарии автора обзора не поддерживаю. Критика очевидная, использовать solid принципы, не использовать пакеты, форматировать код. Заученные слова, которые мог дать любой. Особенно, где-то деструктивная даже идея - что главное, показать себя, и чуть ли не писать "велосипед" для этого, а не пользоваться пакетами. Неопытный ум послушав это может воспринять это буквально и понтоваться там, где можно написать просто и элегантно. При всей критике внешних пакетов, ваш канал пестрит заголовками "обзор пакета". Да и ваша админка это тоже пакет, который вы предлагаете к использованию. Т.е. расходится то, что вы даете и то за что в итоге критикуете. Пакеты это инструменты, и если бы автор кода показал грамотное ими пользование, показал то, что бизнес может в разы сократить время разработки и эффективно использовать время, это было бы аргументом и пусть использует этот инструмент, чем писать свой тратя лишнее время. Не могу также согласиться, что автор кода показал себя как хороший пользователь Laravel. Потому что то, как он использует инструменты фреймворка так их использовать не рекомендуется.
Если бы да кабы, но автор не показал себя не как пользователь пакетов, не как изобретатель велосипедов! Благодарю за комментарий
Итальянская кухня хахаах
Слабый код, не понимаю на что человек надеялся, сразу видно что у него нет опыта, надо с годик на джуне посидеть на нормальном проекте и будет щастье. Очень хочется на ревью увидеть что-то более
Нормально там все... вы придираетесь. )
Клиенту плевать, главное, что бы работало. Клиента интересует монетизация, а не где у вас там скобка и тайпхинт.
Рассуждать про SOLID в контроллерах любого фреймворка как минимум глупо)
Любой контроллер уже нарушает первый принцип solid)))
Я стажер/джун не уверен что лучше написал но не которые моменты даже я как стажер/ джун лучше написал бы
👍
Не функционал, а функциональность. Хех
ruclips.net/video/ldoKNm6qHVw/видео.html
Почаще такую критику с обзором🎉🎉🎉🎉
по возможности буду выпускать!
Это код даже не джуна.)) Я бы даже в стажеры не взял. Без обид автору кода
у него все впереди
Это пока вы не пошли на собеседование... )
3500 не мало но и не много, но тут момент что это код близко не мидла. Джун, даже не аппер
да даже не джун, тут слишком много косяков
@@enmaboya а что вы хотите от джунов тогда? Чтоб сразу все знал и писал чисто? так это не джун, а сеньер.
@@user-ct4tl3yp1z чтобы он умел читать документацию хотя бы.
то что в видео - уровень человека который просто копирует решения из интернета, при этом не особо вникая что именно он скопировал и зачем,
но с этого в принципе все начинали, так что ничего страшного не вижу, просто непонятно зачем было собеситься на мидла
👨🎓
@@enmaboya , копировать решение из интернета, тоже уметь надо и далеко не всем дано. )
Даже велосипед изобрести не каждый способен.
Не стоит светить имя его
Автор дал добро на запись видео и я не вижу ничего плохого, человек в процессе обучения
Эта рубрика должна быть и почаще!
👌