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

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

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

это же полная хуйня

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

ну вот как я и говорил, это типичная read model а тем более отчет,

тут вообще надо на ооп забить

так у него там натурально паспорт

ну я как обычно тред не читал, увидел одно последнее твое сообщение)

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

мы тут очень любим давать советы, не вникая в контекст (домен)

мы тут очень любим давать советы, не вникая в контекст (домен)

тут вообще весело) сколько людей столько и мнений

в итоге сделал вот так $passport = $this->document->getContact()->getPassport(); $doc_ser = $this->dom->createElement('doc_ser', $passport->getSeries()); $doc_num = $this->dom->createElement('doc_num', $passport->getNumber()); $org_and_place = $this->dom->createElement('org_and_place', $passport->getPlaceOfIssue());

Вроде легко читается

ну и норм

на публичных свойствах читалось бы еще легче

эм...так ты не убрал ошибку на которую указали же тебе?

он егоне обходит, а создает)

эм...так ты не убрал ошибку на которую указали же тебе?

на ошибку, какую ошибку?

он вообще структур не касается, а тут именно структура, выраженная почему-то через геттеры

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

@AlexSokolskikh вот эта

@AlexSokolskikh вот эта

так там не только номер а все данные

если это сущность то чото дохера в ней всего

выделяй контексты

если отчет то тебе уже сказали

https://habr.com/post/347908/

если это сущность то чото дохера в ней всего

почему дохера? сущьность паспорт в нем серия, номер, место и дата выдачи

попробуй почитать тут

До этого херню написал, не правильно увидел. Почему вообще в этом контексте объект document присутствует? $doc_ser = $this->dom->createElement('doc_ser', $this->document->getContact()->getPassport()->getSeries());В этом объекте у тебя должен быть уже паспорт

Должно быть $doc_ser = $this->dom->createElement('doc_ser', $this->passport->getSeries());

До этого херню написал, не правильно увидел. Почему вообще в этом контексте объект document присутствует? $doc_ser = $this->dom->createElement('doc_ser', $this->document->getContact()->getPassport()->getSeries());В этом объекте у тебя должен быть уже паспорт

он к этому и движется

До этого херню написал, не правильно увидел. Почему вообще в этом контексте объект document присутствует? $doc_ser = $this->dom->createElement('doc_ser', $this->document->getContact()->getPassport()->getSeries());В этом объекте у тебя должен быть уже паспорт

я в итоге исправил, второй вариант как раз такой, выше скинул

я в итоге исправил, второй вариант как раз такой, выше скинул

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

я в итоге исправил, второй вариант как раз такой, выше скинул

$this->documentНе должно быть в твоем объекте

Должен инжектиться паспорт.

Должен инжектиться паспорт.

ок, понял.

тут вообще надо на ооп забить

На ооп не надо, сделать красивый объект отчета, который вытянет данные и отдаст рендерилке

В сущностях не надо делать геттеры

На ооп не надо, сделать красивый объект отчета, который вытянет данные и отдаст рендерилке

так у этого объекта один фетч метод будет

В сущностях не надо делать геттеры

Скрытый пиар?

так у этого объекта один фетч метод будет

Красота!

В сущностях не надо делать геттеры

у енго это не сущности как я понял а уже дтохи готовые

Скрытый пиар?

Change my mind

В сущностях не надо делать геттеры

Егор, перелогинься

в итоге сделал вот так $passport = $this->document->getContact()->getPassport(); $doc_ser = $this->dom->createElement('doc_ser', $passport->getSeries()); $doc_num = $this->dom->createElement('doc_num', $passport->getNumber()); $org_and_place = $this->dom->createElement('org_and_place', $passport->getPlaceOfIssue());

я бы обернул юзера в спец объект , который готовит и отдает данные для xml , потом можно также сделать ещё объект который будет отдавать json и тд, ну и соответственно через фабричный метод получал бы нужный инстанс

Как вы смотрите на декорирование разрешённой зависимости? Т.е мы уже передали некой сущности некую абстракцию в конструктор, а потом нам захотелось её обернуть с помощью декоратора.

Есть что-то такое private IShotBroadcaster _shotBroadcaster; public void Init(IShotBroadcaster broadcaster) { _shotBroadcaster = broadcaster; }

Захотелось мне внезапно в одной из ситуаций добавить отклонение выстрела на основе ветра

Попахивает чем-то не очень хорошим, по этому я передумал так делать и вместо этого встроюсь в то место где зависимость разрешается (у меня там что-то типа сервис локатора)

А сам класс с методом Init как называется?

Второй вариант лучше

Но это зависит от назначения класса с зависимостью

попахивает анемичной моделью :)

но это неточно

Приписка менеджер рудиментарная, такой пока не вытревленный автоматизм

shot or shoot? :)

Сначала было Shoot

Потом я немного почитал словарь и понял что всё-таки Shot

Сам тип только собираюсь переименовать

Как вы смотрите на декорирование разрешённой зависимости? Т.е мы уже передали некой сущности некую абстракцию в конструктор, а потом нам захотелось её обернуть с помощью декоратора.

