current community

your communities

more stack exchange communities

company blog
Stack Exchange Inbox Reputation and Badges
tour help
Join the Stack Overflow Community
Stack Overflow is a community of 6.7 million programmers, just like you, helping each other.
Join them; it only takes a minute:
Sign up
up vote -1 down vote favorite

I'm quite new to PHP and coming from a Java background. So here it goes:

I have this code:

$selected = array();
foreach($this->getSelectedOptions() AS $array) {
   array_push($selected, $array['value']);
}
var_dump($selected);

getSelectedOptions() retrieves an array of arrays containing strings.

The result is 

array
  0 => string 'abc, def' (length=31)

I was expecting something like this though: 

Array
(
    [0] => abc
    [1] => def
)

Why is this happening? How can I make my array look like the latter (without doing any post-processing with commas etc.)

Thanks!

share|improve this question
    
Seems that $array['value'] isn't actually an array (is this coming from a database?). You may need to explode(',', $array['value']);. – Rocket Hazmat May 14 '12 at 17:53
    
What is output of var_dump($this->getSelectedOptions())? – bitoshi.n May 14 '12 at 17:56
up vote 0 down vote accepted

You need to split the comma separated values and loop again like below:

$selected = array();
foreach($this->getSelectedOptions() AS $array) {
    //$array now contains comma seperated values
    //split and loop
    $values = explode(',',$array['value']);
    foreach($values as $value) {
        array_push($selected, $value);
    }
}
share|improve this answer
    
You're mixing up the variables in the loop. You're exploding an array now and for each value that comes out of the exploded array you're adding the entire string into the array. – Daniël Voogsgerd May 14 '12 at 17:58
    
Oh, right. I forgot to change the last variable. Fixed. – xbonez May 14 '12 at 18:18
    
The explode input var is also incorrect... – Daniël Voogsgerd May 14 '12 at 18:25
up vote 1 down vote

This is because the getSelectedOptions() gives you a comma seperated string instead of an array. We don't have the function so we can't do anything with that. The only thing that is possible now is post-processing. Here is some PHP doing the post-processing.

$selected = array();
foreach($this->getSelectedOptions() AS $array) {
   $values = explode(', ', $array['value']);
   array_push($selected, $values);
}
var_dump($selected);
share|improve this answer

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.