помочь упростить функцию, которая добавляет и увеличивает "image []" в массив - PullRequest
Новая
       14

помочь упростить функцию, которая добавляет и увеличивает "image []" в массив

0 голосов
/ 21 октября 2010

В последнее время я много работал с формами и решил создать php-скрипт, чтобы упростить некоторые аспекты, которые я повторяю, я не буду публиковать полное чудовище, которое создал, но вместо этого я попрошу вас: помогите мне упростить следующий код, если это возможно: 

function add($name,$input,&$array)
{
 $p = explode('[',$name);
 if(isset($p[1]))
 {
  list($key) = explode(']',$p[1]);
  if(ctype_digit($key))
  {
    $array['forms'][$p[0]][$key] = $input;
  }else{
    $array['forms'][$p[0]][] = $input;
  } 
 }else{
    $array['forms'][$name] = $input;
 }
}


$array = array();
add('image[]','',$array);
add('image[8]','',$array);
add('image[1]','',$array);
add('image[]','',$array);

echo '<PLAINTEXT>';
print_r($array);

превращает массив в: 

Array
(
    [forms] => Array
        (
            [image] => Array
                (
                    [0] => 
                    [8] => 
                    [1] => 
                    [9] => 
                )

        )

)

проблема здесь в том, что если вы добавляете «изображение» в качестве $ name, то оно должно быть добавлено в массив, как если бы он был опубликован, поэтому это будет массив (image => data), если вы введете изображение [], тогда это будет массив (image => array (0 => data)).

Я считаю, что мой код слишком громоздкий, я нашел parse_str , который анализирует "image []", но это не помогло мне, так как мне нужно, чтобы имена добавлялись отдельно ...

Можно ли сделать эту функцию более элегантной?

пояснение:

Есть ли лучший способ добавить «имя []» в массив, как если бы он был частью списка имен, добавляемых в массив.

, поэтому мне нужна замена parse_str, которая не перезаписывает массив $. Пример: * +1021 * 

$array = array();
parse_str('image[]=test',$array);
parse_str('image[]=test1',$array);
parse_str('image[]=test2',$array);

но результат выглядит так: 

Array
(
    [image] => Array
        (
            [0] => test2
        )

)

но должен выглядеть так: 

Array
(
    [image] => Array
        (
            [0] => test
            [1] => test1
            [2] => test2
        )

)

Это действительно упростит вышеуказанную функцию!

Ответы [ 3 ]

1 голос
/ 22 октября 2010

Возможно, добавление array_merge_recursive может немного упростить ваш код.Если исходить из сигнатуры метода Джона, это может выглядеть примерно так: 

function add($value, &$target) {
    $temp = array();
    parse_str($value, $temp);
    $target = array_merge_recursive($target, $temp);
}

$array = array();
add('image[]=test',$array);
add('image[text8]=test4',$array);
add('image[]=test2',$array);
add('video[test5]',$array);

Что (как и следовало ожидать, я считаю) также дает 

Array
(
    [image] => Array
        (
            [0] => test
            [text8] => test4
            [1] => test2
        )

    [video] => Array
        (
            [test5] => 
        )

)

Надеюсь, это поможет* edit:

Если вы хотите иметь точно такое же поведение (при минимальном количестве строк), вы всегда можете перестроить строку запроса, добавить свое значение и снова проанализировать его.Результирующий код не является оптимальным или красивым, но он делает свою работу;).Иллюстрированный: 

function add($value, &$target) {
    $query = urldecode(http_build_query($target, '', '&'));
    $query = "{$query}&{$value}";
    parse_str($query, $target);
}

$array = array();

add($array, 'image[]');
add($array, 'image[8]=test4');
add($array, 'image[1]=test2');
add($array, 'image[]');

print_r($array);

Результатом будет 

Array
(
    [image] => Array
        (
            [0] => 
            [8] => test4
            [1] => test2
            [9] => 
        )

)
1 голос
/ 22 октября 2010

Я не совсем уверен, почему preg еще не был упомянут, но кажется, что это то, чего вы действительно хотите 

function add( $input, &$target )
{
    // sanity check.  If the brackets are missing, add them.
    if( strpos( $input, '[' ) === FALSE )
    {
        $input = str_replace( '=', '[]=',  $input );
    }
    // The regex means, "Get the starting variable name segment" 
    // (begins with a letter here, you could just make it \w+)
    // followed by the portion between the left and right brackets
    // and finally by the value after the period until the end of the input
    preg_match( '/^([a-zA-Z]\w*)\[(\w*)\]=?(.*)$/', $input, $matches );

    // the first value in the matches array is the original string.
    array_shift( $matches );

    // sanity check -- if the string doesn't match the above regex, ignore it.
    if( !isset( $matches[ 1 ] ) ) return;
    $m1 = $matches[ 1 ];
    $m2 = ( isset( $matches[ 2 ] ) )? $matches[ 2 ]: NULL;

    // numeric keys are translated to the equivalent of array_push.
    if( is_numeric( $m1 ) || $m1 == "" )
    {
        $target[ $matches[ 0 ] ][] = $m2;
    }
    // non-numerics are kept as is.
    else
    {
        $target[ $matches[ 0 ] ][ $m1 ] = $m2;
    }
}
1 голос
/ 21 октября 2010

Правильно, еще одна попытка с вашим разъяснением: 

function add($name,&$array)
{
    $type = explode('[',$name);
    $key = str_replace(array($type['0'], ']=', '[', ']'), '', $name);
    $array[$type['0']][] = $key;
}

$array = array();
add('image[]=test',$array);
add('image[test1]',$array);
add('image[]=test2',$array);
add('video[test5]',$array);

echo '<PLAINTEXT>';
print_r($array);

Вернется: 

Array
(
  [image] => Array
      (
          [0] => test
          [1] => test1
          [2] => test2
      )

  [video] => Array
      (
          [0] => test5
      )

)

ОК, последний раз! Насколько мне известно, не существует подходящей функции, и не так-то просто привести в порядок существующий код, но я сделал все возможное! 

function add($name,&$array)
{
    $type = explode('[',$name);
    $key = (!empty($type[1])) ? explode(']', $type[1]) : false;
    $value = str_replace(array($key[0], $type[0], ']=', '[', ']'), '', $name);
    if ($key[0]) $array[$type['0']][$key[0]] = $value;
    else $array[$type['0']][] = $value;
}

$array = array();
add('image[]=test',$array);
add('image[text8]=test4',$array);
add('image[]=test2',$array);
add('video[test5]',$array);

echo '<PLAINTEXT>';
print_r($array);

Вернется: 

Array
(
  [image] => Array
      (
          [0] => test
          [text8] => test4
          [1] => test2
      )

  [video] => Array
      (
          [test5] => 
      )

)
...