Podcast #128: We chat with Kent C Dodds about why he loves React and discuss what life was like in the dark days before Git. Listen now.

New answers tagged

2

First refactor the implementation to use an internal constructor internal AdPersister(AdRepository<TEntity> adRepository, IAdImagePersister adImagePersister){ _adRepository = adRepository; _adImagePersister = adImagePersister; } Effectively hiding it from use externally and giving you full controll of the class's activation The lower layer ...


3

Your implementation is good, but I would suggest following points to improve: It is really hard to test/mock source2ToCheck.getSource(). Source2 contains a lot logic/functionality (it is source type and factory) it has to be simplified. getSource() creates new course object it is confusing getSource(...) calls getAction(...) that create a source object. ...


Top 50 recent answers are included