Northwind Starter Kit Review: The parents have eaten sour grapes, and the children’s teeth are set on edge
Join the DZone community and get the full member experience.
Join For Freethis is a review of the northwind starter kit project , this review revision 94815 from dec 18 2011.
in my previous posts, i focused mostly on the needlessness of the repositories implementation and why you want to avoid that (especially implementing it multiple times). in this post, i want to talk about other problems regarding the data access. in this case, the sudden urge to wash my hands that occurred when i saw this:
i mean, you are already using an or/m . i don’t like the repository implementation, but dropping down to sql (and unparapetrized one at that) seems uncalled for.
by the way, the most logical reason for this to be done is to avoid mapping the picture column to the category, since the or/m in use here (ef) doesn’t support the notion of lazy properties.
again, this is a problem when you are trying to use multiple or/ms, and that is neither required nor really useful.
okay, enough about the low level data access details. on my next post i’ll deal with how those repositories are actually being used.
Published at DZone with permission of Oren Eini, DZone MVB. See the original article here.
Opinions expressed by DZone contributors are their own.
Trending
-
What to Pay Attention to as Automation Upends the Developer Experience
-
Turbocharge Ab Initio ETL Pipelines: Simple Tweaks for Maximum Performance Boost
-
Tactics and Strategies on Software Development: How To Reach Successful Software [Video]
-
Step Into Serverless Computing
Comments