C++ 向量对象超出范围

Question

我无法理解如何正确构建指针向量（我对 C++ 很陌生）。我认为下面的代码总结了这个问题：

Mok.h

class Mok
{
public:
    Mok(int n) : m_num{n}
    int getNum() {return m_num}
private:
    int m_num;
};

Bar.h

class Bar
{
public:
    Bar();

    void setTiles();
    void printTiles();

private:
    std::vector<Mok*> m_tileSet;
};

Bar.cpp

void Bar::setTiles()
{
    for (int i = 0; i < 4; ++i)
    {
        m_tileSet.push_back(&Mok(i));
    }
}

void Bar::printTiles()
{
    for (const auto &mok : m_tileSet)
    {
        std::cout << mok->getNum() << " ";
    }
}

这个想法是打印存储在向量中的数字，但我得到了疯狂的数字（我知道这对整数没有多大意义，真正的代码更复杂，这是一种简化。想象 Mok 为一个大对象，以及包含数千个对象的向量）。在 setTiles() 内部，一切似乎都可以正常存储（VS 调试器是这样说的），但是当我到达 printTiles() 时，信息丢失了，我得到了像 -858993460.

我猜 Mok 对象超出了范围，所以引用（编辑：我的意思是地址）开始无处指向？这样做的方法是什么？

谢谢！ :)

Answer 1

正如 Sam 和 Chris 在 cmets 中所建议的，一种方法是避免使用指针。以下代码有效：

Mok.h

class Mok
{
public:
    Mok(int n) : m_num{n}
    int getNum() {return m_num}
private:
    int m_num;
};

Bar.h

class Bar
{
public:
    Bar();

    void setTiles();
    void printTiles();

private:
    std::vector<Mok> m_tileSet;
};

Bar.cpp

void Bar::setTiles()
{
    for (int i = 0; i < 4; ++i)
    {
        m_tileSet.push_back(Mok(i));
    }
}

void Bar::printTiles()
{
    for (auto &mok : m_tileSet)
    {
        std::cout << mok.getNum() << " ";
    }
}

保留指针，使用 std::unique_ptr（感谢 cmets 中的 n.m.）：

Bar.h

class Bar
{
public:
    Bar();

    void setTiles();
    void printTiles();

private:
    std::vector<std::unique_ptr<Mok>> m_tileSet;
};

Bar.cpp

void Bar::setTiles()
{
    for (int i = 0; i < 4; ++i)
    {

        m_tileSet.push_back(std::make_unique<Mok>(i)); // C++14
        //m_tileSet.push_back(std::unique_ptr<Mok>(new Mok(i))); // Previous to C++14
    }
}

void Bar::printTiles()
{
    for (auto &mok : m_tileSet)
    {
        std::cout << mok->getNum() << " ";
    }
}