如何将原始指针向量转换为唯一指针向量？

Question

#include <vector>

enum ListOfGameStates
{
    // List of game states
};

class GameState()
{
    public:
        GameStates(); // Initializes protected (global) variables
        virtual ListOfGameStates run() = 0;
    protected:
        // Heavyweigh resource managers containing all resources and other global vars
}

class GameStateManager()
{
    public:
        GameStateManager();  // Creates all game states
        ~GameStateManager(); // Deletes all game states
        void run();          // Switches from one state to another state
    private:
        // A vector of raw pointers to game states. GameState is a base class.
        std::vector<GameState*> game_states_container;
}

我想摆脱原始指针，这样我就不用担心异常和清理。是否有简单的解决方案（我是一个非常愚蠢的青少年）还是不值得？谢谢！

Answer 1

只需将矢量更改为：

std::vector<std::unique_ptr<GameState>> game_states_container;

并删除析构函数中的任何delete。事实上，你可以完全摆脱析构函数，除非它有其他工作要做。

unique_ptr 不可复制但可移动，因此值得对 C++11 移动语义有所了解。当您想将unique_ptr 添加到您的容器时，您可以使用push_back，前提是您传递一个临时值，例如函数的返回值：

game_states_container.push_back(createGameState());
game_states_container.push_back(std::make_unique<GameStateA>());  // C++14

或者，如果您有一个本地 unique_ptr 变量，您可以使用 std::move 将其移动到向量中：

std::unique_ptr<GameState> game_state = std::make_unique<GameStateA>();  // C++14
// auto game_state = std::unique_ptr<GameState>(new GameStateA);  // C++11
...
game_states_container.push_back(std::move(game_state));

最好在new 后立即将原始指针放入unique_ptr（或者最好使用std::make_unique）。否则，如果在unique_ptr 中分配和包装之间引发异常，则您有内存泄漏。

它与unique_ptr 无关，但与您的GameState 类should have a virtual destructor 无关。

Live demo