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

C# ASP.NET MVC Basics Views Displaying a List of Values

Posted by Larry Singleton
Larry Singleton
18,946 Points

ASP.NET MVC Basics

I can't see where I'm going wrong, I followed the video several times...

Link: https://teamtreehouse.com/library/aspnet-mvc-basics/views/displaying-a-list-of-values

Question:

Using the Razor syntax, write the characters string array variable value to the view within the provided <ul> element.

    Use a foreach loop to render each characters string array value within its own <li> element.
    Name your loop value variable character.
    Be sure to remove the placeholder <!-- characters --> HTML comments.

My failing answer:

 @{
    Layout = null;

    var title = "Super Mario 64";
    var description = "Super Mario 64 is a 1996 platform video game developed and published by Nintendo for the Nintendo 64.";
    var characters = new string[]
    {
        "Mario",
        "Princess Peach",
        "Bowser",
        "Toad",
        "Yoshi"
    };
}

<!DOCTYPE html>

<html>
<head>
    <meta name="viewport" content="width=device-width" />
    <title>Video Game Detail</title>
</head>
<body>
    <div>
        <h1>@title</h1>

        <h5>Description:</h5>
        <div>@description</div>
        @if (characters.Length > 0)
    {
                <h5>Characters:</h5>
        <div>
            <ul>
        @foreach(var character in characters)
        {
        <li>@characters</li>

        }

            </ul>
        </div>
    }

    </div>
</body>
</html>

This is just one variation of my attempts.

Detail.cshtml 
@{
    Layout = null;

    var title = "Super Mario 64";
    var description = "Super Mario 64 is a 1996 platform video game developed and published by Nintendo for the Nintendo 64.";
    var characters = new string[]
    {
        "Mario",
        "Princess Peach",
        "Bowser",
        "Toad",
        "Yoshi"
    };
}

<!DOCTYPE html>

<html>
<head>
    <meta name="viewport" content="width=device-width" />
    <title>Video Game Detail</title>
</head>
<body>
    <div>
        <h1>@title</h1>

        <h5>Description:</h5>
        <div>@description</div>
        @if (characters.Length > 0)
    {
                <h5>Characters:</h5>
        <div>
            <ul>
        @foreach(var character in characters)
        {
        <li>@characters</li>

        }

            </ul>
        </div>
    }

    </div>
</body>
</html>

2 Answers

Steven Parker
Steven Parker
229,732 Points
Steven Parker
Steven Parker
229,732 Points

:mailbox_with_mail: Hi, I got your request to answer.

It looks like you have the wrong name .. or maybe just a typo.

Your loop is correct, but when you render the item within the <li> tags, you call it "characters" (the whole array) instead of "character" (the individual item).

Just remove that "s" and you'll pass!

Larry Singleton
Larry Singleton
18,946 Points
Larry Singleton
Larry Singleton
18,946 Points

I did, I get the following error:

Bummer! Did you remove any of the provided markup?
Steven Parker
Steven Parker
229,732 Points
Steven Parker
Steven Parker
229,732 Points

Oh, it looks like you did too much. You should only replace the "<!-- characters -->" line with the loop that generates the <li>'s. Don't make any other changes (like adding that "@if..." line).