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 trial

PHP Enhancing a Simple PHP Application Adding Search: Controller & View Accounting for Empty Results

i dont understand whats wrong with my code below. i want to insert an else block and then write inside paragraph tags

where am i missing it

index.php
<?php

    $recommendations = array();

?><html>
<body>

    <h1>Flavor Recommendations</h1>
<?php 
if(!empty($recommendations))
{?>
    <ul>
        <?php foreach($recommendations as $flavor) { ?>
            <li><?php echo $flavor; ?></li>
        <?php } ?>
    </ul>
<?php }?>
<?php   
else {
 echo "<p>There are no flavor recommendations for you.</p>";
}?>

</body>
</html>

2 Answers

Jeff Lemay
Jeff Lemay
14,268 Points

Your code is technically correct but the challenge is not passing because you close and open a php block when you don't need to. After your closing curly brace for the if statement, you close the php block with ?>. Then you immediately reopen a php block for the else portion.

Some working code

<?php

    $recommendations = array('Avocado Chocolate');

?><html>
<body>

    <h1>Flavor Recommendations</h1>
<?php
if ( ! empty($recommendations)) 
{ ?>
    <ul>
        <?php foreach($recommendations as $flavor) { ?>
            <li><?php echo $flavor; ?></li>
        <?php } ?>
    </ul>
<?php
} else { ?>
  <p>There are no flavor recommendations for you.</p>
<?php
} ?>

</body>
</html>

It's either too many php tags (as mentioned) or you're echo'ing the empty text instead of closing php tags.

Hope this helps!