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

stuck in refactoring

Since the flavors are now ordered by the number of likes, it would be nice to display the number of likes next to each flavor. Instead of showing just the flavor name ("Cookie Dough"), change it to show the flavor name, followed by a space, followed by a plus sign, followed by the number of likes ("Cookie Dough +976"). (Look through flavors.php to see the structure of the flavors array and to figure out how to access the number of likes.)

this is index.php file

<?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>

this is flavors.php file

<?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);

}

?>

1 Answer

Alex Heil
Alex Heil
53,547 Points

hey amin, you're actually pretty close to the right answer. have a look at your foreach loop, you already passed in the right code to get the likes, but you missed 2 things actually:

1) you're missing the space and the plus sign before the like count 2) the closing li tag should be concatenated to the line

at the end the whole line inside the foreach should look like this

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

hope that helps ;)

hey Alex, Thank you very very much for your time and help! :) i just forgot one dot for concatanation at the end of list item! thank you again ;)

Alex Heil
Alex Heil
53,547 Points

you're welcome amin, good to hear this helped you out ;)