Порог вхождения

хз на счёт лёгкости, но от богомерзких табуляций у меня пригорает :(

так-то да, сложного ничего нету

я ещё и операторы типа else с новой строки пишу... страшный грех :( хуже чревоугодия

Гламур в разработке как он есть.

зачем?

Критику я ждал, примерно, такого плана: а зачем у тебя в EventSourcingService сначала идёт поиск event stream, а только затем (если не найдено) создание нового агрегата. Вот это - критика. А скобки не на той линии - ну такое себе, честно.

научись управлять своими ожиданиями, пример который ты привел - это не один час погружения в контекст

да ладно) меня парень за 10 минут вчера на этом впалил)

позавчера, прости

и он был чертовски прав: это тупая идея. Почти всегда event store будет nosql. если взять ту же кассандру, то там запись на порядок быстрее чтения) и лучше отбивать с помощью уникального ключа

возможно для парня это тема достаточно близка.

ну да, он тоже любитель обмазаться сервис-басами

эта хренотень вся не очень близка к пхпшникам. Её надо развернуть и попробовать покидаться командами. Там докер не зря на скотч прилеплен

ну смотря каким. вон фесор вроде тоже пхпшник :)

таких единицы. Это аще справочник ходячий)

https://github.com/mmasiukevich/service-bus/blob/master/src/MessageBus/MessageBusBuilder.php#L93 я ожидал что зависимость будет кофигурироваться явно через конфиг в котором по умолчанию будет NullLogger

да, соглашусь. Это здесь лишнее. осталось со времён, когда саги были отдельным компонентом. Спасибо

тут возможно только 2 типа? может ли появиться третий? https://github.com/mmasiukevich/service-bus/blob/master/src/MessageBus/MessageBusBuilder.php#L154

да, соглашусь. Это здесь лишнее. осталось со времён, когда саги были отдельным компонентом. Спасибо

мне не важно согласишься ты или нет - я не на экзамене

ещё есть тип query, который завернули на текущем этапе

мне не важно согласишься ты или нет - я не на экзамене

да брось :) согласие это ж типа спасиба. чо уж ты

ещё есть тип query, который завернули на текущем этапе

я бы описал дефолтное поведение в котором бы кидал exception, но возможно тут это лишнее

не, у query поведение совсем иное

пока просто не реализовывал. Надо сильно думать, как его сделать правильно

другого значения там быть сейчас не может

да брось :) согласие это ж типа спасиба. чо уж ты

цель codereview - попытка сделать код лучше и послушать отличное от твоего мнение.

https://github.com/mmasiukevich/service-bus/blob/master/src/MessageBus/MessageBusBuilder.php#L243 apply работает с обьектом? обьекты в пхп передаются же по ссылке?

apply возвращает новый объект

задача - immutable объект

меня смущает присваивание в цикле immutable - тогда я бы ожил увидеть в коде clone и слово immutable в имени класса

если бы я ещё immutable в постфикс писал, Адель бы меня на кол насадил) но идея как-то это обозначить имеет право на жизнь

ImmutableSetterlessAndPrettyCoolTask

ну я думаю это веяние дейттайма. Что бы явно их разделить

а еще этот таск написан на PHP. это тоже надо обозначить

не, простого коммента будет более, чем достаточно, мне кажется. постфикс immutable не по пср)

при каких условиях сработает это условие? https://github.com/mmasiukevich/service-bus/blob/master/src/MessageBus/MessageBusBuilder.php#L227

кажется, в дело пошёл phpstan. не при каких. И $messageNamespace там никогда не будет неймспейсом. Опять-таки, наследие. Сейчас прибью. Спасибо

не, я даже git clone не делал. просто читаю сверху вниз

есть языковые конструкции которые обычно в коде не встречаются - за них глаз и зацепляется

свежий взгляд - всегда хорошо. поправил там по списку из того, что скидывал

кстати конструкторы у тебя private, а __clone приватным почему не сделан? на это есть причины?

а какие причины делать приватным кон?

клон

clone немного про другое

я не знаю логики, поэтому и спрашиваю

у меня не было цели запретить клонирование объектов. Оно мне никак не мешает. А вот увеличить гибкость кода за счёт именованных конструкторов - почему нет.

а какие причины делать приватным кон?

например ты хочешь сделать singleton :)

про именованные конструкторы - я не спорю что они нужны, просто если один конструктор - то я предпочел бы видеть средства языка, а вот если уже 2 способа создать обьект - то имеет смысл. наиболее похожий пример - класс с 1 методом - количество абстракций вырастает и получается что сложнее анализировать код в голове (тк количество обьектов и связей сильно растет).

тут тонкая грань на самом деле. Помимо всего прочего, тебе нужно ещё и обеспечить консистентность восприятия

есть дтошки, где он нафиг не нужен. Но если часть будет через new object(), а часть через именованые - это чушь.

https://github.com/mmasiukevich/service-bus/blob/master/src/Transport/RabbitMqTransport/RabbitMqConsumer.php#L86 побить бы на методы

это да, боль. Но я в целом побью эту промисовую срань при первой же возможности, заменив на корутины

