GKarman/hw19 rabbit-mq #1165
GKarman/hw19 rabbit-mq #1165
Conversation
README.md
Outdated
| docker-compose up -d | ||
| ``` | ||
|
|
||
| 3. Зайти внутрь php-fpm и запустить скрипт consumer |
There was a problem hiding this comment.
Было бы лучше если бы сам скрипт стартовал. Для этого можно создать отдельный контейнер в котором он будет запускаться.
| @@ -0,0 +1,24 @@ | |||
| <?php | |||
| use Gkarman\Rabbitmq\Modules\OrderReport\Domain\ValueObject\ReportDateFrom; | ||
| use Gkarman\Rabbitmq\Modules\OrderReport\Domain\ValueObject\ReportDateTo; | ||
|
|
||
| class OrderReportRequest |
There was a problem hiding this comment.
Можно воспользоваться readonly классом
| */ | ||
| private function init(): void | ||
| { | ||
| $configs = parse_ini_file('src/Configs/rabbitMQ.ini'); |
There was a problem hiding this comment.
Лучше выделить в отдельный класс, в котором разобрать файл и разложить по свойствам. Объект этого класса передать сюда через конструктор данного класса.
|
|
||
| class RabbitMQConfigs | ||
| { | ||
| private readonly ?string $host; |
| { | ||
| public function run() | ||
| { | ||
| $configs = parse_ini_file('src/Configs/rabbitMQ.ini'); |
| public function run() | ||
| { | ||
| $configs = parse_ini_file('src/Configs/rabbitMQ.ini'); | ||
| $connection = new AMQPStreamConnection($configs['host'], $configs['port'], $configs['user'], $configs['password']); |
There was a problem hiding this comment.
Лучше создать вне данного класса и сюда прокинуть как зависимость.
| { | ||
| try { | ||
| $generateOrderRequest = GenerateOrderRequest::createFromArray($request); | ||
| $useCase = new GenerateReportUseCase(new RabbitMQQueueRepository()); |
There was a problem hiding this comment.
Лучше создать вне текущего класса и сюда прокинуть как зависимость, через конструктор.
| */ | ||
| private function init(RabbitMqConfigs $rabbitMQConfigs): void | ||
| { | ||
| $this->connection = new AMQPStreamConnection( |
There was a problem hiding this comment.
Лучше создать вне текущего класса и сюда прокинуть как зависимость, через конструктор.
No description provided.