Take a look at the following code
(myWeakReference.Target as SomeClass).DoSomething();
Do you see the mistake? The WeakReference may return “true” for IsAlive, but because the garbage collector runs within its own thread the value may actually get collected before the next line (because WeakReference.Target does not prevent the GC from collecting the value). I’ve been using WeakReferences quite a lot recently so I was happy to see that the following changes fixed the occasional NullReferenceException occurencies I had been seeing which were quite difficult to track down!
SomeClass myInstance = (SomeClass)myWeakReference.Target;
if (myInstance != null)
The IsAlive property in my opinion is utterly useless (it is the same implementation as Target). I think MS should just remove it, you could say it would break existing code, but I say it would force people to fix it!