1

我以前没有使用过boost,所以如果我做了一些愚蠢的事情,请原谅我。我有一个包含 lua_State 的类。我有一个 boost::shared_ptr 向量,我 push_back 新状态如下:

class Lua_State
{
lua_State *L;
std::string m_scr;

public:
Lua_State() : L(luaL_newstate())
{
    lua_register(L, "test", test);
    luaL_openlibs(L);
}

~Lua_State() {
    lua_close(L);
}

inline bool LoadScript(const char* script)
{
    if (!boost::filesystem::exists(script))
        return false;

    m_scr = fs::path(std::string(script)).filename().string();

    chdir(fs::path(scr).parent_path().string().c_str());

    if (luaL_loadfile(L, m_scr.c_str()))
        return false;

    // prime
    if (lua_pcall(L, 0, 0, 0))
        return false;

    return true;
}
};

typedef boost::shared_ptr<Lua_State> LUASTATEPTR;
class Scripts
{
private:
std::vector<LUASTATEPTR> m_Scripts;
public:
    Scripts() { }

    void TestLoad()
{
    m_Scripts.push_back(LUASTATEPTR(new Lua_State()));
    LUASTATEPTR pState = m_Scripts[0];
    pState->LoadScript("C:/test.lua");
}
};

代码可以正常工作并添加 Lua 状态,但几秒钟后应用程序崩溃。我不知道为什么会这样。当我手动操作时它工作正常(没有 shared_ptrs 和手动取消引用)。

4

1 回答 1

3

你违反了 3 1的规则。您创建了一个重要的析构函数并分配构造函数,而没有禁用或编写复制构造函数和operator=.

可能当您创建时,shared_ptr您正在复制上述课程。然后临时的被丢弃,事情变得繁荣起来。

所以,首先禁用LuaState::operator=(LuaState const&)LuaState(LuaState const&)构造函数(制作一个私有的非实现版本,或者在 C++11delete中),或者实现它。

接下来,用于make_shared<LuaState>()创建您的shared_ptr<LuaState>实例。这将“就地”创建它们并删除副本。

1我所说的 3 法则是什么?请参阅以下链接:三法则(维基百科)什么是三法则?

于 2012-12-06T16:17:57.083 回答