views:

126

answers:

2

We have a WPF application, based on Unity with MMVVVM pattern. In application life cycle there can be several project life cycles, after each project life cycle we do a manual Tear Down and try to free all the reference of ViewModels. For event subscriptions with Unity we are using Weak references. So we are assuming that after tear down, we may call GC Collect, so that all the garbage objects are garbage collected. We have another option of manually un-subscribing all the events, but we are preferring garbage collection because it will clear around 200MB for us, which will facilitate new project loading.

With one instance, we are observing that, If i call GC.Collect only one time, its reference still remains in memory for sometime.

GC.Collect();
GC.WaitForPendingFinalizers(); 

But if i try Calling GC twice in a row, its cleans up everything nicely.

GC.Collect();
GC.WaitForPendingFinalizers();
GC.Collect();
GC.WaitForPendingFinalizers();

Any thoughts or pointers will be highly appreciated.

Update:

There are no Finalizers defined in Class.

Now i am also considering a case, in which this object is referred in another object which might have a finalizer. In Our framework, we have finalizer only for DBProvider, so i don't think, even this is the case.

+3  A: 

Sounds like you have something with a finalizer, basically - if you only call GC.Collect() once, the finalizers are finishing but the finalized objects aren't being collected.

Whether or not that represents a bug is a different matter. Generally it's not a good idea for there to be finalizers which actually need to be executing, but maybe it's okay in your case.

Jon Skeet
I checked with that particular class, as far i can see there are no finalizers.
Nitin Midha
Now i am considering a case, in which this object is referred in another object which might have a finalizer.
Nitin Midha
If any object has a finalizer, any objects to which it holds direct or indirect references will at least partially survive the next GC (finalizers, if any, will run on such objects; WeakReferences may survive, but I wouldn't trust that they always will).
supercat
Now, at another place, even when i am calling GC.Collect twice, its still not cleaning up the reference.
Nitin Midha
@Nitin: That sounds like you've still got a reference to the object then.
Jon Skeet
When i try to reproduce the behavior with Memory Profiler(yourkit) running on the application, it works just fine as expected, the viewmodel is not in the memory any more. So if i have a reference somewhere, then it should have been in Memroy Profiler also.
Nitin Midha
@Nitin: Do you still have the calls to WaitForPendingFinalizers? To be honest, this sort of investigation isn't really easy to do via sites like Stack Overflow :(
Jon Skeet
A: 

@Nitin, I am not sure whether my suggestion would help you or not, but as a rule of thumb we should avoid explicitly calling GC.Collect(). because it will likely to cause performance problem. Instead try to follow the proper disposal pattern.

Karthik