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

[@tochivk]

Хотелось бы, чтобы в следующий раз, вы сразу в видое показывали, как правильно должен выглядеть код. То есть, исправлять его прям в видео.

[@vi_ju]

Видео кайф, но мне кажется было бы прикольно, если показывал бы наглядно как нужно было бы написать (не во всех моментах конечно 😅, а то будет очень долго). Ставлю лайк

[@ЭролСезгин]

абсолютно поддерживаю! а если не лень, то репу в гит, с первым комитом, До и вторым После, прям большое спасибо сказать будет можно =)))

[@ИванГоловатый-ж2и]

Очень полезная рубрика! Обязательно продолжай. И почаще!

[@YazmyratAtayew]

полезная ! Обязательно продолжай. И почаще!

[@rpy6ocTb]

Рубрика отличная, давай на постоянку её =) и как уже писали, было бы хорошо показать, как лучше сделать, но не прям очевидные моменты

[@CutCodeRu]

Снимем и такое

[@Pavelbrov]

Я за, особенно за то, чтоб показывалось до/после

[@CutCodeRu]

Может после этого ролика, автор исправит ошибки и посмотрим тогда "после"

[@Edvard-Aliev]

@@CutCodeRu Нужно от Вас мастер класс! Ждем от Вас! Очень хочется увитился!

[@dobrydyadka]

Бро, спасибо за разбор! Повеселил ))

[@turdakhov]

Больше таких видосов! Спасибо!

[@ArabicLang.online]

Рубрика супер, однозначно нужна и для начинающих, и для опытных.

[@CutCodeRu]

💪

[@pepper-fm]

топ рубрика я бы ещё сослался на roadmap от Повиласа (laravel daily) о том, что нужно знать разрабу на какой уровень

[@CutCodeRu]

у нас свой Roadmap - cutcode.dev/roadmap . попробуйте!

[@АлександрАндреев-г3ь]

Рубрика огонь! Спасибо! )

[@kind1y]

Рубрика - пушка! Хотелось бы ещё сразу видеть исправления.

[@CutCodeRu]

посмотрим на будущее

[@friendvane7612]

Отличная рубрика, почаще говорите о лучших практиках)

[@CutCodeRu]

спасибо!

[@artem-web-developer]

Интересно)

[@CutCodeRu]

🔥

[@molxs5339]

Капец, я еще даже не джун, но про такие вещи как DI знаю и активно использую. А тут целый мидл жесть….

[@fukurou48]

Спасибо, полезно!

[@CutCodeRu]

🔥

[@asadaa3095]

Спасибо большое, хотелось бы смотреть и на сенёров тоже

[@ИльяСорокин-д9ц]

Всё по делу, но очень сильно не согласен на 9.52. Я даже не представляю такую ситуацию при которой на рабочем проекте в продакшене меняют id uint64 на uuid string. Обычно uuid добавляют надстройкой сверху, например, для привязки к внешним системам, а внутри продолжают использовать id. Да и id() это int|string что плохо для проекта в части типизации. Благо если автотесты подскажут проблему заранее или стат анализатор.

[@CutCodeRu]

спасибо за комментарий

[@soversus5374]

Когда диалог строится: лучше было бы так... Не понятно, кому лучше, чем лучше... и почему бы и нет. )

[@vitaercx]

Не возьмусь защищать автора кода, но и рекомендации и комментарии автора обзора не поддерживаю. Критика очевидная, использовать solid принципы, не использовать пакеты, форматировать код. Заученные слова, которые мог дать любой. Особенно, где-то деструктивная даже идея - что главное, показать себя, и чуть ли не писать "велосипед" для этого, а не пользоваться пакетами. Неопытный ум послушав это может воспринять это буквально и понтоваться там, где можно написать просто и элегантно. При всей критике внешних пакетов, ваш канал пестрит заголовками "обзор пакета". Да и ваша админка это тоже пакет, который вы предлагаете к использованию. Т.е. расходится то, что вы даете и то за что в итоге критикуете. Пакеты это инструменты, и если бы автор кода показал грамотное ими пользование, показал то, что бизнес может в разы сократить время разработки и эффективно использовать время, это было бы аргументом и пусть использует этот инструмент, чем писать свой тратя лишнее время. Не могу также согласиться, что автор кода показал себя как хороший пользователь Laravel. Потому что то, как он использует инструменты фреймворка так их использовать не рекомендуется.

[@CutCodeRu]

Если бы да кабы, но автор не показал себя не как пользователь пакетов, не как изобретатель велосипедов! Благодарю за комментарий

[@gamingplay5415]

это нужно показыват) какие никакие подсказки можно подсказывать😊

[@modestusmunh93]

Рубрика нужна!

[@CutCodeRu]

💯

[@areaweb]

Согласен с Данилом, код прям плохой, нужно тренироваться ))

[@pavelhol9652]

Тренируются спортсмены, разрабы - учатся, потому как программирование, это область знаний

[@CutCodeRu]

💯

[@zCubEz]

Эта рубрика должна быть и почаще!

[@CutCodeRu]

👌

[@turdakhov]

Обзор с рефакторингом не на словах был бы еще круче. Спасибо!

[@modestusmunh93]

Согласен! Весь проект рефакторить не нужно, но пару-тройку методов/контроллеров можно было бы коснуться.

