Foreach loop inside foreach loop?

Question

I have a situation where only way seems to use loop inside a loop. Is that a good idea?

Like everybody else, Im looking for the most optimized solution possible.

Situation:

I need to assing different markers for each type (these are taxonomies).
Yes, these markers are markers on Google Map API.
My code below is inside while loop that gets all posts I need.
URL from code is then used in Google Map JS to assing marker image.

What I've got:

// Types   
$car_type = get_the_terms( $post->ID, 'car-type' );


foreach ( $car_type as $type ) { 


    $type_slug = $type->slug;

    foreach ( $type_slug as $type_slug ) {
        $marker_image = 'https://www.my-site.com/media/markers/marker_' . $type_slug  . '.png';   
    }
}

Pieter Goosen · Accepted Answer · 2015-09-27 19:00:52Z

0

Your 2nd foreach loop is a bug. $slug is a string with a single value, and you cannot pass a string to a foreach loop. If you set debug to true, you will see an error message telling you just this.

You can simply just do

//Different markers for different types    
$car_type = get_the_terms( $post->ID, 'car-type' );

//Apparently it don't get the slug if it's not in foreach loop 
foreach ( $car_type as $type ) { 

    $slug = $type->slug;
    $marker_image = 'https://www.my-site.com/media/markers/marker_' . $slug . '.png';  
}

answered Sep 27, 2015 at 19:00

Pieter Goosen

55.5k23 gold badges118 silver badges215 bronze badges

Thanks! If works perfectly. Im not that good with loops yet & Im having trouble to fully understand them. Do I need to use break; somewhere too?

N00b
– N00b

2015-09-27 19:28:09 +00:00
Commented Sep 27, 2015 at 19:28
You do not need to break;, except if you only need the first result. You can do the same exact thing for another taxonomy term. I'm off to bed now, if you do run into an issue, feel free to start a new question:-)

Pieter Goosen
– Pieter Goosen

2015-09-27 19:39:32 +00:00
Commented Sep 27, 2015 at 19:39

Add a comment |

Stack Exchange Network

Foreach loop inside foreach loop?

1 Answer 1

Your Answer

Hot Network Questions

Foreach loop inside foreach loop?

1 Answer 1

Your Answer

Sign up or log in

Post as a guest

Related

Hot Network Questions