Видео кайф, но мне кажется было бы прикольно, если показывал бы наглядно как нужно было бы написать (не во всех моментах конечно 😅, а то будет очень долго). Ставлю лайк
Интересное видео. Да, код нужно улучшить. Меня два года назад с таким возмоно не много лучше кодом на джуна брать не хотели. Сейчас уже на мидла люди пытаются. Если автор читает. .... Не сдавайся проходи больше собиседований и будешь видеть слабые моменты. И фикси их ..)))
Выглядит как Джун решил пройти собеседование на мидла. Ну или тестовое сделано на отье***. Понатыкано пакетов, стандартных методов из коробки, многие из которых вообще не используются. Как сказал автор канала делал задание не разработчик, а пользователь пакетов ларавель. А подписчику желаю удачи в развитии, у всех были и будут ошибки.
Всё по делу, но очень сильно не согласен на 9.52. Я даже не представляю такую ситуацию при которой на рабочем проекте в продакшене меняют id uint64 на uuid string. Обычно uuid добавляют надстройкой сверху, например, для привязки к внешним системам, а внутри продолжают использовать id. Да и id() это int|string что плохо для проекта в части типизации. Благо если автотесты подскажут проблему заранее или стат анализатор.
Не возьмусь защищать автора кода, но и рекомендации и комментарии автора обзора не поддерживаю. Критика очевидная, использовать solid принципы, не использовать пакеты, форматировать код. Заученные слова, которые мог дать любой. Особенно, где-то деструктивная даже идея - что главное, показать себя, и чуть ли не писать "велосипед" для этого, а не пользоваться пакетами. Неопытный ум послушав это может воспринять это буквально и понтоваться там, где можно написать просто и элегантно. При всей критике внешних пакетов, ваш канал пестрит заголовками "обзор пакета". Да и ваша админка это тоже пакет, который вы предлагаете к использованию. Т.е. расходится то, что вы даете и то за что в итоге критикуете. Пакеты это инструменты, и если бы автор кода показал грамотное ими пользование, показал то, что бизнес может в разы сократить время разработки и эффективно использовать время, это было бы аргументом и пусть использует этот инструмент, чем писать свой тратя лишнее время. Не могу также согласиться, что автор кода показал себя как хороший пользователь Laravel. Потому что то, как он использует инструменты фреймворка так их использовать не рекомендуется.
Слабый код, не понимаю на что человек надеялся, сразу видно что у него нет опыта, надо с годик на джуне посидеть на нормальном проекте и будет щастье. Очень хочется на ревью увидеть что-то более
Нормально там все... вы придираетесь. ) Клиенту плевать, главное, что бы работало. Клиента интересует монетизация, а не где у вас там скобка и тайпхинт.
Увидел валидацию в контроллере (вместо форм реквеста) и работу с моделями (вместо сервисов) и подумал что тут максимум Junior+ с натяжкой P.S. Нестрогое сравнение дальше. Погорячился с оценкой. Рекомендую начать с курсов основ (желательно не Лаврика) и документации
@@ДмитрийКондрашов-ц2о чтобы он умел читать документацию хотя бы. то что в видео - уровень человека который просто копирует решения из интернета, при этом не особо вникая что именно он скопировал и зачем, но с этого в принципе все начинали, так что ничего страшного не вижу, просто непонятно зачем было собеситься на мидла
Хотелось бы, чтобы в следующий раз, вы сразу в видое показывали, как правильно должен выглядеть код. То есть, исправлять его прям в видео.
Эта рубрика должна быть и почаще!
Видео кайф, но мне кажется было бы прикольно, если показывал бы наглядно как нужно было бы написать (не во всех моментах конечно 😅, а то будет очень долго). Ставлю лайк
абсолютно поддерживаю!
а если не лень, то репу в гит, с первым комитом, До и вторым После, прям большое спасибо сказать будет можно =)))
Очень полезная рубрика! Обязательно продолжай. И почаще!
Интересное видео. Да, код нужно улучшить. Меня два года назад с таким возмоно не много лучше кодом на джуна брать не хотели. Сейчас уже на мидла люди пытаются. Если автор читает. .... Не сдавайся проходи больше собиседований и будешь видеть слабые моменты. И фикси их ..)))
полезная ! Обязательно продолжай. И почаще!
Бро, спасибо за разбор! Повеселил ))
Больше таких видосов! Спасибо!
прикольно подивився і дещо нове дізнався прям захтілось одразу побігти і свій код поправити ))))
Рубрика огонь! Спасибо! )
Рубрика - пушка! Хотелось бы ещё сразу видеть исправления.
посмотрим на будущее
Я за, особенно за то, чтоб показывалось до/после
Может после этого ролика, автор исправит ошибки и посмотрим тогда "после"
@@CutCodeRu Нужно от Вас мастер класс! Ждем от Вас! Очень хочется увитился!
Рубрика отличная, давай на постоянку её =) и как уже писали, было бы хорошо показать, как лучше сделать, но не прям очевидные моменты
Снимем и такое
Спасибо, полезно!
🔥
Рубрика супер, однозначно нужна и для начинающих, и для опытных.
💪
топ рубрика
я бы ещё сослался на roadmap от Повиласа (laravel daily) о том, что нужно знать разрабу на какой уровень
у нас свой Roadmap - cutcode.dev/roadmap . попробуйте!
Интересно)
🔥
Отличная рубрика, почаще говорите о лучших практиках)
спасибо!
Еще не смотрел видос, но это 100% отличная идея.
👍
Интересно! В группе задавали как раз задавали вопрос, где бы найти такую услугу как код ревью от сеньора
думаете, будет востребовано?
@@CutCodeRu не знаю насчет востребованности, но интерес то есть
Тема интересная, но ожидал в конце ролика исправленные куски кода, как было на старых видео этого канала 😅
Это другая рубрика)
@@CutCodeRu всё же, было бы интересно посмотреть на исправленный вариант кода в таких случаях)
@@Василий-ф2б4р думаю плохой, хороший код еще обязательно выйдет
Рассуждать про SOLID в контроллерах любого фреймворка как минимум глупо)
Любой контроллер уже нарушает первый принцип solid)))
Правильно понял это на мидла ТЗ?
сейчас копайлот в миллион раз чище напишет, какой там джун
Решение даже на джуна не тянет
Выглядит как Джун решил пройти собеседование на мидла. Ну или тестовое сделано на отье***. Понатыкано пакетов, стандартных методов из коробки, многие из которых вообще не используются. Как сказал автор канала делал задание не разработчик, а пользователь пакетов ларавель. А подписчику желаю удачи в развитии, у всех были и будут ошибки.
Я за, побольше таких видео
Спасибо за фидбек
Спасибо большое, хотелось бы смотреть и на сенёров тоже
Всё по делу, но очень сильно не согласен на 9.52. Я даже не представляю такую ситуацию при которой на рабочем проекте в продакшене меняют id uint64 на uuid string. Обычно uuid добавляют надстройкой сверху, например, для привязки к внешним системам, а внутри продолжают использовать id. Да и id() это int|string что плохо для проекта в части типизации. Благо если автотесты подскажут проблему заранее или стат анализатор.
спасибо за комментарий
Капец, я еще даже не джун, но про такие вещи как DI знаю и активно использую. А тут целый мидл жесть….
Когда диалог строится: лучше было бы так...
Не понятно, кому лучше, чем лучше... и почему бы и нет. )
Круто, спасибо большое! Продолжайте пожалуйста) понимаю, куда расти после такого видео 🙏
это нужно показыват) какие никакие подсказки можно подсказывать😊
Рубрика нужна!
💯
Согласен с Данилом, код прям плохой, нужно тренироваться ))
Тренируются спортсмены, разрабы - учатся, потому как программирование, это область знаний
💯
Не возьмусь защищать автора кода, но и рекомендации и комментарии автора обзора не поддерживаю. Критика очевидная, использовать solid принципы, не использовать пакеты, форматировать код. Заученные слова, которые мог дать любой. Особенно, где-то деструктивная даже идея - что главное, показать себя, и чуть ли не писать "велосипед" для этого, а не пользоваться пакетами. Неопытный ум послушав это может воспринять это буквально и понтоваться там, где можно написать просто и элегантно. При всей критике внешних пакетов, ваш канал пестрит заголовками "обзор пакета". Да и ваша админка это тоже пакет, который вы предлагаете к использованию. Т.е. расходится то, что вы даете и то за что в итоге критикуете. Пакеты это инструменты, и если бы автор кода показал грамотное ими пользование, показал то, что бизнес может в разы сократить время разработки и эффективно использовать время, это было бы аргументом и пусть использует этот инструмент, чем писать свой тратя лишнее время. Не могу также согласиться, что автор кода показал себя как хороший пользователь Laravel. Потому что то, как он использует инструменты фреймворка так их использовать не рекомендуется.
Если бы да кабы, но автор не показал себя не как пользователь пакетов, не как изобретатель велосипедов! Благодарю за комментарий
Обзор с рефакторингом не на словах был бы еще круче. Спасибо!
Согласен! Весь проект рефакторить не нужно, но пару-тройку методов/контроллеров можно было бы коснуться.
учту!
IDE спалил имя автора 😂
🙂
Слабый код, не понимаю на что человек надеялся, сразу видно что у него нет опыта, надо с годик на джуне посидеть на нормальном проекте и будет щастье. Очень хочется на ревью увидеть что-то более
Нормально там все... вы придираетесь. )
Клиенту плевать, главное, что бы работало. Клиента интересует монетизация, а не где у вас там скобка и тайпхинт.
Почаще такую критику с обзором🎉🎉🎉🎉
по возможности буду выпускать!
Я стажер/джун не уверен что лучше написал но не которые моменты даже я как стажер/ джун лучше написал бы
👍
Не функционал, а функциональность. Хех
ua-cam.com/video/ldoKNm6qHVw/v-deo.html
Итальянская кухня хахаах
Увидел валидацию в контроллере (вместо форм реквеста) и работу с моделями (вместо сервисов) и подумал что тут максимум Junior+ с натяжкой
P.S. Нестрогое сравнение дальше. Погорячился с оценкой. Рекомендую начать с курсов основ (желательно не Лаврика) и документации
Это код даже не джуна.)) Я бы даже в стажеры не взял. Без обид автору кода
у него все впереди
Это пока вы не пошли на собеседование... )
Нам нужно больше ревью
буду выпускать
3500 не мало но и не много, но тут момент что это код близко не мидла. Джун, даже не аппер
да даже не джун, тут слишком много косяков
@@enmaboya а что вы хотите от джунов тогда? Чтоб сразу все знал и писал чисто? так это не джун, а сеньер.
@@ДмитрийКондрашов-ц2о чтобы он умел читать документацию хотя бы.
то что в видео - уровень человека который просто копирует решения из интернета, при этом не особо вникая что именно он скопировал и зачем,
но с этого в принципе все начинали, так что ничего страшного не вижу, просто непонятно зачем было собеситься на мидла
👨🎓
@@enmaboya , копировать решение из интернета, тоже уметь надо и далеко не всем дано. )
Даже велосипед изобрести не каждый способен.
Не стоит светить имя его
Автор дал добро на запись видео и я не вижу ничего плохого, человек в процессе обучения
Эта рубрика должна быть и почаще!
👌