Перебор и замена данных в многомерном массиве?

Question

[enigma2030]

Добрый день. Подскажите пожалуйста как можно более изящно (при этом не снижая производительность) сделать следующее.

У нас есть некий массив

$a = [
		1 => [
			3 => [
				"logo" => "logo_3_1",
				"cover" => "cover_3_1"
			],
			4 => [
				"logo" => "logo_4_1",
			]
		],
		2 => [
			5 => [
				"logo" => "logo_5_2",
				"cover" => "cover_5_2"
			]
		]
	];

В нем хранятся определенные сущности, у которых есть идентификаторы (logo_3_1, cover_3_1 и т.д).

Задача.
Нужно пройтись по этому массиву и выбрать эти идентификаторы. Потому на основании их будет сделана выборка и мы получим пути к изображениям.

Сейчас это реализована через foreach вложенный в foreach и т.д. Что для выборки идентификаторов что для замены их на пути.

Попробовал использовать итераторы

$r = [];

$ai = new RecursiveArrayIterator($a);
$ari = new RecursiveIteratorIterator($ai);

foreach ($ari as $key => $value) {
	array_push($r, $value);
}

В результате в $r у нас есть все идентификаторы.

А вот тоже самое провернуть с обновлением исходного массива не получается. Т.е так же самая конструкция не позволяет нам обновить значение.

function simpleIterator($iterator, $result)
	{
		while ($iterator->valid()) { 
	        if ($iterator->hasChildren()) {
  				simpleIterator($iterator->getChildren(), $result); 
	        } 
	        else {
	        	if ($result[$iterator->current()]) {
	        		$iterator->offsetSet($iterator->key(), $result[$iterator->current()]);
	        	};
	        }

	        $iterator->next(); 
	    }
	};

iterator_apply($ai, 'simpleIterator', [$ai, $images]);

Подскажите пожалуйста, что я делаю не так, возможно выбранный подход тупиковый.

Answer 1

[Евгений Вольф]

Нужно пройтись по этому массиву и выбрать эти идентификаторы. Потому на основании их будет сделана выборка и мы получим пути к изображениям.

array_map() не пойдёт?

P.S. Там справа ещё пачка функий на эту же тему.

Comments

[enigma2030]

Вы рекомендуете использовать ее и для выборки и для последующей замены? Рекурсивно вызывая ее же для "углубления"?

[Евгений Вольф]

enigma2030: если я правильно понял, Вам нужны не числовые индексы массива, если так - легко проверить, чисто это или нет, и когда "не число" - получаем нужные данные, сохраняем их куда-то, потом используем и т.д.

[enigma2030]

Евгений Вольф: Извините, но немного не понял о чем вы. Про "извлечение" или уже про "замену"?

[Евгений Вольф]

enigma2030: изначально - про извлечение, потом в принципе можно попробовать и заменить (про замену точно не знаю, нужно подумать)

[enigma2030]

Евгений Вольф: Начну с того что array_map() не подошло. array_walk_recursive то самое. А во вторых "не числовые индексы" сами по себе не обязательны. Я привел это как пример. Вместо logo, cover могут быть любые числа которые идентифицируют эти logo и cover. Которые в принципе при извлечении роли вообще и играют.

[Евгений Вольф]

enigma2030:

P.S. Там справа ещё пачка функий на эту же тему.

array_walk_recursive то самое.

Ну вот, частично мы Ваш код сократили/ускорили... :)

"не числовые индексы" сами по себе не обязательны.

Ещё раз, посмотрел внимательнее, у Вас в значениях может быть либо массив, либо "не массив", соотв., если это не массив, значит, это то самое значение, которое нам нужно. В принципе, его можно и получать, и изменять...

[enigma2030]

Евгений Вольф: Жалко что array_walk_recursive не возвращает значений) а то с помощью нее получилось сделать замену значений как описал ниже Сергей Семенко. А вот получить в упор не получается.

function arrayGetIterator($element, $key, $ids)
	{
		$ids[] = $element;
	}

$ids = [];

	array_walk_recursive($a, "arrayGetIterator", $ids);

	echo "<pre>";
	print_r($ids);
	echo "</pre>";

$ids в упор пустой. Но если добавил global $ids в arrayGetIterator. То все работает нормально.

[Евгений Вольф]

enigma2030:

Если требуется, чтобы функция callback изменила значения в массиве, определите первый параметр callback как ссылку. Тогда все изменения будут применены к элементам массива.

(с) Оригинал

[enigma2030]

Евгений Вольф:

Если требуется, чтобы функция callback изменила значения в массиве, определите первый параметр callback как ссылку. Тогда все изменения будут применены к элементам массива.

Это я как раз и использую для изменения, а вот получить значения не получается.

[Евгений Вольф]

enigma2030: я наверное опять что-то непонимаю, но вот-так работает:

//Изменить
array_walk_recursive($a, function(&$b) {
    $b = rand(1, 1000);
});
//Получить
array_walk_recursive($a, function(&$b) {
    var_dump($b);
});

[enigma2030]

Евгений Вольф:

array_walk_recursive($a, function(&$b) {
var_dump($b);
});

Как это присвоить переменной вне массива.

array_walk_recursive($a, function(&$b) {
$b = rand(1, 1000);
});

С этим проблем нет. Тут уже все хорошо благодаря Сергею Семенко. Разобран даже вариант если функция заполнения находится в классе. И используется оттуда.

[Евгений Вольф]

enigma2030:

Как это присвоить переменной вне массива.

$c = 1;

array_walk_recursive($a, function($b) use (&$c) {    
    $c = $b;
});

var_dump($c);

Как-то так, как вариант...

[enigma2030]

Евгений Вольф: Так у меня получилось сделать. А вот когда у нас функция в классе то это не работает. array_walk_recursive пробую так же как при изменение заполнять 3й параметр в этой функции. Но ей все равно.

[Евгений Вольф]

enigma2030: функция в классе - это как? Переменная в этой же функции? Можно пример посмотреть?

[enigma2030]

Евгений Вольф:

function bla($value, $key, &$ids)
	{
		$ids[] = $value;
	}

	array_walk_recursive($a, "bla", $ids);

Вот такой пример я имел ввиду

[Евгений Вольф]

enigma2030: так можно array_walk в эту функцию засунуть... или я опять что-то упустил? :)

[enigma2030]

Евгений Вольф:

функция в классе - это как

class A {
function bla () {}
}

[Евгений Вольф]

enigma2030: так что мешает запускать array_walk внутри этой функции?

[enigma2030]

Евгений Вольф:

так можно array_walk в эту функцию засунуть... или я опять что-то упустил? :)

А зачем он нам, если исходить из того что

function bla(&$value, $key, $logos)
  {
    if (array_key_exists($value, $logos)) {
			$value = $logos[$value];
		};
  }

  array_walk_recursive($a, "bla", $logos);

У нас изменит исходный массив. То почему при таком же подходе, но

$ids = [];
function bla($value, $key, &$ids)
  {
    $ids[] = $value;
  }

  array_walk_recursive($a, "bla", $ids);

У нас не заполнится $ids.

Вроде делаем тоже самое, "топаем" по массиву, но только не переопределяем его, а на основании его данных заполняем другой массив.

Походу я каких то тонкостей не знаю.

[Евгений Вольф]

enigma2030: function bla($value, $key, &$ids) - array_walk_recursive:

Typically, funcname takes on two parameters. The input parameter's value being the first, and the key/index second.

[enigma2030]

Евгений Вольф: php.net/manual/ru/function.array-walk-recursive.php

bool array_walk_recursive ( array &$array , callable $callback [, mixed $userdata = NULL ] )

Если указан необязательный параметр userdata, то он будет передан третьим параметром функции callback.

[Евгений Вольф]

enigma2030: дык это ж не та функция, в которую можно 3-й параметр передавать. Это 3-й параметр для array_walk_recursive(), а я говорю про callback-функцию, у нее всего 2 параметра. В примере выше - это функция bla() - у нее может быть 2 параметра, у array_walk_recursive() - 3.

[Евгений Вольф]

enigma2030: вот это примерно выглядеть (исключительно для примера):

<?php
$a = [
    1 => [
      3 => [
        "logo" => "logo_3_1",
        "cover" => "cover_3_1"
      ],
      4 => [
        "logo" => "logo_4_1",
      ]
    ],
    2 => [
      5 => [
        "logo" => "logo_5_2",
        "cover" => "cover_5_2"
      ]
    ]
  ];

class A
{
    public function bla($array)
    {    
        $ids = [];    
        array_walk_recursive($array, function($value, $key) use (&$ids, $array) {        
            $ids[] = $value;
        });

        return $ids;
    }
}

$class = new A();
var_dump($class->bla($a));

[Евгений Вольф]

enigma2030: а то, что Вы пытаетесь сделать, должно было примерно выглядеть примерно так:
1. У функции bla - 2 параметра
2. array_walk_recursive($a, "bla" use (&$ids), $ids);
Но, я пока не представляю, как вкрутить use( ... ) в функцию, вызываемую из строки.

[enigma2030]

Евгений Вольф:

дык это ж не та функция, в которую можно 3-й параметр передавать. Это 3-й параметр для array_walk_recursive(), а я говорю про callback-функцию, у нее всего 2 параметра. В примере выше - это функция bla() - у нее может быть 2 параметра, у array_walk_recursive() - 3.

Вы видимо не прочитали то что в документации)
Все верно у array_walk_recursive 3 параметра, а у callback по умолчанию 2, но если же мы в array_walk_recursive передадим 3й параметр, то он добавиться в callback как раз таки 3м параметром.

Я же писал вам