[@CutCodeRu]

учту!

[@GreatOsmanSultan]

Почаще такую критику с обзором🎉🎉🎉🎉

[@CutCodeRu]

по возможности буду выпускать!

[@modestusmunh93]

Итальянская кухня хахаах

[@iddqd2285]

Слабый код, не понимаю на что человек надеялся, сразу видно что у него нет опыта, надо с годик на джуне посидеть на нормальном проекте и будет щастье. Очень хочется на ревью увидеть что-то более

[@soversus5374]

Нормально там все... вы придираетесь. ) Клиенту плевать, главное, что бы работало. Клиента интересует монетизация, а не где у вас там скобка и тайпхинт.

[@TsA1ex]

Увидел валидацию в контроллере (вместо форм реквеста) и работу с моделями (вместо сервисов) и подумал что тут максимум Junior+ с натяжкой P.S. Нестрогое сравнение дальше. Погорячился с оценкой. Рекомендую начать с курсов основ (желательно не Лаврика) и документации

[@fil-d1d]

IDE спалил имя автора 😂

[@CutCodeRu]

🙂

[@АнтонГусев-ъ7у]

Я стажер/джун не уверен что лучше написал но не которые моменты даже я как стажер/ джун лучше написал бы

[@CutCodeRu]

👍

[@egorow4innikow565]

Не функционал, а функциональность. Хех kzbin.info/www/bejne/opXSfIGja9Z7jNk

[@gian_tiaga]

Это код даже не джуна.)) Я бы даже в стажеры не взял. Без обид автору кода

[@CutCodeRu]

у него все впереди

[@soversus5374]

Это пока вы не пошли на собеседование... )

[@itdev4680]

3500 не мало но и не много, но тут момент что это код близко не мидла. Джун, даже не аппер

[@enmaboya]

да даже не джун, тут слишком много косяков

[@ДмитрийКондрашов-ц2о]

@@enmaboya а что вы хотите от джунов тогда? Чтоб сразу все знал и писал чисто? так это не джун, а сеньер.

[@enmaboya]

@@ДмитрийКондрашов-ц2о чтобы он умел читать документацию хотя бы. то что в видео - уровень человека который просто копирует решения из интернета, при этом не особо вникая что именно он скопировал и зачем, но с этого в принципе все начинали, так что ничего страшного не вижу, просто непонятно зачем было собеситься на мидла

[@CutCodeRu]

👨‍🎓

[@soversus5374]

@@enmaboya , копировать решение из интернета, тоже уметь надо и далеко не всем дано. ) Даже велосипед изобрести не каждый способен.

[@itdev4680]

Не стоит светить имя его

[@CutCodeRu]

Автор дал добро на запись видео и я не вижу ничего плохого, человек в процессе обучения

[@daloopdaloop]

Эта рубрика должна быть и почаще!

[@plutuss]

Интересное видео. Да, код нужно улучшить. Меня два года назад с таким возмоно не много лучше кодом на джуна брать не хотели. Сейчас уже на мидла люди пытаются. Если автор читает. .... Не сдавайся проходи больше собиседований и будешь видеть слабые моменты. И фикси их ..)))

[@snikitin32]

Рассуждать про SOLID в контроллерах любого фреймворка как минимум глупо) Любой контроллер уже нарушает первый принцип solid)))

[@Aleksvin8888]

прикольно подивився і дещо нове дізнався прям захтілось одразу побігти і свій код поправити ))))

[@bulletproof1304]

Выглядит как Джун решил пройти собеседование на мидла. Ну или тестовое сделано на отье***. Понатыкано пакетов, стандартных методов из коробки, многие из которых вообще не используются. Как сказал автор канала делал задание не разработчик, а пользователь пакетов ларавель. А подписчику желаю удачи в развитии, у всех были и будут ошибки.

[@Василий-ф2б4р]

Тема интересная, но ожидал в конце ролика исправленные куски кода, как было на старых видео этого канала 😅

[@CutCodeRu]

Это другая рубрика)

[@Василий-ф2б4р]

@@CutCodeRu всё же, было бы интересно посмотреть на исправленный вариант кода в таких случаях)

[@CutCodeRu]

@@Василий-ф2б4р думаю плохой, хороший код еще обязательно выйдет

[@syracuse4612]

Правильно понял это на мидла ТЗ?

[@syracuse4612]

сейчас копайлот в миллион раз чище напишет, какой там джун

[@DenysHona]

Решение даже на джуна не тянет

[@Amedomaroful]

Я за, побольше таких видео

[@CutCodeRu]

Спасибо за фидбек

[@iddqd2285]

Еще не смотрел видос, но это 100% отличная идея.

[@CutCodeRu]

👍

[@АртемЮров-в4з]

Интересно! В группе задавали как раз задавали вопрос, где бы найти такую услугу как код ревью от сеньора

[@CutCodeRu]

думаете, будет востребовано?

[@АртемЮров-в4з]

@@CutCodeRu не знаю насчет востребованности, но интерес то есть

[@АленаЕршова-ъ5ю]

Круто, спасибо большое! Продолжайте пожалуйста) понимаю, куда расти после такого видео 🙏

[@Diqeeeeeeeeeei]

Нам нужно больше ревью

[@CutCodeRu]

буду выпускать