Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trialguillermovega
2,058 PointsWhat is wrong with my code for this exercise?
Please see my code and advice. I do not see anything wrong, maybe the system is expecting another way to do it? or Am I misunderstanding the task I have to do?
Thank you
<?php
$books["978-0743261690"] = "Gilgamesh";
$books["978-0060931957"] = "The Odyssey";
$books["978-0192840509"] = "Aesop's Fables";
$books["978-0520227040"] = "Mahabharta";
$books["978-0393320978"] = "Beowulf";
?><html>
<head>
<title>Five Great Books</title>
</head>
<body>
<h1>Five Great Books</h1>
<ul>
<?php foreach($books as $key => $book) { ?>
<?php $isbn = $key; ?>
<li><?php echo $book; ?></li>
<?php } ?>
</ul>
</body>
</html>
1 Answer
Joel Bardsley
31,249 PointsHi Guillermo,
Note the wording of the question: Modify the foreach command so that, as it loops through the books, it loads the ISBN for each book into a working variable called $isbn.
While the code you've posted does work in practice, it is unnecessary for this question to go through the extra step of $isbn = $key when you could instead declare the $isbn variable in the foreach loop itself:
<?php
foreach ($books as $isbn => $book) {
I understand the PHP code challenges can be particularly picky, even if your output is correct, so if all else fails double check what the challenge is asking you to do. Good luck with the rest of the course.
guillermovega
2,058 Pointsguillermovega
2,058 PointsHi Joel,
Thank you!
Got it.