можешь тоже набросить чего)

чуть позже прочту, я тут что-то отупел резко сижу)

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

по структуре проекта, забыл ссылку скинуть: https://gist.github.com/fesor/76d39b19b18f7103a7c058301dc6a8fe

Хорошая статья. Только почему имена папок то во множественном числе, то в единственном?

Но пока только думаю)

Хорошая статья. Только почему имена папок то во множественном числе, то в единственном?

ты про RequestMapper и Service?

хз... как-то так сложилось)

это просто пример

А чем графы рисовал?

Notifications - этот можно спокойно переименовать в единственное число и тогда в передах модуля все будет консистентно. Но называть модуль управления заказа Order я не хочу

А чем графы рисовал?

draw.io

можно было бы назвать OrderManagment но длинно.

или что-то более приближенное к домену

А есть ли инструменты, которые по коду такое сгенерят? В PHPStorm генерируются только связи наследования

А есть ли инструменты, которые по коду такое сгенерят? В PHPStorm генерируются только связи наследования

есть php dependency analyzer вроде

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

хотя это можно руку набить)

@fes0r у меня ощущение, что я как то не так использую генераторы. https://github.com/astra50/astra50.ru/pull/19/files#diff-4da5d77fff040fdbc1c6d6c4fffc6e99R58

Есть что сказать?)

тебе типа надо смерджить два генератора?

ну мол один из одного, один из второго и только потому к следующему

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

Эх, на JS-е бы интерактивне что-то, чтобы можно было кружочки подвигать.

Эх, на JS-е бы интерактивне что-то, чтобы можно было кружочки подвигать.

я видел что-то подобное. но давно дело было

в phpmetrics можешь дергать)

но не рисовать

Меня парят вызовы current() и next()

Меня парят вызовы current() и next()

норм

ну то есть... я бы больше парился что у тебя такой обход по коллекциями интересный

а так в целом конкретно для задачи - почему бы и нет

ну то есть... я бы больше парился что у тебя такой обход по коллекциями интересный

А что именно?

foreach ($this->reportTypeAndFinancing() as [$reportType, $financing]) { // code }

я бы так сделал)

или я не правильно задачу понял?

ну мол...

у тебя есть N репорт тайпов и M финансингов и надо получить N*M репортов?

foreach ($reportTypes as $type) { foreach ($financings as $financing) { yield [$type, $financing]; } }

типа

меня смущают твои бесконечные циклы

мой вариант - более явный но возможно тебе другое что-то надо в плане поведения

У меня такой логики нет, они просто друг за другом генерятся. То есть по мне должно быть что-то простое типа $report->setType($this->generateType()); А тут на 3 строки кода разложено

так ты скажи, у тебя будет min(N, M) репортов или N*M?

Такой задачи не было

понял, тебе просто надо сгенерить 30 репортов

угу

непонятно зачем тебе ++$index; если ты можешь в фориче ключик взять

или еще я часто делаю

foreach (range(0, self::REPORT_NUMBER) as $i) { $report = $this->generateReport(); $em->persist($report); } $em->flush();

а для типов репорта я бы тупо делал array_rand

Я про ключ в foreach не подумал, никогда не брал ключ если там возвращается генератор)

генераторы тут как по мне излишнее усложнение - ими я бы только репорт возвращал уже готовый

Ну я об этом и говорю

$reportTypes = ReportType::all(); $finanings = Financing::all(); while(true) { $report ->setType($reportTypes[array_rand($reportTypes)]) ->setFinancing($finanings[array_rand($finanings)]) yield $report; }

ну и в купе с foreach + range вместо while + break - будет проще восприниматься

ну while(true) внутри генератора же

ну while(true) внутри генератора же

да

ну не "генератора" а функции которая генератор возвращает

а, я понял о чём ты

https://github.com/acelaya/doctrine-enum-type

вот еще прикольная штука

а ну тебе такое не особо канает

почему все так любят трейтами обмазываться?

trait Identity

Надо к своей уже доку написать, всё лень

trait UpdatedAt - ух

Ответ в описании что такое трейты)

Ответ в описании что такое трейты)

goto в мире ооп

ERROR: S client not available

wat?

у тебя же тут нет дублирования

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

а может быть тебе понадобится updatedAt в другой сущности по другому обозвать? скорее всего ты просто заюзаешь трейт и даже не будешь об этом думать

Не значит, но такие общие вещи как id, createdAt мне нравится прятать в тейрты, чтобы при чтении сущности видеть только те поля ради которых она была создана

wat?

так же как goto заставляют тебя "прыгать" по flow of control, трейты заставляют тебя прыгать по зависимостям (файлам)

Не значит, но такие общие вещи как id, createdAt мне нравится прятать в тейрты, чтобы при чтении сущности видеть только те поля ради которых она была создана

эх... для чего только в IDE фолдинги сделали

Прятать в folding каждый раз при открытии файла ещё более не удобно чем лицезреть лишнее

Прятать в folding каждый раз при открытии файла ещё более не удобно чем лицезреть лишнее

defaultstate="collapsed"

если ты знаком со штормовским //<editor-fold

не знаком

//<editor-fold defaultstate="collapsed" description="id, timestamps and stuff..."> private $id; private $createdAt; private $updatedAt; //</editor-fold>

и каждый раз когда ты будешь открывать файлик в шторме

Это куда писать?

Вот я дичь родил private function generateReportType(): Closure { /** @var Generator $generator */ $generator = \call_user_func(function () { while (true) { foreach (ReportType::all() as $type) { yield $type; } } }); return function() use ($generator) { $generator->next(); return $generator->current(); }; }

$report->setType($typeGenerator());

Вот я дичь родил private function generateReportType(): Closure { /** @var Generator $generator */ $generator = \call_user_func(function () { while (true) { foreach (ReportType::all() as $type) { yield $type; } } }); return function() use ($generator) { $generator->next(); return $generator->current(); }; }

бедный тот джун который будет это читать

Это куда писать?

выделяешь код, Surrowd with -> editor-fold

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

class Test { //<editor-fold defaultstate="collapsed" desc="ID, TimeStamps, Stuff..."> private $id; private $createdAt; private $updatedAt; //</editor-fold> }

хм, интересная штука

может пригодится

Правда обмазываться комментами как то такое себе

ну.... зато если надо - просто открываешь фолдинг и смотришь)

не теряя контекста

Что то defaultstate="collapsed" не работает

Или текущее состояние не сбрасывается от переоткрытия таба?

не сбрасыватеся)

открыл - закрой)

а у других будет по умолчанию закрыт

(только что проверил в другой IDE)