Simple if empty conditional Task 1 of 3.

Question

Hi,

Task is:

The following code contains a foreach loop that displays a list of ice cream flavor recommendations. It works great when the $recommendations array has at least one element, but we end up with some extra HTML when we don't have any recommendations. Write a conditional that makes sure the unordered list tags and the foreach loop get executed only if the array has at least one element.

My code:

<?php

$recommendations = array();

?><html> <body>

<h1>Flavor Recommendations</h1>

<?php if(isset($recommendations)) { echo "<ul>"; foreach($recommendations as $flavor) { echo "<li>"; echo $flavor; echo "</li>"; } echo "</ul>";

} else {'<p>"Do nothing"</p>'; }

?>

Getting an error saying that the ul tags are still appearing.

I have a feeling this will be a simple one that I've just completely overlooked, feeling very overworked on php today :(

Thanks,

Adam

Answer 1 · 2014-07-25T16:30:30Z

July 25, 2014 4:30pm

Hi Adam,

The isset function won't work as it's designed to check whether or not a variable has been defined in the current scope, the function you want to use is either sizeof (alias of count) or count which do the same thing, see the below.

<?php if (sizeof($$recommendations) > 0) { ?>
<ul>
    <?php foreach($recommendations as $flavor) { ?>
        <li><?php echo $flavor; ?></li>
    <?php } ?>
</ul>
<?php } ?>

PHP Ref: count - sizeof

Welcome to the Treehouse Community

Looking to learn something new?

Adam Duffield

Adam Duffield

Simple if empty conditional Task 1 of 3.

1 Answer

Chris Shaw

Chris Shaw

Adam Duffield

Adam Duffield