Add map projection - Githubissues

kbilsted / StatePrinter

Automating unit testing and ToString() coding

Apache License 2.0

93 stars 32 forks source link

Add map projection #35

Closed janusschmidt closed 9 years ago

janusschmidt commented 9 years ago

Adds functionality to apply a map that takes a lambda function. The lambdamust returns an anonymous object. The printer will then proceed to print the anonymous object, whenever the target type is printed.

Fixes #17

kbilsted commented 9 years ago

Awesome!!

A few things to consider

We need to update the wiki
maybe "select" is a better name than "map" so we use the linq terminology?
maybe the current general project field harvester is too general in that it holds several lists and validates. Maybe each call to map/include/... Should simply instantiate a new object which is then added to the stack (separate PR) for that.??
split the creategetter run time codegen in 2 methods for each use case

kbilsted commented 9 years ago

Looks good. Since we are planning to obsolete the Include-projection it would be nice with one or two performance tests showing if there is any overhead with this approach. Before we get ahead of ourselves dreaming up scenarios, a simple test could quite possibly show that the introspection and string-building phases dwarf the overhead of the instantiation of the anonymous type..

kbilsted commented 9 years ago

merged onto feature/mapprojection