Как в цикле foreach удалить массив из многомерного массива по ключу?

Question

[nictur1817]

Есть массив. В нем массивы, содержащие fbo и fbs могут располагаться в разных порядках [0] или [1]:

Array (
[0] => Array (
    [product_id] => 111
    [offer_id] => AR-001
    [stocks] => Array (
        [0] => Array (
            [type] => fbo
            [present] => 0
            [reserved] => 0
            )
        [1] => Array (
            [type] => fbs
            [present] => 10
            [reserved] => 0
            )
        )
    )
[1] => Array (
    [product_id] => 222
    [offer_id] => AR-002
    [stocks] => Array (
        [0] => Array (
            [type] => fbs
            [present] => 0
            [reserved] => 0
            )
        [1] => Array (
            [type] => fbo
            [present] => 20
            [reserved] => 0
            )
        )
    )
)

В цикле foreach хочу удалить внутренние массивы, где есть fbo. Чтобы на выходе получился массив, содержащий только fbs. Вот так:

Array (
[0] => Array (
    [product_id] => 111
    [offer_id] => AR-001
    [stocks] => Array (
        [0] => Array (
            [type] => fbs
            [present] => 10
            [reserved] => 0
            )
        )
    )
[1] => Array (
    [product_id] => 222
    [offer_id] => AR-002
    [stocks] => Array (
        [0] => Array (
            [type] => fbs
            [present] => 0
            [reserved] => 0
            )
         )
    )
)

Пробую так:

foreach ($items as $k => $v) {
  if ($v['stocks']['type'] == 'fbo') {
        unset($items[$k]);
    }

Не удаляется. Уже голову сломала.

Как это можно сделать?

Answer 1

[Immortal_pony]

Если есть вероятность, что ключа stocks не будет на месте, то не забудьте добавить соответствующую проверку.
Аналогично с type

foreach ($items as $itemIndex=>$item) {
    foreach($item['stocks'] as $stockIndex=>$stock) {
        if ($stock['type'] === "fbo") {
            unset($items[$itemIndex]['stocks'][$stockIndex]);
            $items[$itemIndex]['stocks'] = array_values($items[$itemIndex]['stocks']);
        }
    }
}

Comments

[nictur1817]

Это работает, теперь остались только fbs. Но возникла другая проблема.
Хочу поместить результат в массив stocks. Но на выходе получаю:

Array ( [stocks] => Array ( [0] => Array ( [offer_id] => AR-001 [stock] => 50 [warehouse_id] => 2222 )[1] => Array ( [offer_id] => AR-002 [stock] => 0 [warehouse_id] => 2222 ) ) )

Т.е. значения [stock] не верны. Там, где 0 должно быть количество, там где 10 должен быть 0. Что делаю не так?

$stocks = [
'stocks' => []  // в массиве $stocks создаем один элемент
];

foreach ($items as $itemIndex=>$item) {

    foreach($item['stocks'] as $stockIndex=>$stock) {
        if ($stock['type'] === "fbo") {
            unset($items[$itemIndex]['stocks'][$stockIndex]);
        }
    }
$stocks['stocks'][] = [
'offer_id'  =>  $item['offer_id'],
'stock' => $item['stocks'][1]['present'],
'warehouse_id' => '2222',
];
}

print_r($stocks);

[Immortal_pony]

Не совсем понял проблему и совсем не понял как вы пытаетесь её решить.
Если речь о том, что в массиве stocks не надо сохранять ключи, то и переиндексируйте этот массив:

$items[$itemIndex]['stocks'] = array_values($items[$itemIndex]['stocks']);

Обновил ответ.

[nictur1817]

Immortal_pony, Спасибо огромное! Универсальное решение. Работает и на php5 и на php8

Answer 2

[Rsa97]

Так и сделать. Циклом обойти внешний массив и для каждого его элемента отфильтровать содержимое подмассива.

Comments

[nictur1817]

А как, не подскажите?

[Сергей delphinpro]

nictur1817, вам же написали

Циклом обойти внешний массив и для каждого его элемента отфильтровать содержимое подмассива.

[nictur1817]

Сергей delphinpro,
Сделала так, теперь остались только fbs. Но возникла другая проблема.
Хочу поместить результат в массив stocks. Но на выходе получаю:

Array ( [stocks] => Array ( [0] => Array ( [offer_id] => AR-001 [stock] => 50 [warehouse_id] => 2222 )[1] => Array ( [offer_id] => AR-002 [stock] => 0 [warehouse_id] => 2222 ) ) )

Т.е. значения [stock] не верны. Там, где 0 должно быть количество, там где 10 должен быть 0. Что делаю не так?

$stocks = [
'stocks' => []  // в массиве $stocks создаем один элемент
];

foreach ($items as $itemIndex=>$item) {

    foreach($item['stocks'] as $stockIndex=>$stock) {
        if ($stock['type'] === "fbo") {
            unset($items[$itemIndex]['stocks'][$stockIndex]);
        }
    }
$stocks['stocks'][] = [
'offer_id'  =>  $item['offer_id'],
'stock' => $item['stocks'][1]['present'],
'warehouse_id' => '2222',
];
}

print_r($stocks);

[Rsa97]

nictur1817, То, что вам написал Immortal_pony не совсем верно.
Посмотрите, что получается в результате работы его кода и найдите ошибку:

Array (
    [0] => Array (
            [product_id] => 111
            [offer_id] => AR-001
            [stocks] => Array (
                    [1] => Array (
                            [type] => fbs
                            [present] => 10
                            [reserved] => 0
                        )
                )
        )
    [1] => Array (
            [product_id] => 222
            [offer_id] => AR-002
            [stocks] => Array (
                    [0] => Array (
                            [type] => fbs
                            [present] => 0
                            [reserved] => 0
                        )
                )
        )
)

Подсказка

Обратите внимание на индексы в подмассивах stocks.

Объяснение

Все массивы в PHP на самом деле ассоциативные. Поэтому, при удалении элемента с каким-либо индексом, индексы остальных элементов не меняются.
Чтобы сбросить их и заново проиндексировать массив с нуля необходимо использовать функцию array_values.

foreach ($items as &$item) {
    $item['stocks'] = array_values(array_filter(
        $item['stocks'],
        fn($el) => $el['type'] === 'fbs'
    ));
}

P.S. Написание кода методом копипасты без понимания, как этот код работает, - это один из классических методов стрелять себе в ногу.

[nictur1817]

Rsa97, Спасибо большое. Да, потом обратила внимание, что индексы остались прежними.
Решила это, обратившись к индексу через key:

$stocks = [
'stocks' => []  // в массиве $stocks создаем один элемент
];

foreach ($items as $itemIndex=>$item) {
    foreach($item['stocks'] as $stockIndex=>$stock) {
        if ($stock['type'] === "fbo") {
            unset($items[$itemIndex]['stocks'][$stockIndex]);
        }
    }
}

foreach ($items as $k => $v) {
$stocks['stocks'][] = [
'offer_id'  =>  $v['offer_id'],
'stock' => $v['stocks'][key($v['stocks'])]['present'],
'warehouse_id' => '2222',
];
}

print_r($stocks);

[nictur1817]

Rsa97, Почему-то выдает ошибку:

Fatal error: Uncaught TypeError: array_filter(): Argument #1 ($array) must be of type array, null given in ****/www/test50.php:71 Stack trace: #0 ****/www/test50.php(71): array_filter(NULL, Object(Closure)) #1 {main} thrown in ****/www/test50.php on line 71

$stocks = [
'stocks' => []  // в массиве $stocks создаем один элемент
];

foreach ($items as $itemIndex=>$item) {

    foreach($item['stocks'] as $stockIndex=>$stock) {
        if ($stock['type'] === "fbo") {
            unset($items[$itemIndex]['stocks'][$stockIndex]);
        }
    }
}

foreach ($items as &$item) {
    $item['stocks'] = array_values(array_filter(
        $item['stocks'],
        fn($el) => $el['type'] === 'fbs'
    ));
}

print_r($items);

[nictur1817]

[Rsa97]

nictur1817, Не надо копипастить всё подряд. Разберитесь, что у вас делает каждая часть кода.

[nictur1817]

Rsa97, Вы ранее написали "Чтобы сбросить их и заново проиндексировать массив с нуля необходимо использовать функцию array_values."

foreach ($items as &$item) {
    $item['stocks'] = array_values(array_filter(
        $item['stocks'],
        fn($el) => $el['type'] === 'fbs'
    ));
}

Код от Immortal_pony делает в принципе то что нужно, но индексы не меняются. Я использовала далее ваш код, чтобы заново проиндексировать массив, но выдает ошибку.
Ошибка мне непонятна, т.к. мне незнакомо такое понятие, как стрелочные функции. Подскажите, пожалуйста, как исправить ошибку? Очень нужно, целый день голову ломаю. Дело в том, что решение с использованием key не работает на сервере с php8.1

[Rsa97]

nictur1817, Мой код самостоятельный. Он и фильтрует массив и нормализует индексы.

[nictur1817]

Rsa97, Спасибо! Разобралась. Всё работает как надо теперь, с использованием вашего кода. Лаконично и изящно. Жаль только, что на php5 не работает