Как избегать foreach внутри foreach?

Question

[Jesse Pinkman]

Есть массив

Array
(
    [0] => Array
        (
            [:stock_type] => tests
        )

    [1] => Array
        (
            [stock_name] => test
        )

)

Массив заполняется динамически.
$arr[] = $var;

Что бы получить ключ и значение нужно сделать сделать foreach два раза

Очень часто вижу на разных форумах, в коммах в вк и тд юзать цикл внутри цикла не правильно, но как этого избегать никто не пишет, по крайней мере я не нашел.

Каким образом можно сделать это по другому(правильно) ?

spoiler

Это мой третий вопрос на тостере и надеюсь, что, бог действительно любит троицу и хотя бы на этот вопрос я найду решение.

Comments

[Adamos]

Бог может любить хоть шестеренку, а Тостер любит вопросы типа "мне не нравится мой код" с кодом, который не нравится, а не растеканием по теории. Этот достоин только удаления.

[Дмитрий]

А можно пример потолще?

[Сергей delphinpro]

В вашем случае этот массив лучше изначально делать плоским

Array
(
            [:stock_type] => tests
            [stock_name] => test
)

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

[Jesse Pinkman]

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

Каким образом можно сделать это по другому(правильно) ?

Ответ, если ты стремишься помочь а не просто оставить свой след и уйти, доволно простой - Либо делать только так как у меня либо свой ответ. Иначе твоя и тебе подобных попытки провоцировать срачь уже достало, в самом деле, просто проигнорь

[Jesse Pinkman]

Сергей delphinpro, не ужели у вас в практике не было случая, когда нужно было юзать цикл внутри цикла ? Как вы с этим боролись, я могу привести кучу примеров, но моего примера достаточно, что бы дать вразумительный ответ.

[Сергей delphinpro]

Jesse Pinkman, в том-то и дело, что вашего примера недостаточно. И я указал почему.
В других случаях в использовании вложенных циклов нет ничего плохого.

[Jesse Pinkman]

Дмитрий, А что изменится ? Если я приведу пример потолще, добавим еще один цикл внутри тех двух ? И это поможет дать ответ ?

Answer 1

[index0h]

Как избегать foreach внутри foreach?

Сделать его не явным

Очень часто вижу на разных форумах, в коммах в вк и тд юзать цикл внутри цикла не правильно

Глупости. Если у вас структура данных - массив массивов даже заюзав какой-нить array_map/array_values/array_keys/array_merge/... вы все равно будете использовать двойной цикл, пусть и не явно. В случае рекурсивного обхода у вас будет точно так же цикл внутри цикла и тоже не явно.

Каким образом можно сделать это по другому(правильно) ?

Написать цикл внутри цикла и не выпендриваться.

Answer 2

[Vladislav]

Бред, если контекст задачи требует это, то делай.
К примеру. У нас есть JSON c артистами у каждого из которого есть треки.

$artists = [
            [
                'name' => 'Artist name1',
                'tracks' => [
                    [
                        'track_name' => 'My first song'
                    ],
                    [
                        'track_name' => 'My second song'
                    ]
                ]
            ],
            [
                'name' => 'Artist name1',
                'tracks' => [
                    [
                        'track_name' => 'My first song'
                    ],
                    [
                        'track_name' => 'My second song'
                    ]
                ]
            ],
        ];

Нам нужно собрать все треки всех артистов в один массив
И ты начинаешь перебирать артистов у каждого из которого далее перебираешь треки и заносишь в отдельный массив.

$all_tracks = [];
        foreach($artists as $artist)
        {
            foreach($artist['tracks'] as $track)
            {
                $all_tracks[] = $track['track_name'];
            }
        }
        var_dump($all_tracks);

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

$all_tracks = [];
        foreach($artists as $artist)
        {
            $all_tracks = array_merge($all_tracks, array_column($artist['tracks'], 'track_name'));
        }
        var_dump($all_tracks);

Хотя по моему скромному мнению он хуже. Т.к функция array_column внутри себя тоже содержит цикл, и array_merge тоже. И уже не 2, а 3 получилось цикла

Answer 3

[nokimaro]

не ужели у вас в практике не было случая, когда нужно было юзать цикл внутри цикла ? Как вы с этим боролись

Каждый день такие случаи, и я никак с этим не борюсь - потому что это не проблема. Проблема это когда вместо понимания сути ориентируешься на

Очень часто вижу на разных форумах, в коммах в вк и тд юзать цикл внутри цикла не правильно

Если хочется понять почему стоит избегать вложенных циклов, то можно почитать, и потом в конкретном своём случае принять решение стоит или не стоит...
https://ru.wikipedia.org/wiki/Временная_сложность_...
https://tproger.ru/articles/computational-complexi...

Ниже решение вопроса без вложенного foreach(), но на вскидку такой код будет не быстрее или равным по скорости для решения с вложенными циклами.

<?php
$arr = [
    0 => ['key1' => 'value1'],
    1 => ['key2' => 'value2'],
    2 => ['key3' => 'value3'],
];

foreach($arr as $data)
{
    $key = array_keys($data)[0];
    $val = $data[$key];
    
    echo "{$key}: {$val}\n";
}

//key1: value1
//key2: value2
//key3: value3

Answer 4

[HighTechLowLife]

1. Если подмассивы имеют одинаковый формат, то можно сделать что-то вроде

$stockNames = array_column($arr, 'stock_name', 'id');

Потом обращаться как echo $stockNames[$stockId] ?? '';

2. Если разный, то смысла в этом нет. Тут только два foreach'a (ну, не будем брать во внимание `array_filter`, `array_map`).

Answer 5

[Slava Rozhnev]

Посмотрите, возможно это Вам поможет:

<?php
$input = [
    ['stock_type' => 'tests'],
    ['stock_name' => 'test', 'one_more_key' => 'one_more_value']
];

$result = [];

foreach($input as $i) {
	$result = array_merge($result, $i);
}

var_export($result);

Проверить код РНР здесь

Comments

[HighTechLowLife]

1. Все это можно сократить до var_export(array_merge(...$input));
2. Если ключи в подмассивах одинаковые, то смысла мало.

[Slava Rozhnev]

HighTechLowLife, я думаю стоит это опубликовать как отдельный ответ

Answer 6

[Владислав Лысков]

Очень часто вижу на разных форумах, в коммах в вк и тд юзать цикл внутри цикла не правильно, но как этого избегать никто не пишет, по крайней мере я не нашел.

дак сам у них спроси, потом отпишись здесь в комментах

Comments

[Jesse Pinkman]

я и спросил

[Владислав Лысков]

Jesse Pinkman, и что они ответили?

[Jesse Pinkman]

Владислав Лысков,

Очень часто вижу на разных форумах, в коммах в вк и тд юзать цикл внутри цикла не правильно, но как этого избегать никто не пишет

[Владислав Лысков]

Jesse Pinkman, кароч, правильный ответ - никак этого не избежать
в данном конкретном случае можно делать так

foreach($arr[0] as $key => $value) {
echo($key);
echo($value);
}