promise hell то какой

он самый

js на php ?

как со временем полегче будет, я промисы выпилю нафиг.

там самое забавное в другом: на методы-то 1 чёрт особо не разбить. Никак не разбить

https://github.com/mmasiukevich/service-bus/blob/master/src/Transport/RabbitMqTransport/RabbitMqTransport.php#L95 а если я захочу в другом месте обрабатывать сигналы?

в плане в другом месте?

https://github.com/mmasiukevich/service-bus/blob/master/src/Transport/RabbitMqTransport/RabbitMqTransport.php#L157 о_О

и что тут удивляет? демон остановлен

меня удивляет что этот механизм спрятан в самых "глубинах кода"

ты где то стартуешь демона, так вот рядом бы и хотелось видеть как ты его останавливаешь

это нормально. Никто, кроме транспортного уровня, не знает ни про твой эвент луп, ни про конкретный транспорт

всё приложение - демон

https://github.com/mmasiukevich/service-bus-demo/blob/master/testDaemon.php

у тебя не в 1 месте используется __destruct, у меня нет уверенности что exit - будет корректно отработан в таком случае, поддержка сигналов может быть вынесена на самый верх и передана как зависимость к нижним уровням

я могу каким то образом передать свой паблишер? https://github.com/mmasiukevich/service-bus/blob/master/src/Transport/RabbitMqTransport/RabbitMqTransport.php#L291 почему хочется что то создавать в getter?

сюда, нет

https://github.com/mmasiukevich/service-bus/blob/master/src/Transport/RabbitMqTransport/RabbitMqTransportConfig.php#L23 обажаю магические константы :) еще больше обажаю патчить чужие билиотеки, я надеюсь эту библиотеку не требуется патчить чтобы присоединиться к другому порту кролика?

во избежание ненужных мыслей представь, что у тебя в виде транспорта используется mysql select'om ты получаешь задачи, insert'om отправляешь обратно. Здесь зависимость прямая и обоснованная контекстом

https://github.com/mmasiukevich/service-bus/blob/master/src/Transport/RabbitMqTransport/RabbitMqTransportConfig.php#L23 обажаю магические константы :) еще больше обажаю патчить чужие билиотеки, я надеюсь эту библиотеку не требуется патчить чтобы присоединиться к другому порту кролика?

нет

Использование кучи private напоминает мне yii1, где хрен что переопределишь, даже если очень нужно. Только в помойку выкинуть ?

Использование кучи private напоминает мне yii1, где хрен что переопределишь, даже если очень нужно. Только в помойку выкинуть ?

Это как раз в копилку того, почему нет сеттеров

Используйте protected вместо private в библиотеках, пнятненько

protected'ом можно себе ноги отстрелить

нет, в библиотеках как раз не принято protected использовать

А можно и не отстрелить а ловко изменить функциональность и быстро решить задачу.

Ну или страдать и покрывать матами того кто понасовал приватов

это как раз основы этого вашего ооп. удивлён, что даже мерзкий юи так делает, но да ладно.

во избежание ненужных мыслей представь, что у тебя в виде транспорта используется mysql select'om ты получаешь задачи, insert'om отправляешь обратно. Здесь зависимость прямая и обоснованная контекстом

тогда смысл в getter то ее держать, задай эту зависимость в конструкторе

это как раз основы этого вашего ооп. удивлён, что даже мерзкий юи так делает, но да ладно.

Он потому и мерзкий, что так делает. А кое-кто везде тупо расставляет private даже не понимая что это контекстуальный инструмент

да нет) это ты не просто не понимаешь, почему private)

Но пока об этом не расскажет какой нибудь окрамиус на 40 минутном докладе, люди сами не дойдут

Надо тогда репозиторий сделать private, а то вдруг кто-то себе ногу отстрелит ;)

надо просто понимать к чему ведут protected свойства, или protected методы. Особенно во фреймворках или библиотеках. У тебя есть публичный апи, его и используй

protected в yii1 привел бы к тому что удалось бы запилить туда нормальный di контейнер и фреймворк зажил бы новой жизнью.

А так - тоже можно запилить, просто переписав пару десятков тысяч строк кода.

di контейнер как-то улучшает характеристики фреймворка, который ведь забит синглтонами?) серьёзно?)

Да, поверх можно построить целый новый дивный мир, не переписывая 200cloc легаси проекта

да и как dic поможет я лично понятия не имею. тебе никто не мешает для клиентской части кода его использовать. Зачем он тебе вбитый ногами в фреймворк?

я не вижу связи, если честно.

паш, протектед во фреймворке - это апи. со всем вытекающим BC

по хорошему, там аще все классы final должны быть, что бы отбить все наркоманские фантазии. Код фреймворка самодостаточен

а все влажные мечты программиста покрываются композицией

Ну понятно что самодостаточен, вот он никому уже и не нужен )

он не нужен потому, что сам фреймворк донный. Но, повторюсь, если тебе так хотелось DIC, что мешало его для своего кода прикрутить?

паш, протектед во фреймворке - это апи. со всем вытекающим BC

Да не, это точно не yii1-way. Они не особо следили за BC