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

时间:2015-02-01 12:44:13

标签: c++ smart-pointers raii raw-pointer

#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;
}

我想摆脱原始指针,这样我就不用担心异常和清理了。是否有简单的简单解决方案(我是一个非常愚蠢的青少年)或者它不值得吗?谢谢!

1 个答案:

答案 0 :(得分:8)

只需将矢量更改为:

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));

最好在unique_ptr new时将原始指针放在std::make_unique中(或者最好使用unique_ptr)。否则,如果在unique_ptr中分配和换行之间抛出异常,则会发生内存泄漏。

它与GameState无关,但与{{1}}班should have a virtual destructor无关。

Live demo