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 Using ViewBag

Thomas Beaudry
Thomas Beaudry
29,084 Points

ASP.NET MVC Basics

Challenge Task 2 of 2

Update the Detail.cshtml view to use the ViewBag data. Remove the title, description, and characters variables. Replace the title, description, and characters references with the respective ViewBag properties.

VideoGamesController.cs
using System.Web.Mvc;

namespace Treehouse.Controllers
{
    public class VideoGamesController : Controller
    {
        public ActionResult Detail()
        {
          ViewBag.Title = "Super Mario 64";
          ViewBag.Description = "Super Mario 64 is a 1996 platform video game developed and published by Nintendo for the Nintendo 64.";
          ViewBag.Characters = new string[]
    {
        "Mario",
        "Princess Peach",
        "Bowser",
        "Toad",
        "Yoshi"
    };  
            return View();
        }
    }
}
Detail.cshtml
@{
    Layout = null;

}

<!DOCTYPE html>

<html>
<head>
    <meta name="viewport" content="width=device-width" />
    <title>Video Game Detail</title>
</head>
<body>
    <div>
        <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>
    </div>
</body>
</html>
Allan Clark
Allan Clark
10,810 Points

what error are you getting?

Thomas Beaudry
Thomas Beaudry
29,084 Points

Bummer! Did you replace the 'title' variable reference with the 'ViewBag.Title' property?

4 Answers

Steven Parker
Steven Parker
229,967 Points

You're really close, but I see two issues:

  • C# is case sensitive, and you wrote "Viewbag" (with a lower-case "b") instead of "ViewBag" (capital "B")
  • on the foreach line, you don't need a second "@" symbol, the first one affects the entire line
Thomas Beaudry
Thomas Beaudry
29,084 Points

OMG, I can't believe I missed that, I was trying to figure out how this was wrong, I tried not adding the second @ previously, but it didnt work due to my lowercase "b"...Thank you for pointing this out :)

Steven Parker
Steven Parker
229,967 Points

Sometimes it just takes an extra pair of eyes.   :eyes:   Glad to help.
And happy coding!.

Allan Clark
Allan Clark
10,810 Points
<h1>@Viewbag.Title</h1>

Check your capitalization, change it to ViewBag.Title.

Thomas Beaudry
Thomas Beaudry
29,084 Points

Thanks for ur help also Allen Clark :)

Thomas Beaudry
Thomas Beaudry
29,084 Points

Steven Parker, I marked you as the "Best Answer" Thanks you so much ;) !

As well you have another mistake here: @foreach (var character in @ViewBag.Characters). Before ViewBag.Chaacters you don't need @, because you have @ before foreach

Steven Parker
Steven Parker
229,967 Points

That was the second of the two issues I pointed out 3 years ago. :wink: