C++ & Lua Api, размещение на месте с заводской функцией

Question

У меня есть класс, который я использую в своем коде на C++ и в некоторых сценариях Lua. Соответствующая часть класса выглядит следующим образом:

typedef boost::shared_ptr<Thing> ThingPtr; // convenient 

class Thing 
{ 
public: 
    Thing() { /* do some stuff */ } 
    ~virtual Thing() { } 

    ThingPtr createThing() 
    { 
     ThingPtr thing(new Thing); 

     // initialization can't be done in constructor 
     thing->doSomeInit(); 

     return thing; 
    }  

// other stuff.... 

}; 

Я разоблачить этот класс в Lua (без привязки или что-нибудь «фантазии»). Прежде чем я добавил функцию фабрики, моя функция Lua, чтобы создать Thing выглядел как:

int MyLua::newThing(lua_State* L) 
{ 
    int size = sizeof(Thing); 

    // allocate a new Thing object in place 
    new ((Thing*)lua_newuserdata(L, size)) Thing(); 

    luaL_setmetatable(L, "MyStuff.thing"); 

    return 1; 
} 

После того, как я добавил функцию заводскую я что-то вроде:

int MyLua::newThing(lua_State* L) 
{ 
    int size = sizeof(Thing); 

    // allocate a new Thing object in place 
    Thing* thing = new ((Thing*)lua_newuserdata(L, size)) Thing(); 
    thing->doSomeInit(); 

    luaL_setmetatable(L, "MyStuff.thing"); 

    return 1; 
} 

Это прекрасно, казалось хорошо, за исключением что теперь я хочу, чтобы конструктор Thing был закрыт для обеспечения использования фабричной функции в других местах кода C++. Итак, теперь у меня есть что-то вроде:

int MyLua::newThing(lua_State* L) 
{ 
    int size = sizeof(Thing); 
    ThingPtr thing = Thing::createThing(); 

    void* space = lua_newuserdata(L, size); 
    memcpy(space, client.get(), size); 

    luaL_setmetatable(L, "MyStuff.thing"); 

    return 1; 
} 

Мой вопрос: есть ли лучший способ сделать это? Призыв к memcpy заставляет меня чувствовать себя некомфортно.

Answer 1

Вы не можете передать права собственности на объекты C++ на Lua.

Ваш исходный код является ошибочным, потому что он никогда не вызовет деструктор для вашего Thing s. Хотя Lua будет мусором собирать память, выделенную через lua_newuserdata, она не будет вызывать деструктор объекта (просто потому, что Lua, будучи библиотекой C, не знает о концепции деструкторов).

Таким образом, вам понадобится отдельная конструкция на стороне C++, которая управляет временем жизни ваших объектов и передает только необработанные указатели на Lua для отображения в качестве пользовательских данных.

Answer 2

Это должно сделать вас неудобным; memcpy допускается только для типов с возможностью копирования (Thing не такой тип). Я даже не уверен, что new (lua_newuserdata(L, size)) Thing() разрешен, потому что Lua использует realloc, чтобы требовать новую память по умолчанию, и это может привести к перемещению памяти (т. Е. realloc может memcpy это так или иначе).

Решением, ИМО, является динамически выделять ваш Thing (который, кажется, ваш createThing завода действительно сделать, но с помощью смарт-указателя) и хранить указатель C на объект в пользовательских данных с __gc Метаметодом, которая очищает ваш объект , С умным указателем это сложнее, но это потребует хранения копии умного указателя в куче, сохранения указателя C на интеллектуальном указателе в пользовательских данных, а затем освобождения умного указателя в метатете __gc.