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