function bla(&$value, $key, $logos)
  {
    if (array_key_exists($value, $logos)) {
      $value = $logos[$value];
    };
  }

  array_walk_recursive($a, "bla", $logos);

Это работает как надо. Но такой же подход не позволяет нам "заполнить" 3й параметр. Только в этом и вся загвоздка.

[enigma2030]

Евгений Вольф:

class A
{
public function bla($array)
{
$ids = [];
array_walk_recursive($array, function($value, $key) use (&$ids, $array) {
$ids[] = $value;
});

return $ids;
}
}

$class = new A();
var_dump($class->bla($a));

И я не хочу array_walk_recursive запихивать функцию класса, я хочу в array_walk_recursive вызвать функцию из класса

[Евгений Вольф]

enigma2030: я примерно об этом и говорил, что бы так работало, должно быть:
array_walk_recursive($a, "bla" use (&$logos));
судя по всему...

Answer 2

[Сергей Семенко]

<?php

$a = [
    1 => [
        3 => [
            'logo' => 'logo_3_1',
            'cover' => 'cover_3_1',
        ],
        4 => [
            
        ]
    ],
    2 => [
        5 => [
            'logo' => 'logo_5_2',
            'cover' => 'cover_5_2',
        ]
    ]
];

$logos = [
    'logo_3_1' => 'L31',
    'logo_4_1' => 'L41',
];

array_walk_recursive($a, function (&$item, $key) use ($logos) {
    if (in_array($key, ['logo', 'cover'])) {
        if (array_key_exists($item, $logos)) {
            $item = $logos[$item];
        }
    }
});

echo '<pre>';
var_dump($a);
echo '</pre>';

Comments

[enigma2030]

Благодарю за вариант.

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

К примеру logo_3_1 у нас будет /upload/section/random_id.png
Вот как используя ваш подход в array_walk_recursive передать массив содержащий те самые /upload/section/random_id.png.

[enigma2030]

function arrayIterator(&$element, $key)
{
global $logos;

if ($logos[$element]) {
$element = $logos[$element];
};
}

Где "arrayIterator" callback для array_walk_recursive решает проблему с подменой. Но соответствует ли он варианту хорошего решения, для меня вопрос)

[Сергей Семенко]

enigma2030: в каком виде тот самый массив?

[enigma2030]

Сергей Семенко:

$logos = [
		"logo_3_1" => "L31",
		"logo_4_1" => "L41",
	];

[enigma2030]

Сергей Семенко: Сергей, отличный вариант! А подскажите как быть если мы не можем callback прописывать в array_walk_recursive. Предположим это это функция некого класса, как тогда быть при таком подходе?

[Сергей Семенко]

enigma2030: array_walk_recursive($a, [$object, 'method']);

[enigma2030]

Сергей Семенко: Я скорее как быть с "use ($logos)"

[Сергей Семенко]

enigma2030: можно передать этот массив в объект заранее

[enigma2030]

3 параметр в array_walk_recursive)

[Сергей Семенко]

enigma2030: действительно )

[enigma2030]

Сергей Семенко: php.net/manual/ru/function.array-walk-recursive.php документация наше все, но без дефолтного примера, только прочтение "построчно" и "вдумчиво" позволило найти это. А не как обычно сходу в пример смотреть)
Благодарю за ответ. Думаю на этом и остановлюсь. Правда так и не получилось разобраться как используя итераторы можно сделать то же самое)

[Сергей Семенко]

enigma2030: использовать ссылки

[Сергей Семенко]

enigma2030: обратите внимание на это: &$item

[enigma2030]

Сергей Семенко: Чет не работает

function simpleIterator(&$iterator, $logos)
	{
		while ($iterator->valid()) { 
	        if ($iterator->hasChildren()) {
  				simpleIterator($iterator->getChildren(), $logos); 
	        } 
	        else {
	        	if ($logos[$iterator->current()]) {
	        		$iterator->offsetSet($iterator->key(), $logos[$iterator->current()]);
	        	};
	        }

	        $iterator->next(); 
	    }
	};

$ai = new RecursiveArrayIterator($a);
$ari = new RecursiveIteratorIterator($ai);

iterator_apply($ai, 'simpleIterator', [$ai, $logos]);

[Сергей Семенко]

enigma2030: а зачем вообще так делать?

Можно попробовать вот так:
$ai = new RecursiveArrayIterator(&$a);

[enigma2030]

Сергей Семенко: Пробовал
syntax error, unexpected '&'
Не нравится)

[Сергей Семенко]

enigma2030: это я уже не соображаю :/ В общем не вижу смысла использовать Iterators

[enigma2030]

Сергей Семенко: )) наверное и не буду. Просто один специалист сказал что они очень удобны при таких реализациях. И вот хотелось прощупать что это за итераторы из SPL. В итоге для выборки они зашли на ура, а вот для изменения я застрял.