ericdc1/Dapper.SimpleCRUD — pull request #1 by YogirajA (2013-03-17 13:26:12+00:00)
Title: Changed to avoid possible multiple enumerations of IEnumerable

Hi Eric,

First of all, I like this project! It is less opinionated about the database.

I changed the SimpleCRUD class to avoid what subject line says. I did this by calling ToList or ToArray whichever was appropriate. It makes things more efficient.

You can read more about it at
[JetBrains site](http://confluence.jetbrains.com/display/ReSharper/Possible+multiple+enumeration+of+IEnumerable)

Jon Skeet has explained it more in this [StackOverflow answer](http://stackoverflow.com/questions/15041994/linq-tolist-toarray-todictionary-performance). He also has links to his own relevant blog posts.

I also removed some of the variables and usings that were not being used from Test.cs and P

I used to resharper to help me through process. If you are interested in that, they offer free copy for open source projects, just saying.

I hope to make use of this more and show it off in my presentation at Hartford,CT codecamp.

Please let me know if you what you think.

Thanks,

Yogi