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

Rachel Xie
Rachel Xie
5,824 Points

Cannot implicitly convert type `Treehouse.Models.VideoGame' to `Treehouse.Data.VideoGamesRepository'

Could someone please tell me where is wrong here? Much appreciation!

VideoGamesRepository.cs
using Treehouse.Models;

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

        // 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 + ")";
            }
        }
    }
}

1 Answer

Steven Parker
Steven Parker
231,248 Points

:point_right: It looks like you might be confusing task 1 and task 2.

Did you somehow peek ahead at task 2? But let's concentrate on task 1 for the moment. Here's a few hints:

  • Your code has the method taking an int argument, but...
  • The challenge said "The GetVideoGames method shouldn't accept any parameters."
  • Your code has the method returning a VideoGamesRepository, but...
  • The challenge also said it "...should return the _videoGames private static field." — and...
  • The type of the _videoGames field is VideoGame[].
  • You only need to return the field, you won't need any looping or conditionals.
Rachel Xie
Rachel Xie
5,824 Points

Ahha~ It seems something with my network. I've been shown the taks2 requirement driectly, without knowing the task1. That makes a lot more sense. Thank you, Steven~!