unique_ptr 列表中的访问冲突

Question

我有一个 list 或 unique-ptrs，在清除列表后，会出现以下错误：

Unhandled exception at 0x013EA350 in Last.exe: 0xC0000005: Access violation reading location 0xFEEEFEEE.

当迭代元素时，这在 std::list 文件中触发。这个问题有点独特：

1. MenuManager 有一个 unique_ptr<Menu> 对象列表。
2. Menu 有一个 unique_ptr<MenuItem> 对象列表。
3. 第一个Menu 及其所有MenuItems 完美破坏，但是在清除MenuManager 列表中的第二个Menu 时触发此错误，与完全独立的MenuItems。看起来第二个unique_ptr<Menu> 在第一个销毁后变得未定义，尽管它在MenuManager 的销毁开始时存在且非空。

这是什么原因造成的？ （具体来说，不是我的代码在哪里导致了这个错误。我想知道这可能是什么原因）

MenuManager.h

class MenuManager
{
public:
    /*MenuManager(list<Menu*> menus) 
        : m_menus(menus) { SetMenu( menus.front() ); }*/
    MenuManager() : m_currentMenu(nullptr) {}


    // Get the menu that the manager is pointing to.
    Menu* GetCurrentMenu(void) { return m_currentMenu; }

    // Set the current menu based on its name.
    void SetMenu(string menuName) { SetMenu( GetMenuByName(menuName) ); }
    // Add a menu to the list of menus managed by this object.
    void AddMenu(Menu* menu);

    // Render the current menu.
    void RenderMenu(void) { m_currentMenu->Render(); }

private:
    list< unique_ptr<Menu> > m_menus; // The list of menus managed by this object
    Menu* m_currentMenu; // A pointer to the menu in current use

    Menu* GetMenuByName(string menuName);
    void SetMenu(Menu* menu); // Set the menu based on the menu argument
};

Menu 是另一个基本上不相关的类的子类。但是，它的列表定义为list< unique_ptr<MenuItem> >。 MenuItem 的定义无关紧要。

Answer 1

您没有显示足够多的代码来查明错误，但错误说明了一切......

Last.exe 中 0x013EA350 处未处理的异常：0xC0000005： 访问冲突读取位置 0xFEEEFEEE。

在 Visual Studio 下的调试模式下，被释放的内存设置为代码 0xFEEE。所以指针被释放了两次。

请注意，我看到管理器有一个 std::list 指针，它实际上并没有管理：你有一个带有裸指针的 AddMenu()。

我强烈建议您在任何地方都将std::unique_ptr 替换为std::shared_ptr，并且永远不要在任何地方使用裸指针（除非调用需要此类裸指针的较低级别的API。）共享指针的巨大优势在于你可以有许多类持有对它的引用，如果一个类死亡，其他类仍然有一个有效的指针。如果您有父/子引用，请确保使用std::weak_ptr 作为子级中的父级指针。