eval(), не колхоз ли делать такую конструкцию?

Привет.

Задача такая: динамически объединять двумерные массивы с целью получения одномерного. Остановился на использовании array_merge, но наткнулся на статью (http://p2k.ru/archives/44), где по результатам тестов не рекомендуется многократно активировать array_merge, а советуют сначала сделать набор из массивов, а потом запихнуть его в array_merge при помощи eval().

Примерно вот так. Сделал вроде работает, вроде быстрее. Но как-то не по программистки что-ли... Что думаете?

$str[1][]=1;
$str[1][]=2;
$str[1][]=3;

$str[2][]=4;
$str[2][]=5;

$str2=array();

foreach($str as $i=>$val)$str2[]='$str['.$i.']';
eval('$str3=array_merge('.implode(',',$str2).');');
foreach($str3 as $val)echo$val;

"PHP5" Леон Аткинсон, Зеев Сураски 2006 - Избегайте использования функции eval.

Перед тем как углубиться в детали, запомните, что "eval() - это зло". ......

eval() == evil )

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

Вместо eval(), лучше применить анонимную функцию.

xlebosol
Гм ... Интересная идея, как то их не использовал ни разу

xlebosol
Нет не подойдет....это все равно вызовет многократное использование array_merge.

eval = зло, но анонимку тут не применить насколько я вижу, поскольку кол-во массивов переменно.

ABC
смотрите сами на то, как это отражается на быстродействии.

Для сравнения было бы неплохо видеть и второй ваш вариант решения без eval. Как бы то ни было, если плохой eval оказывается здесь заметно быстрее необруганного array_merge (а быстродействие - то что всегда ставится в минус eval'у), то, очевидно, ругают не то.

Тогда добро пожаловать в неизведанный мир итераторов (iteratorIterator, ArrayIterator) и т.д.. Что то array_merge не то мержит, в данном случае и без эвалов и array_merge можно обойтись.

Вот по тестил. Все же eval немного быстрее (не значительно, так что можно пренебречь). За то второй вариант красивее )

function genArray($a = 10, $b = 10)
{
    $result = array();
    for ($i = 0; $i < $a; ++$i) {

        $result[] = array_fill(0, $b, hash('adler32', rand(0, 99)));
    }
    return $result;
}

$array = genArray(500, 500);

//eval
$start = microtime(true);
$str2 = array();
foreach ($array as $i => $val) $str2[] = '$array[' . $i . ']';
eval('$str3=array_merge(' . implode(',', $str2) . ');');
var_dump(microtime(true) - $start);

//reflection
$start = microtime(true);
$reflection = new ReflectionFunction('array_merge');
$result = $reflection->invokeArgs($array);
var_dump(microtime(true) - $start);

результат

+ За проделанную работу, но разница важна.

$str[1][]=1;
$str[1][]=2;
$str[1][]=3;

$str[2][]=4;
$str[2][]=5;

//....
//$str[N]=N1;
//$str[N]=N2;

$str2=array();
foreach($str as $i=>$val)
{
    $str2=array_merge($str2,$val);//Результат:1,2,3,4,5
    //$str2=$str2+$val;//Результат:1,2,3
}
foreach($str2 as $val)echo $val;

Вот здесь исходник статьи о которой я упомянул в начале поста, там результаты тестов:
http://p2k.ru/archives/44

Может я устал уже за день и туго соображаю. Мне не понятно, как тут можно обойтись без этого:

Я же объясняю использовать для сложения не затерев ключи можно только:array_merge

Многократное использование array_merge тяжелее однократного. Для однократного нужно динамически перечислить массивы сделать это можно тогда когда array_merge вместе с аргументами будет просто строкой, а eval выполнит эту команду со стороны.

ABC
, тот пост я удалил, по тому что код оказался не совсем рабочим, а точнее совсем не рабочим ;) .

постом выше предложил вариант с reflection

$reflection = new ReflectionFunction('array_merge');
$result = $reflection->invokeArgs($array);

kaww
Разве у тебя не применяется многократная инициализация array_merge? Все просто внутри класса происходит. Так ведь?

Я не спорю, но в той статье ключевым помоему является вот эта правильная мысль:

<?php
$str[1][]=1;
$str[1][]=2;
$str[1][]=3;

$str[2][]=4;
$str[2][]=5;

$str2=array();

foreach($str as $i=>$val) {
    $str2[] = $str[$i];
}

$it = new RecursiveIteratorIterator( new RecursiveArrayIterator($str2));

foreach ($it as $key=>$val)
echo $val."\n";
?>

ABC
Как это происходит нужно спросить у разработчиков пхп http://www.php.net/manual/ru/class.reflectionfunction.php