Disadvantages of Lazy<T>?
I'll expand a bit on my comment, which reads:
I've just started using Lazy, and find that it's often indicative of bad design; or laziness on the part of the programmer. Also, one disadvantage is that you have to be more vigilant with scoped up variables, and create proper closures.
For example, I've used Lazy<T>
to create the pages the user can see in my (sessionless) MVC app. It's a guiding wizard, so the user might want to go to a random previous step. When the handshake is made, an array of Lazy<Page>
objects is crated, and if the user specifies as step, that exact page is evaluated. I find it delivers good performance, but there are some aspects to it that I don't like, for example many of my foreach
constructs now look like this:
foreach(var something in somethings){
var somethingClosure = something;
list.Add(new Lazy<Page>(() => new Page(somethingClosure));
}
I.e. you have to deal with the problem of closures very proactively. Otherwise I don't think it's such a bad performance hit to store a lambda and evaluate it when needed.
On the other hand this might be indicative that the programmer is being a Lazy<Programmer>
, in the sense that you'd prefer not thinking through your program now, and instead let the proper logic evaluate when needed, as with example in my case - instead of building that array, I could just figure out just what that specific requested page would be; but I chose to be lazy, and do an all in approach.
EDIT
It occurs to me that Lazy<T>
also has a few peculiars when working with concurrency. For example there's a ThreadLocal<T>
for some scenarios, and several flag configurations for your particular multi-threaded scenario. You can read more on msdn.
In my opinion, you should always have a reason for choosing Lazy. There are several alternatives depending on the use case and there are definitely cases where this structure is appropriate. But don't use it just because it's cool.
For example I don't get the point in the page selection example in one of the other answers. Using a list of Lazy for selecting a single element can be well done with a list or dictionary of delegates directly without using Lazy or with a simple switch statement.
So the most obvious alternatives are
- direct instantiation for cheap data structures or structures that are anyway needed
- delegates for things that are needed zero to few times in some algorithm
- some caching structure for items that should free the memory when not being used for some time
- some kind of "future" structure like Task that already may start initializing asynchronously before actual usage consuming idle CPU time in cases where the probability is quite high that the structure will be required later on
In contrast to that, Lazy is often suitable when
- computationally intense data structures
- are needed zero to many times in some algorithm where the zero case has a significant probability
- and the data is local to some method or class and can be garbage collected when not in use any more or the data should be kept in memory for the whole program's runtime
Here's not quite a negative aspect, but a gotcha for lazy people :).
Lazy initializers are like static initializers. They get run once. If an exception is thrown, the exception is cached and subsequent calls to .Value would throw the same exception. This is by design and is mentioned in the docs ... http://msdn.microsoft.com/en-us/library/dd642329.aspx:
Exceptions that are thrown by valueFactory are cached.
Hence, code as below will never return a value:
bool firstTime = true;
Lazy<int> lazyInt = new Lazy<int>(() =>
{
if (firstTime)
{
firstTime = false;
throw new Exception("Always throws exception the very first time.");
}
return 21;
});
int? val = null;
while (val == null)
{
try
{
val = lazyInt.Value;
}
catch
{
}
}