A yamaliev/hw19 #1207
A yamaliev/hw19 #1207
Conversation
app/nginx/Dockerfile
Outdated
| @@ -0,0 +1,5 @@ | |||
| FROM nginx | |||
There was a problem hiding this comment.
В базовом образе nginx уже всё это есть, нет необходимость создавать свой. Просто используйте базовый.
| if ($model->load(Yii::$app->request->post()) && $model->validate()) { | ||
| $message = ['start_time' => $model->start_time, 'end_time' => $model->end_time]; | ||
|
|
||
| $rabbit = new RabbitMq('bank_statement'); |
There was a problem hiding this comment.
Мы каждый раз создаём соединение с кроликом. Более того, создаём класс внутри друго-го класса. Создайте это единожды и передайте сюда как зависимость через конструктор или агрумент этого метода.
| */ | ||
| public function actionIndex(): void | ||
| { | ||
| $rabbit = new RabbitMq('bank_statement'); |
There was a problem hiding this comment.
Мы каждый раз создаём соединение с кроликом. Более того, создаём класс внутри друго-го класса. Создайте это единожды и передайте сюда как зависимость через конструктор или агрумент этого метода.
| public function actionIndex(): void | ||
| { | ||
| $rabbit = new RabbitMq('bank_statement'); | ||
| (new RabbitMqConsumer($rabbit))(); |
There was a problem hiding this comment.
Лучше сделать метод для запуска и тогда можно будет создать экземпляр этого класса один раз и передать сюда как зависимость.
| $message = ['start_time' => $model->start_time, 'end_time' => $model->end_time]; | ||
|
|
||
| $rabbit = new RabbitMq('bank_statement'); | ||
| (new RabbitMqProducer($rabbit))($message); |
There was a problem hiding this comment.
Лучше сделать метод для запуска и тогда можно будет создать экземпляр этого класса один раз и передать сюда как зависимость.
No description provided.