Как правильно делать запросы в цикле?

Question

[Ilya Loopashko]

У меня есть контроллер, в котором осуществляется запрос к внешнему API. Необходимо выполнить 4 запроса. Они выполняются в цикле foreach, после каждого запроса стоит пауза в 5 секунд. Однако я не уверен, что это правильное решение. Какие способы могли бы улучшить данный функционал?

foreach ($cityList as $cityName => $coordinates) {
        $cityData[$cityName] = getCityDataFromApiService($cityName, $coordinates['latitude'], $coordinates['longitude']);
        sleep(5);
    }

Comments

[Сергей delphinpro]

Если это контроллер страницы, то пользователю придется ждать ответа страницы минимум 4*5 = 20 секунд

[Ilya Loopashko]

Сергей delphinpro, Сергей delphinpro, Нет, это небольшой парсинг данных. Данные складываю в базу данных и работаю с ними оттуда.

[Сергей delphinpro]

Ilya Loopashko, а слип зачем? Ограничение у апи?

[Ilya Loopashko]

Сергей delphinpro, Ага

Answer 1

[Максим Федоров]

it depends

• если запросы зависимые, то в цикле
  
• если запросы можно выполнить "параллельно", то можно асинхронно сделать сразу все 4 запроса и подождать самый медленный
  

например через multi curl, тогда все 4 запроса будут выполняться за время самого медленного из них, а не за сумму времени, удобно через Guzzle Async

$promise1 = $client->getAsync('http://www.example.com/foo1');
$promise2 = $client->getAsync('http://www.example.com/foo2');
$promise3 = $client->getAsync('http://www.example.com/foo3');
$promises = [$promise1, $promise2, $promise3];

$results = GuzzleHttp\Promise\settle($promises)->wait(); // тут все результаты

в обоих случаях не понятно, зачем таймаут, тк синхронно второй запрос начнется только ПОСЛЕ выполнения предыдущего, а в асинхронном варианте они просто выполнятся "одновременно" и вы получите результат сразу от всех запросов... таймаут ни к селу ни к городу

Comments

[Ilya Loopashko]

Отлично. Это то, что нужно!