Как убрать вложенный array merge?

Question

[redisdev]

Как убрать вложенный array merge ? Нужно что-то вроде сливания списка массивов после завершения цикал, то есть генерируем массивы и далее за один раз сливаем сипользуя array merge. Код который нуэно улучшить:

foreach ($contentIds as $contentId) {
                $commentsLists = array_merge($commentsLists, $this->commentGetter->getCommentsRedis(...)
                ));
}

Comments

[Геннадий]

Почему просто разом по массиву id не получить список комментариев?
Правильно ли я понимаю, что $this->commentGetter->getCommentsFromStorage(...) обращается к БД и по id получает информацию о комментарии? Если так, то зачем вы перебираете их в цикле и для каждого отдельно вызываете метод, если это можно сделать одним запросом?

[Adamos]

Есть подозрение, что решается проблема XY.
Борьба с array_merge в ситуации, когда вместо одного запроса к БД в нее долбится "цикал".

[redisdev]

Геннадий, Получаю из Redis комментарии

[redisdev]

Вопрос именно в том как убрать array merge в данном случае

[Vitsliputsli]

redisdev,

Код который нуэно улучшить

А какая цель? Что нужно улучшить? Скорость работы? Использование памяти? Читаемость?

[redisdev]

скорость

[Vitsliputsli]

redisdev, т.е. это место создает проблему для всего проекта? изза того что эти 3 строчки отрабатывают очень медленно наступает какая то беда? Я к тому, что не решаете проблему, если ее нет.
Если она есть, то получайте из редиса сразу пачкой, а не отдельными запросами. Если это невозможно, то проверьте, почему array_merge, возможно там не нужно сравнение ключей и переиндексация, т.е. нужен оператор +, или вообще второй массив это массив из 1 значения и достаточно запушить только его. Т.е. нужно анализировать данные которые обрабатываются.

[Ипатьев]

Vitsliputsli, ну вообще, это "алгоритм маляра Шлемиэля", и делать мёрж на каждой итерации, пожалуй, действительно неэффективно.

Другое дело, что как и во всех других случаях "плохой производительности" проблема не в алгоритме, а в объеме обрабатываемых данных. То есть он зачем-то достаёт офигиллиард записей из редиса и кладёт в массив. Соответственно, сначала он должен ответить себе на вопрос - а нафига?

Answer 1

[androidmedia]

Создай новый массив в цикле его заполни, а после цикла уже объедини два массива.

Answer 2

[Александр Черняев]

$comments = []; 

foreach ($contentIds as $contentId) {
    $comments[] = $this->commentGetter->getCommentsRedis(...);
}

$comments = array_merge(...$comments);