Looping through multiple arrays and display select options

Question

I have a multi-select box for country selection. I want to select any countries which are associated, meaning an array I get from the database.

Here's the code I have:

                        <?php
                        foreach($countries as $country){
                            if(!empty($offer_countries)){
                                foreach($offer_countries as $key => $offer_country){
                                    if(isset($offer_country['country_id']) && ($offer_country['country_id'] == $country['id'])){
                                        echo '<option value="'.$country['id'].'" selected>'.$country['name'].'</option>';

                                    }else{
                                        echo '<option value="'.$country['id'].'">'.$country['name'].'</option>';

                                    }
                                }
                            }else{

                                echo '<option value="'.$country['id'].'">'.$country['name'].'</option>';

                            }
                        }
                    ?>

The $offer_countries array, looks like this:

Array
(
    [0] => Array
        (
            [country_id] => 1
        )

    [1] => Array
        (
            [country_id] => 2
        )

    [2] => Array
        (
            [country_id] => 3
        )

)

I'm looping all countries to display them, then I have a nested foreach to see if the country is already set, if so, make the option box selected.

The problem with this is that let's say I have 3 items selected, it'll display 3 of the same country, based on the number of items in the array. So if United States should be checked, it'll show it three times, with the last one checked.

Ok, sorry for the looong explanation, it's probably fairly self explanatory, but any help would be awesome!

in_array and array_search wont work because it's arrays within an array — dzm
– dzm, Commented May 12, 2011 at 21:42
I suppose array_uintersect could work, but I doubt there'd be a performance improvement in this case. — Kevin Peno
– Kevin Peno, Commented May 12, 2011 at 22:03

dzm · Accepted Answer · 2011-05-12 21:50:13Z

This solved it:

                        <?php
                        foreach($countries as $country){
                            $i = 0;
                            if(!empty($offer_countries)){
                                foreach($offer_countries as $key => $offer_country){
                                    if($offer_country['country_id'] == $country['id']){
                                        echo '<option value="'.$country['id'].'" selected>'.$country['name'].'</option>';
                                        $i = 1;
                                        break;
                                    }
                                }
                                if($i == 0){
                                    echo '<option value="'.$country['id'].'">'.$country['name'].'</option>';
                                }
                            }else{

                                echo '<option value="'.$country['id'].'">'.$country['name'].'</option>';

                            }
                        }
                    ?>

RonLugge · Accepted Answer · 2011-05-12 21:25:27Z

0

Your inner 'foreach' statement is going to output 'something' whether or not the value is set, and it does so based on the $country variable set up in the outer foreach loop.

So what happens is that you output on the outer 'foreach' loop once for each time it runs on the inner foreach loop.

answered May 12, 2011 at 21:25

RonLugge

5,1845 gold badges39 silver badges64 bronze badges

2 Comments

dzm Over a year ago

How do I fix that? I tried added breaks after each output on the inner loop, but then it only selects the last item in the array

RonLugge Over a year ago

Looks like you solved it while I was busy elsewhere. You actually did it more or less the way I would have suggested, using an external boolean value as a flag to track what happened inside the inner foreach loop.

Collectives™ on Stack Overflow

Looping through multiple arrays and display select options

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related