Question

I’d like to know whether this approach is correct or if their are better ways of doing this.

I have what is basically a Person class which has a number of other classes as variables, each of the custom classes is instantiated by passing the Person ID and then that class retrieves the data it needs using that ID. I expose the variable classes via Properties.

Currently I am instancing the variable classes when I create an instance of the Person class, and the large number of these mean that the time it takes to instantiate the Person class is growing. My idea was to move the instancing of the variable classes to the Propertie declaration and then using an If statement here to instantiate it if it hasn’t yet been done.

As I said above is this approach correct or is their a better way of doing this?

Thanks

Answer 1

There is a term for the technique you’re describing; it’s called ‘lazy-loaded properties’. It should definitely help spread out your load on this object away from a ‘front-loaded’ constructor.

On a different note, it sounds like what you’re describing is going to result in a terribly tightly-coupled object model (if you don’t have one already) which is likely to have a negative impact on this code’s maintainability. However, I don’t think that a serious dissertation on that topic and how to work otherwise is really within the scope of this question.