PHP array extract

Question

I have a querystring :

"condition=good;condition=not-good&features=ABS&features=ESP&features=ENT&brand=Honda&model=Traffic"

*please note duplicate parameter

I use this function to convert and - get also duplicate key - to array :

function proper_parse_str($str) {
  # result array
  $arr = array();

  # split on outer delimiter
  $pairs = explode('&', $str);

  # loop through each pair
  foreach ($pairs as $i) {
    # split into name and value
    list($name,$value) = explode('=', $i, 2);

    # if name already exists
    if( isset($arr[$name]) ) {
      # stick multiple values into an array
      if( is_array($arr[$name]) ) {
        $arr[$name][] = $value;
      }
      else {
        $arr[$name] = array($arr[$name], $value);
      }
    }
    # otherwise, simply stick it in a scalar
    else {
      $arr[$name] = $value;
    }
  }

  # return result array
  return $arr;
}

In order to echo html I use this :

//using the above function
$array=proper_parse_str($string);
foreach ($array as $key => $value) {
    if (is_array($value)) {
        foreach($value as $t) {
            $e .="<li>".$t."</li>";
        }
        $mkey .="<ul><li><b>".$key."</b><ul>".$e."</ul></li></ul>";
    } else {
        $tt ="<li>".$value."</li>";
        $mkey .="<ul><li><b>".$key."</b><ul>".$tt."</ul></li></ul>";
    }
}  
echo $mkey;

to get :

Condition
   good
   not-good
Features
   ABS
   ESP
   ENT
Brand
   Honda
Model
   Traffic

but I get :

Condition
   good
   not-good
Features
   **good
   **not-good
   ABS
   ESP
   ENT
Brand
   Honda
Model
   Traffic

Please help me..

Why are the first two pairs separated by a ; instead of a &? — Nedec
– Nedec, Commented Dec 13, 2010 at 14:27
Do a print_r( $arr ) to check if your construction is actually correct. The print-out code looks a bit incomplete. — poke
– poke, Commented Dec 13, 2010 at 14:29
@Nadec: I would say type since there's nothing in the code to look for it. — Brad Christie
– Brad Christie, Commented Dec 13, 2010 at 14:31

ircmaxell · Accepted Answer · 2010-12-13 14:29:01Z

5

You never initialized $e in your echoing code. So it will always just append the new values rather than resetting. Try this:

$array=proper_parse_str($string);
$mkey = '';
foreach ($array as $key => $value)  {
    if (is_array($value)) { 
        $e = '';
        foreach($value as $t){
            $e .="<li>".$t."</li>"; 
        }  
        $mkey .="<ul><li><b>".$k."</b><ul>".$e."</ul></li></ul>";
    }  else {        
        $tt ="<li>".$value."</li>";
        $mkey .="<ul><li><b>".$key."</b><ul>".$tt."</ul></li></ul>";
    }   
}  
echo $mkey;

The moral of the story: always initialize your variables...

answered Dec 13, 2010 at 14:29

ircmaxell

166k36 gold badges269 silver badges316 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Brad Christie · Accepted Answer · 2010-12-13 14:32:08Z

1

Why not use parse_str? (Assuming what you're doing is typical to parsing a GET argument string).

answered Dec 13, 2010 at 14:32

Brad Christie

102k16 gold badges160 silver badges200 bronze badges

3 Comments

ircmaxell Over a year ago

It's not identical (I was going to say the same thing, but then realized the difference). Basically this simulates array syntax without the need for []. With parse_str, identical keys overwrite, so foo=bar&foo=baz would only result in foo=baz. But his implementation would transform it to foo[]=bar&foo[]=baz. Slightly different (and not fitting with the spec), but it is different...

Brad Christie Over a year ago

Ah, good catch. Seems almost easier to catch these duplicates and adjust them, then parse with the native function. I guess to each their own.

ircmaxell Over a year ago

That's what I would likely do if I had to solve this problem myself. That way you're still coming up with valid query strings, and have the benefit of the decoding/etc that the native function gives...

Poelinca Dorin · Accepted Answer · 2010-12-13 14:36:19Z

0

function proper_parse_str($str) {
  # result array
  $arr = array();

  # split on outer delimiter
  $pairs = explode('&', $str);

  # loop through each pair
  foreach ($pairs as $i) {
    # split into name and value
    list($name,$value) = explode('=', $i, 2);
    $arr[$name][] = $value;
  }

  # return result array
  return $arr;
}
//using the above function
        $array=proper_parse_str("condition=good&condition=not-good&features=ABS&features=ESP&features=ENT&brand=Honda&model=Traffic");
echo "<ul>";
foreach ($array as $key => $value)
{
    echo "<li>$key<ol>";
    foreach ( $value as $k => $v )
    {
        echo "<li>$v</li>";
    }
    echo "</ol></li>";
}  
echo "</ul>";

answered Dec 13, 2010 at 14:36

Poelinca Dorin

9,7052 gold badges41 silver badges43 bronze badges

1 Comment

morowind Over a year ago

Manny smart / experts guys here...thank's this solution works too.

Aether · Accepted Answer · 2010-12-13 14:32:27Z

0

Can you change your query string? If so, you may want to look at using arrays. E.g.

condition[]=good&condition[]=not-good&features[]=ABS&features[]=ESP&features[]=ENT&brand[]=Honda&model[]=Traffic

gives:

array
  'condition' => 
    array
      0 => string 'good' (length=4)
      1 => string 'not-good' (length=8)
  'features' => 
    array
      0 => string 'ABS' (length=3)
      1 => string 'ESP' (length=3)
      2 => string 'ENT' (length=3)
  'brand' => 
    array
      0 => string 'Honda' (length=5)
  'model' => 
    array
      0 => string 'Traffic' (length=7)

answered Dec 13, 2010 at 14:32

Aether

3642 silver badges5 bronze badges

Comments

poke · Accepted Answer · 2010-12-13 14:33:43Z

0

You can do the string generation a bit more simple using an array join.

$mkey = '';
foreach ( proper_parse_str( $string ) as $key => $value )
{
    if ( is_array( $value ) )
        $value = implode( '</li><li>', $value );

    $mkey .= '<li><b>' . $key . '</b><ul><li>' . $value . '</li></ul></li>';
}
echo $mkey;

Regardless of that, I would suggest you to always use an array as the value holder in your array to make your structure more static.

answered Dec 13, 2010 at 14:33

poke

392k80 gold badges596 silver badges632 bronze badges

Collectives™ on Stack Overflow

PHP array extract

5 Answers 5

Comments

3 Comments

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

3 Comments

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related