如何手动销毁wxMenu？解决PopupMenu内存泄漏与崩溃问题

问题场景 #

wxAuiToolBar按钮点击后弹出包含最近打开文件子菜单的菜单，功能正常但存在两个核心问题：

• 怀疑m_MainMenu存在内存泄漏：PopupMenu不接管菜单所有权，手动管理内存存疑
• 调用delete m_MainMenu触发访问违规崩溃，且无法将m_MainMenu分配到栈上

崩溃根源 #

调试确认崩溃源于将wxString强制转换为wxObject绑定事件，这种非法类型转换会导致事件处理时的内存访问错误。

解决与优化方案 #

1. 修复崩溃问题 #

采用以下两种方式之一绑定事件，避免类型转换错误：

• 自定义继承wxObject的类，用于传递文件路径数据
• 使用Lambda表达式绑定事件（更简洁高效）

2. 解决内存泄漏 #

将m_MainMenu在堆上分配，复用固定菜单项，仅销毁重建动态变化的最近文件子菜单：

• 初始化时创建主菜单并添加固定菜单项，仅执行一次
• 每次点击按钮时，先销毁旧的最近文件子菜单，再重建新的子菜单
• 窗口销毁时统一释放主菜单内存

3. Lambda绑定注意事项 #

使用Lambda绑定事件时，必须捕获文件路径的副本而非引用，防止路径对象失效后引发的内存访问问题。

优化后的代码示例 #

// 类成员变量声明
wxMenu* m_MainMenu;
std::vector<wxString> m_RecentFiles;

// 初始化（如窗口构造函数中）
m_MainMenu = new wxMenu;
// 添加固定菜单项
m_MainMenu->Append(ID_FIXED_ACTION, _("固定操作"));

// 工具栏按钮点击事件处理
void OnToolbarBtnClick(wxCommandEvent&)
{
    // 清理旧的最近文件子菜单
    wxMenu* oldRecentMenu = m_MainMenu->FindSubMenu(_("最近文件"));
    if (oldRecentMenu)
    {
        m_MainMenu->Destroy(oldRecentMenu);
    }

    // 重建最近文件子菜单
    wxMenu* recentMenu = new wxMenu;
    for (const auto& filePath : m_RecentFiles)
    {
        // Lambda捕获filePath副本，避免引用失效
        recentMenu->Append(wxID_ANY, filePath)->Bind(wxEVT_MENU,
            [this, filePath](wxCommandEvent&) {
                OpenTargetFile(filePath);
            });
    }
    m_MainMenu->AppendSubMenu(recentMenu, _("最近文件"));

    // 弹出菜单
    PopupMenu(m_MainMenu);
}

// 窗口销毁事件处理
void OnWindowClose(wxCloseEvent& evt)
{
    delete m_MainMenu; // 此时子菜单已被Destroy，可安全释放
    evt.Skip();
}

内容的提问来源于stack exchange，提问作者macroland