Хотелось бы, чтобы в следующий раз, вы сразу в видое показывали, как правильно должен выглядеть код. То есть, исправлять его прям в видео.
@daloopdaloop Жыл бұрын
Эта рубрика должна быть и почаще!
@vi_ju Жыл бұрын
Видео кайф, но мне кажется было бы прикольно, если показывал бы наглядно как нужно было бы написать (не во всех моментах конечно 😅, а то будет очень долго). Ставлю лайк
@ЭролСезгин Жыл бұрын
абсолютно поддерживаю! а если не лень, то репу в гит, с первым комитом, До и вторым После, прям большое спасибо сказать будет можно =)))
@ИванГоловатый-ж2и Жыл бұрын
Очень полезная рубрика! Обязательно продолжай. И почаще!
@plutuss Жыл бұрын
Интересное видео. Да, код нужно улучшить. Меня два года назад с таким возмоно не много лучше кодом на джуна брать не хотели. Сейчас уже на мидла люди пытаются. Если автор читает. .... Не сдавайся проходи больше собиседований и будешь видеть слабые моменты. И фикси их ..)))
@kind1y Жыл бұрын
Рубрика - пушка! Хотелось бы ещё сразу видеть исправления.
@CutCodeRu Жыл бұрын
посмотрим на будущее
@Aleksvin8888 Жыл бұрын
прикольно подивився і дещо нове дізнався прям захтілось одразу побігти і свій код поправити ))))
@gambleriches Жыл бұрын
полезная ! Обязательно продолжай. И почаще!
@dobrydyadka Жыл бұрын
Бро, спасибо за разбор! Повеселил ))
@turdakhov Жыл бұрын
Больше таких видосов! Спасибо!
@АлександрАндреев-г3ь Жыл бұрын
Рубрика огонь! Спасибо! )
@Pavelbrov Жыл бұрын
Я за, особенно за то, чтоб показывалось до/после
@CutCodeRu Жыл бұрын
Может после этого ролика, автор исправит ошибки и посмотрим тогда "после"
@Edvard-Aliev Жыл бұрын
@@CutCodeRu Нужно от Вас мастер класс! Ждем от Вас! Очень хочется увитился!
@rpy6ocTb Жыл бұрын
Рубрика отличная, давай на постоянку её =) и как уже писали, было бы хорошо показать, как лучше сделать, но не прям очевидные моменты
@CutCodeRu Жыл бұрын
Снимем и такое
@pepper-fm Жыл бұрын
топ рубрика я бы ещё сослался на roadmap от Повиласа (laravel daily) о том, что нужно знать разрабу на какой уровень
@CutCodeRu Жыл бұрын
у нас свой Roadmap - cutcode.dev/roadmap . попробуйте!
@ArabicLang.online Жыл бұрын
Рубрика супер, однозначно нужна и для начинающих, и для опытных.
@CutCodeRu Жыл бұрын
💪
@friendvane7612 Жыл бұрын
Отличная рубрика, почаще говорите о лучших практиках)
@CutCodeRu Жыл бұрын
спасибо!
@iddqd2285 Жыл бұрын
Еще не смотрел видос, но это 100% отличная идея.
@CutCodeRu Жыл бұрын
👍
@artem-web-developer Жыл бұрын
Интересно)
@CutCodeRu Жыл бұрын
🔥
@fukurou48 Жыл бұрын
Спасибо, полезно!
@CutCodeRu Жыл бұрын
🔥
@Василий-ф2б4р Жыл бұрын
Тема интересная, но ожидал в конце ролика исправленные куски кода, как было на старых видео этого канала 😅
@CutCodeRu Жыл бұрын
Это другая рубрика)
@Василий-ф2б4р Жыл бұрын
@@CutCodeRu всё же, было бы интересно посмотреть на исправленный вариант кода в таких случаях)
@CutCodeRu Жыл бұрын
@@Василий-ф2б4р думаю плохой, хороший код еще обязательно выйдет
@asadaa3095 Жыл бұрын
Спасибо большое, хотелось бы смотреть и на сенёров тоже
@АртемЮров-в4з Жыл бұрын
Интересно! В группе задавали как раз задавали вопрос, где бы найти такую услугу как код ревью от сеньора
@CutCodeRu Жыл бұрын
думаете, будет востребовано?
@АртемЮров-в4з Жыл бұрын
@@CutCodeRu не знаю насчет востребованности, но интерес то есть
@bulletproof1304 Жыл бұрын
Выглядит как Джун решил пройти собеседование на мидла. Ну или тестовое сделано на отье***. Понатыкано пакетов, стандартных методов из коробки, многие из которых вообще не используются. Как сказал автор канала делал задание не разработчик, а пользователь пакетов ларавель. А подписчику желаю удачи в развитии, у всех были и будут ошибки.
@molxs5339 Жыл бұрын
Капец, я еще даже не джун, но про такие вещи как DI знаю и активно использую. А тут целый мидл жесть….
@syracuse4612 Жыл бұрын
Правильно понял это на мидла ТЗ?
@syracuse4612 Жыл бұрын
сейчас копайлот в миллион раз чище напишет, какой там джун
@DenysHona5 ай бұрын
Решение даже на джуна не тянет
@snikitin329 ай бұрын
Рассуждать про SOLID в контроллерах любого фреймворка как минимум глупо) Любой контроллер уже нарушает первый принцип solid)))
@ИльяСорокин-д9ц Жыл бұрын
Всё по делу, но очень сильно не согласен на 9.52. Я даже не представляю такую ситуацию при которой на рабочем проекте в продакшене меняют id uint64 на uuid string. Обычно uuid добавляют надстройкой сверху, например, для привязки к внешним системам, а внутри продолжают использовать id. Да и id() это int|string что плохо для проекта в части типизации. Благо если автотесты подскажут проблему заранее или стат анализатор.
@CutCodeRu Жыл бұрын
спасибо за комментарий
@Amedomaroful Жыл бұрын
Я за, побольше таких видео
@CutCodeRu Жыл бұрын
Спасибо за фидбек
@АленаЕршова-ъ5ю Жыл бұрын
Круто, спасибо большое! Продолжайте пожалуйста) понимаю, куда расти после такого видео 🙏
@soversus53748 ай бұрын
Когда диалог строится: лучше было бы так... Не понятно, кому лучше, чем лучше... и почему бы и нет. )
@modestusmunh93 Жыл бұрын
Рубрика нужна!
@CutCodeRu Жыл бұрын
💯
@turdakhov Жыл бұрын
Обзор с рефакторингом не на словах был бы еще круче. Спасибо!
@modestusmunh93 Жыл бұрын
Согласен! Весь проект рефакторить не нужно, но пару-тройку методов/контроллеров можно было бы коснуться.
@CutCodeRu Жыл бұрын
учту!
@gamingplay5415 Жыл бұрын
это нужно показыват) какие никакие подсказки можно подсказывать😊
@areaweb Жыл бұрын
Согласен с Данилом, код прям плохой, нужно тренироваться ))
@pavelhol9652 Жыл бұрын
Тренируются спортсмены, разрабы - учатся, потому как программирование, это область знаний
@CutCodeRu Жыл бұрын
💯
@vitaercx Жыл бұрын
Не возьмусь защищать автора кода, но и рекомендации и комментарии автора обзора не поддерживаю. Критика очевидная, использовать solid принципы, не использовать пакеты, форматировать код. Заученные слова, которые мог дать любой. Особенно, где-то деструктивная даже идея - что главное, показать себя, и чуть ли не писать "велосипед" для этого, а не пользоваться пакетами. Неопытный ум послушав это может воспринять это буквально и понтоваться там, где можно написать просто и элегантно. При всей критике внешних пакетов, ваш канал пестрит заголовками "обзор пакета". Да и ваша админка это тоже пакет, который вы предлагаете к использованию. Т.е. расходится то, что вы даете и то за что в итоге критикуете. Пакеты это инструменты, и если бы автор кода показал грамотное ими пользование, показал то, что бизнес может в разы сократить время разработки и эффективно использовать время, это было бы аргументом и пусть использует этот инструмент, чем писать свой тратя лишнее время. Не могу также согласиться, что автор кода показал себя как хороший пользователь Laravel. Потому что то, как он использует инструменты фреймворка так их использовать не рекомендуется.
@CutCodeRu Жыл бұрын
Если бы да кабы, но автор не показал себя не как пользователь пакетов, не как изобретатель велосипедов! Благодарю за комментарий
@GreatOsmanSultan Жыл бұрын
Почаще такую критику с обзором🎉🎉🎉🎉
@CutCodeRu Жыл бұрын
по возможности буду выпускать!
@fil-d1d Жыл бұрын
IDE спалил имя автора 😂
@CutCodeRu Жыл бұрын
🙂
@TsA1ex Жыл бұрын
Увидел валидацию в контроллере (вместо форм реквеста) и работу с моделями (вместо сервисов) и подумал что тут максимум Junior+ с натяжкой P.S. Нестрогое сравнение дальше. Погорячился с оценкой. Рекомендую начать с курсов основ (желательно не Лаврика) и документации
@iddqd2285 Жыл бұрын
Слабый код, не понимаю на что человек надеялся, сразу видно что у него нет опыта, надо с годик на джуне посидеть на нормальном проекте и будет щастье. Очень хочется на ревью увидеть что-то более
@soversus53748 ай бұрын
Нормально там все... вы придираетесь. ) Клиенту плевать, главное, что бы работало. Клиента интересует монетизация, а не где у вас там скобка и тайпхинт.
@АнтонГусев-ъ7у Жыл бұрын
Я стажер/джун не уверен что лучше написал но не которые моменты даже я как стажер/ джун лучше написал бы
@CutCodeRu Жыл бұрын
👍
@modestusmunh93 Жыл бұрын
Итальянская кухня хахаах
@egorow4innikow565 Жыл бұрын
Не функционал, а функциональность. Хех kzbin.info/www/bejne/opXSfIGja9Z7jNk
@gian_tiaga Жыл бұрын
Это код даже не джуна.)) Я бы даже в стажеры не взял. Без обид автору кода
@CutCodeRu Жыл бұрын
у него все впереди
@soversus53748 ай бұрын
Это пока вы не пошли на собеседование... )
@Diqeeeeeeeeeei Жыл бұрын
Нам нужно больше ревью
@CutCodeRu Жыл бұрын
буду выпускать
@itdev4680 Жыл бұрын
3500 не мало но и не много, но тут момент что это код близко не мидла. Джун, даже не аппер
@enmaboya Жыл бұрын
да даже не джун, тут слишком много косяков
@ДмитрийКондрашов-ц2о Жыл бұрын
@@enmaboya а что вы хотите от джунов тогда? Чтоб сразу все знал и писал чисто? так это не джун, а сеньер.
@enmaboya Жыл бұрын
@@ДмитрийКондрашов-ц2о чтобы он умел читать документацию хотя бы. то что в видео - уровень человека который просто копирует решения из интернета, при этом не особо вникая что именно он скопировал и зачем, но с этого в принципе все начинали, так что ничего страшного не вижу, просто непонятно зачем было собеситься на мидла
@CutCodeRu Жыл бұрын
👨🎓
@soversus53748 ай бұрын
@@enmaboya , копировать решение из интернета, тоже уметь надо и далеко не всем дано. ) Даже велосипед изобрести не каждый способен.
@itdev4680 Жыл бұрын
Не стоит светить имя его
@CutCodeRu Жыл бұрын
Автор дал добро на запись видео и я не вижу ничего плохого, человек в процессе обучения