Question

We have a Windows Forms application that contains thousands of forms.

Many of these are temporarily displayed as dialogs via the ShowDialog() method.

This application has been around for years and we’ve discovered that many of the forms are not getting garbage collected in a timely manner due to various resource leaks in the form or the controls that it uses.

Specifically, we’ve found examples of GDI+ resources that aren’t being disposed of properly, although there may be other types of resource leaks that have not yet been characterized.

Although the right way to resolve this is obviously to go through every form and every control and eliminate all of the resource problems. This will take some time to accomplish.

As an short term alternative, we have found that explicitly calling Dispose() on the form seems to initiate the garbage collection process and the form and its resources are deallocated immediately.

My question is whether is would be a reasonable workaround to wrap each form’s ShowDialog() block in a using statement so that Dispose() is called after the form has been displayed, and also would this be a good practice to institute in general?

For example, change the existing code from this:

public void ShowMyForm()
{
    MyForm myForm = new MyForm();
    myForm.ShowDialog();
}

To this:

public void ShowMyForm()
{
    using (MyForm myForm = new MyForm())
    {
        myForm.ShowDialog();
    }
}

In our testing, MyForm’s Dispose() method never gets called for the first example, but it gets called immediately for the second example.

Does this seem like a reasonable approach as a short term workaround while we spend the time tracking down each of the specific resource issues?

Are there other approaches that we could consider for a short term workaround and/or methodologies for identifying and resolving these types of resource issues?

Answer 1

According to MSDN, you must explicitly call Dispose on forms shown using ShowDialog (unlike with the Show method):

When a form is displayed as a modal dialog box, clicking the Close
button (the button with an X at the upper-right corner of the form)
causes the form to be hidden and the DialogResult property to be set
to DialogResult.Cancel. Unlike non-modal forms, the Close method is
not called by the .NET Framework when the user clicks the close form
button of a dialog box or sets the value of the DialogResult property.
Instead the form is hidden and can be shown again without creating a
new instance of the dialog box. Because a form displayed as a dialog
box is hidden instead of closed, you must call the Dispose method of
the form when the form is no longer needed by your application.