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 trialBen Atkinson
6,540 PointsI'm not sure what I'm doing wrong, I didnt think I was trying to recast or anything, please help..
using Treehouse.Models;
namespace Treehouse.Data { public class VideoGamesRepository { // TODO Add GetVideoGames method
public VideoGame GetVideoGames()
{
VideoGame[] videoToReturn = null;
foreach (var _videoGame in _videoGames)
{
videoToReturn = _videoGame;
}
return videoToReturn;
}
// 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
}
};
}
}
using Treehouse.Models;
namespace Treehouse.Data
{
public class VideoGamesRepository
{
// TODO Add GetVideoGames method
public VideoGame GetVideoGames()
{
VideoGame[] videoToReturn = null;
foreach (var _videoGame in _videoGames)
{
videoToReturn = _videoGame;
}
return videoToReturn;
}
// 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
}
};
}
}
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
Edward Ries
7,388 PointsLets take a look at this method. Your return value is VideoGame but your return statement is returning videoToReturn which is oof time VideoGame[] which is an array of video games. I believe you over complicated this method. Try changing the contents to return _videoGames; and change the return type on the method to VideoGame[]
public VideoGame GetVideoGames() { VideoGame[] videoToReturn = null; foreach (var _videoGame in _videoGames) { videoToReturn = _videoGame; }
return videoToReturn;
Ben Atkinson
6,540 PointsThanks for the help, I tried a heap of things, and I actually thought I tried that early on but I obviously didn't. Thanks again ?