I\'m using Simple Injector, but maybe what I need is more of a conceptual answer.
Here\'s the deal, suppose I have an interface with my application settings:
You are definitely heading the wrong way here.
Some years ago I built an application that contained an interface much like your IApplicationSettings
. I believe I named it IApplicationConfiguration
, but it contained all application's configuration values as well.
Although it helped me make my application testable at first, after some time the design started to get in the way. A lot of implementations depended on that interface, but it kept changing a lot and with it the implementation, and the test version.
Just like you I implemented some lazy loading, but this had a terrible down side. When one of the configuration values was missing, I only found out that it did when the value was called for the first time. This resulted in a configuration that was hard to verify.
It took me a couple of iterations of refactoring to find out what the core of the problem was. Big interfaces are a problem. My IApplicationConfiguration
class was violating the Interface Segregation Principle and the result was poor maintainability.
In the end I found out that this interface was completely useless. Besides violating the ISP, those configuration values described an implementation detail and instead of making an application wide abstraction, it is much better to supply each implementation directly with the configuration value they need, and only the values they need.
When you do this, the easiest thing to do is to wrap those values into a Parameter Object (even if it is just one value), and inject those configuration values into the constructor. Here's an ecample:
var enableLogging =
Convert.ToBoolean(ConfigurationManager.AppSettings["EnableLogging"]);
container.RegisterSingleton(new LoggerSettings(loggingEnabled: enableLogging));
In this case, LoggerSettings
is a configuration object specific to Logger
, which requires it as constructor argument.
When doing this, the enableLogging
value is read just once from the configuration file and is done so during application startup. This makes it fast and makes it fail at application startup when the value is missing.