Как вам шаблон, коллеги? Видео, кстати, теперь и в 2К доступно.
@alexcc333
3 жыл бұрын
Как и все шаблоны со своим приколом)
@onlybestmusic4185
3 жыл бұрын
Спасибо что услышал мое предложение по использованию примеров при объяснении паттернов. Время от времени пересматриваю для закрепления в голове и незабывания, вроде патерны знаешь но насколько же всё-таки приятно смотреть с примерами :) хотения и терпения тебе
@awsdevbackend1783
3 жыл бұрын
Очень много кода. Я понимаю, что это характерно для данного шаблона, но всегда подмывает пойти более простым пусть и не совсем правильным путем и решить все просто при помощи большого количества условий)
@sashasss7114
3 жыл бұрын
Дмитрий, добрый день. Можете сделать актуальное видео по работе с WebSocket-ами?
@DmitryAfanasyev
3 жыл бұрын
Возьму на заметку 👍
@eaglesm1993
3 жыл бұрын
Вот это заморочился, спасибо, было интересно и полезно)
@topalek
3 жыл бұрын
Дмитрий, спасибо вам за вашу работу
@DmitryAfanasyev
3 жыл бұрын
🙏
@user-qq2in3lh1i
Жыл бұрын
Спасибо огромное ! Все понятно - и по полочкам разложено !
@user-dg3vy5cq6k
3 жыл бұрын
Я пришел с курса Эксперn PHP, почти закончил, видимо поверил в себя! Ребята старый смарти рулит!
@borisoffdenis
3 жыл бұрын
Дмитрий крут!
@ithandbook
3 жыл бұрын
Спасибо! Классное видео! По поводу интриги у меня два предположения: 1. Возможно что-то не то с методом actionMain(), который в свою очередь запускает метод run() декораторов и основного декорируемого класса. У меня получилась такая несимметричная матрёшка, когда я переопределил методы actionBefore, actionMain и actionAfter во всех декораторах: [2021-03-28 16:14:36] local.INFO: Decorator [2021-03-28 16:14:36] local.INFO: Notify managers before [2021-03-28 16:14:36] local.INFO: Notify managers immediately before calling run() action [2021-03-28 16:14:36] local.INFO: Notify users before [2021-03-28 16:14:36] local.INFO: Notify users immediately before calling run() action [2021-03-28 16:14:36] local.INFO: Log Before [2021-03-28 16:14:36] local.INFO: Log immediately before calling run() action [2021-03-28 16:14:36] local.INFO: Base update [2021-03-28 16:14:36] local.INFO: Log After [2021-03-28 16:14:36] local.INFO: Notify users after [2021-03-28 16:14:36] local.INFO: Notify managers after Не скажу, что это прям ошибка, но если бы сообщения с "run()" были бы рядом с "Base update", было бы красивее:) 2. Возможно что-то с наследованием? У Вас свернута папочка "Inheritance" в каталоге с паттерном и Вы не показывали, что в ней:))
@shnircourier
3 жыл бұрын
спасибо за видео
@nlookorg
2 жыл бұрын
Хорошая подача
@DrTopk
3 жыл бұрын
спасибо за твои видео!
@tiikhomirovilia6070
2 жыл бұрын
Thank You!
@alicenNorwood
3 жыл бұрын
Великолепный паттерн, надёжный, блять, как швейцарские часы
@alicenNorwood
3 жыл бұрын
Не туда написал, это к компоновщику
@DmitryAfanasyev
3 жыл бұрын
Прочел голосом Гоблина 😁
@alicenNorwood
3 жыл бұрын
@@DmitryAfanasyev написал голосом Гоблина
@user-et2cn3is9y
10 ай бұрын
Уффф...последние три паттерна, смысл то понятен. Словами на котиках могу объяснить и даже словами на примерах из реальных кейсов разработки. Но написать самому код...не смогу, все-таки опыта пока не хватает.
@nickfist9187
3 жыл бұрын
Спасибо за Вашу работу! Смотрится легко. Вопрос: планируете ли Вы добавить в курс "Laravel" аутентификацию и авторизацию пользователей?
@DmitryAfanasyev
3 жыл бұрын
Будет отдельным миникурсом
@dimitro.cardellini
3 жыл бұрын
1. Классное видео -- посмотрю предыдущие. 2. С ошибкой -- интгрига ...! ) Круто У меня вот есть вопрос к АбстрактДекоратору, а именно: - разбиение обертки на actionBefore, mainAction, actionAfter конечно, экономит на расширении декораторов, но оно провоцирует на добавление в декоратор мутируемого состояния: в actionBefore что-то считаем, записываем в контекст (он же this), потом в mainAction -- это читаем, принимаем решения тоже что-то считаем и тоже скорее всего что-то запишем, чтобы потом в actionAfter что-то вывести. Хотя по факту, если бы мы все сделали в переопределенном методе run, то контекст бы не понадобился. Я не о том, что все надо писать в одном методе, -- конечно же, внутри конерктного декоратора все надо декомпозировать в соответствии с его семантикой так, чтобы переопределенный метод run было легко прочитать. Сейчас это все выглядит, как экономия на "this->decoratedObject->run($order, $orderData)". В итоге все выльется, либо в то, что наследники будут переопределять mainAction точно так же, как они бы переопределяли сам метод run (тогда почему бы так не сделать сразу), либо будет использоваться мутрируемое состояние экземпляра декоратора, либо ... и то, и другое сразу. Лучше в абстрактном декораторе ограничиться только конструктором. И да, определения в англоязыной вики немного другие -- имеет смысл их тоже принимать во внимание.
@enmaboya
3 жыл бұрын
так что за ошибка то была в итоге?
@MrUndeadline
3 жыл бұрын
На примере какой нить игры, где нужно сотне разных противников под общим интерфейсом добавить какое то поведение, очень накладно создавать еще 100 подклассов для добавления строчки кода
@user-wu7mg1cu4e
Жыл бұрын
вот-вот, для других целей, они и не нужны
@romankovin8566
3 жыл бұрын
Большая просьба в видео раскрывать в дереве проекта те папки в которых лежат файлы. Поможет определить общий объем темы, сколько классов будет создано, сколько интерфейсов и т.д.
@DmitryAfanasyev
3 жыл бұрын
Да вроде всегда показываю всю используемую иерархию...
@diatm1506
10 ай бұрын
А чем этот декоратор отличается например от реализации в typescript или java? @decorator
@viv81ster
Жыл бұрын
Немного непонятное описание потому что на ум сразу приходит Стратегия а не Декоратор Мы же должны расширить метод какого-то объедка а не просто объедок. А в описании вроде основная цель поцепить новое поведение конкретно объдку
@alexalex6643
3 жыл бұрын
самая большая проблема с порядком выполнения. В вашем примере нельзя просто сначала например залогироватьДО, потом написать юзеру, потом залогировать после, если мы делаем что-то типа $logger($user($entity)) проблема №2, как по мне прекрасно описана в том же рефакториг гуру, где он описывал read/write данных. Там если не знать, какой цепочкой данные в декоратор собрались(допустим, что это красиво настраивается через UI гдето, и, внезапно, программистский баг, и таблица настроек слетает), непонятно будет, как потом исходный поток байт развернуть. И опять же, проблема 1, надо в нужной последовательности развернуть данные
@vitall789
Жыл бұрын
Мне кажется использовать методом стека может быть не оптимально! Лучше написать каждый декоратор под свою задачу, при создании декоратора передавая параметры объектов к примеру "logger"... если это нужно.
@onlybestmusic4185
3 жыл бұрын
В глаза бросаются две проблемы 1) $orderData передаётся массивом, целостность массива ничем не гарантируется, нужен какой-то вэлуе обжект или dtо (кстати можно отдельное видео сделать и в нём объединить простой патерн DTO и как его правильно использовать в ларавеле и передавать между слоями и так далее. Есть интересное видео по использованию DTО в symfony, но там на аннотациях завязано, так бесит...) 2) в конфиге нет порядка выполнения, кто-то случайно поменяет порядок или вставит свою запись где-то в верх или в середину и вся логика декоратора пойдёт по бороде. Надо вводить жёсткий порядок выполнения и сортировать коллекцию по этому порядку а уже потом производить все эти декорирования.
@user-lk1fw1lp8b
3 жыл бұрын
DTO это анти-паттерн
@DmitryAfanasyev
3 жыл бұрын
1) почему? Пруфлинк в студию 2) синглтон антипаттерн, и?.
@user-lk1fw1lp8b
3 жыл бұрын
@@DmitryAfanasyev 1. Попытаюсь объяснить своими словами: ДТО полностью ломают инкапсуляцию, т к являются просто контейнерами данных и позволяют им(данным) свободно утекать в другие части программы. Они не самодостаточны 2. Синглтон антипаттерн, и поэтому мы не должны его использовать. Его следует знать как то, что необходимо избегать
@DmitryAfanasyev
3 жыл бұрын
Надо Тейлору написать, а он в ларавеле внедрил и оптимизировал создание синглтонов. Не разбирается наверное в шаблонах.
@DmitryAfanasyev
3 жыл бұрын
Про дто не понял объяснения.
@ant3413
3 жыл бұрын
будут ли примеры взаимодействие с Vue?
@DmitryAfanasyev
3 жыл бұрын
Да
@ardixq
2 жыл бұрын
А можно где то взять этот код, что б самому посмотреть?
@DmitryAfanasyev
2 жыл бұрын
Код не выкладываю. Вредит обучению. Надо руками набирать самому.
@alexanderk8992
3 жыл бұрын
Возникли два вопроса: 1. как в декоратор прокинуть объект логера? 2. почему ты в конфиге прописал классы без нэймспейсов?
@DmitryAfanasyev
3 жыл бұрын
В ларе ведь логгер из коробки - не надо прокидывать. Неймспейсы выше в секции use.
@alexanderk8992
3 жыл бұрын
@@DmitryAfanasyev Почему не надо? а если нужно тест написать? Неймспейсы выше - это дичь, поскольку пользователи конфига могут не знать такую особенность, могут тупо забыть, лучше прописывать полностью в конфиге
@DmitryAfanasyev
3 жыл бұрын
Напиши Тейлору про дичь нэймспейсов, а то он их тоже в конфигах вверху херачит.
@alexanderk8992
3 жыл бұрын
@@DmitryAfanasyev а, то есть вверху кофига? тогда да, норм. не увидел в видосе, где именно use используется. поэтому и спросил про репу, чтобы более наглядно видеть код.
@alexanderk8992
3 жыл бұрын
Интрига заключалась как раз в дто?) Думаю, стоит записать видосы про дто вэлью обжект и основы ооп
@DmitryAfanasyev
3 жыл бұрын
Нет, не в дто
@alexanderk8992
3 жыл бұрын
@@DmitryAfanasyev ну, получилось, что так. Кстати, не думал замутить подкаст не с уроками, а где будешь приглашать гостей поговорить про код и программирование?
@DmitryAfanasyev
3 жыл бұрын
Типа айтиБорода? Не думал о таком формате.
@alexanderk8992
3 жыл бұрын
@@DmitryAfanasyev ну, более неофициально. Собираются челы и говорят о чем-то. Можно по темам или просто об айти в целом. Как пойдет
@_LEXX_
3 жыл бұрын
Не плохо, но как то криповато получается вложенность, new в new в new ... похоже на callback hell
@DmitryAfanasyev
3 жыл бұрын
Так и есть. Можно неплохо так усложнить код.
@alexalex6643
3 жыл бұрын
я в своё время, как только начинал учить патерны, видел такую реализацию: $order = new Sugar(new Sugar(new Coffee()));
Пікірлер: 121