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 Refactoring the Codebase Separating Concerns: MVC

Mazen Lawand
Mazen Lawand
10,105 Points
Posted by Mazen Lawand
Mazen Lawand
10,105 Points

Why is my statement wrong?

I am getting an error when I input this code for the challenge, but everything seems correct.

foreach ($flavors as $flavor) { echo "<li>" . $flavor["name"] . " " . "+" . $flavor["likes"] . "</li>"; }

index.php 
<?php

// INCLUDE MODEL FILE
require_once("flavors.php");



// CONTROLLER CODE
$flavors = get_flavors_by_likes(3);



// VIEW CODE
?><html>
<body>

    <h1>Ye Olde Ice Cream Shoppe</h1>

    <p>We sell amazing flavors of ice cream.</p>

    <h2>Most Popular Flavors</h2>

    <ul>
}

    <?php
        foreach ($flavors as $flavor) {
            echo "<li>" . $flavor["name"] . " " . "+" . $flavor["likes"] . "</li>";
        }
    ?>
    </ul>

</body>
</html>
flavors.php 
<?php

// This function returns all the flavors
// ordered by the date they have been added.
function get_all_flavors() {

    $flavors = array(
        array("name" => "Vanilla", "likes" => 312),
        array("name" => "Cookie Dough", "likes" => 976),
        array("name" => "Peppermint", "likes" => 12),        
        array("name" => "Cake Batter", "likes" => 598),
        array("name" => "Avocado Chocolate", "likes" => 6),        
        array("name" => "Jalapeno So Spicy", "likes" => 3),        
    );

    return $flavors;

}

// This function receives a number as an argument.
// It returns the newest flavors, the most recent
// flavor first. (The number of flavors returned
// corresponds to the number in the argument.)
function get_recent_flavors($number) {

    $recent = array();
    $all = get_all_flavors();

    $total_flavors = count($all);
    $position = 0;

    foreach($all as $flavor) {
        $position = $position + 1;
        if ($total_flavors - $position < $number) {
            $recent[] = $flavor;
        }
    }

    return array_reverse($recent);

}

// This function receives a number as an argument.
// It returns the most popular flavors based on
// the number of likes, the most popular flavor
// first. (The number of flavors returned corresponds
// to the number in the argument.)
function get_flavors_by_likes($number) {

    $all = get_all_flavors();

    $total_flavors = count($all);
    $position = 0;

    $popular = $all;
    usort($popular, function($a, $b) {
        return $b['likes'] - $a['likes'];
    });

    return array_slice($popular, 0, $number);

}

?>

3 Answers

Renato Nobre
Renato Nobre
5,805 Points
Renato Nobre
Renato Nobre
5,805 Points

Try doing the last statement like this:

  echo "<li>" . $flavor["name"] . " +" . $flavor["likes"] . "</li>";

Your code would work, but I think the challenge just accept like that

thomascawthorn
thomascawthorn
22,986 Points
thomascawthorn
thomascawthorn
22,986 Points

Which statement! It's always best when asking questions to be as specific as possible.

Mazen Lawand
Mazen Lawand
10,105 Points
Mazen Lawand
Mazen Lawand
10,105 Points

My apologies, I went ahead and edited the question. 

<?php

foreach ($flavors as $flavor) {
     echo "<li>" . $flavor["name"] . " " . "+" . $flavor["likes"] . "</li>";
}
thomascawthorn
thomascawthorn
22,986 Points

Moorree information! What's the quesiton you're stuck on? What are you trying to do? What's the error you're getting back?