Refactor to StringDeserializer and ResponseParser

7digital / SevenDigital.Api.Wrapper

A fluent c# wrapper for the 7digital API

MIT License

16 stars 29 forks source link

Closed AnthonySteele closed 10 years ago

AnthonySteele commented 10 years ago

use new T() not Activator.CreateInstance to make an object
Add the generic constraint that DTOS much have a parameterless constructor. It did anyway, just not visibly: previous use of Activator.CreateInstance() depended on new() being present. Now we will fail at compiler-time not runtime if it's not there.
StringDeserializer is generic on the method not the class
the generic on ResponseParser can also move down to the Parse method

gregsochanik commented 10 years ago

Accidentally merged this before #165, so reverted and then merged #165. Is it possible to reopen this PR?