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 shawn khah
shawn khah
15,082 Points

need help

Challenge Task 2 of 2

Update the Detail.cshtml view to be a strongly typed view. Add a model directive to strongly type the view using the Treehouse.Models.VideoGame data model type. Replace the ViewBag Title, Description, and Characters property references with the respective Model properties.

Bummer! Did you remove any of the provided markup?

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>Favorite: @Model.Favorite</h5>
<div>@Model.Description</div>

<h5>Characters: @Model.Characters</h5>
<div>
    <ul>
       >
        @foreach (var character in Model.Characters)
        {
            <li>@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 + ")";
            }
        }
    }
}

3 Answers

James Churchill
STAFF
James Churchill
Treehouse Teacher
James Churchill
James Churchill
Treehouse Teacher

Shawn,

Evan's advice is spot on. Give that a try and let us know if that works for you.

Thanks ~James

shawn khah
shawn khah
15,082 Points

thank's james

Evan Demaris
Evan Demaris
64,262 Points
Evan Demaris
Evan Demaris
64,262 Points

Hi Shawn,

It looks like the problem is with the extra markup added to your Detail.cshtml. All that's needed for part 2 of the Challenge is to add the @model Treehouse.Models.VideoGame to the beginning, then replace every instance of ViewBag with Model.

Hope that helps!

shawn khah
shawn khah
15,082 Points

thank's evan

Lee Puckett
Lee Puckett
12,539 Points

I did what you said but it didn't work.

@model Treehouse.Models.VideoGame
@{
    Model.PageTitle = "Video Game Detail";
}

<h1>@Model.Title</h1>

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

<h5>Characters: @Model.Characters</h5>
<div>
    <ul>
       >
        @foreach (var character in Model.Characters)
        {
            <li>@character</li>
        }
    </ul>
</div>
James Churchill
James Churchill
Treehouse Teacher
James Churchill
James Churchill
Treehouse Teacher

Lee,

You've changed more of the markup in the view than what is required. Also, the code challenge will complain if you add extra markup, so you'll need to avoid doing that too.

Here's a copy of the original Detail.cshtml file. 

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

<h1>@ViewBag.Title</h1>

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

<h5>Characters:</h5>
<div>
    <ul>
        @foreach (var character in ViewBag.Characters)
        {
            <li>@character</li>
        }
    </ul>
</div>

Copy and paste that code into your Detail.cshtml file and then do the following:

1) Add the @model Treehouse.Models.VideoGame at the top of the file
2) Replace every instance of ViewBag with Model

Thanks ~James

Marcus Stigall
Marcus Stigall
13,252 Points
Marcus Stigall
Marcus Stigall
13,252 Points

@model Treehouse.Models.VideoGame @{ Model.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>@character</li> } </ul> </div>

getting the error message Bummer! Did you remove the 'ViewBag.Title' property assignment to 'Video Game Detail'?

Marcus Stigall
Marcus Stigall
13,252 Points

i figured it out i wasn't supposed to change pagetitle to model