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 Adding Repository Methods

Kyle Plancich
Kyle Plancich
7,482 Points
Posted by Kyle Plancich
Kyle Plancich
7,482 Points

Please help me understand this!

I just don't get what I'm doing wrong with this code. I don't under stand how I can set the method to not accept variables and not throw an error?! I have watched the video several times. I just don't get it!

VideoGamesRepository.cs 
using Treehouse.Models;

namespace Treehouse.Data
{
    public class VideoGamesRepository
    {
        // TODO Add GetVideoGames method
      public  VideoGame GetVideoGames()
      {
          VideoGame videoGameReturn = null;
          foreach (var videoGame in _videoGames)
          {
            if (videoGame.Id == id)
            {
                videoGameReturn = videoGames;
                break;
            }
          }
        return videoGameReturn;

      }
        // TODO Add GetVideoGame method

        private static VideoGame[] _videoGames = new VideoGame[]
        {
            new VideoGame()
            {
                Id = 1,
                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"
                },
                Publisher = "Nintendo",
                Favorite = true
            },
            new VideoGame()
            {
                Id = 2,
                Title = "Mario Kart 64",
                Description = "Mario Kart 64 is a 1996 go-kart racing game developed and published by Nintendo for the Nintendo 64 video game console.",
                Characters = new string[]
                {
                    "Mario",
                    "Princess Peach",
                    "Bowser",
                    "Toad",
                    "Yoshi"
                },
                Publisher = "Nintendo",
                Favorite = false
            }
        };
    }
}
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 bool Favorite { get; set; }

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

2 Answers

tjgrist
tjgrist
6,553 Points
tjgrist
tjgrist
6,553 Points

You haven't made a method that returns the _videoGames list. Do that first. It's simple, write the first method here:

        public VideoGame[] GetVideoGames(){
            return _videoGames;
        }

        public VideoGame GetVideoGame(int id){
            foreach (VideoGame game in _videoGames){
                if (game.Id == id){
                    return game;
                }
            }
            return null;
        }

The second method is for searching for a videogame. You are close but you have a typo in the "if" statement. If the video game is not found, you can return null.

So the first method is saying that you are returning an array of videogames: 

public VideoGame[] GetVideoGames()

The second method you are returning just one VideoGame. 

public VideoGame GetVideoGame()

Does that make sense? Let me know if you have any questions.
Kyle Plancich
Kyle Plancich
7,482 Points
Kyle Plancich
Kyle Plancich
7,482 Points

TJ Grist, Thank you for helping me. I was so lost with that!