内存泄漏 - 返回而不处理“new CheckBox(...)”

Question

protected override void SetValueFromControl()
{
CheckBox checkbox = new CheckBox();

if (this.comboBox_Meta.SelectedItem != null)
{
    if (this.comboBox_Meta.SelectedIndex != 1)
    {
        checkbox.Checked = true;
    }
    else
    {
       checkbox.Checked = false;
    }

    this.my_Helper.SetValueFromCheckBox("xxx", checkbox);
}
}

我的 ANT Profiler 说有资源泄漏，如下错误

返回而不处理“new CheckBox(...)”。

在我的 SetValueFromCheckBox 方法中，我使用以下条件。

some value = _checkbox.Checked;

注意：现在在社区提供有用的反馈后，我添加了“使用”并等待下一次构建以进行错误验证。

Answer 1

不，错误意味着您应该调用Dispose。

或将其包装在 using 块中：

using (var chk = new CheckBox())
{
    // your code
} // at this line `Dispose` is called automatically, even in case of exception

每个 Windows 窗体控件都会创建一堆不会自动释放的句柄等。它们中的大多数仅在显示控件时创建，因此您可能没有 dispose 调用就可以了，但因为探查器不知道它会抱怨。最好的做法是始终处置实现 IDisposable 的对象 - 您永远不知道实现是否会更改，并且对象会自行创建句柄。

文档：http://msdn.microsoft.com/en-us/library/system.idisposable.aspx

Answer 2

一个常见的误解是，将事物设置为 null 实际上会处理它们。它不是。要处理CheckBox，请调用其Dispose 方法。当您按照其他人的建议使用 using 语句时，这会自动完成。

但是，当您实际将控件添加到控件树时，这不是必需的！在这种情况下，控件将在表单被释放时被释放。

Answer 3

这意味着你应该对它调用 dispose 或使用 using 语句。

using(CheckBox checkbox = new CheckBox())
{

    if (this.comboBox_Meta.SelectedItem != null)
    {
        if (this.comboBox_Meta.SelectedIndex != 1)
        {
            checkbox.Checked = true;
        }
        else
        {
           checkbox.Checked = false;

        }

        this.my_Helper.SetValueFromCheckBox("xxx", checkbox);
    }
}

Answer 4

当您创建动态控件时，您最终应该将控件添加到控件树中。

如果你不这样做，那么创建这样的控制是没有意义的。

不将控件添加到树控件的直接副作用是您将错过控件树的开箱即用处置。当一个控件被释放时（顶层的窗体），它的子控件将被递归释放。

其他人建议添加using 声明，但我认为这是错误的。它将删除警告，但实际问题是您没有正确使用控件树。