Cohort 1.5 exercise

[northernseagull]

• not sure done the datastore initializing/abstraction properly
• used http methods to get the post vars as didn't know how to get from martini
• not sure if the methods i added (GetBookById etc) are right way to do things?

[tomw-1]

• not sure done the datastore initializing/abstraction properly
• used http methods to get the post vars as didn't know how to get from martini
• not sure if the methods i added (GetBookById etc) are right way to do things?

• The datastore init is done fine. Each request is essentially in its own thread, so init'ing the context and client each time is ok. You could keep a local instance variable for each and only create if these are nil for marginal improvements
• You can use bindings to get out of the http request a little cleaner, but I'm not sure if they work with request params rather than request body
• The methods are right. But possibly in the wrong place. Having it in the models package/folder feels off. I would prefer it to have the model as being just the struct definition, then having a book service/interface which handle the get/save methods