用户关闭非模态 window 时如何确保垃圾回收？

Question

在我的 C# Winforms 应用程序中，我有以下内容（显示的代码最少）

Form1 是用户用来做事的主要应用程序。 Form2 显示了一个帮助文件，解释了如何使用 Form1 上的功能来做事。 只要 Form1 可见，我希望用户能够随意显示（无模式）和关闭帮助文件。

我还担心用户打开和关闭 Form2 时可能发生的 memory 泄漏。 因此，当用户关闭 Form2 时，它会引发 Form1 订阅的事件。 当调用 Form1 事件方法时，调用 Form2 上的 Dispose()，将 Form2 object 设置为 null 并调用垃圾收集器。

这是否会消除因用户打开和关闭 Form2 而导致的 memory 泄漏的可能性？ 是不是矫枉过正？ 是否有更好的方法确保在 Form2 关闭的时间点进行垃圾回收？ 我不想依赖 Windows 稍后决定

更新

Jimi 指出我不需要 Form2 Closed 事件的自定义事件处理程序。 他是对的。 在我的 Form1 class 中，我现在为我的 Form2 使用标准的 FormClosedEventHandler。 然而，代码本身几乎保持不变。 但是，当我删除对 GC.Collect() 的调用时，我使用任务管理器看到了 memory 泄漏的证据。

这是我的数据：

Run #1. Form2_FormClosed method has:
------------------------------------
f_HelpForm.Dispose();
f_HelpForm = null;
GC.Collect();

Start App
Task Manager Memory for app: 6.7 MB
Open and Close Form2 20 times
Task Manager Memory for app: 8.2 MB

Run #2. Form2_FormClosed method has:
------------------------------------
f_HelpForm.Dispose();
f_HelpForm = null;
//GC.Collect();

Start App
Task Manager Memory for app: 6.9 MB
Open and Close Form2 20 times
Task Manager Memory for app: 18.9 MB

Run #3. Form2_FormClosed method has:
------------------------------------
//f_HelpForm.Dispose();
f_HelpForm = null;
//GC.Collect();

Start App
Task Manager Memory for app: 6.9 MB
Open and Close Form2 20 times
Task Manager Memory for app: 18.1 M

在没有调用 GC.Collect() 的情况下，无论是否调用 Dispose()，与包含调用 GC.collect() 的代码相比，占用空间增加了 100%。

我听到你们在说什么，但是............我想我会把我的代码留在它的“运行＃1”配置中

代码

注意：我承认设置 form2 = null 对幕后垃圾收集有直接影响。 但是，我设置 form2 = null 的目的是向Form2Button_Click方法提供一个信号，该方法可用于决定是否实例化 Form2 object

public partial class Form1 : Form
{
    Form2 form2;
    
    public Form1()
    {
        form2 = null;
    }  

    private void Form2Button_Click(object sender, EventArgs e)
    {
        if (form2 == null)
        {
            form2 = new ClsHelpForm(this);
            form2.Form2Closed += Form2_FormClosed;
        }

        form2.Select();
        form2.Show();
    }

    //When this user clicks the Help button on Form1, this method is invoked
    private void Form2_FormClosed(object sender, EventArgs e)
    {
        form2.Form2Closed -= Form2_FormClosed;
        form2.Dispose();
        form2 = null;
        GC.Collect();
    }   
{

public partial class Form2 : Form
{
    public event EventHandler Form2Closed;

    public Form2()
    {
    }

    //When the user clicks the "X" button on Form2, this method is invoked
    private void Form2_FormClosed(object sender, Form2EventArgs e)
    {
        Form2Closed?.Invoke(this, EventArgs.Empty);
    }
}

Answer 1

我还担心用户打开和关闭 Form2 时可能发生的 memory 泄漏。

您为什么担心 memory 泄漏？ 当没有证据表明存在问题时，不要尝试优化。 只要 Form2 及其所有子对象在调用 Dispose 时实际清理资源，就不会有 memory 泄漏。

是否有更好的方法确保在 Form2 关闭的时间点进行垃圾回收？ 我不想依赖 Windows 稍后决定

这似乎是不必要的偏执狂。 只要确保 Form2 在其 Dispose 方法中进行清理并让垃圾回收自然发生即可。 除了删除 GC.Collect() 调用外，一切看起来都很好。

Answer 2

如果您将垃圾放入厨房垃圾桶，5 分钟后垃圾仍然在那里，这并不意味着垃圾会永远在那里。 这只是意味着垃圾桶还没有被清空。 一旦它足够满，它最终会被清空。 如果您将垃圾放在地板上，就会出现“泄漏”，在这种情况下，当垃圾桶装满时垃圾不会被捡起。 （不是一个完美的类比，因为有人（希望）会在某个时候拿起它，但你明白了）

您的观察结果可能显示 100% 的增长，因为没有 memory 要求收集的压力。 仅当 memory从未被 GC 释放时才会发生 memory 泄漏，而强制垃圾收集无法修复。 memory 泄漏的一个示例是对非托管资源的引用，该资源在处理（未关闭）表单时未释放。

自己调用GC.Collect不会修复 memory 泄漏——它只是在系统需要的时候更早地清理 memory。

Answer 3

Form2的成员实例不会在堆上占用太多空间，而且似乎没有什么理由在用户每次想要显示它时创建和销毁它的句柄。

为什么不在应用程序关闭之前阻止Form2句柄的破坏？

public partial class Form2 : Form
{
    public Form2(Form owner)
    {
        InitializeComponent();
        Owner = owner;
        StartPosition = FormStartPosition.Manual;
    }
    protected override void OnVisibleChanged(EventArgs e)
    {
        base.OnVisibleChanged(e);
        if(Visible)
        {
            Location = new Point(
                Owner.Location.X + Owner.Width + 10,
                Owner.Location.Y);
        }
    }
    protected override void OnFormClosing(FormClosingEventArgs e)
    {
        base.OnFormClosing(e);
        if(e.CloseReason.Equals(CloseReason.UserClosing))
        {
            e.Cancel = true;
            Hide();
        }
    }
}

当 form2 循环（通过自动化）100 次时，监控进程 memory 显示零次 GC，并且与显示的次数没有直接关系。

在哪里：

public partial class Form1 : Form
{
    Form2 _form2;
    public Form1()
    {
        InitializeComponent();
        _form2 = new Form2(this);
        Disposed += (sender, e) => _form2.Dispose();
        buttonShowHelp.Click += async (sender, e) =>
        {
            for (int i = 0; i < numericUpDown.Value; i++)
            {
                _form2.Visible = true;
                await Task.Delay(500);
                _form2.Visible = false;
                await Task.Delay(500);
            }
            // But leave it open after cycling.
            _form2.Visible = true;
        };
    }
    protected override void OnFormClosing(FormClosingEventArgs e)
    {
        base.OnFormClosing(e);
    }
}