PHP nested foreach

Question

I thought that I understood php arrays, but it seems I don't :( This is my code:

<?php

// Create a magazines array
$magazines = array();

// Put 5 magazines on it
for($x=0;$x<5;$x++)
    $magazines[] = "Magazine " . $x ;


// Associate articles array to each magazine
foreach($magazines as $magazine){
    $articles[$magazine] = array();
    for($x=0;$x<3;$x++)
        $articles[$magazine] = $magazine . " - Article " . $x ;
}

// List them all
foreach($magazines as $magazine){
    echo $magazine . " has these articles: <br>";
    foreach($articles[$magazine] as $article)
        echo $article . "</br>";
}

?>

It only prints the magazines, not the articles inside each magazine. It's clear there is something I don't get about nested foreach loops. Could you please help me? What am I doing wrong? Thanks in advance! :)

Did you mean to put the articles inside a magazine? 'cos your code's the other way around! — Wayne
– Wayne, Commented Jul 23, 2011 at 6:44

Bill the Lizard · Accepted Answer · 2011-11-06 12:20:42Z

4

You missed the [] in the nested loop:

<?php

// Create a magazines array
$magazines = array();

// Put 5 magazines on it
for($x=0;$x<5;$x++)
    $magazines[] = "Magazine " . $x ;


// Associate articles array to each magazine
foreach($magazines as $magazine){
    $articles[$magazine] = array();
    for($x=0;$x<3;$x++)
        $articles[$magazine][] = $magazine . " - Article " . $x ;
}

// List them all
foreach($magazines as $magazine){
    echo $magazine . " has these articles: <br>";
    foreach($articles[$magazine] as $article)
        echo $article . "</br>";
}

?>

This snippet should work

edited Nov 6, 2011 at 12:20

Bill the Lizard

407k213 gold badges579 silver badges892 bronze badges

answered Jul 23, 2011 at 6:41

Fabrizio D'Ammassa

4,77928 silver badges32 bronze badges

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

2 Comments

Jorge Over a year ago

Hi Fabrizio, It works! Now I understand how it works and what my mistake was. Thank you so much!

vstm Over a year ago

I came to the same conclusion (+1). It's also worth mentioning that this error could've been easily detected by setting error_reporting to E_ALL | E_STRICT.

czerasz · Accepted Answer · 2011-07-23 06:41:44Z

2

Try changing this code part:

$articles[$magazine][] = $magazine . " - Article " . $x ;

answered Jul 23, 2011 at 6:41

czerasz

14.4k10 gold badges56 silver badges63 bronze badges

Collectives™ on Stack Overflow

PHP nested foreach

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related