Porting MVC Music Store to RavenStoreController
We will start with the Index() method:
There are one this in this code that bothers me, and that is that this code is going to perform two DB queries. But that is beside the point, since we are going to modify the whole thing.
And here is my port:
As you can see, it is pretty much the same, and not really that interesting. Let us see what else we have:
Something that is important to note here is that we are doing a search on the name of a genre. The problem is that the genre name isn’t the primary key, worse, there isn’t even an index on the name column. Now, admittedly, the genre table contains ten rows, but it is the principal of the thing. (If you are smart, you only have to be read the riot act by the DBA about non index queries in production once).
Now, it would be trivial for us to implement this in Raven using the same approach, but I don’t see a reason to do this. The genre that we get in the Browse method is dependant on the data that we return from the Index method, so there is no reason no to pass the id of the genre directly. I modified the Index() action to pass the entire genre, not just the genre name, and to pass the id back to the Browse() action, not the name.
I gotten started implementing this, but I got stuck on the association of Albums from the genre.
Document database doesn’t normally have associations, and they don’t have joins. So how can we do this?
By now, you should be pretty familiar with the answer, we need to define an index :-)
// AlbumsByGenre from album in docs.Albums where album.Genre != null select new { Genre = album.Genre.Id }
And this index allows us to write this code:
And finally, we have the GenreMenu:
Which we can port very easily:
And that is all for the StoreController
More posts in "Porting MVC Music Store to Raven" series:
- (31 May 2010) StoreManagerController, part 2
- (29 May 2010) StoreManagerController
- (28 May 2010) Porting the checkout process
- (25 May 2010) StoreController
- (24 May 2010) Advanced Migrations
- (23 May 2010) Migrations
- (22 May 2010) Porting the HomeController, the Right Way
- (21 May 2010) Porting the HomeController, the map/reduce way
- (20 May 2010) Data migration
- (19 May 2010) Setting up the application
- (18 May 2010) The data model
Comments
Note to the uninitiated, this is now out of date and it is now possible to do strongly typed queries
.Where(x=>x.Genre == genre);
:)
I think the index system is very nice. Simple, maintainable and fast.
A few questions. (I haven't looked at the actual example but I believe I understand what's going on)
First, isn't the parameter to Browse passed in the url? In that case I'd much rather see /browse/jazz than /browse/17, wouldn't you?
Second, what is the static type of session.Query <genre?
And last, can you define a string primary key in Raven?
configurator,
a) the key I used was the one the original application used.
b) yes, jazz is easier than 17, but I followed the path of the app.
c) all keys in Raven are strings
"There are one this in this code that bothers me, and that is that this code is going to perform two DB queries. But that is beside the point, since we are going to modify the whole thing."
"One thing that bothers me is that this code is going to perform two DB queries. This is beside the point however, since we are going to modify the whole approach."
Comment preview