我如何解决由ContextMenuStrips不处理引起的内存泄漏

Question

我在这里问了这个问题（我该如何解决由ContextMenuStrip引用的对象引起的内存泄漏 ），并认为已经解决了该问题，但是明显的修复方法无效。 我已经搜索了该问题，并发现了许多有关由于ContextMenuStrips而导致的内存泄漏的问题和博客，但没有找到具体的答案。 我创建了一个小的WinForms测试应用程序，该应用程序出现了问题。 创建项目后，我在设计器的Form1中添加一个TreeView并按如下所示修改Form1.cs：

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
        treeView1.NodeMouseClick += OnNodeMouseClick;

        var object1 = new MyObject("Object1");
        var object2 = new MyObject("Object2");
        var object3 = new MyObject("Object3");

        var treeNode1 = new TreeNode(object1.Name) { Tag = object1 };
        var treeNode2 = new TreeNode(object2.Name) { Tag = object2 };
        var treeNode3 = new TreeNode(object3.Name) { Tag = object3 };
        treeView1.Nodes.Add(treeNode1);
        treeView1.Nodes.Add(treeNode2);
        treeView1.Nodes.Add(treeNode3);
    }

    private void OnNodeMouseClick(object sender, TreeNodeMouseClickEventArgs e)
    {
        if (treeView1.SelectedNode == null)
            return;

        if (e.Button != MouseButtons.Right)
            return;

        MyObject targetObject = (MyObject)e.Node.Tag;

        if (targetObject == null)
            return;

        var point = new Point(e.X + 20, e.Y);
        var popupMenu = targetObject.PopupMenu;
        popupMenu.Show(this, point);
    }
}

我也有MyObject类：

class MyObject
{
    public MyObject(string name)
    {
        Name = name;
    }
    public string Name { get; set; }
    public ContextMenuStrip PopupMenu
    {
        get
        {
            ContextMenuStrip myPopup = new ContextMenuStrip();
            myPopup.Items.Add(ItemDelete);
            return myPopup;
        }
    }

    public ToolStripMenuItem ItemDelete
    {
        get
        {
            ToolStripMenuItem itemDelete = new ToolStripMenuItem("Delete " + Name);
            itemDelete.Click += ItemDelete_Click;
            return itemDelete;
        }
    }

    public void ItemDelete_Click(object sender, EventArgs e)
    {
    }
}

我尚未实现delete方法，因为不需要看到该问题。 每当右键单击树中的对象时，都会创建一个新的ContextMenuStrip。 菜单永远不会被删除。 在我的实际应用程序中，上下文菜单可以引用一些大对象。 dotMemory中显示的保留路径具有由UserPreferenceChangeEventHandler引用的ContextMenuStrip。 关闭菜单后如何处置？

Answer 1

将您的OnNodeMouseClick更改为此：

    private void OnNodeMouseClick(object sender, TreeNodeMouseClickEventArgs e)
    {
        if (treeView1.SelectedNode == null)
            return;
        if (e.Button != MouseButtons.Right)
            return;
        MyObject targetObject = (MyObject)e.Node.Tag;
        if (targetObject == null)
            return;
        e.Node.ContextMenuStrip = MyObject.SharedPopup;
    }

和您的MyObject类：

class MyObject
{
    private static ContextMenuStrip sharedPopup;
    public static ContextMenuStrip SharedPopup
    {
        get
        {
            if (null == sharedPopup)
            {
                sharedPopup = new ContextMenuStrip();
                ToolStripMenuItem deleteItem = new ToolStripMenuItem();
                deleteItem.Click += ItemDelete_Click;
                deleteItem.Paint += deleteItem_Paint;
                sharedPopup.Items.Add(deleteItem);
            }
            return sharedPopup;
        }
    }

    static void deleteItem_Paint(object sender, PaintEventArgs e)
    {
        ToolStripMenuItem item = sender as ToolStripMenuItem;
        item.Text = "Delete " + ((item.Owner as ContextMenuStrip).SourceControl as TreeView).SelectedNode.Text;
    }

    public MyObject(string name)
    {
        Name = name;
    }
    public string Name { get; set; }

    static void ItemDelete_Click(object sender, EventArgs e)
    {

    }
}

这将重用相同的上下文菜单，并仅在需要时更新文本。

Answer 2

我接受了Loathing的建议，但进行了修改。 我的示例太简单了，没有显示实际应用程序中可能出现的所有不同对象类型和所有不同菜单构建。 我需要快速修复，并且修改构建菜单的过程不切实际。 我修改了我的处理程序，如下所示，现在一次只有一个ContextMenuStrip。 当创建一个新的旧的时，旧的被删除。

    private void OnNodeMouseClick(object sender, TreeNodeMouseClickEventArgs e)
    {
        if (treeView1.SelectedNode == null)
            return;

        if (e.Button != MouseButtons.Right)
            return;

        MyObject targetObject = (MyObject)e.Node.Tag;

        if (targetObject == null)
            return;

        var point = new Point(e.X + 20, e.Y);
        var popupMenu = targetObject.PopupMenu;

        if (ContextMenuStrip != null)
        {
            ContextMenuStrip.Dispose();
        }

        ContextMenuStrip = popupMenu;
        ContextMenuStrip.Show(this, point);
    }

顺便说一句，我没有开发此代码。 该应用程序的这一部分非常古老，我只尝试修复内存泄漏之类的非常糟糕的事情。