private IEnumerable<Book> GetBooks(ViewBooksLinkModel model)
{
var shouldDefaultToAllGenres = ShouldDefaultToAllGenres(model);
return shouldDefaultToAllGenres
? bookRetriever.GetReviewedBooks()
: bookRetriever.GetReviewedBooks(model.Genre);
}
public ViewBooksViewModel Get(ViewBooksLinkModel model)
{
var books = GetBooks(model);
var models = books.ToList().Select(b => new BookListView(b));
var wishlistBooks = bookRetriever.GetWishlistBooks().ToList().Select(b => new BookListView(b));
return new ViewBooksViewModel(models, genreRetriever.GetAll(), model.Genre, wishlistBooks);
}
// TODO - Should this logic live here? 100% yes - it is application behaviour specific to this view
private bool ShouldDefaultToAllGenres(ViewBooksLinkModel model)
{
return string.IsNullOrWhiteSpace(model.Genre)
|| !genreRetriever.CanFindGenreWith(model.Genre);
}