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 Modeling and Presenting Data Using Strongly Typed Views

Posted by Don-Alex Antoine
Don-Alex Antoine
8,207 Points

Replace ViewBag.Character

I believe i did this correctly. I don't know what else to do

VideoGamesController.cs 
using System.Web.Mvc;
using Treehouse.Models;

namespace Treehouse.Controllers
{
    public class VideoGamesController : Controller
    {
        public ActionResult Detail()
        {
            var videoGame = new VideoGame()
            {
                Title =  "Super Mario 64",
                Description = "Super Mario 64 is a 1996 platform video game developed and published by Nintendo for the Nintendo 64.",

                Characters = new string[]
                {
                    "Mario",
                    "Princess Peach",
                    "Bowser",
                    "Toad",
                    "Yoshi"
                }
            };

            return View(videoGame);
        }
    }
}
Detail.cshtml 
@model Treehouse.Models.VideoGame

@{
    ViewBag.PageTitle = "Video Game Detail";
}

<h1>@Model.Title</h1>

<h5>Description:</h5>
<div>@Model.Description</div>

<h5>Characters:</h5>
<div>
    <ul>
        @foreach (var character in @Model.Characters)
        {
            <li>@Model.character</li>
        }
    </ul>
</div>
VideoGame.cs 
namespace Treehouse.Models
{
    // Don't make any changes to this class!
    public class VideoGame
    {
        public int Id { get; set; }
        public string Title { get; set; }
        public string Description { get; set; }
        public string[] Characters { get; set; }
        public string Publisher { get; set; }

        public string DisplayText
        {
            get
            {
                return Title + " (" + Publisher + ")";
            }
        }
    }
}

1 Answer

Rick Gleitz
Rick Gleitz
47,876 Points
Rick Gleitz
Rick Gleitz
47,876 Points

Hi Don-Alex,

Whew, that was fun. I don't know much about this course, and I haven't progressed very far in C#, but I was able to debug this, I think.

It looks like your problem is in the second task, because your code passed the first one just fine.

In the second task, in your <ul> tag for the characters, you have:

    <ul>
        @foreach (var character in @Model.Characters)
        {
            <li>@Model.character</li>
        }
    </ul>

In the foreach loop, you've created a var called character referencing all the characters in @Model.Characters. Then you are generating a list comprised of each individual character. So your <li> should just have character in it versus @ModelCharacter.

I hope I worded this properly. I'm quite surprised I was able to figure it out. Now I have to end up doing the course, since I passed the challenge and now it's in my "In Progress" list.

Hope this helps!