так а декоратор ты откуда берешь? тоже его прокидывашеь?

да. нынче сложно менять имена классов... автоматически

да. нынче сложно менять имена классов... автоматически

Да, в моей среде с этим сложности

При переименование типа слетает вся сериализация

так а декоратор ты откуда берешь? тоже его прокидывашеь?

Метод GetShotBroadcaster переопределяется в производном типе

Там всё и декарируется

попахивает анемичной моделью :)

Раскрой пожалуйста это

UE4 ?

UE4 ?

Юнити

UE4 ?

Там же плюсы

Там же плюсы

Кусочек шарпа есть

В виде сдк

Там же плюсы

Да, плюсы.

Скорее всего этот холивар уже поднимался (тогда плиз киньте ссыль на тред). Валидация в домене, как вы обрабатываете ошибки инвариатов, exception или что-то вроде Notification Фаулера со сбором ошибок, чтобы клиент мог увидеть все несоответствия требованиям. К примеру, что перевод не возможен тк 1. Лимит переводов исчерпан 2. Баланс на нуле

Скорее всего этот холивар уже поднимался (тогда плиз киньте ссыль на тред). Валидация в домене, как вы обрабатываете ошибки инвариатов, exception или что-то вроде Notification Фаулера со сбором ошибок, чтобы клиент мог увидеть все несоответствия требованиям. К примеру, что перевод не возможен тк 1. Лимит переводов исчерпан 2. Баланс на нуле

как правило исключения, но можно упороться и делать все ивентами

что бы можно было честно flow of control мутить

как правило исключения, но можно упороться и делать все ивентами

А что на счет объекта который собирает эти ошибки. А потом exception вместе с ним. Это портит чистоту проверок инвариатов в домене?

Или делать Either

Я хз по поводу архетиктуры и паттернрв сюда или нет. Кто подскажет как можно назвать такую архетиктуру или модель сервеоа когда : клиент отправляет запрос на сервер. Сервер тот же отвечает что запрос принят и кладет в очередь на обработку. Далее клиент переодически опрашивает сервер есть ли результат по его ревесту.

Я хз по поводу архетиктуры и паттернрв сюда или нет. Кто подскажет как можно назвать такую архетиктуру или модель сервеоа когда : клиент отправляет запрос на сервер. Сервер тот же отвечает что запрос принят и кладет в очередь на обработку. Далее клиент переодически опрашивает сервер есть ли результат по его ревесту.

асинхронная обработка, poling

А что на счет объекта который собирает эти ошибки. А потом exception вместе с ним. Это портит чистоту проверок инвариатов в домене?

смерись с тем что домен никогда не будет кристально чистым. Лучше сосредоточься на том, добавил ты там проверку потому что бизнесу это надо или нет

смерись с тем что домен никогда не будет кристально чистым. Лучше сосредоточься на том, добавил ты там проверку потому что бизнесу это надо или нет

Валидацию домена и всего остально я разделяю разумеется.

асинхронная обработка, poling

Согласен. Но тут не веб сокетов не полингов. Клиент тупо долбит с вопросом - есть для меня ответ?

Согласен. Но тут не веб сокетов не полингов. Клиент тупо долбит с вопросом - есть для меня ответ?

это и называется long-poling

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

Согласен. Но тут не веб сокетов не полингов. Клиент тупо долбит с вопросом - есть для меня ответ?

это и есть long polling один из примеров апи телеги

Валидацию домена и всего остально я разделяю разумеется.

странно зачем

проверка инвариантов это часть домена

это и есть long polling один из примеров апи телеги

Что? В телеге такое используется? Да лаааадно

Что? В телеге такое используется? Да лаааадно

https://core.telegram.org/bots/api#getupdates

Что? В телеге такое используется? Да лаааадно

вконтач и фэйсбук довольно долгое время юзали да

проверка инвариантов это часть домена

Я имею ввиду что разделяю проверку инвариантов домена и там проверку размера строчки для хранения в базе

Я имею ввиду что разделяю проверку инвариантов домена и там проверку размера строчки для хранения в базе

то есть у тебя база данных налагает ограничения которые не налагает домен? нахера?)

вконтач и фэйсбук довольно долгое время юзали да

Стоп я на всякий случай уточню. Клиент переодически спрашивает о том есть ли что для него аосредством нового запроса. Типо ддосит. Мы про одно и тоже говорим?

Стоп я на всякий случай уточню. Клиент переодически спрашивает о том есть ли что для него аосредством нового запроса. Типо ддосит. Мы про одно и тоже говорим?

именно

то есть у тебя база данных налагает ограничения которые не налагает домен? нахера?)

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

Стоп я на всякий случай уточню. Клиент переодически спрашивает о том есть ли что для него аосредством нового запроса. Типо ддосит. Мы про одно и тоже говорим?

то что ты описал - это просто полинг. Лонг полинг это когда ты подключаешься к серверу и соединение держится какое-то время (например 30 секунд) и если у сервера чего есть для клиента - выплевывается. А если нет - тайпаут и реконнект.

Ну опять же тут это все от контекста зависит